All of lore.kernel.org
 help / color / mirror / Atom feed
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
> 


WARNING: multiple messages have this Message-ID (diff)
From: Anna Schumaker <Anna.Schumaker-HgOvQuBEEgTQT0dZR+AlfA@public.gmane.org>
To: Chuck Lever <chuck.lever-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.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-FYjufvaPoItvLzlybtyyYzGyq/o6K9yX@public.gmane.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
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2015-01-30 17:21 UTC|newest]

Thread overview: 44+ 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:01 ` Chuck Lever
2015-01-21 16:02 ` [PATCH v3 01/20] xprtrdma: human-readable completion status Chuck Lever
2015-01-21 16:02   ` Chuck Lever
2015-01-21 16:02 ` [PATCH v3 02/20] xprtrdma: Modernize htonl and ntohl Chuck Lever
2015-01-21 16:02   ` 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   ` Chuck Lever
2015-01-21 16:02 ` [PATCH v3 04/20] xprtrdma: Clean up hdrlen Chuck Lever
2015-01-21 16:02   ` 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   ` Chuck Lever
2015-01-21 16:02 ` [PATCH v3 06/20] xprtrdma: Remove rpcrdma_ep::rep_ia Chuck Lever
2015-01-21 16:02   ` 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:02   ` 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   ` 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   ` 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   ` 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   ` 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   ` 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   ` 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:03   ` 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   ` 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   ` Chuck Lever
2015-01-21 16:04 ` [PATCH v3 17/20] xprtrdma: Allocate RPC/RDMA " Chuck Lever
2015-01-21 16:04   ` 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   ` 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   ` Chuck Lever
2015-01-21 16:04 ` [PATCH v3 20/20] xprtrdma: Clean up after adding regbuf management Chuck Lever
2015-01-21 16:04   ` Chuck Lever
2015-01-30 17:21 ` Anna Schumaker [this message]
2015-01-30 17:21   ` [PATCH v3 00/20] NFS/RDMA client for 3.20 Anna Schumaker

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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.