[SCSI] qla4xxx: Fix double reset in case of firmware hung for ISP83XX
authorVikas Chaudhary <vikas.chaudhary@qlogic.com>
Thu, 7 Mar 2013 10:43:10 +0000 (05:43 -0500)
committerJames Bottomley <JBottomley@Parallels.com>
Wed, 10 Apr 2013 18:45:16 +0000 (11:45 -0700)
In case of firmware hung we need to call "mailbox_premature_completion" to
complete any pending mbox command as firmware is not alive.

Signed-off-by: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/qla4xxx/ql4_os.c

index b3ccb1ad8c025ef77faadef9becc59902a7365d9..ade1ba6b345181617d435eeb55961a62a456a7ba 100644 (file)
@@ -2543,6 +2543,7 @@ static void qla4_8xxx_process_fw_error(struct scsi_qla_host *ha)
 void qla4_8xxx_watchdog(struct scsi_qla_host *ha)
 {
        uint32_t dev_state;
+       uint32_t idc_ctrl;
 
        /* don't poll if reset is going on */
        if (!(test_bit(DPC_RESET_ACTIVE, &ha->dpc_flags) ||
@@ -2561,10 +2562,23 @@ void qla4_8xxx_watchdog(struct scsi_qla_host *ha)
                        qla4xxx_wake_dpc(ha);
                } else if (dev_state == QLA8XXX_DEV_NEED_RESET &&
                           !test_bit(DPC_RESET_HA, &ha->dpc_flags)) {
+
+                       ql4_printk(KERN_INFO, ha, "%s: HW State: NEED RESET!\n",
+                                  __func__);
+
+                       if (is_qla8032(ha)) {
+                               idc_ctrl = qla4_83xx_rd_reg(ha,
+                                                       QLA83XX_IDC_DRV_CTRL);
+                               if (!(idc_ctrl & GRACEFUL_RESET_BIT1)) {
+                                       ql4_printk(KERN_INFO, ha, "%s: Graceful reset bit is not set\n",
+                                                  __func__);
+                                       qla4xxx_mailbox_premature_completion(
+                                                                           ha);
+                               }
+                       }
+
                        if (is_qla8032(ha) ||
                            (is_qla8022(ha) && !ql4xdontresethba)) {
-                               ql4_printk(KERN_INFO, ha, "%s: HW State: "
-                                   "NEED RESET!\n", __func__);
                                set_bit(DPC_RESET_HA, &ha->dpc_flags);
                                qla4xxx_wake_dpc(ha);
                        }