[SCSI] mpt2sas: when Async scanning is enabled then while scanning, devices are remov...
authorSreekanth Reddy <Sreekanth.Reddy@lsi.com>
Thu, 25 Jul 2013 06:01:12 +0000 (11:31 +0530)
committerJames Bottomley <JBottomley@Parallels.com>
Tue, 3 Sep 2013 14:27:52 +0000 (07:27 -0700)
When Async scanning mode is enabled and device scanning is in progress then
devices should not be removed. But in actuality, devices are removed but
their transport layer entries are not removed. This causes error to add
the same device to the transport layer after host reset or diagnostic
reset.

So, in this patch, modified the code in such a way that device is not removed
when Async scanning mode is enabled and device scanning is in progress.

Signed-off-by: Sreekanth Reddy <Sreekanth.Reddy@lsi.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/mpt2sas/mpt2sas_scsih.c

index 9854385deadf551735c3cdbc266434d52d3a7de9..481afe60bc91e4ff4ff9928da782fb909825fe8b 100644 (file)
@@ -628,11 +628,12 @@ _scsih_sas_device_add(struct MPT2SAS_ADAPTER *ioc,
                 * devices while scanning is turned on due to an oops in
                 * scsi_sysfs_add_sdev()->add_device()->sysfs_addrm_start()
                 */
-               if (!ioc->is_driver_loading)
+               if (!ioc->is_driver_loading) {
                        mpt2sas_transport_port_remove(ioc,
                        sas_device->sas_address,
                        sas_device->sas_address_parent);
-               _scsih_sas_device_remove(ioc, sas_device);
+                       _scsih_sas_device_remove(ioc, sas_device);
+               }
        }
 }
 
@@ -7916,10 +7917,12 @@ _scsih_probe_boot_devices(struct MPT2SAS_ADAPTER *ioc)
                    sas_device->sas_address_parent)) {
                        _scsih_sas_device_remove(ioc, sas_device);
                } else if (!sas_device->starget) {
-                       if (!ioc->is_driver_loading)
-                               mpt2sas_transport_port_remove(ioc, sas_address,
+                       if (!ioc->is_driver_loading) {
+                               mpt2sas_transport_port_remove(ioc,
+                                       sas_address,
                                        sas_address_parent);
-                       _scsih_sas_device_remove(ioc, sas_device);
+                               _scsih_sas_device_remove(ioc, sas_device);
+                       }
                }
        }
 }
@@ -7972,14 +7975,14 @@ _scsih_probe_sas(struct MPT2SAS_ADAPTER *ioc)
                        kfree(sas_device);
                        continue;
                } else if (!sas_device->starget) {
-                       if (!ioc->is_driver_loading)
+                       if (!ioc->is_driver_loading) {
                                mpt2sas_transport_port_remove(ioc,
                                        sas_device->sas_address,
                                        sas_device->sas_address_parent);
-                       list_del(&sas_device->list);
-                       kfree(sas_device);
-                       continue;
-
+                               list_del(&sas_device->list);
+                               kfree(sas_device);
+                               continue;
+                       }
                }
                spin_lock_irqsave(&ioc->sas_device_lock, flags);
                list_move_tail(&sas_device->list, &ioc->sas_device_list);