From: Anna Schumaker <Anna.Schumaker@netapp.com>
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
Date: Fri, 30 Jan 2015 12:21:16 -0500 [thread overview]
Message-ID: <54CBBD8C.7090400@Netapp.com> (raw)
In-Reply-To: <20150121155303.17297.47423.stgit@manet.1015granger.net>
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
>
prev parent reply other threads:[~2015-01-30 17:21 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-21 16:01 [PATCH v3 00/20] NFS/RDMA client for 3.20 Chuck Lever
2015-01-21 16:02 ` [PATCH v3 01/20] xprtrdma: human-readable completion status Chuck Lever
2015-01-21 16:02 ` [PATCH v3 02/20] xprtrdma: Modernize htonl and ntohl Chuck Lever
2015-01-21 16:02 ` [PATCH v3 03/20] xprtrdma: Display XIDs in host byte order Chuck Lever
2015-01-21 16:02 ` [PATCH v3 04/20] xprtrdma: Clean up hdrlen Chuck Lever
2015-01-21 16:02 ` [PATCH v3 05/20] xprtrdma: Rename "xprt" and "rdma_connect" fields in struct rpcrdma_xprt Chuck Lever
2015-01-21 16:02 ` [PATCH v3 06/20] xprtrdma: Remove rpcrdma_ep::rep_ia Chuck Lever
2015-01-21 16:02 ` [PATCH v3 07/20] xprtrdma: Remove rl_mr field, and the mr_chunk union Chuck Lever
2015-01-21 16:03 ` [PATCH v3 08/20] xprtrdma: Move credit update to RPC reply handler Chuck Lever
2015-01-21 16:03 ` [PATCH v3 09/20] xprtrdma: Remove rpcrdma_ep::rep_func and ::rep_xprt Chuck Lever
2015-01-21 16:03 ` [PATCH v3 10/20] xprtrdma: Free the pd if ib_query_qp() fails Chuck Lever
2015-01-21 16:03 ` [PATCH v3 11/20] xprtrdma: Take struct ib_device_attr off the stack Chuck Lever
2015-01-21 16:03 ` [PATCH v3 12/20] xprtrdma: Take struct ib_qp_attr and ib_qp_init_attr " Chuck Lever
2015-01-21 16:03 ` [PATCH v3 13/20] xprtrdma: Simplify synopsis of rpcrdma_buffer_create() Chuck Lever
2015-01-21 16:03 ` [PATCH v3 14/20] xprtrdma: Refactor rpcrdma_buffer_create() and rpcrdma_buffer_destroy() Chuck Lever
2015-01-21 16:04 ` [PATCH v3 15/20] xprtrdma: Add struct rpcrdma_regbuf and helpers Chuck Lever
2015-01-21 16:04 ` [PATCH v3 16/20] xprtrdma: Allocate RPC send buffer separately from struct rpcrdma_req Chuck Lever
2015-01-21 16:04 ` [PATCH v3 17/20] xprtrdma: Allocate RPC/RDMA " Chuck Lever
2015-01-21 16:04 ` [PATCH v3 18/20] xprtrdma: Allocate RPC/RDMA receive buffer separately from struct rpcrdma_rep Chuck Lever
2015-01-21 16:04 ` [PATCH v3 19/20] xprtrdma: Allocate zero pad separately from rpcrdma_buffer Chuck Lever
2015-01-21 16:04 ` [PATCH v3 20/20] xprtrdma: Clean up after adding regbuf management Chuck Lever
2015-01-30 17:21 ` Anna Schumaker [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54CBBD8C.7090400@Netapp.com \
--to=anna.schumaker@netapp.com \
--cc=chuck.lever@oracle.com \
--cc=linux-nfs@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).