Commit 63ab6cb5 authored by Quinn Tran's avatar Quinn Tran Committed by Martin K. Petersen
Browse files

scsi: qla2xxx: edif: Fix I/O timeout due to over-subscription 

The current edif code does not keep track of FW IOCB resources.  This led
to IOCB queue full on error recovery (I/O timeout).  Make use of the
existing code that tracks IOCB resources to prevent over-subscription.

Link: https://lore.kernel.org/r/20220608115849.16693-2-njavali@marvell.com


Reviewed-by: default avatarHimanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: default avatarQuinn Tran <qutran@marvell.com>
Signed-off-by: default avatarNilesh Javali <njavali@marvell.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 3fd3a52c

drivers/scsi/qla2xxx/qla_edif.c

+7 −0
Original line number Diff line number Diff line
@@ -2951,6 +2951,12 @@ qla28xx_start_scsi_edif(srb_t *sp) 


	tot_dsds = nseg;

	req_cnt = qla24xx_calc_iocbs(vha, tot_dsds);



	sp->iores.res_type = RESOURCE_INI;

	sp->iores.iocb_cnt = req_cnt;

	if (qla_get_iocbs(sp->qpair, &sp->iores))

		goto queuing_error;



	if (req->cnt < (req_cnt + 2)) {

		cnt = IS_SHADOW_REG_CAPABLE(ha) ? *req->out_ptr :

		    rd_reg_dword(req->req_q_out);
@@ -3142,6 +3148,7 @@ qla28xx_start_scsi_edif(srb_t *sp) 
		mempool_free(sp->u.scmd.ct6_ctx, ha->ctx_mempool);

		sp->u.scmd.ct6_ctx = NULL;

	}

	qla_put_iocbs(sp->qpair, &sp->iores);

	spin_unlock_irqrestore(lock, flags);



	return QLA_FUNCTION_FAILED;