diff options
| author |   Chuck Lever <chuck.lever@oracle.com> | 2019-04-24 09:40:31 -0400 |
|---|---|---|
| committer |   Anna Schumaker <Anna.Schumaker@Netapp.com> | 2019-04-25 15:40:09 -0400 |
| commit | 5f2311f5bd359d6d810922bf25c238053a449f2a (patch) | |
| tree | f206e962a324e19660bbc7803d7ea3ab6fc6eb9e /net/sunrpc/xprtrdma/verbs.c | |
| parent | xprtrdma: Eliminate struct rpcrdma_create_data_internal (diff) | |
| download | linux-dev-5f2311f5bd359d6d810922bf25c238053a449f2a.tar.xz linux-dev-5f2311f5bd359d6d810922bf25c238053a449f2a.zip | |
xprtrdma: Remove pr_err() call sites from completion handlers
Clean up: rely on the trace points instead.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to '')
| -rw-r--r-- | net/sunrpc/xprtrdma/verbs.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c index 0d0c3356f34e..fcbcd4afaa5a 100644 --- a/net/sunrpc/xprtrdma/verbs.c +++ b/net/sunrpc/xprtrdma/verbs.c @@ -135,11 +135,6 @@ rpcrdma_wc_send(struct ib_cq *cq, struct ib_wc *wc) /* WARNING: Only wr_cqe and status are reliable at this point */ trace_xprtrdma_wc_send(sc, wc); - if (wc->status != IB_WC_SUCCESS && wc->status != IB_WC_WR_FLUSH_ERR) - pr_err("rpcrdma: Send: %s (%u/0x%x)\n", - ib_wc_status_msg(wc->status), - wc->status, wc->vendor_err); - rpcrdma_sendctx_put_locked(sc); } @@ -177,10 +172,6 @@ rpcrdma_wc_receive(struct ib_cq *cq, struct ib_wc *wc) return; out_flushed: - if (wc->status != IB_WC_WR_FLUSH_ERR) - pr_err("rpcrdma: Recv: %s (%u/0x%x)\n", - ib_wc_status_msg(wc->status), - wc->status, wc->vendor_err); rpcrdma_recv_buffer_put(rep); } |