qla4xxx: Export sysfs DDBs from DPC handler
authorNilesh Javali <nilesh.javali@qlogic.com>
Thu, 10 Apr 2014 01:12:25 +0000 (21:12 -0400)
committerChristoph Hellwig <hch@lst.de>
Mon, 19 May 2014 17:12:19 +0000 (19:12 +0200)
Issue:
modprobe qla4xxx is killed by systemd due to timeout.

Solution:
The exporting of sysfs DDBs from qla4xxx_probe_adapter added delay of
approximately 15s due to which system-udevd killed the modprobe of the
driver. Added fix to export the sysfs DDBs from the DPC handler.

Signed-off-by: Nilesh Javali <nilesh.javali@qlogic.com>
Signed-off-by: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/qla4xxx/ql4_def.h
drivers/scsi/qla4xxx/ql4_glbl.h
drivers/scsi/qla4xxx/ql4_os.c

index 73a502288bde6aa003154682a0f3d1da534a210a..8f6d0fb2cd807255a66e962c3cb7c4c8633d4d77 100644 (file)
@@ -601,6 +601,7 @@ struct scsi_qla_host {
 #define DPC_HA_NEED_QUIESCENT          22 /* 0x00400000 ISP-82xx only*/
 #define DPC_POST_IDC_ACK               23 /* 0x00800000 */
 #define DPC_RESTORE_ACB                        24 /* 0x01000000 */
+#define DPC_SYSFS_DDB_EXPORT           25 /* 0x02000000 */
 
        struct Scsi_Host *host; /* pointer to host data */
        uint32_t tot_ddbs;
index 20354754faa2076b1406920a5d8b694063961937..5f58b451327e8a79040facd208b041c2a43cdb8f 100644 (file)
@@ -281,6 +281,7 @@ int qla4_83xx_get_port_config(struct scsi_qla_host *ha, uint32_t *config);
 int qla4_83xx_set_port_config(struct scsi_qla_host *ha, uint32_t *config);
 int qla4_8xxx_check_init_adapter_retry(struct scsi_qla_host *ha);
 int qla4_83xx_is_detached(struct scsi_qla_host *ha);
+int qla4xxx_sysfs_ddb_export(struct scsi_qla_host *ha);
 
 extern int ql4xextended_error_logging;
 extern int ql4xdontresethba;
index bb160409b7cb90279481567b75c2b79770c7fb2f..9be829bb90dba795d12ff3be871bd833aae62412 100644 (file)
@@ -4565,6 +4565,7 @@ static void qla4xxx_timer(struct scsi_qla_host *ha)
             test_bit(DPC_LINK_CHANGED, &ha->dpc_flags) ||
             test_bit(DPC_HA_UNRECOVERABLE, &ha->dpc_flags) ||
             test_bit(DPC_HA_NEED_QUIESCENT, &ha->dpc_flags) ||
+            test_bit(DPC_SYSFS_DDB_EXPORT, &ha->dpc_flags) ||
             test_bit(DPC_AEN, &ha->dpc_flags)) {
                DEBUG2(printk("scsi%ld: %s: scheduling dpc routine"
                              " - dpc flags = 0x%lx\n",
@@ -5429,6 +5430,11 @@ dpc_post_reset_ha:
                                qla4xxx_relogin_all_devices(ha);
                }
        }
+       if (test_and_clear_bit(DPC_SYSFS_DDB_EXPORT, &ha->dpc_flags)) {
+               if (qla4xxx_sysfs_ddb_export(ha))
+                       ql4_printk(KERN_ERR, ha, "%s: Error exporting ddb to sysfs\n",
+                                  __func__);
+       }
 }
 
 /**
@@ -8406,7 +8412,7 @@ exit_ddb_del:
  *
  * Export the firmware DDB for all send targets and normal targets to sysfs.
  **/
-static int qla4xxx_sysfs_ddb_export(struct scsi_qla_host *ha)
+int qla4xxx_sysfs_ddb_export(struct scsi_qla_host *ha)
 {
        struct dev_db_entry *fw_ddb_entry = NULL;
        dma_addr_t fw_ddb_entry_dma;
@@ -8844,11 +8850,8 @@ skip_retry_init:
                ql4_printk(KERN_ERR, ha,
                           "%s: No iSCSI boot target configured\n", __func__);
 
-       if (qla4xxx_sysfs_ddb_export(ha))
-               ql4_printk(KERN_ERR, ha,
-                          "%s: Error exporting ddb to sysfs\n", __func__);
-
-               /* Perform the build ddb list and login to each */
+       set_bit(DPC_SYSFS_DDB_EXPORT, &ha->dpc_flags);
+       /* Perform the build ddb list and login to each */
        qla4xxx_build_ddb_list(ha, INIT_ADAPTER);
        iscsi_host_for_each_session(ha->host, qla4xxx_login_flash_ddb);
        qla4xxx_wait_login_resp_boot_tgt(ha);