Linux NFS development
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: Devesh Sharma <devesh.sharma@avagotech.com>
Cc: Chuck Lever <chuck.lever@oracle.com>,
	linux-rdma@vger.kernel.org,
	Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v4 00/11] NFS/RDMA server patches for v4.5
Date: Wed, 23 Dec 2015 16:00:15 -0500	[thread overview]
Message-ID: <20151223210015.GA29650@fieldses.org> (raw)
In-Reply-To: <CANjDDBjypu0jX22fZ-Nf-bNNeam2MM0MADaZY9C-o9ihJPiseg@mail.gmail.com>

On Wed, Dec 16, 2015 at 05:40:09PM +0530, Devesh Sharma wrote:
> iozone passed on ocrdma device.

What other testing has there been of this patchset?

Connectathon, xfstests, and pynfs make more of an effort to test corner
cases, iozone isn't much of a test of correctness.

--b.

> Link bounce fails to recover iozone
> traffic, however failure is not related to this patch series. I am in
> processes of finding out the patch which broke it.
> 
> Tested-By: Devesh Sharma <devesh.sharma@avagotech.com>
> 
> On Tue, Dec 15, 2015 at 3:00 AM, Chuck Lever <chuck.lever@oracle.com> wrote:
> > Here are patches to support server-side bi-directional RPC/RDMA
> > operation (to enable NFSv4.1 on RPC/RDMA transports). Thanks to
> > all who reviewed v1, v2, and v3. This version has some significant
> > changes since the previous one.
> >
> > In preparation for Doug's final topic branch, Bruce, I've rebased
> > these on Christoph's ib_device_attr branch. There were some merge
> > conflicts which I've fixed and tested. These are ready for your
> > review.
> >
> > Also available in the "nfsd-rdma-for-4.5" 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/nfsd-rdma-for-4.5
> >
> >
> > Changes since v3:
> > - Rebased on Christoph's ib_device_attr branch
> > - Backchannel patches have been squashed together
> > - Memory allocation overhaul to prevent blocking allocation
> >   when sending backchannel calls
> >
> >
> > Changes since v2:
> > - Rebased on v4.4-rc4
> > - Backchannel code in new source file to address dprintk issues
> > - svc_rdma_get_context() now uses a pre-allocated cache
> > - Dropped svc_rdma_send clean up
> >
> >
> > Changes since v1:
> >
> > - Rebased on v4.4-rc3
> > - Removed the use of CONFIG_SUNRPC_BACKCHANNEL
> > - Fixed computation of forward and backward max_requests
> > - Updated some comments and patch descriptions
> > - pr_err and pr_info converted to dprintk
> > - Simplified svc_rdma_get_context()
> > - Dropped patch removing access_flags field
> > - NFSv4.1 callbacks tested with for-4.5 client
> >
> > ---
> >
> > Chuck Lever (11):
> >       svcrdma: Do not send XDR roundup bytes for a write chunk
> >       svcrdma: Clean up rdma_create_xprt()
> >       svcrdma: Clean up process_context()
> >       svcrdma: Improve allocation of struct svc_rdma_op_ctxt
> >       svcrdma: Improve allocation of struct svc_rdma_req_map
> >       svcrdma: Remove unused req_map and ctxt kmem_caches
> >       svcrdma: Add gfp flags to svc_rdma_post_recv()
> >       svcrdma: Remove last two __GFP_NOFAIL call sites
> >       svcrdma: Make map_xdr non-static
> >       svcrdma: Define maximum number of backchannel requests
> >       svcrdma: Add class for RDMA backwards direction transport
> >
> >
> >  include/linux/sunrpc/svc_rdma.h            |   37 ++-
> >  net/sunrpc/xprt.c                          |    1
> >  net/sunrpc/xprtrdma/Makefile               |    2
> >  net/sunrpc/xprtrdma/svc_rdma.c             |   41 ---
> >  net/sunrpc/xprtrdma/svc_rdma_backchannel.c |  371 ++++++++++++++++++++++++++++
> >  net/sunrpc/xprtrdma/svc_rdma_recvfrom.c    |   52 ++++
> >  net/sunrpc/xprtrdma/svc_rdma_sendto.c      |   34 ++-
> >  net/sunrpc/xprtrdma/svc_rdma_transport.c   |  284 ++++++++++++++++-----
> >  net/sunrpc/xprtrdma/transport.c            |   30 +-
> >  net/sunrpc/xprtrdma/xprt_rdma.h            |   20 +-
> >  10 files changed, 730 insertions(+), 142 deletions(-)
> >  create mode 100644 net/sunrpc/xprtrdma/svc_rdma_backchannel.c
> >
> > --
> > Signature
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2015-12-23 21:00 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-14 21:30 [PATCH v4 00/11] NFS/RDMA server patches for v4.5 Chuck Lever
2015-12-14 21:30 ` [PATCH v4 01/11] svcrdma: Do not send XDR roundup bytes for a write chunk Chuck Lever
2015-12-21 21:07   ` J. Bruce Fields
2015-12-21 21:15     ` Chuck Lever
2015-12-21 21:29       ` J. Bruce Fields
2015-12-21 22:11         ` Chuck Lever
2015-12-23 19:59           ` J. Bruce Fields
2015-12-14 21:30 ` [PATCH v4 02/11] svcrdma: Clean up rdma_create_xprt() Chuck Lever
2015-12-14 21:30 ` [PATCH v4 03/11] svcrdma: Clean up process_context() Chuck Lever
2015-12-14 21:30 ` [PATCH v4 04/11] svcrdma: Improve allocation of struct svc_rdma_op_ctxt Chuck Lever
2015-12-14 21:30 ` [PATCH v4 05/11] svcrdma: Improve allocation of struct svc_rdma_req_map Chuck Lever
2015-12-14 21:30 ` [PATCH v4 06/11] svcrdma: Remove unused req_map and ctxt kmem_caches Chuck Lever
2015-12-14 21:30 ` [PATCH v4 07/11] svcrdma: Add gfp flags to svc_rdma_post_recv() Chuck Lever
2015-12-14 21:31 ` [PATCH v4 08/11] svcrdma: Remove last two __GFP_NOFAIL call sites Chuck Lever
2015-12-14 21:31 ` [PATCH v4 09/11] svcrdma: Make map_xdr non-static Chuck Lever
2015-12-14 21:31 ` [PATCH v4 10/11] svcrdma: Define maximum number of backchannel requests Chuck Lever
2015-12-14 21:31 ` [PATCH v4 11/11] svcrdma: Add class for RDMA backwards direction transport Chuck Lever
2015-12-16 12:10 ` [PATCH v4 00/11] NFS/RDMA server patches for v4.5 Devesh Sharma
2015-12-23 21:00   ` J. Bruce Fields [this message]
2015-12-24  9:57     ` Chuck Lever

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=20151223210015.GA29650@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=chuck.lever@oracle.com \
    --cc=devesh.sharma@avagotech.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