On 06/16/2017 07:56 AM, James Smart wrote:
> 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.kenn...@broadcom.com>
> Signed-off-by: James Smart <james.sm...@broadcom.com>
> ---
> drivers/scsi/lpfc/lpfc_init.c | 21 ++++++++++++---------
> 1 file changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/scsi/lpfc/lpfc_init.c b/drivers/scsi/lpfc/lpfc_init.c
> index 9d3a12636455..77283705eb8d 100644
> --- a/drivers/scsi/lpfc/lpfc_init.c
> +++ b/drivers/scsi/lpfc/lpfc_init.c
> @@ -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]) {
>
Reviewed-by: Hannes Reinecke <h...@suse.com>
Cheers,
Hannes
--
Dr. Hannes Reinecke Teamlead Storage & Networking
h...@suse.de +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
