Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <162637843471.728653.5920517086867549998.stgit@manet.1015granger.net>
 <02B12E59-E014-4CF6-9A5B-58E5F426F964@oracle.com>
In-Reply-To: <02B12E59-E014-4CF6-9A5B-58E5F426F964@oracle.com>
From:   Anna Schumaker <schumaker.anna@gmail.com>
Date:   Tue, 10 Aug 2021 14:01:44 -0400
Message-ID: <CAFX2JfkSG5Qe4_svtunwRhMvwdN=bNP0VtuSFtU6siDihjWpZA@mail.gmail.com>
Subject: Re: [PATCH v2 0/7] XDR overhaul of NFS callback service
To:     Chuck Lever III <chuck.lever@oracle.com>
Cc:     Trond Myklebust <trondmy@hammerspace.com>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Chuck,

On Fri, Jul 30, 2021 at 3:53 PM Chuck Lever III <chuck.lever@oracle.com> wrote:
>
> Hi Trond-
>
> > On Jul 15, 2021, at 3:52 PM, Chuck Lever <chuck.lever@oracle.com> wrote:
> >
> > Trond, please let me know if you want to take these or if I may
> > handle them through the NFSD tree for v5.15. Thanks.
>
> I've included these in the NFSD for-next topic branch:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git/commit/?h=for-next
>
> They can be removed if you would like to take them through
> your tree instead.

These look to be mostly client-side changes, so it would make sense to
take them through the NFS tree. Would that cause problems now that
they've been in your for-next branch for a while?

Anna

>
> If I am to take these, Bruce and I would like an Acked-by:
> from you.
>
>
> > The purpose of this series is to prepare for the optimization of
> > svc_process_common() to handle NFSD workloads more efficiently. In
> > other words, NFSD should be the lubricated common case, and callback
> > is the use case that takes exceptional paths.
> >
> > Changes since RFC:
> > - Removed RQ_DROPME test from nfs_callback_dispatch()
> > - Restored .pc_encode call-outs to prevent dropped replies
> > - Fixed whitespace damage
> >
> > ---
> >
> > Chuck Lever (7):
> >      SUNRPC: Add svc_rqst::rq_auth_stat
> >      SUNRPC: Set rq_auth_stat in the pg_authenticate() callout
> >      SUNRPC: Eliminate the RQ_AUTHERR flag
> >      NFS: Add a private local dispatcher for NFSv4 callback operations
> >      NFS: Remove unused callback void decoder
> >      NFS: Extract the xdr_init_encode/decode() calls from decode_compound
> >      NFS: Clean up the synopsis of callback process_op()
> >
> >
> > fs/lockd/svc.c                    |  2 +
> > fs/nfs/callback.c                 |  4 ++
> > fs/nfs/callback_xdr.c             | 61 ++++++++++++++++---------------
> > include/linux/sunrpc/svc.h        |  3 +-
> > include/linux/sunrpc/svcauth.h    |  4 +-
> > include/trace/events/sunrpc.h     |  9 ++---
> > net/sunrpc/auth_gss/svcauth_gss.c | 47 +++++++++++++-----------
> > net/sunrpc/svc.c                  | 39 ++++++--------------
> > net/sunrpc/svcauth.c              |  8 ++--
> > net/sunrpc/svcauth_unix.c         | 18 +++++----
> > 10 files changed, 96 insertions(+), 99 deletions(-)
> >
> > --
> > Chuck Lever
> >
>
> --
> Chuck Lever
>
>
>