diff options
| author | Chuck Lever | 2015-08-03 19:03:20 +0200 |
|---|---|---|
| committer | Anna Schumaker | 2015-08-05 22:21:26 +0200 |
| commit | e531dcabec8dc2ee141aab01ddf20ca87c52d916 (patch) | |
| tree | 2f8a96a351564e5bb09b3eeb2bd67f71981cc60f /net/sunrpc/xprtrdma/xprt_rdma.h | |
| parent | xprtrdma: Don't fall back to PHYSICAL memory registration (diff) | |
| download | kernel-qcow2-linux-e531dcabec8dc2ee141aab01ddf20ca87c52d916.tar.gz kernel-qcow2-linux-e531dcabec8dc2ee141aab01ddf20ca87c52d916.tar.xz kernel-qcow2-linux-e531dcabec8dc2ee141aab01ddf20ca87c52d916.zip | |
xprtrdma: Remove last ib_reg_phys_mr() call site
All HCA providers have an ib_get_dma_mr() verb. Thus
rpcrdma_ia_open() will either grab the device's local_dma_key if one
is available, or it will call ib_get_dma_mr(). If ib_get_dma_mr()
fails, rpcrdma_ia_open() fails and no transport is created.
Therefore execution never reaches the ib_reg_phys_mr() call site in
rpcrdma_register_internal(), so it can be removed.
The remaining logic in rpcrdma_{de}register_internal() is folded
into rpcrdma_{alloc,free}_regbuf().
This is clean up only. No behavior change is expected.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Reviewed-by: Devesh Sharma <devesh.sharma@avagotech.com>
Reviewed-By: Sagi Grimberg <sagig@mellanox.com>
Tested-by: Devesh Sharma <devesh.sharma@avagotech.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'net/sunrpc/xprtrdma/xprt_rdma.h')
| -rw-r--r-- | net/sunrpc/xprtrdma/xprt_rdma.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/sunrpc/xprtrdma/xprt_rdma.h b/net/sunrpc/xprtrdma/xprt_rdma.h index abee4728f885..ce4e79e4cf1d 100644 --- a/net/sunrpc/xprtrdma/xprt_rdma.h +++ b/net/sunrpc/xprtrdma/xprt_rdma.h @@ -119,7 +119,6 @@ struct rpcrdma_ep { struct rpcrdma_regbuf { size_t rg_size; struct rpcrdma_req *rg_owner; - struct ib_mr *rg_mr; struct ib_sge rg_iov; __be32 rg_base[0] __attribute__ ((aligned(256))); }; |