Commit ed288d74 authored by Bart Van Assche's avatar Bart Van Assche Committed by Jason Gunthorpe

net/xprtrdma: Simplify ib_post_(send|recv|srq_recv)() calls

Instead of declaring and passing a dummy 'bad_wr' pointer, pass NULL
as third argument to ib_post_(send|recv|srq_recv)().
Signed-off-by: default avatarBart Van Assche <bart.vanassche@wdc.com>
Reviewed-by: default avatarChuck Lever <chuck.lever@oracle.com>
Acked-by: default avatarAnna Schumaker <Anna.Schumaker@netapp.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent 2e3bbe46
...@@ -279,9 +279,7 @@ fmr_op_map(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mr_seg *seg, ...@@ -279,9 +279,7 @@ fmr_op_map(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mr_seg *seg,
static int static int
fmr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req) fmr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req)
{ {
struct ib_send_wr *bad_wr; return ib_post_send(ia->ri_id->qp, &req->rl_sendctx->sc_wr, NULL);
return ib_post_send(ia->ri_id->qp, &req->rl_sendctx->sc_wr, &bad_wr);
} }
/* Invalidate all memory regions that were registered for "req". /* Invalidate all memory regions that were registered for "req".
......
...@@ -464,7 +464,7 @@ frwr_op_map(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mr_seg *seg, ...@@ -464,7 +464,7 @@ frwr_op_map(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mr_seg *seg,
static int static int
frwr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req) frwr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req)
{ {
struct ib_send_wr *post_wr, *bad_wr; struct ib_send_wr *post_wr;
struct rpcrdma_mr *mr; struct rpcrdma_mr *mr;
post_wr = &req->rl_sendctx->sc_wr; post_wr = &req->rl_sendctx->sc_wr;
...@@ -486,7 +486,7 @@ frwr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req) ...@@ -486,7 +486,7 @@ frwr_op_send(struct rpcrdma_ia *ia, struct rpcrdma_req *req)
/* If ib_post_send fails, the next ->send_request for /* If ib_post_send fails, the next ->send_request for
* @req will queue these MWs for recovery. * @req will queue these MWs for recovery.
*/ */
return ib_post_send(ia->ri_id->qp, post_wr, &bad_wr); return ib_post_send(ia->ri_id->qp, post_wr, NULL);
} }
/* Handle a remotely invalidated mr on the @mrs list /* Handle a remotely invalidated mr on the @mrs list
......
...@@ -229,11 +229,10 @@ void svc_rdma_recv_ctxt_put(struct svcxprt_rdma *rdma, ...@@ -229,11 +229,10 @@ void svc_rdma_recv_ctxt_put(struct svcxprt_rdma *rdma,
static int __svc_rdma_post_recv(struct svcxprt_rdma *rdma, static int __svc_rdma_post_recv(struct svcxprt_rdma *rdma,
struct svc_rdma_recv_ctxt *ctxt) struct svc_rdma_recv_ctxt *ctxt)
{ {
struct ib_recv_wr *bad_recv_wr;
int ret; int ret;
svc_xprt_get(&rdma->sc_xprt); svc_xprt_get(&rdma->sc_xprt);
ret = ib_post_recv(rdma->sc_qp, &ctxt->rc_recv_wr, &bad_recv_wr); ret = ib_post_recv(rdma->sc_qp, &ctxt->rc_recv_wr, NULL);
trace_svcrdma_post_recv(&ctxt->rc_recv_wr, ret); trace_svcrdma_post_recv(&ctxt->rc_recv_wr, ret);
if (ret) if (ret)
goto err_post; goto err_post;
......
...@@ -329,7 +329,7 @@ static int svc_rdma_post_chunk_ctxt(struct svc_rdma_chunk_ctxt *cc) ...@@ -329,7 +329,7 @@ static int svc_rdma_post_chunk_ctxt(struct svc_rdma_chunk_ctxt *cc)
do { do {
if (atomic_sub_return(cc->cc_sqecount, if (atomic_sub_return(cc->cc_sqecount,
&rdma->sc_sq_avail) > 0) { &rdma->sc_sq_avail) > 0) {
ret = ib_post_send(rdma->sc_qp, first_wr, &bad_wr); ret = ib_post_send(rdma->sc_qp, first_wr, NULL);
trace_svcrdma_post_rw(&cc->cc_cqe, trace_svcrdma_post_rw(&cc->cc_cqe,
cc->cc_sqecount, ret); cc->cc_sqecount, ret);
if (ret) if (ret)
......
...@@ -291,7 +291,6 @@ static void svc_rdma_wc_send(struct ib_cq *cq, struct ib_wc *wc) ...@@ -291,7 +291,6 @@ static void svc_rdma_wc_send(struct ib_cq *cq, struct ib_wc *wc)
*/ */
int svc_rdma_send(struct svcxprt_rdma *rdma, struct ib_send_wr *wr) int svc_rdma_send(struct svcxprt_rdma *rdma, struct ib_send_wr *wr)
{ {
struct ib_send_wr *bad_wr;
int ret; int ret;
might_sleep(); might_sleep();
...@@ -311,7 +310,7 @@ int svc_rdma_send(struct svcxprt_rdma *rdma, struct ib_send_wr *wr) ...@@ -311,7 +310,7 @@ int svc_rdma_send(struct svcxprt_rdma *rdma, struct ib_send_wr *wr)
} }
svc_xprt_get(&rdma->sc_xprt); svc_xprt_get(&rdma->sc_xprt);
ret = ib_post_send(rdma->sc_qp, wr, &bad_wr); ret = ib_post_send(rdma->sc_qp, wr, NULL);
trace_svcrdma_post_send(wr, ret); trace_svcrdma_post_send(wr, ret);
if (ret) { if (ret) {
set_bit(XPT_CLOSE, &rdma->sc_xprt.xpt_flags); set_bit(XPT_CLOSE, &rdma->sc_xprt.xpt_flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment