diff options
author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2005-12-06 13:57:06 -0500 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.(none)> | 2005-12-07 09:30:49 -0500 |
commit | 23443b1d6130eff8e1335e4f84eaf0577a331dcf (patch) | |
tree | d6bc184f97f74114cd1904a51a45d94527118ba8 | |
parent | bb58596f6802a4959c2cea02acd272245e671c1d (diff) |
[SCSI] qla2xxx: Correct mis-handling of AENs.
A regression in a recent change
33135aa2a568ec1a30e734f18e5315e10516e4f3 caused the driver
to mistakenly drop handling of AENs. Due to the incorrect
handling, ports would not reappear after RSCNs and LIPs.
Drops unused/incorrect compound #define from qla_def.h.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r-- | drivers/scsi/qla2xxx/qla_def.h | 10 | ||||
-rw-r--r-- | drivers/scsi/qla2xxx/qla_init.c | 6 |
2 files changed, 4 insertions, 12 deletions
diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drivers/scsi/qla2xxx/qla_def.h index 7096945ea234..7b3efd531297 100644 --- a/drivers/scsi/qla2xxx/qla_def.h +++ b/drivers/scsi/qla2xxx/qla_def.h | |||
@@ -2476,17 +2476,9 @@ typedef struct scsi_qla_host { | |||
2476 | */ | 2476 | */ |
2477 | #define LOOP_TRANSITION(ha) \ | 2477 | #define LOOP_TRANSITION(ha) \ |
2478 | (test_bit(ISP_ABORT_NEEDED, &ha->dpc_flags) || \ | 2478 | (test_bit(ISP_ABORT_NEEDED, &ha->dpc_flags) || \ |
2479 | test_bit(LOOP_RESYNC_NEEDED, &ha->dpc_flags)) | 2479 | test_bit(LOOP_RESYNC_NEEDED, &ha->dpc_flags) || \ |
2480 | |||
2481 | #define LOOP_NOT_READY(ha) \ | ||
2482 | ((test_bit(ISP_ABORT_NEEDED, &ha->dpc_flags) || \ | ||
2483 | test_bit(ABORT_ISP_ACTIVE, &ha->dpc_flags) || \ | ||
2484 | test_bit(LOOP_RESYNC_NEEDED, &ha->dpc_flags) || \ | ||
2485 | test_bit(LOOP_RESYNC_ACTIVE, &ha->dpc_flags)) || \ | ||
2486 | atomic_read(&ha->loop_state) == LOOP_DOWN) | 2480 | atomic_read(&ha->loop_state) == LOOP_DOWN) |
2487 | 2481 | ||
2488 | #define LOOP_RDY(ha) (!LOOP_NOT_READY(ha)) | ||
2489 | |||
2490 | #define TGT_Q(ha, t) (ha->otgt[t]) | 2482 | #define TGT_Q(ha, t) (ha->otgt[t]) |
2491 | 2483 | ||
2492 | #define to_qla_host(x) ((scsi_qla_host_t *) (x)->hostdata) | 2484 | #define to_qla_host(x) ((scsi_qla_host_t *) (x)->hostdata) |
diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c index 2d720121a0d3..c46d2469b85f 100644 --- a/drivers/scsi/qla2xxx/qla_init.c +++ b/drivers/scsi/qla2xxx/qla_init.c | |||
@@ -1259,7 +1259,7 @@ qla2x00_configure_hba(scsi_qla_host_t *ha) | |||
1259 | rval = qla2x00_get_adapter_id(ha, | 1259 | rval = qla2x00_get_adapter_id(ha, |
1260 | &loop_id, &al_pa, &area, &domain, &topo); | 1260 | &loop_id, &al_pa, &area, &domain, &topo); |
1261 | if (rval != QLA_SUCCESS) { | 1261 | if (rval != QLA_SUCCESS) { |
1262 | if (LOOP_NOT_READY(ha) || atomic_read(&ha->loop_down_timer) || | 1262 | if (LOOP_TRANSITION(ha) || atomic_read(&ha->loop_down_timer) || |
1263 | (rval == QLA_COMMAND_ERROR && loop_id == 0x7)) { | 1263 | (rval == QLA_COMMAND_ERROR && loop_id == 0x7)) { |
1264 | DEBUG2(printk("%s(%ld) Loop is in a transition state\n", | 1264 | DEBUG2(printk("%s(%ld) Loop is in a transition state\n", |
1265 | __func__, ha->host_no)); | 1265 | __func__, ha->host_no)); |
@@ -1796,7 +1796,7 @@ qla2x00_configure_loop(scsi_qla_host_t *ha) | |||
1796 | } | 1796 | } |
1797 | 1797 | ||
1798 | if (rval == QLA_SUCCESS && test_bit(RSCN_UPDATE, &flags)) { | 1798 | if (rval == QLA_SUCCESS && test_bit(RSCN_UPDATE, &flags)) { |
1799 | if (LOOP_NOT_READY(ha)) { | 1799 | if (LOOP_TRANSITION(ha)) { |
1800 | rval = QLA_FUNCTION_FAILED; | 1800 | rval = QLA_FUNCTION_FAILED; |
1801 | } else { | 1801 | } else { |
1802 | rval = qla2x00_configure_fabric(ha); | 1802 | rval = qla2x00_configure_fabric(ha); |
@@ -2369,7 +2369,7 @@ qla2x00_find_all_fabric_devs(scsi_qla_host_t *ha, struct list_head *new_fcports) | |||
2369 | if (qla2x00_is_reserved_id(ha, loop_id)) | 2369 | if (qla2x00_is_reserved_id(ha, loop_id)) |
2370 | continue; | 2370 | continue; |
2371 | 2371 | ||
2372 | if (atomic_read(&ha->loop_down_timer) || LOOP_NOT_READY(ha)) | 2372 | if (atomic_read(&ha->loop_down_timer) || LOOP_TRANSITION(ha)) |
2373 | break; | 2373 | break; |
2374 | 2374 | ||
2375 | if (swl != NULL) { | 2375 | if (swl != NULL) { |