Re: [PATCH v2 0/7] XDR overhaul of NFS callback service

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On Aug 10, 2021, at 2:01 PM, Anna Schumaker <schumaker.anna@xxxxxxxxx> wrote:
> 
> Hi Chuck,
> 
> On Fri, Jul 30, 2021 at 3:53 PM Chuck Lever III <chuck.lever@xxxxxxxxxx> wrote:
>> 
>> Hi Trond-
>> 
>>> On Jul 15, 2021, at 3:52 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> 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?

I've dropped them.


> 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

--
Chuck Lever







[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux