linux-kernel - Re: [PATCH 08/12] nfsd: allow parallel creates from nfsd

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <165645779028.15378.2009203210771986783@noble.neil.brown.name>
Date:   Wed, 29 Jun 2022 09:09:50 +1000
From:   "NeilBrown" <neilb@...e.de>
To:     "Chuck Lever III" <chuck.lever@...cle.com>
Cc:     "Al Viro" <viro@...iv.linux.org.uk>,
        "Daire Byrne" <daire@...g.com>,
        "Trond Myklebust" <trond.myklebust@...merspace.com>,
        "Linux NFS Mailing List" <linux-nfs@...r.kernel.org>,
        "linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
        "LKML" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 08/12] nfsd: allow parallel creates from nfsd

On Wed, 29 Jun 2022, Chuck Lever III wrote:
> 
> > On Jun 13, 2022, at 7:18 PM, NeilBrown <neilb@...e.de> wrote:
> > diff --git a/fs/nfsd/nfsfh.c b/fs/nfsd/nfsfh.c
> > index c29baa03dfaf..a50db688c60d 100644
> > --- a/fs/nfsd/nfsfh.c
> > +++ b/fs/nfsd/nfsfh.c
> > @@ -616,7 +616,7 @@ fh_update(struct svc_fh *fhp)
> >  * @fhp: file handle to be updated
> >  *
> >  */
> > -void fh_fill_pre_attrs(struct svc_fh *fhp)
> > +void fh_fill_pre_attrs(struct svc_fh *fhp, bool atomic)
> 
> Hi Neil, just noticed this:
> 
>   CC [M]  fs/nfsd/nfsfh.o
>   CHECK   /home/cel/src/linux/linux/fs/nfsd/nfsfh.c
> /home/cel/src/linux/linux/fs/nfsd/nfsfh.c:621: warning: Function parameter or member 'atomic' not described in 'fh_fill_pre_attrs'

Thanks.  I"ll address that, and also the other issues that you raised in
your patch-by-patch review.  Thanks for those.

> 
> And... do you intend to repost this series with the supplemental
> fixes applied?

I've been a bit distracted this week, but my current plan is to
reorganise the patches to put as many of the NFS and NFSD patches as
possible before the VFS patches.  There should then just be one each for
NFS and NFSD after the VFS changes.  I hope to post that series early
next week.

> 
> Should we come up with a plan to merge these during the next
> window, or do you feel more work is needed?

I think it would be reasonable to merge the preliminary NFS and NFSD
patches in the next window.  I'd really like to hear from Al before
pushing the rest too hard.  Probably after rc1 I'll post the remainder
of the series and include Linus if we haven't heard from Al.  I'd be
perfectly happy if the main content of the series landed in the
subsequent merge window.

Thanks,
NeilBrown