scsi: lpfc: Fix duplicate NVME rport entries and namespaces.
authorDick Kennedy <dick.kennedy@broadcom.com>
Wed, 23 Aug 2017 23:55:37 +0000 (16:55 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 25 Aug 2017 02:29:38 +0000 (22:29 -0400)
After lip, the driver sometimes would have two rports for the same
device, allowing the namespaces to be duplicated by nvme.

In lpfc_plogi_confirm_nport() the driver was not swapping the nrport
maintained by the ndlp's undergoing address swapping. This allowed the
2nd rport to sneak in as it was considered a separate device.

This patch adds the fixes to Swap the nrport in each ndlp and take care
of the reference counts on the ndlps similar to FCP rports.

Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: James Smart <james.smart@broadcom.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc_els.c

index 19af0e6e7667f104b030e006e270eae95b437aee..ffbd3eddeabb4369cebc15ff15063fe43c2b91ea 100644 (file)
@@ -1527,6 +1527,7 @@ lpfc_plogi_confirm_nport(struct lpfc_hba *phba, uint32_t *prsp,
        uint8_t  name[sizeof(struct lpfc_name)];
        uint32_t rc, keepDID = 0, keep_nlp_flag = 0;
        uint16_t keep_nlp_state;
+       struct lpfc_nvme_rport *keep_nrport = NULL;
        int  put_node;
        int  put_rport;
        unsigned long *active_rrqs_xri_bitmap = NULL;
@@ -1624,6 +1625,10 @@ lpfc_plogi_confirm_nport(struct lpfc_hba *phba, uint32_t *prsp,
        keep_nlp_state = new_ndlp->nlp_state;
        lpfc_nlp_set_state(vport, new_ndlp, ndlp->nlp_state);
 
+       /* interchange the nvme remoteport structs */
+       keep_nrport = new_ndlp->nrport;
+       new_ndlp->nrport = ndlp->nrport;
+
        /* Move this back to NPR state */
        if (memcmp(&ndlp->nlp_portname, name, sizeof(struct lpfc_name)) == 0) {
                /* The new_ndlp is replacing ndlp totally, so we need
@@ -1646,6 +1651,13 @@ lpfc_plogi_confirm_nport(struct lpfc_hba *phba, uint32_t *prsp,
                        }
                        new_ndlp->nlp_type = ndlp->nlp_type;
                }
+
+               /* Fix up the nvme rport */
+               if (ndlp->nrport) {
+                       ndlp->nrport = NULL;
+                       lpfc_nlp_put(ndlp);
+               }
+
                /* We shall actually free the ndlp with both nlp_DID and
                 * nlp_portname fields equals 0 to avoid any ndlp on the
                 * nodelist never to be used.
@@ -1690,6 +1702,14 @@ lpfc_plogi_confirm_nport(struct lpfc_hba *phba, uint32_t *prsp,
                        keep_nlp_state = NLP_STE_NPR_NODE;
                lpfc_nlp_set_state(vport, ndlp, keep_nlp_state);
 
+               /* Previous ndlp no longer active with nvme host transport.
+                * Remove reference from earlier registration unless the
+                * nvme host took care of it.
+                */
+               if (ndlp->nrport)
+                       lpfc_nlp_put(ndlp);
+               ndlp->nrport = keep_nrport;
+
                /* Fix up the rport accordingly */
                rport = ndlp->rport;
                if (rport) {