[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <0A08512F-E081-4451-976F-498611D9365E@oracle.com>
Date: Mon, 4 Jul 2022 17:17:16 +0000
From: Chuck Lever III <chuck.lever@...cle.com>
To: Neil Brown <neilb@...e.de>
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 Jun 28, 2022, at 7:09 PM, NeilBrown <neilb@...e.de> wrote:
>
> 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.
Please base your patches on the "for-next" branch in this repo:
https://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git
Thanks!
--
Chuck Lever
Powered by blists - more mailing lists