From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mikulas Patocka Subject: [PATCH] dm-mpath: do not change SCSI device handler Date: Tue, 2 Apr 2013 20:04:17 -0400 (EDT) Message-ID: Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: "Alasdair G. Kergon" , Mike Snitzer Cc: dm-devel@redhat.com List-Id: dm-devel.ids Hi This fixes BZ 912245 and 902595. Mikulas --- dm-mpath: do not change SCSI device handler This patch prevents the multipath target from changing the device handler. This fixes a kernel crash that can happen when changing the device handler. When we reload a multipath device, there are two instances of the multipath target - the first instance that is active and the second instance that is being constructed with "ctr" method. If the multipath constructor finds out that the device is using a different device handler, it detaches the existing handler and attaches a new handler. However, the first instance of the multipath target still exists and processes requests. If the first instance sends some path-management request with scsi_dh_activate and the second instance detaches the device handler while the path-management request is in flight, a crash happens. The reason for the crash is that the endio routine for the path-management request is working with structures that were freed when the handler was detached. There is no practical need to change device handlers on an active device, so this patch disables it. Signed-off-by: Mikulas Patocka --- drivers/md/dm-mpath.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) Index: linux-3.9-rc5-fast/drivers/md/dm-mpath.c =================================================================== --- linux-3.9-rc5-fast.orig/drivers/md/dm-mpath.c 2013-04-02 21:54:25.000000000 +0200 +++ linux-3.9-rc5-fast/drivers/md/dm-mpath.c 2013-04-03 01:15:04.000000000 +0200 @@ -618,12 +618,13 @@ static struct pgpath *parse_path(struct */ r = scsi_dh_attach(q, m->hw_handler_name); if (r == -EBUSY) { - /* - * Already attached to different hw_handler: - * try to reattach with correct one. - */ - scsi_dh_detach(q); - r = scsi_dh_attach(q, m->hw_handler_name); + ti->error = "a different device handler is already " + "attached"; + DMERR("A different device handler for device %s is " + "attached. You need to deactivate " + "the device to change device handler.", + p->path.dev->name); + goto bad; } if (r < 0) {