diff options
| author | Chuck Lever | 2014-05-28 16:33:16 +0200 |
|---|---|---|
| committer | Anna Schumaker | 2014-06-04 14:56:41 +0200 |
| commit | 7f1d54191ed6fa0f79f584fe3ebf6519738e817f (patch) | |
| tree | 56497b34c0cdd720b803108b231bf7312e76c29d /net/sunrpc/xprtrdma/verbs.c | |
| parent | xprtrdma: Simplify rpcrdma_deregister_external() synopsis (diff) | |
| download | kernel-qcow2-linux-7f1d54191ed6fa0f79f584fe3ebf6519738e817f.tar.gz kernel-qcow2-linux-7f1d54191ed6fa0f79f584fe3ebf6519738e817f.tar.xz kernel-qcow2-linux-7f1d54191ed6fa0f79f584fe3ebf6519738e817f.zip | |
xprtrdma: Make rpcrdma_ep_destroy() return void
Clean up: rpcrdma_ep_destroy() returns a value that is used
only to print a debugging message. rpcrdma_ep_destroy() already
prints debugging messages in all error cases.
Make rpcrdma_ep_destroy() return void instead.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Tested-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'net/sunrpc/xprtrdma/verbs.c')
| -rw-r--r-- | net/sunrpc/xprtrdma/verbs.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c index 0cbc83c0fbe4..edc951e1f1d9 100644 --- a/net/sunrpc/xprtrdma/verbs.c +++ b/net/sunrpc/xprtrdma/verbs.c @@ -748,11 +748,8 @@ out1: * Disconnect and destroy endpoint. After this, the only * valid operations on the ep are to free it (if dynamically * allocated) or re-create it. - * - * The caller's error handling must be sure to not leak the endpoint - * if this function fails. */ -int +void rpcrdma_ep_destroy(struct rpcrdma_ep *ep, struct rpcrdma_ia *ia) { int rc; @@ -782,8 +779,6 @@ rpcrdma_ep_destroy(struct rpcrdma_ep *ep, struct rpcrdma_ia *ia) if (rc) dprintk("RPC: %s: ib_destroy_cq returned %i\n", __func__, rc); - - return rc; } /* |