Commit c3725bdc authored by James Smart's avatar James Smart Committed by Martin K. Petersen
Browse files

scsi: lpfc: Fix driver handling of nvme resources during unload 



During driver unload, the driver may crash due to NULL pointers.  The
NULL pointers were due to the driver not protecting itself sufficiently
during some of the teardown paths.  Additionally, the driver was not
waiting for and cleanup up nvme io resources. As such, the driver wasn't
making the callbacks to the transport, stalling the transports
association teardown.

This patch waits for io clean up before tearding down and adds checks
for possible NULL pointers.

Cc: <stable@vger.kernel.org> # 4.12+
Signed-off-by: default avatarDick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: default avatarJames Smart <james.smart@broadcom.com>
Reviewed-by: default avatarHannes Reinecke <hare@suse.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 3386f4bd

drivers/scsi/lpfc/lpfc_crtn.h

+2 −0
Original line number Diff line number Diff line
@@ -254,6 +254,8 @@ void lpfc_nvmet_ctxbuf_post(struct lpfc_hba *phba, 
			    struct lpfc_nvmet_ctxbuf *ctxp);

int lpfc_nvmet_rcv_unsol_abort(struct lpfc_vport *vport,

			       struct fc_frame_header *fc_hdr);

void lpfc_sli_flush_nvme_rings(struct lpfc_hba *phba);

void lpfc_nvme_wait_for_io_drain(struct lpfc_hba *phba);

void lpfc_sli4_build_dflt_fcf_record(struct lpfc_hba *, struct fcf_record *,

			uint16_t);

int lpfc_sli4_rq_put(struct lpfc_queue *hq, struct lpfc_queue *dq,

drivers/scsi/lpfc/lpfc_init.c

+18 −0
Original line number Diff line number Diff line
@@ -10130,6 +10130,16 @@ lpfc_sli4_xri_exchange_busy_wait(struct lpfc_hba *phba) 
	int fcp_xri_cmpl = 1;

	int els_xri_cmpl = list_empty(&phba->sli4_hba.lpfc_abts_els_sgl_list);



	/* Driver just aborted IOs during the hba_unset process.  Pause

	 * here to give the HBA time to complete the IO and get entries

	 * into the abts lists.

	 */

	msleep(LPFC_XRI_EXCH_BUSY_WAIT_T1 * 5);



	/* Wait for NVME pending IO to flush back to transport. */

	if (phba->cfg_enable_fc4_type & LPFC_ENABLE_NVME)

		lpfc_nvme_wait_for_io_drain(phba);



	if (phba->cfg_enable_fc4_type & LPFC_ENABLE_FCP)

		fcp_xri_cmpl =

			list_empty(&phba->sli4_hba.lpfc_abts_scsi_buf_list);
@@ -11653,6 +11663,10 @@ lpfc_sli4_prep_dev_for_reset(struct lpfc_hba *phba) 
	/* Flush all driver's outstanding SCSI I/Os as we are to reset */

	lpfc_sli_flush_fcp_rings(phba);



	/* Flush the outstanding NVME IOs if fc4 type enabled. */

	if (phba->cfg_enable_fc4_type & LPFC_ENABLE_NVME)

		lpfc_sli_flush_nvme_rings(phba);



	/* stop all timers */

	lpfc_stop_hba_timers(phba);
@@ -11684,6 +11698,10 @@ lpfc_sli4_prep_dev_for_perm_failure(struct lpfc_hba *phba) 


	/* Clean up all driver's outstanding SCSI I/Os */

	lpfc_sli_flush_fcp_rings(phba);



	/* Flush the outstanding NVME IOs if fc4 type enabled. */

	if (phba->cfg_enable_fc4_type & LPFC_ENABLE_NVME)

		lpfc_sli_flush_nvme_rings(phba);

}



