ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nvme/rdma: Clean pointer to nvme_request

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

Signed-off-by: Alexey Marchuk <alexeymar@mellanox.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1771 (master)

(cherry picked from commit f11989385e)
Change-Id: Ie2ad001da701e25ef984ae57da850fb84d51b734
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2641
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
This commit is contained in:
Alexey Marchuk 2020-04-09 13:14:41 +03:00 committed by Tomasz Zawadzki
parent 2381516ecc
commit 0acac18cfa
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/nvme/nvme_rdma.c
View File

@ -255,6 +255,7 @@ static void
nvme_rdma_req_put(struct nvme_rdma_qpair *rqpair, struct spdk_nvme_rdma_req *rdma_req) nvme_rdma_req_put(struct nvme_rdma_qpair *rqpair, struct spdk_nvme_rdma_req *rdma_req)
{ {
rdma_req->completion_flags = 0; rdma_req->completion_flags = 0;
rdma_req->req = NULL;
TAILQ_REMOVE(&rqpair->outstanding_reqs, rdma_req, link); TAILQ_REMOVE(&rqpair->outstanding_reqs, rdma_req, link);
TAILQ_INSERT_HEAD(&rqpair->free_reqs, rdma_req, link); TAILQ_INSERT_HEAD(&rqpair->free_reqs, rdma_req, link);
} }
@ -1499,6 +1500,7 @@ nvme_rdma_req_init(struct nvme_rdma_qpair *rqpair, struct nvme_request *req,
struct spdk_nvme_ctrlr *ctrlr = rqpair->qpair.ctrlr; struct spdk_nvme_ctrlr *ctrlr = rqpair->qpair.ctrlr;
int rc; int rc;
assert(rdma_req->req == NULL);
rdma_req->req = req; rdma_req->req = req;
req->cmd.cid = rdma_req->id; req->cmd.cid = rdma_req->id;