]> www.pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - drivers/scsi/lpfc/lpfc_els.c
[SCSI] lpfc 8.1.2: Added support for FAN
[linux-2.6-omap-h63xx.git] / drivers / scsi / lpfc / lpfc_els.c
index 056f9157d2c8572b236444c10609038b96dd26ac..a88a1477b5528d12846886cda2fa54821a49e7e8 100644 (file)
@@ -92,7 +92,7 @@ lpfc_els_chk_latt(struct lpfc_hba * phba)
                }
        }
 
-       return (1);
+       return 1;
 
 }
 
@@ -235,7 +235,7 @@ lpfc_prep_els_iocb(struct lpfc_hba * phba,
                                ndlp->nlp_DID, icmd->ulpIoTag, cmdSize);
        }
 
-       return (elsiocb);
+       return elsiocb;
 }
 
 
@@ -446,9 +446,10 @@ lpfc_cmpl_els_flogi(struct lpfc_hba * phba,
                lpfc_printf_log(phba,
                                KERN_INFO,
                                LOG_ELS,
-                               "%d:0100 FLOGI failure Data: x%x x%x\n",
+                               "%d:0100 FLOGI failure Data: x%x x%x x%x\n",
                                phba->brd_no,
-                               irsp->ulpStatus, irsp->un.ulpWord[4]);
+                               irsp->ulpStatus, irsp->un.ulpWord[4],
+                               irsp->ulpTimeout);
                goto flogifail;
        }
 
@@ -517,7 +518,7 @@ lpfc_issue_els_flogi(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        cmdsize = (sizeof (uint32_t) + sizeof (struct serv_parm));
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_FLOGI)) == 0) {
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -552,9 +553,9 @@ lpfc_issue_els_flogi(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 int
@@ -611,29 +612,21 @@ lpfc_initial_flogi(struct lpfc_hba * phba)
 {
        struct lpfc_nodelist *ndlp;
 
-       /* First look for Fabric ndlp on the unmapped list */
-
-       if ((ndlp =
-            lpfc_findnode_did(phba, NLP_SEARCH_UNMAPPED,
-                              Fabric_DID)) == 0) {
+       /* First look for the Fabric ndlp */
+       ndlp = lpfc_findnode_did(phba, NLP_SEARCH_ALL, Fabric_DID);
+       if (!ndlp) {
                /* Cannot find existing Fabric ndlp, so allocate a new one */
-               if ((ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL))
-                   == 0) {
-                       return (0);
-               }
+               ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL);
+               if (!ndlp)
+                       return 0;
                lpfc_nlp_init(phba, ndlp, Fabric_DID);
-       }
-       else {
-               phba->fc_unmap_cnt--;
-               list_del(&ndlp->nlp_listp);
-               spin_lock_irq(phba->host->host_lock);
-               ndlp->nlp_flag &= ~NLP_LIST_MASK;
-               spin_unlock_irq(phba->host->host_lock);
+       } else {
+               lpfc_nlp_list(phba, ndlp, NLP_JUST_DQ);
        }
        if (lpfc_issue_els_flogi(phba, ndlp, 0)) {
                mempool_free( ndlp, phba->nlp_mem_pool);
        }
-       return (1);
+       return 1;
 }
 
 static void
@@ -675,22 +668,23 @@ lpfc_cmpl_els_plogi(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
 
        irsp = &rspiocb->iocb;
        ndlp = (struct lpfc_nodelist *) cmdiocb->context1;
-       spin_lock_irq(phba->host->host_lock);
-       ndlp->nlp_flag &= ~NLP_PLOGI_SND;
-       spin_unlock_irq(phba->host->host_lock);
 
        /* Since ndlp can be freed in the disc state machine, note if this node
         * is being used during discovery.
         */
        disc = (ndlp->nlp_flag & NLP_NPR_2B_DISC);
+       spin_lock_irq(phba->host->host_lock);
+       ndlp->nlp_flag &= ~(NLP_PLOGI_SND | NLP_NPR_2B_DISC);
+       spin_unlock_irq(phba->host->host_lock);
        rc   = 0;
 
        /* PLOGI completes to NPort <nlp_DID> */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
                        "%d:0102 PLOGI completes to NPort x%x "
-                       "Data: x%x x%x x%x x%x\n",
+                       "Data: x%x x%x x%x x%x x%x\n",
                        phba->brd_no, ndlp->nlp_DID, irsp->ulpStatus,
-                       irsp->un.ulpWord[4], disc, phba->num_disc_nodes);
+                       irsp->un.ulpWord[4], irsp->ulpTimeout, disc,
+                       phba->num_disc_nodes);
 
        /* Check to see if link went down during discovery */
        if (lpfc_els_chk_latt(phba)) {
@@ -722,7 +716,7 @@ lpfc_cmpl_els_plogi(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                   ((irsp->un.ulpWord[4] == IOERR_SLI_ABORTED) ||
                   (irsp->un.ulpWord[4] == IOERR_LINK_DOWN) ||
                   (irsp->un.ulpWord[4] == IOERR_SLI_DOWN))) {
-                       disc = (ndlp->nlp_flag & NLP_NPR_2B_DISC);
+                       rc = NLP_STE_FREED_NODE;
                }
                else {
                        rc = lpfc_disc_state_machine(phba, ndlp, cmdiocb,
@@ -747,18 +741,11 @@ lpfc_cmpl_els_plogi(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                lpfc_more_plogi(phba);
        }
 
-       if (rc != NLP_STE_FREED_NODE) {
+       if (phba->num_disc_nodes == 0) {
                spin_lock_irq(phba->host->host_lock);
-               ndlp->nlp_flag &= ~NLP_NPR_2B_DISC;
+               phba->fc_flag &= ~FC_NDISC_ACTIVE;
                spin_unlock_irq(phba->host->host_lock);
-       }
 
-       if (phba->num_disc_nodes == 0) {
-               if(disc) {
-                       spin_lock_irq(phba->host->host_lock);
-                       phba->fc_flag &= ~FC_NDISC_ACTIVE;
-                       spin_unlock_irq(phba->host->host_lock);
-               }
                lpfc_can_disctmo(phba);
                if (phba->fc_flag & FC_RSCN_MODE) {
                        /* Check to see if more RSCNs came in while we were
@@ -796,10 +783,10 @@ lpfc_issue_els_plogi(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        pring = &psli->ring[LPFC_ELS_RING];     /* ELS ring */
 
        cmdsize = (sizeof (uint32_t) + sizeof (struct serv_parm));
-       if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
-                                         ndlp, ELS_CMD_PLOGI)) == 0) {
-               return (1);
-       }
+       elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry, ndlp,
+                                                       ELS_CMD_PLOGI);
+       if (!elsiocb)
+               return 1;
 
        icmd = &elsiocb->iocb;
        pcmd = (uint8_t *) (((struct lpfc_dmabuf *) elsiocb->context2)->virt);
@@ -824,10 +811,10 @@ lpfc_issue_els_plogi(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
                ndlp->nlp_flag &= ~NLP_PLOGI_SND;
                spin_unlock_irq(phba->host->host_lock);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
-       return (0);
+       return 0;
 }
 
 static void
@@ -851,9 +838,10 @@ lpfc_cmpl_els_prli(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
        /* PRLI completes to NPort <nlp_DID> */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
                        "%d:0103 PRLI completes to NPort x%x "
-                       "Data: x%x x%x x%x\n",
+                       "Data: x%x x%x x%x x%x\n",
                        phba->brd_no, ndlp->nlp_DID, irsp->ulpStatus,
-                       irsp->un.ulpWord[4], phba->num_disc_nodes);
+                       irsp->un.ulpWord[4], irsp->ulpTimeout,
+                       phba->num_disc_nodes);
 
        phba->fc_prli_sent--;
        /* Check to see if link went down during discovery */
@@ -906,7 +894,7 @@ lpfc_issue_els_prli(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        cmdsize = (sizeof (uint32_t) + sizeof (PRLI));
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_PRLI)) == 0) {
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -943,11 +931,11 @@ lpfc_issue_els_prli(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
                ndlp->nlp_flag &= ~NLP_PRLI_SND;
                spin_unlock_irq(phba->host->host_lock);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
        phba->fc_prli_sent++;
-       return (0);
+       return 0;
 }
 
 static void
@@ -1016,21 +1004,22 @@ lpfc_cmpl_els_adisc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
 
        irsp = &(rspiocb->iocb);
        ndlp = (struct lpfc_nodelist *) cmdiocb->context1;
-       spin_lock_irq(phba->host->host_lock);
-       ndlp->nlp_flag &= ~NLP_ADISC_SND;
-       spin_unlock_irq(phba->host->host_lock);
 
        /* Since ndlp can be freed in the disc state machine, note if this node
         * is being used during discovery.
         */
        disc = (ndlp->nlp_flag & NLP_NPR_2B_DISC);
+       spin_lock_irq(phba->host->host_lock);
+       ndlp->nlp_flag &= ~(NLP_ADISC_SND | NLP_NPR_2B_DISC);
+       spin_unlock_irq(phba->host->host_lock);
 
        /* ADISC completes to NPort <nlp_DID> */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
                        "%d:0104 ADISC completes to NPort x%x "
-                       "Data: x%x x%x x%x x%x\n",
+                       "Data: x%x x%x x%x x%x x%x\n",
                        phba->brd_no, ndlp->nlp_DID, irsp->ulpStatus,
-                       irsp->un.ulpWord[4], disc, phba->num_disc_nodes);
+                       irsp->un.ulpWord[4], irsp->ulpTimeout, disc,
+                       phba->num_disc_nodes);
 
        /* Check to see if link went down during discovery */
        if (lpfc_els_chk_latt(phba)) {
@@ -1054,13 +1043,10 @@ lpfc_cmpl_els_adisc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                }
                /* ADISC failed */
                /* Do not call DSM for lpfc_els_abort'ed ELS cmds */
-               if ((irsp->ulpStatus == IOSTAT_LOCAL_REJECT) &&
-                  ((irsp->un.ulpWord[4] == IOERR_SLI_ABORTED) ||
-                  (irsp->un.ulpWord[4] == IOERR_LINK_DOWN) ||
-                  (irsp->un.ulpWord[4] == IOERR_SLI_DOWN))) {
-                       disc = (ndlp->nlp_flag & NLP_NPR_2B_DISC);
-               }
-               else {
+               if ((irsp->ulpStatus != IOSTAT_LOCAL_REJECT) ||
+                  ((irsp->un.ulpWord[4] != IOERR_SLI_ABORTED) &&
+                  (irsp->un.ulpWord[4] != IOERR_LINK_DOWN) &&
+                  (irsp->un.ulpWord[4] != IOERR_SLI_DOWN))) {
                        lpfc_disc_state_machine(phba, ndlp, cmdiocb,
                                        NLP_EVT_CMPL_ADISC);
                }
@@ -1112,9 +1098,6 @@ lpfc_cmpl_els_adisc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        }
                }
        }
-       spin_lock_irq(phba->host->host_lock);
-       ndlp->nlp_flag &= ~NLP_NPR_2B_DISC;
-       spin_unlock_irq(phba->host->host_lock);
 out:
        lpfc_els_free_iocb(phba, cmdiocb);
        return;
