diff options
| author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2008-04-24 18:21:30 -0400 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-04-27 13:19:59 -0400 |
| commit | 666301e673e192c87a40e07a8357d6996b57b70f (patch) | |
| tree | 833dc6ed2b01f86c94c3ce886e0bdb91fc5d2e05 | |
| parent | c6852c4c5984fff130a859792d4b26d30c85c54b (diff) | |
[SCSI] qla2xxx: Correct regression in relogin code.
Commit 63a8651f2548c6bb5132c0b4e7dad4f57a9274db ([SCSI] qla2xxx:
Correct infinite-login-retry issue.) introduced a small
regression where a successful relogin would result in an fcport's
loop_id to be incorrectly reset to FC_NO_LOOP_ID. Only clear-out
loopid, if retries have been 'truly' exhausted.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Cc: Stable Tree <stable@kernel.org>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
| -rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 878a37ac2a55..3223fd16bcfe 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
| @@ -2373,7 +2373,7 @@ qla2x00_do_dpc(void *data) | |||
| 2373 | } else { | 2373 | } else { |
| 2374 | fcport->login_retry = 0; | 2374 | fcport->login_retry = 0; |
| 2375 | } | 2375 | } |
| 2376 | if (fcport->login_retry == 0) | 2376 | if (fcport->login_retry == 0 && status != QLA_SUCCESS) |
| 2377 | fcport->loop_id = FC_NO_LOOP_ID; | 2377 | fcport->loop_id = FC_NO_LOOP_ID; |
| 2378 | } | 2378 | } |
| 2379 | if (test_bit(LOOP_RESYNC_NEEDED, &ha->dpc_flags)) | 2379 | if (test_bit(LOOP_RESYNC_NEEDED, &ha->dpc_flags)) |