SUNRPC: Check a return result
authorChuck Lever <chuck.lever@oracle.com>
Fri, 26 Oct 2007 17:30:54 +0000 (13:30 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 30 Jan 2008 07:05:42 +0000 (02:05 -0500)
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>
net/sunrpc/xprtrdma/verbs.c

index 44b0fb942e8db5834ddaebf81fe4fc1e3fbd5530..ffbf22a1d2ca0e1a24c76f9e7231732ff7ce8dfd 100644 (file)
@@ -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;
 }