@@ -1138,7 +1121,7 @@ lpfc_issue_els_adisc(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        cmdsize = (sizeof (uint32_t) + sizeof (ADISC));
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_ADISC)) == 0) {
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -1163,10 +1146,10 @@ lpfc_issue_els_adisc(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
                ndlp->nlp_flag &= ~NLP_ADISC_SND;
                spin_unlock_irq(phba->host->host_lock);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
-       return (0);
+       return 0;
 }
 
 static void
@@ -1190,9 +1173,10 @@ lpfc_cmpl_els_logo(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
        /* LOGO completes to NPort <nlp_DID> */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
                        "%d:0105 LOGO completes to NPort x%x "
-                       "Data: x%x x%x x%x\n",
+                       "Data: x%x x%x x%x x%x\n",
                        phba->brd_no, ndlp->nlp_DID, irsp->ulpStatus,
-                       irsp->un.ulpWord[4], phba->num_disc_nodes);
+                       irsp->un.ulpWord[4], irsp->ulpTimeout,
+                       phba->num_disc_nodes);
 
        /* Check to see if link went down during discovery */
        if (lpfc_els_chk_latt(phba))
@@ -1217,12 +1201,10 @@ lpfc_cmpl_els_logo(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                                        NLP_EVT_CMPL_LOGO);
                }
        } else {
-               /* Good status, call state machine */
+               /* Good status, call state machine.
+                * This will unregister the rpi if needed.
+                */
                lpfc_disc_state_machine(phba, ndlp, cmdiocb, NLP_EVT_CMPL_LOGO);
-
-               if (ndlp->nlp_flag & NLP_DELAY_TMO) {
-                       lpfc_unreg_rpi(phba, ndlp);
-               }
        }
 
 out:
@@ -1247,7 +1229,7 @@ lpfc_issue_els_logo(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
        cmdsize = 2 * (sizeof (uint32_t) + sizeof (struct lpfc_name));
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_LOGO)) == 0) {
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -1268,10 +1250,10 @@ lpfc_issue_els_logo(struct lpfc_hba * phba, struct lpfc_nodelist * ndlp,
                ndlp->nlp_flag &= ~NLP_LOGO_SND;
                spin_unlock_irq(phba->host->host_lock);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
-       return (0);
+       return 0;
 }
 
 static void
@@ -1286,9 +1268,10 @@ lpfc_cmpl_els_cmd(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
        lpfc_printf_log(phba,
                        KERN_INFO,
                        LOG_ELS,
-                       "%d:0106 ELS cmd tag x%x completes Data: x%x x%x\n",
+                       "%d:0106 ELS cmd tag x%x completes Data: x%x x%x x%x\n",
                        phba->brd_no,
-                       irsp->ulpIoTag, irsp->ulpStatus, irsp->un.ulpWord[4]);
+                       irsp->ulpIoTag, irsp->ulpStatus,
+                       irsp->un.ulpWord[4], irsp->ulpTimeout);
 
        /* Check to see if link went down during discovery */
        lpfc_els_chk_latt(phba);
@@ -1310,16 +1293,16 @@ lpfc_issue_els_scr(struct lpfc_hba * phba, uint32_t nportid, uint8_t retry)
        psli = &phba->sli;
        pring = &psli->ring[LPFC_ELS_RING];     /* ELS ring */
        cmdsize = (sizeof (uint32_t) + sizeof (SCR));
-       if ((ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL)) == 0) {
-               return (1);
-       }
+       ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL);
+       if (!ndlp)
+               return 1;
 
        lpfc_nlp_init(phba, ndlp, nportid);
 
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_SCR)) == 0) {
                mempool_free( ndlp, phba->nlp_mem_pool);
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -1339,11 +1322,11 @@ lpfc_issue_els_scr(struct lpfc_hba * phba, uint32_t nportid, uint8_t retry)
                spin_unlock_irq(phba->host->host_lock);
                mempool_free( ndlp, phba->nlp_mem_pool);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
        mempool_free( ndlp, phba->nlp_mem_pool);
-       return (0);
+       return 0;
 }
 
 static int
@@ -1363,15 +1346,15 @@ lpfc_issue_els_farpr(struct lpfc_hba * phba, uint32_t nportid, uint8_t retry)
        psli = &phba->sli;
        pring = &psli->ring[LPFC_ELS_RING];     /* ELS ring */
        cmdsize = (sizeof (uint32_t) + sizeof (FARP));
-       if ((ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL)) == 0) {
-               return (1);
-       }
+       ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL);
+       if (!ndlp)
+               return 1;
        lpfc_nlp_init(phba, ndlp, nportid);
 
        if ((elsiocb = lpfc_prep_els_iocb(phba, 1, cmdsize, retry,
                                          ndlp, ELS_CMD_RNID)) == 0) {
                mempool_free( ndlp, phba->nlp_mem_pool);
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -1405,11 +1388,11 @@ lpfc_issue_els_farpr(struct lpfc_hba * phba, uint32_t nportid, uint8_t retry)
                spin_unlock_irq(phba->host->host_lock);
                mempool_free( ndlp, phba->nlp_mem_pool);
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
        spin_unlock_irq(phba->host->host_lock);
        mempool_free( ndlp, phba->nlp_mem_pool);
-       return (0);
+       return 0;
 }
 
 void
@@ -1450,8 +1433,9 @@ lpfc_els_retry_delay_handler(struct lpfc_nodelist *ndlp)
 
        phba = ndlp->nlp_phba;
        spin_lock_irq(phba->host->host_lock);
