From: James Smart <jsmart2021@gmail.com>

Date: Fri, 14 May 2021 12:55:50 -0700

Subject: scsi: lpfc: Fix non-optimized ERSP handling

Patch-mainline: Queued in subsystem maintainer repository

Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git

Git-commit: fa21189db9ab022080e056cc6da219975da48fd6

References: bsc#1186451

When processing an NVMe ERSP IU which didn't match the optimized CQE-only

path, the status was being left to the WQE status. WQE status is non-zero

as it is indicating a non-optimized completion that needs to be handled by

the driver.

Fix by clearing the status field when falling into the non-optimized

case. Log message added to track optimized vs non-optimized debug.

Link: https://lore.kernel.org/r/20210514195559.119853-3-jsmart2021@gmail.com

Co-developed-by: Justin Tee <justin.tee@broadcom.com>

Signed-off-by: Justin Tee <justin.tee@broadcom.com>

Signed-off-by: James Smart <jsmart2021@gmail.com>

Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

Acked-by: Daniel Wagner <dwagner@suse.de>

---

 drivers/scsi/lpfc/lpfc_nvme.c |   14 ++++++++++++--

 1 file changed, 12 insertions(+), 2 deletions(-)

--- a/drivers/scsi/lpfc/lpfc_nvme.c

+++ b/drivers/scsi/lpfc/lpfc_nvme.c

@@ -1049,9 +1049,19 @@ lpfc_nvme_io_cmd_wqe_cmpl(struct lpfc_hb

 			nCmd->transferred_length = wcqe->total_data_placed;

 			nCmd->rcv_rsplen = wcqe->parameter;

 			nCmd->status = 0;

-			/* Sanity check */

-			if (nCmd->rcv_rsplen == LPFC_NVME_ERSP_LEN)

+

+			/* Check if this is really an ERSP */

+			if (nCmd->rcv_rsplen == LPFC_NVME_ERSP_LEN) {

+				lpfc_ncmd->status = IOSTAT_SUCCESS;

+				lpfc_ncmd->result = 0;

+

+				lpfc_printf_vlog(vport, KERN_INFO, LOG_NVME,

+					 "6084 NVME Completion ERSP: "

+					 "xri %x placed x%x\n",

+					 lpfc_ncmd->cur_iocbq.sli4_xritag,

+					 wcqe->total_data_placed);

 				break;

+			}

 			lpfc_printf_vlog(vport, KERN_ERR, LOG_TRACE_EVENT,

 					 "6081 NVME Completion Protocol Error: "

 					 "xri %x status x%x result x%x "