scsi: aacraid: Add periodic checks to see IOP reset status
authorRaghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com>
Wed, 10 May 2017 16:39:45 +0000 (09:39 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 13 Jun 2017 00:47:59 +0000 (20:47 -0400)
Added function that waits with a timeout for the ctrl to be up and running
after triggering an IOP reset. Also removed 30 sec sleep as it is not
needed.

Signed-off-by: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com>
Reviewed-by: David Carroll <david.carroll@microsemi.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/aacraid/aacraid.h
drivers/scsi/aacraid/src.c

index c74e1aa18a20a9b6467b517624d42b0bff4d0b1e..bd0afdd7f2e41a99951ebf048d7e06694734912e 100644 (file)
@@ -2516,6 +2516,7 @@ struct aac_hba_info {
 
 #define        SELF_TEST_FAILED                0x00000004
 #define        MONITOR_PANIC                   0x00000020
+#define        KERNEL_BOOTING                  0x00000040
 #define        KERNEL_UP_AND_RUNNING           0x00000080
 #define        KERNEL_PANIC                    0x00000100
 #define        FLASH_UPD_PENDING               0x00002000
index 39ecb58762a7820eeccf32f9ed7cc5b7b17446d6..cde4160c3e476a2ad984d292202e5872153b0503 100644 (file)
@@ -694,6 +694,37 @@ static void aac_dump_fw_fib_iop_reset(struct aac_dev *dev)
                        0, 0, 0,  0, 0, 0, NULL, NULL, NULL, NULL, NULL);
 }
 
+static bool aac_is_ctrl_up_and_running(struct aac_dev *dev)
+{
+       bool ctrl_up = true;
+       unsigned long status, start;
+       bool is_up = false;
+
+       start = jiffies;
+       do {
+               schedule();
+               status = src_readl(dev, MUnit.OMR);
+
+               if (status == 0xffffffff)
+                       status = 0;
+
+               if (status & KERNEL_BOOTING) {
+                       start = jiffies;
+                       continue;
+               }
+
+               if (time_after(jiffies, start+HZ*SOFT_RESET_TIME)) {
+                       ctrl_up = false;
+                       break;
+               }
+
+               is_up = status & KERNEL_UP_AND_RUNNING;
+
+       } while (!is_up);
+
+       return ctrl_up;
+}
+
 static void aac_notify_fw_of_iop_reset(struct aac_dev *dev)
 {
        aac_adapter_sync_cmd(dev, IOP_RESET_ALWAYS, 0, 0, 0, 0, 0, 0, NULL,
@@ -709,8 +740,6 @@ static void aac_send_iop_reset(struct aac_dev *dev)
        aac_set_intx_mode(dev);
 
        src_writel(dev, MUnit.IDR, IOP_SRC_RESET_MASK);
-
-       msleep(30000);
 }
 
 static void aac_send_hardware_soft_reset(struct aac_dev *dev)
@@ -726,6 +755,7 @@ static void aac_send_hardware_soft_reset(struct aac_dev *dev)
 static int aac_src_restart_adapter(struct aac_dev *dev, int bled, u8 reset_type)
 {
        unsigned long status, start;
+       bool is_ctrl_up;
 
        if (bled < 0)
                goto invalid_out;
@@ -745,6 +775,16 @@ static int aac_src_restart_adapter(struct aac_dev *dev, int bled, u8 reset_type)
        switch (reset_type) {
        case IOP_HWSOFT_RESET:
                aac_send_iop_reset(dev);
+
+               /*
+                * Creates a delay or wait till up and running comes thru
+                */
+               is_ctrl_up = aac_is_ctrl_up_and_running(dev);
+               if (!is_ctrl_up)
+                       dev_err(&dev->pdev->dev, "IOP reset failed\n");
+               else
+                       goto set_startup;
+
                /*
                 * Check to see if KERNEL_UP_AND_RUNNING
                 * Wait for the adapter to be up and running.
@@ -780,6 +820,7 @@ invalid_out:
        if (src_readl(dev, MUnit.OMR) & KERNEL_PANIC)
                return -ENODEV;
 
+set_startup:
        if (startup_timeout < 300)
                startup_timeout = 300;