> On Jun 12, 2023, at 10:24 AM, Jeff Layton <jlayton@xxxxxxxxxx> wrote: > > On Mon, 2023-06-12 at 10:10 -0400, Chuck Lever wrote: >> From: Chuck Lever <chuck.lever@xxxxxxxxxx> >> >> Pre-requisite for releasing pages in the send completion handler. >> Reverted by hand: patch -R would not apply cleanly. >> > > I'm guessing because there were other patches to this area in the > interim that you didn't want to revert? Correct. >> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> >> --- >> net/sunrpc/xprtrdma/svc_rdma_sendto.c | 5 ----- >> 1 file changed, 5 deletions(-) >> >> diff --git a/net/sunrpc/xprtrdma/svc_rdma_sendto.c b/net/sunrpc/xprtrdma/svc_rdma_sendto.c >> index a35d1e055b1a..8e7ccef74207 100644 >> --- a/net/sunrpc/xprtrdma/svc_rdma_sendto.c >> +++ b/net/sunrpc/xprtrdma/svc_rdma_sendto.c >> @@ -975,11 +975,6 @@ int svc_rdma_sendto(struct svc_rqst *rqstp) >> ret = svc_rdma_send_reply_msg(rdma, sctxt, rctxt, rqstp); >> if (ret < 0) >> goto put_ctxt; >> - >> - /* Prevent svc_xprt_release() from releasing the page backing >> - * rq_res.head[0].iov_base. It's no longer being accessed by >> - * the I/O device. */ >> - rqstp->rq_respages++; >> return 0; >> >> reply_chunk: >> >> > > -- > Jeff Layton <jlayton@xxxxxxxxxx> -- Chuck Lever
