Message-ID: <54CBBD8C.7090400@Netapp.com>
Date: Fri, 30 Jan 2015 12:21:16 -0500
From: Anna Schumaker <Anna.Schumaker@netapp.com>
MIME-Version: 1.0
To: Chuck Lever <chuck.lever@oracle.com>
CC: <linux-rdma@vger.kernel.org>, <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v3 00/20] NFS/RDMA client for 3.20
References: <20150121155303.17297.47423.stgit@manet.1015granger.net>
In-Reply-To: <20150121155303.17297.47423.stgit@manet.1015granger.net>
Content-Type: text/plain; charset="utf-8"
Sender: linux-nfs-owner@vger.kernel.org

Hi Chuck,

I've applied these patches and the GPF flags patch for 3.20.  Let me know if you need anything else!

Anna
On 01/21/2015 11:01 AM, Chuck Lever wrote:
> The following series of patches for the Linux NFS client breaks up
> the per-transport buffer pool data structures. The goal is to allow
> transport operation to scale better as the size of SEND/RECV buffers
> (the inline thresholds), the maximum NFS r/wsize, and the number of
> RDMA credits (concurrent RPC requests) is increased.
> 
> The primary change is that the header send buffers have been split
> from struct rpcrdma_req. Specific benefits are outlined in the patch
> descriptions.
> 
> More pre-requisites are required. Changes to raise the maximum
> r/wsize and other limits are left for a future merge window.
> 
> See the topic branch "nfs-rdma-for-3.20" at:
> 
>  git://git.linux-nfs.org/projects/cel/cel-2.6.git
> 
> Anna, I think these are ready for you to merge.
> 
> Changes since v2:
> - Rebased on v3.19-rc5
> - Several patch descriptions clarified, based on review comments
> - New size of client receive buffers was needlessly large
> 
> Changes since v1:
> - Rebased on v3.19-rc4
> - One short description fixed
> - linux-rdma included this time (sorry for the noise)
> 
> ---
> 
> Chuck Lever (20):
>       xprtrdma: human-readable completion status
>       xprtrdma: Modernize htonl and ntohl
>       xprtrdma: Display XIDs in host byte order
>       xprtrdma: Clean up hdrlen
>       xprtrdma: Rename "xprt" and "rdma_connect" fields in struct rpcrdma_xprt
>       xprtrdma: Remove rpcrdma_ep::rep_ia
>       xprtrdma: Remove rl_mr field, and the mr_chunk union
>       xprtrdma: Move credit update to RPC reply handler
>       xprtrdma: Remove rpcrdma_ep::rep_func and ::rep_xprt
>       xprtrdma: Free the pd if ib_query_qp() fails
>       xprtrdma: Take struct ib_device_attr off the stack
>       xprtrdma: Take struct ib_qp_attr and ib_qp_init_attr off the stack
>       xprtrdma: Simplify synopsis of rpcrdma_buffer_create()
>       xprtrdma: Refactor rpcrdma_buffer_create() and rpcrdma_buffer_destroy()
>       xprtrdma: Add struct rpcrdma_regbuf and helpers
>       xprtrdma: Allocate RPC send buffer separately from struct rpcrdma_req
>       xprtrdma: Allocate RPC/RDMA send buffer separately from struct rpcrdma_req
>       xprtrdma: Allocate RPC/RDMA receive buffer separately from struct rpcrdma_rep
>       xprtrdma: Allocate zero pad separately from rpcrdma_buffer
>       xprtrdma: Clean up after adding regbuf management
> 
> 
>  include/linux/sunrpc/rpc_rdma.h |   14 +
>  include/linux/sunrpc/svc_rdma.h |    2 
>  net/sunrpc/xprtrdma/rpc_rdma.c  |  108 ++++++----
>  net/sunrpc/xprtrdma/transport.c |  179 +++++++----------
>  net/sunrpc/xprtrdma/verbs.c     |  411 ++++++++++++++++++++++++---------------
>  net/sunrpc/xprtrdma/xprt_rdma.h |  111 +++++++----
>  6 files changed, 478 insertions(+), 347 deletions(-)
> 
> --
> Chuck Lever
>