Commit 454073c9 authored by Santosh Vernekar's avatar Santosh Vernekar Committed by James Bottomley

[SCSI] qla2xxx: Add loopback IDC-TIME-EXTEND aen handling support.

Earlier IDC-TIME-EXTEND aen was a nop and ignored by driver.
We now have to handle the aen so that other protocol drivers can use time
extension during some loopback operations.
Signed-off-by: default avatarSantosh Vernekar <santosh.vernekar@qlogic.com>
Signed-off-by: default avatarSaurav Kashyap <saurav.kashyap@qlogic.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Parallels.com>
parent f875cd4c
...@@ -627,6 +627,7 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config, ...@@ -627,6 +627,7 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config,
{ {
int ret = 0; int ret = 0;
int rval = 0; int rval = 0;
unsigned long rem_tmo = 0, current_tmo = 0;
struct qla_hw_data *ha = vha->hw; struct qla_hw_data *ha = vha->hw;
if (!IS_QLA81XX(ha) && !IS_QLA8031(ha) && !IS_QLA8044(ha)) if (!IS_QLA81XX(ha) && !IS_QLA8031(ha) && !IS_QLA8044(ha))
...@@ -652,8 +653,19 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config, ...@@ -652,8 +653,19 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config,
} }
/* Wait for DCBX complete event */ /* Wait for DCBX complete event */
if (!wait_for_completion_timeout(&ha->dcbx_comp, current_tmo = DCBX_COMP_TIMEOUT * HZ;
(DCBX_COMP_TIMEOUT * HZ))) { while (1) {
rem_tmo = wait_for_completion_timeout(&ha->dcbx_comp,
current_tmo);
if (!ha->idc_extend_tmo || rem_tmo) {
ha->idc_extend_tmo = 0;
break;
}
current_tmo = ha->idc_extend_tmo * HZ;
ha->idc_extend_tmo = 0;
}
if (!rem_tmo) {
ql_dbg(ql_dbg_user, vha, 0x7022, ql_dbg(ql_dbg_user, vha, 0x7022,
"DCBX completion not received.\n"); "DCBX completion not received.\n");
ret = qla81xx_reset_loopback_mode(vha, new_config, 0, 0); ret = qla81xx_reset_loopback_mode(vha, new_config, 0, 0);
...@@ -678,6 +690,7 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config, ...@@ -678,6 +690,7 @@ qla81xx_set_loopback_mode(scsi_qla_host_t *vha, uint16_t *config,
} }
ha->notify_dcbx_comp = 0; ha->notify_dcbx_comp = 0;
ha->idc_extend_tmo = 0;
done_set_internal: done_set_internal:
return rval; return rval;
......
...@@ -26,7 +26,7 @@ ...@@ -26,7 +26,7 @@
* | | | 0x3036,0x3038 | * | | | 0x3036,0x3038 |
* | | | 0x303a | * | | | 0x303a |
* | DPC Thread | 0x4022 | 0x4002,0x4013 | * | DPC Thread | 0x4022 | 0x4002,0x4013 |
* | Async Events | 0x5086 | 0x502b-0x502f | * | Async Events | 0x5087 | 0x502b-0x502f |
* | | | 0x5047,0x5052 | * | | | 0x5047,0x5052 |
* | | | 0x5040,0x5075 | * | | | 0x5040,0x5075 |
* | | | 0x503d,0x5044 | * | | | 0x503d,0x5044 |
......
...@@ -3292,6 +3292,7 @@ struct qla_hw_data { ...@@ -3292,6 +3292,7 @@ struct qla_hw_data {
/* QLA83XX IDC specific fields */ /* QLA83XX IDC specific fields */
uint32_t idc_audit_ts; uint32_t idc_audit_ts;
uint32_t idc_extend_tmo;
/* DPC low-priority workqueue */ /* DPC low-priority workqueue */
struct workqueue_struct *dpc_lp_wq; struct workqueue_struct *dpc_lp_wq;
......
...@@ -282,25 +282,38 @@ qla81xx_idc_event(scsi_qla_host_t *vha, uint16_t aen, uint16_t descr) ...@@ -282,25 +282,38 @@ qla81xx_idc_event(scsi_qla_host_t *vha, uint16_t aen, uint16_t descr)
"%04x %04x %04x %04x %04x %04x %04x.\n", "%04x %04x %04x %04x %04x %04x %04x.\n",
event[aen & 0xff], mb[0], mb[1], mb[2], mb[3], event[aen & 0xff], mb[0], mb[1], mb[2], mb[3],
mb[4], mb[5], mb[6]); mb[4], mb[5], mb[6]);
if ((aen == MBA_IDC_COMPLETE && mb[1] >> 15)) { switch (aen) {
vha->hw->flags.idc_compl_status = 1; /* Handle IDC Error completion case. */
if (vha->hw->notify_dcbx_comp) case MBA_IDC_COMPLETE:
complete(&vha->hw->dcbx_comp); if (mb[1] >> 15) {
} vha->hw->flags.idc_compl_status = 1;
if (vha->hw->notify_dcbx_comp)
/* Acknowledgement needed? [Notify && non-zero timeout]. */ complete(&vha->hw->dcbx_comp);
timeout = (descr >> 8) & 0xf; }
if (aen != MBA_IDC_NOTIFY || !timeout) break;
return;
ql_dbg(ql_dbg_async, vha, 0x5022, case MBA_IDC_NOTIFY:
"%lu Inter-Driver Communication %s -- ACK timeout=%d.\n", /* Acknowledgement needed? [Notify && non-zero timeout]. */
vha->host_no, event[aen & 0xff], timeout); timeout = (descr >> 8) & 0xf;
ql_dbg(ql_dbg_async, vha, 0x5022,
"%lu Inter-Driver Communication %s -- ACK timeout=%d.\n",
vha->host_no, event[aen & 0xff], timeout);
rval = qla2x00_post_idc_ack_work(vha, mb); if (!timeout)
if (rval != QLA_SUCCESS) return;
ql_log(ql_log_warn, vha, 0x5023, rval = qla2x00_post_idc_ack_work(vha, mb);
"IDC failed to post ACK.\n"); if (rval != QLA_SUCCESS)
ql_log(ql_log_warn, vha, 0x5023,
"IDC failed to post ACK.\n");
break;
case MBA_IDC_TIME_EXT:
vha->hw->idc_extend_tmo = descr;
ql_dbg(ql_dbg_async, vha, 0x5087,
"%lu Inter-Driver Communication %s -- "
"Extend timeout by=%d.\n",
vha->host_no, event[aen & 0xff], vha->hw->idc_extend_tmo);
break;
}
} }
#define LS_UNKNOWN 2 #define LS_UNKNOWN 2
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment