scsi: qla2xxx: Fix stuck session in gpdb

commit 725d3a0d31a51c0debf970011e05f585e805165b upstream.

Fix stuck sessions in get port database. When a thread is in the process of
re-establishing a session, a flag is set to prevent multiple threads /
triggers from doing the same task. This flag was left on, where any attempt
to relogin was locked out. Clear this flag, if the attempt has failed.

Link: https://lore.kernel.org/r/20220110050218.3958-4-njavali@marvell.com
Cc: stable@vger.kernel.org
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: Quinn Tran <qutran@marvell.com>
Signed-off-by: Nilesh Javali <njavali@marvell.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c
index 613e5467b4bc2cc5c056b6c5e50206960b0a98c5..beba4db0734ea1f952151e629d416962b0c7898f 100644 (file)
--- a/drivers/scsi/qla2xxx/qla_init.c
+++ b/drivers/scsi/qla2xxx/qla_init.c
@@ -219,9 +219,9 @@ qla2x00_async_login(struct scsi_qla_host *vha, fc_port_t *fcport,
        if (!vha->flags.online || (fcport->flags & FCF_ASYNC_SENT) ||
            fcport->loop_id == FC_NO_LOOP_ID) {
                ql_log(ql_log_warn, vha, 0xffff,
-                   "%s: %8phC - not sending command.\n",
-                   __func__, fcport->port_name);
-               return rval;
+                   "%s: %8phC online %d flags %x - not sending command.\n",
+                   __func__, fcport->port_name, vha->flags.online, fcport->flags);
+               goto done;
        }
 
        sp = qla2x00_get_sp(vha, fcport, GFP_KERNEL);