Commit f1198938 authored by Alexey Marchuk's avatar Alexey Marchuk Committed by Tomasz Zawadzki
Browse files

nvme/rdma: Clean pointer to nvme_request 



That is done to make sure that scenario described in github
issue #1292 won't happen

Change-Id: Ie2ad001da701e25ef984ae57da850fb84d51b734
Signed-off-by: default avatarAlexey Marchuk <alexeymar@mellanox.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1771


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatar <dongx.yi@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 581e1bb5

lib/nvme/nvme_rdma.c

+2 −0
Original line number Diff line number Diff line
@@ -271,6 +271,7 @@ static void 
nvme_rdma_req_put(struct nvme_rdma_qpair *rqpair, struct spdk_nvme_rdma_req *rdma_req)

{

	rdma_req->completion_flags = 0;

	rdma_req->req = NULL;

	TAILQ_REMOVE(&rqpair->outstanding_reqs, rdma_req, link);

	TAILQ_INSERT_HEAD(&rqpair->free_reqs, rdma_req, link);

}
@@ -1537,6 +1538,7 @@ nvme_rdma_req_init(struct nvme_rdma_qpair *rqpair, struct nvme_request *req, 
	bool icd_supported;

	int rc;



	assert(rdma_req->req == NULL);

	rdma_req->req = req;

	req->cmd.cid = rdma_req->id;

	payload_type = nvme_payload_type(&req->payload);