On Mon, 2023-06-12 at 10:09 -0400, Chuck Lever wrote:
> Return to the behavior of releasing reply buffer pages as part of
> sending an RPC Reply over RDMA. I measured a performance improvement
> (which is documented in 4/4). Matching the page release behavior of
> socket transports also means we should be able to share a little
> more code between transports as MSG_SPLICE_PAGES rolls out.
>
> Changes since v1:
> - Add a related fix
> - Clarify some of the patch descriptions
>
> ---
>
> Chuck Lever (5):
> SUNRPC: Revert cc93ce9529a6 ("svcrdma: Retain the page backing rq_res.head[0].iov_base")
> SUNRPC: Revert 579900670ac7 ("svcrdma: Remove unused sc_pages field")
> svcrdma: Revert 2a1e4f21d841 ("svcrdma: Normalize Send page handling")
> svcrdma: Prevent page release when nothing was received
> SUNRPC: Optimize page release in svc_rdma_sendto()
>
>
> include/linux/sunrpc/svc_rdma.h | 4 +-
> net/sunrpc/xprtrdma/svc_rdma_backchannel.c | 8 +---
> net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 12 ++---
> net/sunrpc/xprtrdma/svc_rdma_sendto.c | 53 ++++++++++++++--------
> 4 files changed, 44 insertions(+), 33 deletions(-)
>
> --
> Chuck Lever
>
Kind of cool that we're getting to the place where micro-optimizations
like this make such a big difference! This all looks fine to me:
Reviewed-by: Jeff Layton <jlayton@xxxxxxxxxx>
