scsi: lpfc: Vport creation is failing with "Link Down" error
authorJames Smart <jsmart2021@gmail.com>
Fri, 16 Jun 2017 05:56:43 +0000 (22:56 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 20 Jun 2017 01:39:37 +0000 (21:39 -0400)
Vport creation fails for SLI-3 adapters.

Mailbox submission fails because mailbox interrupt is disabled. Mailbox
interrupt is disabled during port reset.

Do reset only for physical port.

Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: James Smart <james.smart@broadcom.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc_init.c

index 9d3a12636455bafb9cf25feb5e12da4d6f682fe6..77283705eb8d507c2bcd9f83c34c8db734dfd7c4 100644 (file)
@@ -3691,14 +3691,6 @@ lpfc_get_wwpn(struct lpfc_hba *phba)
        LPFC_MBOXQ_t *mboxq;
        MAILBOX_t *mb;
 
-       if (phba->sli_rev < LPFC_SLI_REV4) {
-               /* Reset the port first */
-               lpfc_sli_brdrestart(phba);
-               rc = lpfc_sli_chipset_init(phba);
-               if (rc)
-                       return (uint64_t)-1;
-       }
-
        mboxq = (LPFC_MBOXQ_t *) mempool_alloc(phba->mbox_mem_pool,
                                                GFP_KERNEL);
        if (!mboxq)
@@ -3852,8 +3844,19 @@ lpfc_create_port(struct lpfc_hba *phba, int instance, struct device *dev)
        int i;
        uint64_t wwn;
        bool use_no_reset_hba = false;
+       int rc;
 
-       wwn = lpfc_get_wwpn(phba);
+       if (lpfc_no_hba_reset_cnt) {
+               if (phba->sli_rev < LPFC_SLI_REV4 &&
+                   dev == &phba->pcidev->dev) {
+                       /* Reset the port first */
+                       lpfc_sli_brdrestart(phba);
+                       rc = lpfc_sli_chipset_init(phba);
+                       if (rc)
+                               return NULL;
+               }
+               wwn = lpfc_get_wwpn(phba);
+       }
 
        for (i = 0; i < lpfc_no_hba_reset_cnt; i++) {
                if (wwn == lpfc_no_hba_reset[i]) {