Commit cd84d800 authored by Jay Bhat's avatar Jay Bhat Committed by Leon Romanovsky
Browse files

RDMA/irdma: Silently consume unsignaled completions 



In case we get an unsignaled error completion, we silently consume the CQE by
pretending the QP does not exist. Without this, bookkeeping for signaled
completions does not work correctly.

Signed-off-by: default avatarJay Bhat <jay.bhat@intel.com>
Signed-off-by: default avatarTatyana Nikolova <tatyana.e.nikolova@intel.com>
Link: https://patch.msgid.link/20251031021726.1003-5-tatyana.e.nikolova@intel.com


Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 15324308

drivers/infiniband/hw/irdma/uk.c

+5 −0
Original line number Diff line number Diff line
@@ -194,6 +194,7 @@ __le64 *irdma_qp_get_next_send_wqe(struct irdma_qp_uk *qp, u32 *wqe_idx, 
	qp->sq_wrtrk_array[*wqe_idx].wrid = info->wr_id;

	qp->sq_wrtrk_array[*wqe_idx].wr_len = total_size;

	qp->sq_wrtrk_array[*wqe_idx].quanta = quanta;

	qp->sq_wrtrk_array[*wqe_idx].signaled = info->signaled;



	return wqe;

}
@@ -1355,6 +1356,10 @@ int irdma_uk_cq_poll_cmpl(struct irdma_cq_uk *cq, 
			info->wr_id = qp->sq_wrtrk_array[wqe_idx].wrid;

			if (!info->comp_status)

				info->bytes_xfered = qp->sq_wrtrk_array[wqe_idx].wr_len;

			if (!qp->sq_wrtrk_array[wqe_idx].signaled) {

				ret_code = -EFAULT;

				goto exit;

			}

			info->op_type = (u8)FIELD_GET(IRDMACQ_OP, qword3);

			IRDMA_RING_SET_TAIL(qp->sq_ring,

					    wqe_idx + qp->sq_wrtrk_array[wqe_idx].quanta);

drivers/infiniband/hw/irdma/user.h

+2 −1
Original line number Diff line number Diff line
@@ -482,7 +482,8 @@ struct irdma_sq_uk_wr_trk_info { 
	u64 wrid;

	u32 wr_len;

	u16 quanta;

	u8 reserved[2];

	u8 signaled;

	u8 reserved[1];

};



struct irdma_qp_quanta {