-       did = (uint32_t) (ndlp->nlp_DID);
-       cmd = (uint32_t) (ndlp->nlp_last_elscmd);
+       did = ndlp->nlp_DID;
+       cmd = ndlp->nlp_last_elscmd;
+       ndlp->nlp_last_elscmd = 0;
 
        if (!(ndlp->nlp_flag & NLP_DELAY_TMO)) {
                spin_unlock_irq(phba->host->host_lock);
@@ -1468,24 +1452,28 @@ lpfc_els_retry_delay_handler(struct lpfc_nodelist *ndlp)
                break;
        case ELS_CMD_PLOGI:
                if (!lpfc_issue_els_plogi(phba, ndlp, retry)) {
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_PLOGI_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_PLOGI_LIST);
                }
                break;
        case ELS_CMD_ADISC:
                if (!lpfc_issue_els_adisc(phba, ndlp, retry)) {
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_ADISC_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_ADISC_LIST);
                }
                break;
        case ELS_CMD_PRLI:
                if (!lpfc_issue_els_prli(phba, ndlp, retry)) {
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_PRLI_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_PRLI_LIST);
                }
                break;
        case ELS_CMD_LOGO:
                if (!lpfc_issue_els_logo(phba, ndlp, retry)) {
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_NPR_NODE;
                        lpfc_nlp_list(phba, ndlp, NLP_NPR_LIST);
                }
@@ -1541,11 +1529,6 @@ lpfc_els_retry(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
 
                case IOERR_SEQUENCE_TIMEOUT:
                        retry = 1;
-                       if ((cmd == ELS_CMD_FLOGI)
-                           && (phba->fc_topology != TOPOLOGY_LOOP)) {
-                               delay = 1;
-                               maxretry = 48;
-                       }
                        break;
 
                case IOERR_NO_RESOURCES:
@@ -1650,36 +1633,41 @@ lpfc_els_retry(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        mod_timer(&ndlp->nlp_delayfunc, jiffies + HZ);
                        ndlp->nlp_flag |= NLP_DELAY_TMO;
 
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_NPR_NODE;
                        lpfc_nlp_list(phba, ndlp, NLP_NPR_LIST);
                        ndlp->nlp_last_elscmd = cmd;
 
-                       return (1);
+                       return 1;
                }
                switch (cmd) {
                case ELS_CMD_FLOGI:
                        lpfc_issue_els_flogi(phba, ndlp, cmdiocb->retry);
-                       return (1);
+                       return 1;
                case ELS_CMD_PLOGI:
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_PLOGI_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_PLOGI_LIST);
                        lpfc_issue_els_plogi(phba, ndlp, cmdiocb->retry);
-                       return (1);
+                       return 1;
                case ELS_CMD_ADISC:
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_ADISC_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_ADISC_LIST);
                        lpfc_issue_els_adisc(phba, ndlp, cmdiocb->retry);
-                       return (1);
+                       return 1;
                case ELS_CMD_PRLI:
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_PRLI_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_PRLI_LIST);
                        lpfc_issue_els_prli(phba, ndlp, cmdiocb->retry);
-                       return (1);
+                       return 1;
                case ELS_CMD_LOGO:
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_NPR_NODE;
                        lpfc_nlp_list(phba, ndlp, NLP_NPR_LIST);
                        lpfc_issue_els_logo(phba, ndlp, cmdiocb->retry);
-                       return (1);
+                       return 1;
                }
        }
 
@@ -1690,7 +1678,7 @@ lpfc_els_retry(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        phba->brd_no,
                        cmd, ndlp->nlp_DID, cmdiocb->retry, ndlp->nlp_flag);
 
-       return (0);
+       return 0;
 }
 
 int
@@ -1739,10 +1727,6 @@ lpfc_cmpl_els_logo_acc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        phba->brd_no, ndlp->nlp_DID, ndlp->nlp_flag,
                        ndlp->nlp_state, ndlp->nlp_rpi);
 
-       spin_lock_irq(phba->host->host_lock);
-       ndlp->nlp_flag &= ~NLP_LOGO_ACC;
-       spin_unlock_irq(phba->host->host_lock);
-
        switch (ndlp->nlp_state) {
        case NLP_STE_UNUSED_NODE:       /* node is just allocated */
                lpfc_nlp_list(phba, ndlp, NLP_NO_LIST);
@@ -1780,11 +1764,12 @@ lpfc_cmpl_els_acc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
        /* ELS response tag <ulpIoTag> completes */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
                        "%d:0110 ELS response tag x%x completes "
-                       "Data: x%x x%x x%x x%x x%x x%x\n",
+                       "Data: x%x x%x x%x x%x x%x x%x x%x\n",
                        phba->brd_no,
                        cmdiocb->iocb.ulpIoTag, rspiocb->iocb.ulpStatus,
-                       rspiocb->iocb.un.ulpWord[4], ndlp->nlp_DID,
-                       ndlp->nlp_flag, ndlp->nlp_state, ndlp->nlp_rpi);
+                       rspiocb->iocb.un.ulpWord[4], rspiocb->iocb.ulpTimeout,
+                       ndlp->nlp_DID, ndlp->nlp_flag, ndlp->nlp_state,
+                       ndlp->nlp_rpi);
 
        if (mbox) {
                if ((rspiocb->iocb.ulpStatus == 0)
@@ -1795,6 +1780,7 @@ lpfc_cmpl_els_acc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        lpfc_unreg_rpi(phba, ndlp);
                        mbox->mbox_cmpl = lpfc_mbx_cmpl_reg_login;
                        mbox->context2 = ndlp;
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_REG_LOGIN_ISSUE;
                        lpfc_nlp_list(phba, ndlp, NLP_REGLOGIN_LIST);
                        if (lpfc_sli_issue_mbox(phba, mbox,
@@ -1809,6 +1795,7 @@ lpfc_cmpl_els_acc(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
                        mempool_free( mbox, phba->mbox_mem_pool);
                        if (ndlp->nlp_flag & NLP_ACC_REGLOGIN) {
                                lpfc_nlp_list(phba, ndlp, NLP_NO_LIST);
+                               ndlp = NULL;
                        }
                }
        }
@@ -1846,7 +1833,8 @@ lpfc_els_rsp_acc(struct lpfc_hba * phba, uint32_t flag,
                if ((elsiocb =
                     lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
                                        ndlp, ELS_CMD_ACC)) == 0) {
-                       return (1);
+                       ndlp->nlp_flag &= ~NLP_LOGO_ACC;
+                       return 1;
                }
                icmd = &elsiocb->iocb;
                icmd->ulpContext = oldcmd->ulpContext;  /* Xri */
@@ -1859,7 +1847,7 @@ lpfc_els_rsp_acc(struct lpfc_hba * phba, uint32_t flag,
                if ((elsiocb =
                     lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
                                        ndlp, ELS_CMD_ACC)) == 0) {
-                       return (1);
+                       return 1;
                }
                icmd = &elsiocb->iocb;
                icmd->ulpContext = oldcmd->ulpContext;  /* Xri */
@@ -1873,7 +1861,7 @@ lpfc_els_rsp_acc(struct lpfc_hba * phba, uint32_t flag,
                memcpy(pcmd, &phba->fc_sparam, sizeof (struct serv_parm));
                break;
        default:
-               return (1);
+               return 1;
        }
 
        if (newnode)
@@ -1889,6 +1877,9 @@ lpfc_els_rsp_acc(struct lpfc_hba * phba, uint32_t flag,
                        ndlp->nlp_flag, ndlp->nlp_state, ndlp->nlp_rpi);
 
        if (ndlp->nlp_flag & NLP_LOGO_ACC) {
+               spin_lock_irq(phba->host->host_lock);
+               ndlp->nlp_flag &= ~NLP_LOGO_ACC;
+               spin_unlock_irq(phba->host->host_lock);
                elsiocb->iocb_cmpl = lpfc_cmpl_els_logo_acc;
        } else {
                elsiocb->iocb_cmpl = lpfc_cmpl_els_acc;
@@ -1900,9 +1891,9 @@ lpfc_els_rsp_acc(struct lpfc_hba * phba, uint32_t flag,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 int
@@ -1924,7 +1915,7 @@ lpfc_els_rsp_reject(struct lpfc_hba * phba, uint32_t rejectError,
        cmdsize = 2 * sizeof (uint32_t);
        if ((elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
                                          ndlp, ELS_CMD_LS_RJT)) == 0) {
-               return (1);
+               return 1;
        }
 
        icmd = &elsiocb->iocb;
@@ -1952,9 +1943,9 @@ lpfc_els_rsp_reject(struct lpfc_hba * phba, uint32_t rejectError,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 int
@@ -1977,7 +1968,7 @@ lpfc_els_rsp_adisc_acc(struct lpfc_hba * phba,
        cmdsize = sizeof (uint32_t) + sizeof (ADISC);
        if ((elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
                                          ndlp, ELS_CMD_ACC)) == 0) {
-               return (1);
+               return 1;
        }
 
        /* Xmit ADISC ACC response tag <ulpIoTag> */
@@ -2010,9 +2001,9 @@ lpfc_els_rsp_adisc_acc(struct lpfc_hba * phba,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 int
@@ -2034,13 +2025,10 @@ lpfc_els_rsp_prli_acc(struct lpfc_hba * phba,
        pring = &psli->ring[LPFC_ELS_RING];     /* ELS ring */
 
        cmdsize = sizeof (uint32_t) + sizeof (PRLI);
-       if ((elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
-                                         ndlp,
-                                         (ELS_CMD_ACC |
-                                          (ELS_CMD_PRLI & ~ELS_RSP_MASK)))) ==
-           0) {
-               return (1);
-       }
+       elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry, ndlp,
+                               (ELS_CMD_ACC | (ELS_CMD_PRLI & ~ELS_RSP_MASK)));
+       if (!elsiocb)
+               return 1;
 
        /* Xmit PRLI ACC response tag <ulpIoTag> */
        lpfc_printf_log(phba, KERN_INFO, LOG_ELS,
@@ -2090,9 +2078,9 @@ lpfc_els_rsp_prli_acc(struct lpfc_hba * phba,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 static int
@@ -2120,7 +2108,7 @@ lpfc_els_rsp_rnid_acc(struct lpfc_hba * phba,
 
        if ((elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, oldiocb->retry,
                                          ndlp, ELS_CMD_ACC)) == 0) {
-               return (1);
+               return 1;
        }
 
        /* Xmit RNID ACC response tag <ulpIoTag> */
@@ -2173,9 +2161,9 @@ lpfc_els_rsp_rnid_acc(struct lpfc_hba * phba,
        spin_unlock_irq(phba->host->host_lock);
        if (rc == IOCB_ERROR) {
                lpfc_els_free_iocb(phba, elsiocb);
-               return (1);
+               return 1;
        }
-       return (0);
+       return 0;
 }
 
 int
@@ -2191,6 +2179,7 @@ lpfc_els_disc_adisc(struct lpfc_hba * phba)
                if (ndlp->nlp_flag & NLP_NPR_2B_DISC) {
                        if (ndlp->nlp_flag & NLP_NPR_ADISC) {
                                ndlp->nlp_flag &= ~NLP_NPR_ADISC;
+                               ndlp->nlp_prev_state = ndlp->nlp_state;
                                ndlp->nlp_state = NLP_STE_ADISC_ISSUE;
                                lpfc_nlp_list(phba, ndlp,
                                        NLP_ADISC_LIST);
@@ -2228,6 +2217,7 @@ lpfc_els_disc_plogi(struct lpfc_hba * phba)
                if ((ndlp->nlp_flag & NLP_NPR_2B_DISC) &&
                   (!(ndlp->nlp_flag & NLP_DELAY_TMO))) {
                        if (!(ndlp->nlp_flag & NLP_NPR_ADISC)) {
+                               ndlp->nlp_prev_state = ndlp->nlp_state;
                                ndlp->nlp_state = NLP_STE_PLOGI_ISSUE;
                                lpfc_nlp_list(phba, ndlp, NLP_PLOGI_LIST);
                                lpfc_issue_els_plogi(phba, ndlp, 0);
@@ -2268,7 +2258,7 @@ lpfc_els_flush_rscn(struct lpfc_hba * phba)
        phba->fc_flag &= ~(FC_RSCN_MODE | FC_RSCN_DISCOVERY);
        spin_unlock_irq(phba->host->host_lock);
        lpfc_can_disctmo(phba);
-       return (0);
+       return 0;
 }
 
 int
@@ -2289,7 +2279,7 @@ lpfc_rscn_payload_check(struct lpfc_hba * phba, uint32_t did)
 
        /* If we are doing a FULL RSCN rediscovery, match everything */
        if (phba->fc_flag & FC_RSCN_DISCOVERY) {
-               return (did);
+               return did;
        }
 
        for (i = 0; i < phba->fc_rscn_id_cnt; i++) {
@@ -2337,7 +2327,7 @@ lpfc_rscn_payload_check(struct lpfc_hba * phba, uint32_t did)
                        }
                }
        }
-       return (match);
+       return match;
 }
 
 static int
@@ -2369,8 +2359,13 @@ lpfc_rscn_recovery_check(struct lpfc_hba * phba)
 
                        lpfc_disc_state_machine(phba, ndlp, NULL,
                                        NLP_EVT_DEVICE_RECOVERY);
+
+                       /* Make sure NLP_DELAY_TMO is NOT running
+                        * after a device recovery event.
+                        */
                        if (ndlp->nlp_flag & NLP_DELAY_TMO) {
                                ndlp->nlp_flag &= ~NLP_DELAY_TMO;
+                               ndlp->nlp_last_elscmd = 0;
                                del_timer_sync(&ndlp->nlp_delayfunc);
                                if (!list_empty(&ndlp->
                                                els_retry_evt.evt_listp))
@@ -2379,7 +2374,7 @@ lpfc_rscn_recovery_check(struct lpfc_hba * phba)
                        }
                }
        }
-       return (0);
+       return 0;
 }
 
 static int
@@ -2415,7 +2410,7 @@ lpfc_els_rcv_rscn(struct lpfc_hba * phba,
        if (phba->hba_state < LPFC_NS_QRY) {
                lpfc_els_rsp_acc(phba, ELS_CMD_ACC, cmdiocb, ndlp, NULL,
                                                                newnode);
-               return (0);
+               return 0;
        }
 
        /* If we are already processing an RSCN, save the received
@@ -2457,7 +2452,7 @@ lpfc_els_rcv_rscn(struct lpfc_hba * phba,
 
                /* send RECOVERY event for ALL nodes that match RSCN payload */
                lpfc_rscn_recovery_check(phba);
-               return (0);
+               return 0;
        }
 
        phba->fc_flag |= FC_RSCN_MODE;
@@ -2476,7 +2471,7 @@ lpfc_els_rcv_rscn(struct lpfc_hba * phba,
        /* send RECOVERY event for ALL nodes that match RSCN payload */
        lpfc_rscn_recovery_check(phba);
 
-       return (lpfc_els_handle_rscn(phba));
+       return lpfc_els_handle_rscn(phba);
 }
 
 int
@@ -2498,40 +2493,41 @@ lpfc_els_handle_rscn(struct lpfc_hba * phba)
 
        /* To process RSCN, first compare RSCN data with NameServer */
        phba->fc_ns_retry = 0;
-       if ((ndlp = lpfc_findnode_did(phba, NLP_SEARCH_UNMAPPED,
-                                     NameServer_DID))) {
+       ndlp = lpfc_findnode_did(phba, NLP_SEARCH_UNMAPPED, NameServer_DID);
+       if (ndlp) {
                /* Good ndlp, issue CT Request to NameServer */
                if (lpfc_ns_cmd(phba, ndlp, SLI_CTNS_GID_FT) == 0) {
                        /* Wait for NameServer query cmpl before we can
                           continue */
-                       return (1);
+                       return 1;
                }
        } else {
                /* If login to NameServer does not exist, issue one */
                /* Good status, issue PLOGI to NameServer */
-               if ((ndlp =
-                    lpfc_findnode_did(phba, NLP_SEARCH_ALL, NameServer_DID))) {
+               ndlp = lpfc_findnode_did(phba, NLP_SEARCH_ALL, NameServer_DID);
+               if (ndlp) {
                        /* Wait for NameServer login cmpl before we can
                           continue */
-                       return (1);
+                       return 1;
                }
-               if ((ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL))
-                   == 0) {
+               ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL);
+               if (!ndlp) {
                        lpfc_els_flush_rscn(phba);
-                       return (0);
+                       return 0;
                } else {
                        lpfc_nlp_init(phba, ndlp, NameServer_DID);
                        ndlp->nlp_type |= NLP_FABRIC;
+                       ndlp->nlp_prev_state = ndlp->nlp_state;
                        ndlp->nlp_state = NLP_STE_PLOGI_ISSUE;
                        lpfc_issue_els_plogi(phba, ndlp, 0);
                        /* Wait for NameServer login cmpl before we can
                           continue */
-                       return (1);
+                       return 1;
                }
        }
 
        lpfc_els_flush_rscn(phba);
-       return (0);
+       return 0;
 }
 
 static int
@@ -2565,7 +2561,7 @@ lpfc_els_rcv_flogi(struct lpfc_hba * phba,
                                "%d:0113 An FLOGI ELS command x%x was received "
                                "from DID x%x in Loop Mode\n",
                                phba->brd_no, cmd, did);
-               return (1);
+               return 1;
        }
 
        did = Fabric_DID;
@@ -2581,7 +2577,7 @@ lpfc_els_rcv_flogi(struct lpfc_hba * phba,
                if (!rc) {
                        if ((mbox = mempool_alloc(phba->mbox_mem_pool,
                                                  GFP_KERNEL)) == 0) {
-                               return (1);
+                               return 1;
                        }
                        lpfc_linkdown(phba);
                        lpfc_init_link(phba, mbox,
@@ -2594,7 +2590,7 @@ lpfc_els_rcv_flogi(struct lpfc_hba * phba,
                        if (rc == MBX_NOT_FINISHED) {
                                mempool_free( mbox, phba->mbox_mem_pool);
                        }
-                       return (1);
+                       return 1;
                }
                else if (rc > 0) {      /* greater than */
                        spin_lock_irq(phba->host->host_lock);
@@ -2610,13 +2606,13 @@ lpfc_els_rcv_flogi(struct lpfc_hba * phba,
                stat.un.b.lsRjtRsnCodeExp = LSEXP_SPARM_OPTIONS;
                stat.un.b.vendorUnique = 0;
                lpfc_els_rsp_reject(phba, stat.un.lsRjtError, cmdiocb, ndlp);
-               return (1);
+               return 1;
        }
 
        /* Send back ACC */
        lpfc_els_rsp_acc(phba, ELS_CMD_PLOGI, cmdiocb, ndlp, NULL, newnode);
 
-       return (0);
+       return 0;
 }
 
 static int
@@ -2654,7 +2650,7 @@ lpfc_els_rcv_rnid(struct lpfc_hba * phba,
                stat.un.b.vendorUnique = 0;
                lpfc_els_rsp_reject(phba, stat.un.lsRjtError, cmdiocb, ndlp);
        }
-       return (0);
+       return 0;
 }
 
 static int
@@ -2702,10 +2698,10 @@ lpfc_els_rsp_rps_acc(struct lpfc_hba * phba, LPFC_MBOXQ_t * pmb)
 
        cmdsize = sizeof(RPS_RSP) + sizeof(uint32_t);
        mempool_free( pmb, phba->mbox_mem_pool);
-       if ((elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, 3,
-                                         ndlp, ELS_CMD_ACC)) == 0) {
+       elsiocb = lpfc_prep_els_iocb(phba, 0, cmdsize, lpfc_max_els_tries,
+                                                       ndlp, ELS_CMD_ACC);
+       if (!elsiocb)
                return;
-       }
 
        icmd = &elsiocb->iocb;
        icmd->ulpContext = xri;
@@ -2926,7 +2922,7 @@ lpfc_els_rcv_farp(struct lpfc_hba * phba,
 
        /* We will only support match on WWPN or WWNN */
        if (fp->Mflags & ~(FARP_MATCH_NODE | FARP_MATCH_PORT)) {
-               return (0);
+               return 0;
        }
 
        cnt = 0;
@@ -2949,6 +2945,7 @@ lpfc_els_rcv_farp(struct lpfc_hba * phba,
                   (ndlp->nlp_state == NLP_STE_MAPPED_NODE)) {
                        /* Log back into the node before sending the FARP. */
                        if (fp->Rflags & FARP_REQUEST_PLOGI) {
+                               ndlp->nlp_prev_state = ndlp->nlp_state;
                                ndlp->nlp_state = NLP_STE_PLOGI_ISSUE;
                                lpfc_nlp_list(phba, ndlp, NLP_PLOGI_LIST);
                                lpfc_issue_els_plogi(phba, ndlp, 0);
@@ -2960,7 +2957,7 @@ lpfc_els_rcv_farp(struct lpfc_hba * phba,
                        }
                }
        }
-       return (0);
+       return 0;
 }
 
 static int
@@ -2993,47 +2990,90 @@ lpfc_els_rcv_farpr(struct lpfc_hba * phba,
 
 static int
 lpfc_els_rcv_fan(struct lpfc_hba * phba, struct lpfc_iocbq * cmdiocb,
-                struct lpfc_nodelist * ndlp)
+                struct lpfc_nodelist * fan_ndlp)
 {
        struct lpfc_dmabuf *pcmd;
        uint32_t *lp;
        IOCB_t *icmd;
-       FAN *fp;
        uint32_t cmd, did;
+       FAN *fp;
+       struct lpfc_nodelist *ndlp, *next_ndlp;
+
+       /* FAN received */
+       lpfc_printf_log(phba, KERN_INFO, LOG_ELS, "%d:265 FAN received\n",
+                                                               phba->brd_no);
 
        icmd = &cmdiocb->iocb;
        did = icmd->un.elsreq64.remoteID;
-       pcmd = (struct lpfc_dmabuf *) cmdiocb->context2;
-       lp = (uint32_t *) pcmd->virt;
+       pcmd = (struct lpfc_dmabuf *)cmdiocb->context2;
+       lp = (uint32_t *)pcmd->virt;
 
        cmd = *lp++;
-       fp = (FAN *) lp;
-
-       /* FAN received */
+       fp = (FAN *)lp;
 
-       /* ACCEPT the FAN request */
-       lpfc_els_rsp_acc(phba, ELS_CMD_ACC, cmdiocb, ndlp, NULL, 0);
+       /* FAN received; Fan does not have a reply sequence */
 
        if (phba->hba_state == LPFC_LOCAL_CFG_LINK) {
-               /* The discovery state machine needs to take a different
-                * action if this node has switched fabrics
-                */
-               if ((memcmp(&fp->FportName, &phba->fc_fabparam.portName,
-                           sizeof (struct lpfc_name)) != 0)
-                   ||
-                   (memcmp(&fp->FnodeName, &phba->fc_fabparam.nodeName,
-                           sizeof (struct lpfc_name)) != 0)) {
-                       /* This node has switched fabrics.  An FLOGI is required
-                        * after the timeout
+               if ((memcmp(&phba->fc_fabparam.nodeName, &fp->FnodeName,
+                       sizeof(struct lpfc_name)) != 0) ||
+                   (memcmp(&phba->fc_fabparam.portName, &fp->FportName,
+                       sizeof(struct lpfc_name)) != 0)) {
+                       /*
+                        * This node has switched fabrics.  FLOGI is required
+                        * Clean up the old rpi's
                         */
-                       return (0);
+
+                       list_for_each_entry_safe(ndlp, next_ndlp,
+                               &phba->fc_npr_list, nlp_listp) {
+
+                               if (ndlp->nlp_type & NLP_FABRIC) {
+                                       /*
+                                        * Clean up old Fabric, Nameserver and
+                                        * other NLP_FABRIC logins
+                                        */
+                                       lpfc_nlp_list(phba, ndlp, NLP_NO_LIST);
+                               }
+                               else if (!(ndlp->nlp_flag & NLP_NPR_ADISC)) {
+                                       /* Fail outstanding I/O now since this
+                                        * device is marked for PLOGI
+                                        */
+                                       lpfc_unreg_rpi(phba, ndlp);
+                               }
+                       }
+
+                       phba->hba_state = LPFC_FLOGI;
+                       lpfc_set_disctmo(phba);
+                       lpfc_initial_flogi(phba);
+                       return 0;
                }
+               /* Discovery not needed,
+                * move the nodes to their original state.
+                */
+               list_for_each_entry_safe(ndlp, next_ndlp, &phba->fc_npr_list,
+                       nlp_listp) {
 
-               /* Start discovery */
+                       switch (ndlp->nlp_prev_state) {
+                       case NLP_STE_UNMAPPED_NODE:
+                               ndlp->nlp_prev_state = NLP_STE_NPR_NODE;
+                               ndlp->nlp_state = NLP_STE_UNMAPPED_NODE;
+                               lpfc_nlp_list(phba, ndlp, NLP_UNMAPPED_LIST);
+                               break;
+
+                       case NLP_STE_MAPPED_NODE:
+                               ndlp->nlp_prev_state = NLP_STE_NPR_NODE;
+                               ndlp->nlp_state = NLP_STE_MAPPED_NODE;
+                               lpfc_nlp_list(phba, ndlp, NLP_MAPPED_LIST);
+                               break;
+
+                       default:
+                               break;
+                       }
+               }
+
+               /* Start discovery - this should just do CLEAR_LA */
                lpfc_disc_start(phba);
        }
-
-       return (0);
+       return 0;
 }
 
 void
@@ -3156,7 +3196,6 @@ lpfc_els_flush_cmd(struct lpfc_hba * phba)
        struct lpfc_dmabuf *pcmd;
        uint32_t *elscmd;
        uint32_t els_command;
-       uint32_t remote_ID;
 
        pring = &phba->sli.ring[LPFC_ELS_RING];
        spin_lock_irq(phba->host->host_lock);
@@ -3179,18 +3218,6 @@ lpfc_els_flush_cmd(struct lpfc_hba * phba)
                elscmd = (uint32_t *) (pcmd->virt);
                els_command = *elscmd;
 
-               if (cmd->ulpCommand == CMD_GEN_REQUEST64_CR) {
-                       struct lpfc_nodelist *ndlp;
-
-                       ndlp = lpfc_findnode_rpi(phba, cmd->ulpContext);
-                       remote_ID = ndlp->nlp_DID;
-                       if (phba->hba_state == LPFC_HBA_READY) {
-                               continue;
-                       }
-               } else {
-                       remote_ID = cmd->un.elsreq64.remoteID;
-               }
-
                list_del(&piocb->list);
                pring->txcmplq_cnt--;
 
@@ -3216,18 +3243,6 @@ lpfc_els_flush_cmd(struct lpfc_hba * phba)
                elscmd = (uint32_t *) (pcmd->virt);
                els_command = *elscmd;
 
-               if (cmd->ulpCommand == CMD_GEN_REQUEST64_CR) {
-                       struct lpfc_nodelist *ndlp;
-
-                       ndlp = lpfc_findnode_rpi(phba, cmd->ulpContext);
-                       remote_ID = ndlp->nlp_DID;
-                       if (phba->hba_state == LPFC_HBA_READY) {
-                               continue;
-                       }
-               } else {
-                       remote_ID = cmd->un.elsreq64.remoteID;
-               }
-
                list_del(&piocb->list);
                pring->txcmplq_cnt--;
 
@@ -3311,10 +3326,11 @@ lpfc_els_unsol_event(struct lpfc_hba * phba,
        }
 
        did = icmd->un.rcvels.remoteID;
-       if ((ndlp = lpfc_findnode_did(phba, NLP_SEARCH_ALL, did)) == 0) {
+       ndlp = lpfc_findnode_did(phba, NLP_SEARCH_ALL, did);
+       if (!ndlp) {
                /* Cannot find existing Fabric ndlp, so allocate a new one */
-               if ((ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL))
-                   == 0) {
+               ndlp = mempool_alloc(phba->nlp_mem_pool, GFP_KERNEL);
+               if (!ndlp) {
                        lpfc_mbuf_free(phba, mp->virt, mp->phys);
                        kfree(mp);
                        drop_cmd = 1;
@@ -3475,8 +3491,9 @@ dropit:
        if (drop_cmd == 1) {
                lpfc_printf_log(phba, KERN_ERR, LOG_ELS,
                                "%d:0111 Dropping received ELS cmd "
-                               "Data: x%x x%x\n", phba->brd_no,
-                               icmd->ulpStatus, icmd->un.ulpWord[4]);
+                               "Data: x%x x%x x%x\n", phba->brd_no,
+                               icmd->ulpStatus, icmd->un.ulpWord[4],
+                               icmd->ulpTimeout);
                phba->fc_stat.elsRcvDrop++;
        }
        return;