diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2007-10-26 13:30:54 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2008-01-30 02:05:42 -0500 |
commit | 5d40a8a525c8165bafed233cf0f137e8d10d7e92 (patch) | |
tree | 57e7117c055bbd48ceabae0ec6c8b6b0e8df3ea8 | |
parent | d4b37ff73540ab90bee57b882a10b21e2f97939f (diff) |
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>
-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, | |||
522 | struct rpcrdma_create_data_internal *cdata) | 522 | struct rpcrdma_create_data_internal *cdata) |
523 | { | 523 | { |
524 | struct ib_device_attr devattr; | 524 | struct ib_device_attr devattr; |
525 | int rc; | 525 | int rc, err; |
526 | 526 | ||
527 | rc = ib_query_device(ia->ri_id->device, &devattr); | 527 | rc = ib_query_device(ia->ri_id->device, &devattr); |
528 | if (rc) { | 528 | if (rc) { |
@@ -648,8 +648,10 @@ rpcrdma_ep_create(struct rpcrdma_ep *ep, struct rpcrdma_ia *ia, | |||
648 | return 0; | 648 | return 0; |
649 | 649 | ||
650 | out2: | 650 | out2: |
651 | if (ib_destroy_cq(ep->rep_cq)) | 651 | err = ib_destroy_cq(ep->rep_cq); |
652 | ; | 652 | if (err) |
653 | dprintk("RPC: %s: ib_destroy_cq returned %i\n", | ||
654 | __func__, err); | ||
653 | out1: | 655 | out1: |
654 | return rc; | 656 | return rc; |
655 | } | 657 | } |