diff options
author | 2007-10-26 13:30:54 -0400 | |
---|---|---|
committer | 2008-01-30 02:05:42 -0500 | |
commit | 5d40a8a525c8165bafed233cf0f137e8d10d7e92 (patch) | |
tree | 57e7117c055bbd48ceabae0ec6c8b6b0e8df3ea8 /net/sunrpc | |
parent | SUNRPC: Fix an unnecessary implicit type cast in rpcrdma_count_chunks() (diff) | |
download | linux-5d40a8a525c8165bafed233cf0f137e8d10d7e92.tar.gz linux-5d40a8a525c8165bafed233cf0f137e8d10d7e92.tar.bz2 linux-5d40a8a525c8165bafed233cf0f137e8d10d7e92.zip |
SUNRPC: Check a return result
Minor: Replace an empty if statement with a debugging dprintk.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Cc: Thomas Talpey <Thomas.Talpey@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc')
-rw-r--r-- | net/sunrpc/xprtrdma/verbs.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c index 44b0fb942e8d..ffbf22a1d2ca 100644 --- a/net/sunrpc/xprtrdma/verbs.c +++ b/net/sunrpc/xprtrdma/verbs.c @@ -522,7 +522,7 @@ rpcrdma_ep_create(struct rpcrdma_ep *ep, struct rpcrdma_ia *ia, struct rpcrdma_create_data_internal *cdata) { struct ib_device_attr devattr; - int rc; + int rc, err; rc = ib_query_device(ia->ri_id->device, &devattr); if (rc) { @@ -648,8 +648,10 @@ rpcrdma_ep_create(struct rpcrdma_ep *ep, struct rpcrdma_ia *ia, return 0; out2: - if (ib_destroy_cq(ep->rep_cq)) - ; + err = ib_destroy_cq(ep->rep_cq); + if (err) + dprintk("RPC: %s: ib_destroy_cq returned %i\n", + __func__, err); out1: return rc; } |