/**

drivers/scsi/lpfc/lpfc_nvme.c

+85 −11
Original line number Diff line number Diff line
@@ -88,6 +88,9 @@ lpfc_nvme_create_queue(struct nvme_fc_local_port *pnvme_lport, 
	struct lpfc_nvme_qhandle *qhandle;

	char *str;



	if (!pnvme_lport->private)

		return -ENOMEM;



	lport = (struct lpfc_nvme_lport *)pnvme_lport->private;

	vport = lport->vport;

	qhandle = kzalloc(sizeof(struct lpfc_nvme_qhandle), GFP_KERNEL);
@@ -140,6 +143,9 @@ lpfc_nvme_delete_queue(struct nvme_fc_local_port *pnvme_lport, 
	struct lpfc_nvme_lport *lport;

	struct lpfc_vport *vport;



	if (!pnvme_lport->private)

		return;



	lport = (struct lpfc_nvme_lport *)pnvme_lport->private;

	vport = lport->vport;
@@ -1265,13 +1271,29 @@ lpfc_nvme_fcp_io_submit(struct nvme_fc_local_port *pnvme_lport, 
	struct lpfc_nvme_buf *lpfc_ncmd;

	struct lpfc_nvme_rport *rport;

	struct lpfc_nvme_qhandle *lpfc_queue_info;

	struct lpfc_nvme_fcpreq_priv *freqpriv = pnvme_fcreq->private;

	struct lpfc_nvme_fcpreq_priv *freqpriv;

#ifdef CONFIG_SCSI_LPFC_DEBUG_FS

	uint64_t start = 0;

#endif



	/* Validate pointers. LLDD fault handling with transport does

	 * have timing races.

	 */

	lport = (struct lpfc_nvme_lport *)pnvme_lport->private;

	if (unlikely(!lport)) {

		ret = -EINVAL;

		goto out_fail;

	}



	vport = lport->vport;



	if (unlikely(!hw_queue_handle)) {

		lpfc_printf_vlog(vport, KERN_INFO, LOG_NVME_ABTS,

				 "6129 Fail Abort, NULL hw_queue_handle\n");

		ret = -EINVAL;

		goto out_fail;

	}



	phba = vport->phba;



	if (vport->load_flag & FC_UNLOADING) {
@@ -1284,13 +1306,9 @@ lpfc_nvme_fcp_io_submit(struct nvme_fc_local_port *pnvme_lport, 
		goto out_fail;

	}



	/* Validate pointers. */

	if (!pnvme_lport || !pnvme_rport || !freqpriv) {

		lpfc_printf_vlog(vport, KERN_INFO, LOG_NVME_IOERR | LOG_NODE,

				 "6117 No Send:IO submit ptrs NULL, lport %p, "

				 "rport %p fcreq_priv %p\n",

				 pnvme_lport, pnvme_rport, freqpriv);

		ret = -ENODEV;

	freqpriv = pnvme_fcreq->private;

	if (unlikely(!freqpriv)) {

		ret = -EINVAL;

		goto out_fail;

	}
@@ -1497,20 +1515,34 @@ lpfc_nvme_fcp_abort(struct nvme_fc_local_port *pnvme_lport, 
	struct lpfc_nvme_lport *lport;

	struct lpfc_vport *vport;

	struct lpfc_hba *phba;

	struct lpfc_nvme_rport *rport;

	struct lpfc_nvme_buf *lpfc_nbuf;

	struct lpfc_iocbq *abts_buf;

	struct lpfc_iocbq *nvmereq_wqe;

	struct lpfc_nvme_fcpreq_priv *freqpriv = pnvme_fcreq->private;

	struct lpfc_nvme_fcpreq_priv *freqpriv;

	union lpfc_wqe *abts_wqe;

	unsigned long flags;

	int ret_val;



	/* Validate pointers. LLDD fault handling with transport does

	 * have timing races.

	 */

	lport = (struct lpfc_nvme_lport *)pnvme_lport->private;

	rport = (struct lpfc_nvme_rport *)pnvme_rport->private;

	if (unlikely(!lport))

		return;



	vport = lport->vport;



	if (unlikely(!hw_queue_handle)) {

		lpfc_printf_vlog(vport, KERN_INFO, LOG_NVME_ABTS,

				 "6129 Fail Abort, HW Queue Handle NULL.\n");

		return;

	}



	phba = vport->phba;

	freqpriv = pnvme_fcreq->private;



	if (unlikely(!freqpriv))

		return;

	if (vport->load_flag & FC_UNLOADING)

		return;
@@ -2677,3 +2709,45 @@ lpfc_sli4_nvme_xri_aborted(struct lpfc_hba *phba, 
			"6312 XRI Aborted xri x%x not found\n", xri);



}



/**

 * lpfc_nvme_wait_for_io_drain - Wait for all NVME wqes to complete

 * @phba: Pointer to HBA context object.

 *

 * This function flushes all wqes in the nvme rings and frees all resources

 * in the txcmplq. This function does not issue abort wqes for the IO

 * commands in txcmplq, they will just be returned with

 * IOERR_SLI_DOWN. This function is invoked with EEH when device's PCI

 * slot has been permanently disabled.

 **/

void

lpfc_nvme_wait_for_io_drain(struct lpfc_hba *phba)

{

	struct lpfc_sli_ring  *pring;

	u32 i, wait_cnt = 0;



	if (phba->sli_rev < LPFC_SLI_REV4)

		return;



	/* Cycle through all NVME rings and make sure all outstanding

	 * WQEs have been removed from the txcmplqs.

	 */

	for (i = 0; i < phba->cfg_nvme_io_channel; i++) {

		pring = phba->sli4_hba.nvme_wq[i]->pring;



		/* Retrieve everything on the txcmplq */

		while (!list_empty(&pring->txcmplq)) {

			msleep(LPFC_XRI_EXCH_BUSY_WAIT_T1);

			wait_cnt++;



			/* The sleep is 10mS.  Every ten seconds,

			 * dump a message.  Something is wrong.

			 */

			if ((wait_cnt % 1000) == 0) {

				lpfc_printf_log(phba, KERN_ERR, LOG_NVME_IOERR,

						"6178 NVME IO not empty, "

						"cnt %d\n", wait_cnt);

			}

		}

	}

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE