diff options
| author |   Sagi Grimberg <sagi@grimberg.me> | 2017-07-05 09:56:13 +0300 |
|---|---|---|
| committer | Sagi Grimberg <sagi@grimberg.me> | 2017-07-06 09:49:43 +0300 |
| commit | 842594c8775b585c58459e044708c0335b6aa6b7 (patch) | |
| tree | 9e2c791f916f7e7d9c6e0c839b4dd0b14e4aefc9 /drivers/nvme | |
| parent | nvme: split nvme_uninit_ctrl into stop and uninit (diff) | |
| download | linux-dev-842594c8775b585c58459e044708c0335b6aa6b7.tar.xz linux-dev-842594c8775b585c58459e044708c0335b6aa6b7.zip | |
nvme-rdma: unconditionally recycle the request mr
When our RDMA queue-pair is torn down with high load
of I/O traffic, we have no way of knowing if the
memory region was actually registered by the reg_mr
work request as it completion flushes with error (hw
might have done it or not).
So in order to not deal with all this uncertanty, we
simply recycle the MR in reinit_request.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Diffstat (limited to 'drivers/nvme')
| -rw-r--r-- | drivers/nvme/host/rdma.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/nvme/host/rdma.c b/drivers/nvme/host/rdma.c index ccdbd99d5a1c..da04df1af231 100644 --- a/drivers/nvme/host/rdma.c +++ b/drivers/nvme/host/rdma.c @@ -272,9 +272,6 @@ static int nvme_rdma_reinit_request(void *data, struct request *rq) struct nvme_rdma_request *req = blk_mq_rq_to_pdu(rq); int ret = 0; - if (!req->mr->need_inval) - goto out; - ib_dereg_mr(req->mr); req->mr = ib_alloc_mr(dev->pd, IB_MR_TYPE_MEM_REG, |