OSDN Git Service

scsi: qla2xxx: Fix stale session
authorQuinn Tran <qutran@marvell.com>
Fri, 30 Aug 2019 22:24:01 +0000 (15:24 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Sat, 7 Sep 2019 19:38:55 +0000 (15:38 -0400)
On fast cable pull, where driver is unable to detect device has disappeared
and came back based on switch info, qla2xxx would not re-login while remote
port has already invalidated the session.  This causes IO timeout.  This
patch would relogin to remote device for RSCN affected port.

Signed-off-by: Quinn Tran <qutran@marvell.com>
Signed-off-by: Himanshu Madhani <hmadhani@marvell.com>
Reviewed-by: Ewan D. Milne <emilne@redhat.com>
Link: https://lore.kernel.org/r/20190830222402.23688-6-hmadhani@marvell.com
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_gs.c

index 03f94eb..dc0e366 100644 (file)
@@ -3628,7 +3628,6 @@ void qla24xx_async_gnnft_done(scsi_qla_host_t *vha, srb_t *sp)
                list_for_each_entry(fcport, &vha->vp_fcports, list) {
                        if (memcmp(rp->port_name, fcport->port_name, WWN_SIZE))
                                continue;
-                       fcport->scan_needed = 0;
                        fcport->scan_state = QLA_FCPORT_FOUND;
                        found = true;
                        /*
@@ -3637,10 +3636,12 @@ void qla24xx_async_gnnft_done(scsi_qla_host_t *vha, srb_t *sp)
                        if ((fcport->flags & FCF_FABRIC_DEVICE) == 0) {
                                qla2x00_clear_loop_id(fcport);
                                fcport->flags |= FCF_FABRIC_DEVICE;
-                       } else if (fcport->d_id.b24 != rp->id.b24) {
+                       } else if (fcport->d_id.b24 != rp->id.b24 ||
+                               fcport->scan_needed) {
                                qlt_schedule_sess_for_deletion(fcport);
                        }
                        fcport->d_id.b24 = rp->id.b24;
+                       fcport->scan_needed = 0;
                        break;
                }