scsi: megaraid_sas: Do not fire DCMDs during PCI shutdown/detach
authorKashyap Desai <kashyap.desai@broadcom.com>
Fri, 21 Oct 2016 13:33:31 +0000 (06:33 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 8 Nov 2016 22:29:57 +0000 (17:29 -0500)
This patch addresses the issue of driver firing DCMDs in PCI
shutdown/detach path irrespective of firmware state.  Driver will now
check whether firmware is in operational state or not before firing
DCMDs. If firmware is in unrecoverable state or does not become
operational within specfied time, driver will skip firing DCMDs.

[mkp: fixed typos]

Signed-off-by: Sumit Saxena <sumit.saxena@broadcom.com>
Signed-off-by: Shivasharan Srikanteshwara <shivasharan.srikanteshwara@broadcom.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/megaraid/megaraid_sas_base.c
drivers/scsi/megaraid/megaraid_sas_fusion.c

index d8b1fbd4c8aafc61e5e5491d5394440651a43a04..d5410a39d956faf1d129a00b2de007dc7778bb09 100644 (file)
@@ -6245,6 +6245,34 @@ fail_reenable_msix:
 #define megasas_resume NULL
 #endif
 
+static inline int
+megasas_wait_for_adapter_operational(struct megasas_instance *instance)
+{
+       int wait_time = MEGASAS_RESET_WAIT_TIME * 2;
+       int i;
+
+       if (atomic_read(&instance->adprecovery) == MEGASAS_HW_CRITICAL_ERROR)
+               return 1;
+
+       for (i = 0; i < wait_time; i++) {
+               if (atomic_read(&instance->adprecovery) == MEGASAS_HBA_OPERATIONAL)
+                       break;
+
+               if (!(i % MEGASAS_RESET_NOTICE_INTERVAL))
+                       dev_notice(&instance->pdev->dev, "waiting for controller reset to finish\n");
+
+               msleep(1000);
+       }
+
+       if (atomic_read(&instance->adprecovery) != MEGASAS_HBA_OPERATIONAL) {
+               dev_info(&instance->pdev->dev, "%s timed out while waiting for HBA to recover.\n",
+                       __func__);
+               return 1;
+       }
+
+       return 0;
+}
+
 /**
  * megasas_detach_one -        PCI hot"un"plug entry point
  * @pdev:              PCI device structure
@@ -6269,9 +6297,14 @@ static void megasas_detach_one(struct pci_dev *pdev)
        if (instance->fw_crash_state != UNAVAILABLE)
                megasas_free_host_crash_buffer(instance);
        scsi_remove_host(instance->host);
+
+       if (megasas_wait_for_adapter_operational(instance))
+               goto skip_firing_dcmds;
+
        megasas_flush_cache(instance);
        megasas_shutdown_controller(instance, MR_DCMD_CTRL_SHUTDOWN);
 
+skip_firing_dcmds:
        /* cancel the delayed work if this work still in queue*/
        if (instance->ev != NULL) {
                struct megasas_aen_event *ev = instance->ev;
@@ -6385,8 +6418,14 @@ static void megasas_shutdown(struct pci_dev *pdev)
        struct megasas_instance *instance = pci_get_drvdata(pdev);
 
        instance->unload = 1;
+
+       if (megasas_wait_for_adapter_operational(instance))
+               goto skip_firing_dcmds;
+
        megasas_flush_cache(instance);
        megasas_shutdown_controller(instance, MR_DCMD_CTRL_SHUTDOWN);
+
+skip_firing_dcmds:
        instance->instancet->disable_intr(instance);
        megasas_destroy_irqs(instance);
 
index 61be7ed73a7c39584247f4238d9687b15f0b7189..2159f6ae5a31462ed4890a1f69ea47fd95d147cc 100644 (file)
@@ -2463,12 +2463,15 @@ irqreturn_t megasas_isr_fusion(int irq, void *devp)
                        /* Start collecting crash, if DMA bit is done */
                        if ((fw_state == MFI_STATE_FAULT) && dma_state)
                                schedule_work(&instance->crash_init);
-                       else if (fw_state == MFI_STATE_FAULT)
-                               schedule_work(&instance->work_init);
+                       else if (fw_state == MFI_STATE_FAULT) {
+                               if (instance->unload == 0)
+                                       schedule_work(&instance->work_init);
+                       }
                } else if (fw_state == MFI_STATE_FAULT) {
                        dev_warn(&instance->pdev->dev, "Iop2SysDoorbellInt"
                               "for scsi%d\n", instance->host->host_no);
-                       schedule_work(&instance->work_init);
+                       if (instance->unload == 0)
+                               schedule_work(&instance->work_init);
                }
        }