Commit 7413e1e4 authored by Shuhei Matsumoto's avatar Shuhei Matsumoto Committed by Jim Harris
Browse files

nvme: Initialize cpl->sqid when aborting requests for RDMA and TCP 



nvme_rdma_qpair_abort_reqs() and nvme_tcp_qpair_abort_reqs() did not
initialize cpl->sqid. Hence, unexpected message was printed by
spdk_nvme_print_completion(). Fix the bugs in this patch.

Fixes #2930

Signed-off-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Change-Id: I8b41166e58b26ce22c453ab85794b46dbe3dd3a2
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17067


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: default avatarMichael Haeuptle <michaelhaeuptle@gmail.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent a475aed5

lib/nvme/nvme_rdma.c

+1 −0
Original line number Diff line number Diff line
@@ -2355,6 +2355,7 @@ nvme_rdma_qpair_abort_reqs(struct spdk_nvme_qpair *qpair, uint32_t dnr) 
	struct spdk_nvme_cpl cpl;

	struct nvme_rdma_qpair *rqpair = nvme_rdma_qpair(qpair);



	cpl.sqid = qpair->id;

	cpl.status.sc = SPDK_NVME_SC_ABORTED_SQ_DELETION;

	cpl.status.sct = SPDK_NVME_SCT_GENERIC;

	cpl.status.dnr = dnr;

lib/nvme/nvme_tcp.c

+1 −0
Original line number Diff line number Diff line
@@ -789,6 +789,7 @@ nvme_tcp_qpair_abort_reqs(struct spdk_nvme_qpair *qpair, uint32_t dnr) 
	struct spdk_nvme_cpl cpl = {};

	struct nvme_tcp_qpair *tqpair = nvme_tcp_qpair(qpair);



	cpl.sqid = qpair->id;

	cpl.status.sc = SPDK_NVME_SC_ABORTED_SQ_DELETION;

	cpl.status.sct = SPDK_NVME_SCT_GENERIC;

	cpl.status.dnr = dnr;