]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
[PATCH] libata: make sure sdev doesn't go away while rescanning
authorTejun Heo <htejun@gmail.com>
Wed, 22 Nov 2006 02:21:31 +0000 (11:21 +0900)
committerTejun Heo <htejun@gmail.com>
Sun, 3 Dec 2006 08:56:24 +0000 (17:56 +0900)
ata_scsi_dev_rescan() doesn't synchronize against SCSI device detach
and the target sdev might go away in the middle.  Fix it.

Signed-off-by: Tejun Heo <htejun@gmail.com>
drivers/ata/libata-scsi.c

index 7ff1adfc729c5a6157820f2db0228ec5f47bdd5d..d250858d201c995ff76e5abbc15da73cfcb2560f 100644 (file)
@@ -3181,15 +3181,27 @@ static int ata_scsi_user_scan(struct Scsi_Host *shost, unsigned int channel,
 void ata_scsi_dev_rescan(void *data)
 {
        struct ata_port *ap = data;
-       struct ata_device *dev;
+       unsigned long flags;
        unsigned int i;
 
+       spin_lock_irqsave(ap->lock, flags);
+
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
-               dev = &ap->device[i];
+               struct ata_device *dev = &ap->device[i];
+               struct scsi_device *sdev = dev->sdev;
 
-               if (ata_dev_enabled(dev) && dev->sdev)
-                       scsi_rescan_device(&(dev->sdev->sdev_gendev));
+               if (!ata_dev_enabled(dev) || !sdev)
+                       continue;
+               if (scsi_device_get(sdev))
+                       continue;
+
+               spin_unlock_irqrestore(ap->lock, flags);
+               scsi_rescan_device(&(sdev->sdev_gendev));
+               scsi_device_put(sdev);
+               spin_lock_irqsave(ap->lock, flags);
        }
+
+       spin_unlock_irqrestore(ap->lock, flags);
 }
 
 /**