From: Anna Schumaker <Anna.Schumaker@netapp.com>
To: Chuck Lever <chuck.lever@oracle.com>,
<linux-rdma@vger.kernel.org>, <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v4 00/16] NFS/RDMA client side for Linux 4.3
Date: Thu, 6 Aug 2015 10:28:01 -0400 [thread overview]
Message-ID: <55C36EF1.2040905@Netapp.com> (raw)
In-Reply-To: <20150803165807.9115.23842.stgit@manet.1015granger.net>
Hi Chuck,
I've applied these patches locally without a problem (and I made sure to remove the pr_warns() in the last patch). Are you planning to submit a v5, or are these ready for me now?
Thanks,
Anna
On 08/03/2015 01:02 PM, Chuck Lever wrote:
> Several important client-side performance and scalability
> improvements are made in this series, proposed for the 4.3
> kernel, including:
>
> - Increase maximum RPC/RDMA credits to 128
> - Increase maximum NFS/RDMA r/wsize to one megabyte
> - Prefer inline rather than reply chunk replies
>
> And these fixes:
>
> - Send NFSv4 WRITE compounds correctly
> - Support RDMA_NOMSG calls
> - Remove support for RDMA_MSGP calls
> - Fix large NFS symlink operations
> - Get inline threshold accounting right
>
> Also available in the "nfs-rdma-for-4.3" topic branch of this git repo:
>
> git://git.linux-nfs.org/projects/cel/cel-2.6.git
>
> Or for browsing:
>
> http://git.linux-nfs.org/?p=cel/cel-2.6.git;a=log;h=refs/heads/nfs-rdma-for-4.3
>
>
> Changes since v3:
> - Rebased on Linux v4.2-rc5
> - Added v2 of Devesh's reboot crash fix
> - Clarified description of NFSv4 WRITE fix
>
>
> Changes since v2:
> - Rebased on Linux v4.2-rc3
> - Corrected RPCRDMA_MAX_IOVS macro
> - Included patch to remove core ib_reg_phys_mr() API
>
>
> Changes since v1:
> - Rebased on Linux v4.2-rc2
> - PHYSICAL registration, being insecure, must now be explicitly selected
> - Further clean-ups were done because ib_reg_phys_mr() is gone
> - Support for RDMA_MSGP type calls has been removed
> - Some patch descriptions have been clarified
>
> ---
>
> Chuck Lever (15):
> xprtrdma: Make xprt_setup_rdma() agnostic to family of server address
> xprtrdma: Raise maximum payload size to one megabyte
> xprtrdma: Increase default credit limit
> xprtrdma: Don't fall back to PHYSICAL memory registration
> xprtrdma: Remove last ib_reg_phys_mr() call site
> xprtrdma: Clean up rpcrdma_ia_open()
> xprtrdma: Remove logic that constructs RDMA_MSGP type calls
> xprtrdma: Account for RPC/RDMA header size when deciding to inline
> xprtrdma: Always provide a write list when sending NFS READ
> xprtrdma: Don't provide a reply chunk when expecting a short reply
> xprtrdma: Fix XDR tail buffer marshalling
> xprtrdma: Fix large NFS SYMLINK calls
> xprtrdma: Clean up xprt_rdma_print_stats()
> xprtrdma: Count RDMA_NOMSG type calls
> core: Remove the ib_reg_phys_mr() and ib_rereg_phys_mr() verbs
>
> Devesh Sharma (1):
> xprtrdma: take HCA driver refcount at client
>
>
> drivers/infiniband/core/verbs.c | 67 ----------
> fs/nfs/nfs3xdr.c | 1
> fs/nfs/nfs4xdr.c | 4 -
> include/linux/sunrpc/xprtrdma.h | 2
> include/rdma/ib_verbs.h | 46 -------
> net/sunrpc/xprtrdma/fmr_ops.c | 19 +++
> net/sunrpc/xprtrdma/frwr_ops.c | 5 +
> net/sunrpc/xprtrdma/physical_ops.c | 25 ++++
> net/sunrpc/xprtrdma/rpc_rdma.c | 197 +++++++++++++++---------------
> net/sunrpc/xprtrdma/transport.c | 77 +++++-------
> net/sunrpc/xprtrdma/verbs.c | 238 +++++++++++++-----------------------
> net/sunrpc/xprtrdma/xprt_rdma.h | 27 ++--
> 12 files changed, 285 insertions(+), 423 deletions(-)
>
> --
> Chuck Lever
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
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>,
linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v4 00/16] NFS/RDMA client side for Linux 4.3
Date: Thu, 6 Aug 2015 10:28:01 -0400 [thread overview]
Message-ID: <55C36EF1.2040905@Netapp.com> (raw)
In-Reply-To: <20150803165807.9115.23842.stgit-FYjufvaPoItvLzlybtyyYzGyq/o6K9yX@public.gmane.org>
Hi Chuck,
I've applied these patches locally without a problem (and I made sure to remove the pr_warns() in the last patch). Are you planning to submit a v5, or are these ready for me now?
Thanks,
Anna
On 08/03/2015 01:02 PM, Chuck Lever wrote:
> Several important client-side performance and scalability
> improvements are made in this series, proposed for the 4.3
> kernel, including:
>
> - Increase maximum RPC/RDMA credits to 128
> - Increase maximum NFS/RDMA r/wsize to one megabyte
> - Prefer inline rather than reply chunk replies
>
> And these fixes:
>
> - Send NFSv4 WRITE compounds correctly
> - Support RDMA_NOMSG calls
> - Remove support for RDMA_MSGP calls
> - Fix large NFS symlink operations
> - Get inline threshold accounting right
>
> Also available in the "nfs-rdma-for-4.3" topic branch of this git repo:
>
> git://git.linux-nfs.org/projects/cel/cel-2.6.git
>
> Or for browsing:
>
> http://git.linux-nfs.org/?p=cel/cel-2.6.git;a=log;h=refs/heads/nfs-rdma-for-4.3
>
>
> Changes since v3:
> - Rebased on Linux v4.2-rc5
> - Added v2 of Devesh's reboot crash fix
> - Clarified description of NFSv4 WRITE fix
>
>
> Changes since v2:
> - Rebased on Linux v4.2-rc3
> - Corrected RPCRDMA_MAX_IOVS macro
> - Included patch to remove core ib_reg_phys_mr() API
>
>
> Changes since v1:
> - Rebased on Linux v4.2-rc2
> - PHYSICAL registration, being insecure, must now be explicitly selected
> - Further clean-ups were done because ib_reg_phys_mr() is gone
> - Support for RDMA_MSGP type calls has been removed
> - Some patch descriptions have been clarified
>
> ---
>
> Chuck Lever (15):
> xprtrdma: Make xprt_setup_rdma() agnostic to family of server address
> xprtrdma: Raise maximum payload size to one megabyte
> xprtrdma: Increase default credit limit
> xprtrdma: Don't fall back to PHYSICAL memory registration
> xprtrdma: Remove last ib_reg_phys_mr() call site
> xprtrdma: Clean up rpcrdma_ia_open()
> xprtrdma: Remove logic that constructs RDMA_MSGP type calls
> xprtrdma: Account for RPC/RDMA header size when deciding to inline
> xprtrdma: Always provide a write list when sending NFS READ
> xprtrdma: Don't provide a reply chunk when expecting a short reply
> xprtrdma: Fix XDR tail buffer marshalling
> xprtrdma: Fix large NFS SYMLINK calls
> xprtrdma: Clean up xprt_rdma_print_stats()
> xprtrdma: Count RDMA_NOMSG type calls
> core: Remove the ib_reg_phys_mr() and ib_rereg_phys_mr() verbs
>
> Devesh Sharma (1):
> xprtrdma: take HCA driver refcount at client
>
>
> drivers/infiniband/core/verbs.c | 67 ----------
> fs/nfs/nfs3xdr.c | 1
> fs/nfs/nfs4xdr.c | 4 -
> include/linux/sunrpc/xprtrdma.h | 2
> include/rdma/ib_verbs.h | 46 -------
> net/sunrpc/xprtrdma/fmr_ops.c | 19 +++
> net/sunrpc/xprtrdma/frwr_ops.c | 5 +
> net/sunrpc/xprtrdma/physical_ops.c | 25 ++++
> net/sunrpc/xprtrdma/rpc_rdma.c | 197 +++++++++++++++---------------
> net/sunrpc/xprtrdma/transport.c | 77 +++++-------
> net/sunrpc/xprtrdma/verbs.c | 238 +++++++++++++-----------------------
> net/sunrpc/xprtrdma/xprt_rdma.h | 27 ++--
> 12 files changed, 285 insertions(+), 423 deletions(-)
>
> --
> Chuck Lever
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
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
next prev parent reply other threads:[~2015-08-06 14:28 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-03 17:02 [PATCH v4 00/16] NFS/RDMA client side for Linux 4.3 Chuck Lever
2015-08-03 17:02 ` Chuck Lever
2015-08-03 17:02 ` [PATCH v4 01/16] xprtrdma: Make xprt_setup_rdma() agnostic to family of server address Chuck Lever
2015-08-03 17:02 ` Chuck Lever
2015-08-03 17:02 ` [PATCH v4 02/16] xprtrdma: Raise maximum payload size to one megabyte Chuck Lever
2015-08-03 17:02 ` Chuck Lever
2015-08-03 17:02 ` [PATCH v4 03/16] xprtrdma: Increase default credit limit Chuck Lever
2015-08-03 17:02 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 04/16] xprtrdma: Don't fall back to PHYSICAL memory registration Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 05/16] xprtrdma: Remove last ib_reg_phys_mr() call site Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 06/16] xprtrdma: Clean up rpcrdma_ia_open() Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 07/16] xprtrdma: Remove logic that constructs RDMA_MSGP type calls Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 08/16] xprtrdma: Account for RPC/RDMA header size when deciding to inline Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:03 ` [PATCH v4 09/16] xprtrdma: Always provide a write list when sending NFS READ Chuck Lever
2015-08-03 17:03 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 10/16] xprtrdma: Don't provide a reply chunk when expecting a short reply Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 11/16] xprtrdma: Fix XDR tail buffer marshalling Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 12/16] xprtrdma: Fix large NFS SYMLINK calls Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 13/16] xprtrdma: Clean up xprt_rdma_print_stats() Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 14/16] xprtrdma: Count RDMA_NOMSG type calls Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:04 ` [PATCH v4 15/16] core: Remove the ib_reg_phys_mr() and ib_rereg_phys_mr() verbs Chuck Lever
2015-08-03 17:04 ` Chuck Lever
2015-08-03 17:05 ` [PATCH v4 16/16] xprtrdma: take HCA driver refcount at client Chuck Lever
2015-08-03 17:05 ` Chuck Lever
2015-08-03 17:07 ` Chuck Lever
2015-08-03 17:07 ` Chuck Lever
2015-08-06 14:28 ` Anna Schumaker [this message]
2015-08-06 14:28 ` [PATCH v4 00/16] NFS/RDMA client side for Linux 4.3 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=55C36EF1.2040905@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.