scsi: lpfc: Fix failure to delete vports when discovery is in progress
authorJustin Tee <justin.tee@broadcom.com>
Wed, 31 Jan 2024 18:51:02 +0000 (10:51 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 6 Feb 2024 01:51:35 +0000 (20:51 -0500)
Requests to delete an NPIV port may fail repeatedly if the initial request
is received during discovery.

If the FC_UNLOADING load_flag is set, then skip CT response processing for
the physical port.  This allows discovery processing for other lpfc_vport
objects to reach their cmpl routines before deleting the vport.

Signed-off-by: Justin Tee <justin.tee@broadcom.com>
Link: https://lore.kernel.org/r/20240131185112.149731-8-justintee8345@gmail.com
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc_ct.c

index baae1f8279e0cbe66128d815974803b56783f09e..315db836404ad0f889915f9769724ab8849feb37 100644 (file)
@@ -943,8 +943,8 @@ lpfc_cmpl_ct_cmd_gid_ft(struct lpfc_hba *phba, struct lpfc_iocbq *cmdiocb,
                goto out;
        }
 
-       /* Don't bother processing response if vport is being torn down. */
-       if (vport->load_flag & FC_UNLOADING) {
+       /* Skip processing response on pport if unloading */
+       if (vport == phba->pport && vport->load_flag & FC_UNLOADING) {
                if (vport->fc_flag & FC_RSCN_MODE)
                        lpfc_els_flush_rscn(vport);
                goto out;
@@ -1166,8 +1166,8 @@ lpfc_cmpl_ct_cmd_gid_pt(struct lpfc_hba *phba, struct lpfc_iocbq *cmdiocb,
                goto out;
        }
 
-       /* Don't bother processing response if vport is being torn down. */
-       if (vport->load_flag & FC_UNLOADING) {
+       /* Skip processing response on pport if unloading */
+       if (vport == phba->pport && vport->load_flag & FC_UNLOADING) {
                if (vport->fc_flag & FC_RSCN_MODE)
                        lpfc_els_flush_rscn(vport);
                goto out;