xprtrdma: Remap Receive buffers after a reconnect
[ Upstream commit 895cedc1791916e8a98864f12b656702fad0bb67 ]
On server-initiated disconnect, rpcrdma_xprt_disconnect() was DMA-
unmapping the Receive buffers, but rpcrdma_post_recvs() neglected
to remap them after a new connection had been established. The
result was immediate failure of the new connection with the Receives
flushing with LOCAL_PROT_ERR.
Fixes: 671c450b6f
("xprtrdma: Fix oops in Receive handler after device removal")
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
8a64aadc29
commit
e827572152
@@ -972,9 +972,6 @@ struct rpcrdma_rep *rpcrdma_rep_create(struct rpcrdma_xprt *r_xprt,
|
|||||||
if (!rep->rr_rdmabuf)
|
if (!rep->rr_rdmabuf)
|
||||||
goto out_free;
|
goto out_free;
|
||||||
|
|
||||||
if (!rpcrdma_regbuf_dma_map(r_xprt, rep->rr_rdmabuf))
|
|
||||||
goto out_free_regbuf;
|
|
||||||
|
|
||||||
xdr_buf_init(&rep->rr_hdrbuf, rdmab_data(rep->rr_rdmabuf),
|
xdr_buf_init(&rep->rr_hdrbuf, rdmab_data(rep->rr_rdmabuf),
|
||||||
rdmab_length(rep->rr_rdmabuf));
|
rdmab_length(rep->rr_rdmabuf));
|
||||||
rep->rr_cqe.done = rpcrdma_wc_receive;
|
rep->rr_cqe.done = rpcrdma_wc_receive;
|
||||||
@@ -987,8 +984,6 @@ struct rpcrdma_rep *rpcrdma_rep_create(struct rpcrdma_xprt *r_xprt,
|
|||||||
list_add(&rep->rr_all, &r_xprt->rx_buf.rb_all_reps);
|
list_add(&rep->rr_all, &r_xprt->rx_buf.rb_all_reps);
|
||||||
return rep;
|
return rep;
|
||||||
|
|
||||||
out_free_regbuf:
|
|
||||||
rpcrdma_regbuf_free(rep->rr_rdmabuf);
|
|
||||||
out_free:
|
out_free:
|
||||||
kfree(rep);
|
kfree(rep);
|
||||||
out:
|
out:
|
||||||
@@ -1425,6 +1420,10 @@ void rpcrdma_post_recvs(struct rpcrdma_xprt *r_xprt, int needed, bool temp)
|
|||||||
rep = rpcrdma_rep_create(r_xprt, temp);
|
rep = rpcrdma_rep_create(r_xprt, temp);
|
||||||
if (!rep)
|
if (!rep)
|
||||||
break;
|
break;
|
||||||
|
if (!rpcrdma_regbuf_dma_map(r_xprt, rep->rr_rdmabuf)) {
|
||||||
|
rpcrdma_rep_put(buf, rep);
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
trace_xprtrdma_post_recv(rep);
|
trace_xprtrdma_post_recv(rep);
|
||||||
rep->rr_recv_wr.next = wr;
|
rep->rr_recv_wr.next = wr;
|
||||||
|
Reference in New Issue
Block a user