From: Chuck Lever <chuck.lever@oracle.com>
To: Takeshi Nishimura <takeshi.nishimura.linux@gmail.com>,
Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: BUG: Linux 6.12 nfsd does not support FATTR4_WORD2_CHANGE_ATTR_TYPE in NFSv4.2 mode!!
Date: Mon, 13 Jan 2025 08:54:02 -0500 [thread overview]
Message-ID: <dcbb68c5-efbe-4edb-8ba9-677bda178efd@oracle.com> (raw)
In-Reply-To: <CALWcw=GGunhwiuGJpA0HarYNiWCAtx8ku-TVZM8jgfDfwaiwcA@mail.gmail.com>
On 1/12/25 11:06 PM, Takeshi Nishimura wrote:
> On Sat, Jan 11, 2025 at 10:17 PM Rick Macklem <rick.macklem@gmail.com> wrote:
>>
>> On Sat, Jan 11, 2025 at 12:08 PM Takeshi Nishimura
>> <takeshi.nishimura.linux@gmail.com> wrote:
>>>
>>> Dear list,
>>>
>>> We tried to use FATTR4_WORD2_CHANGE_ATTR_TYPE with Linux 6.12 nfsd,
>>> but the server does not set that attribute, while it is mandatory for
>>> NFSv4.2.
>> My understand is that nothing is mandatory in NFSv4.2. Everything is considered
>> optional extensions. I doubt any extant 4.2 server supports all of the optional
>> extensions in NFSv4.2.
>
> That can't be true, or would be a bug in the NFSv4.2 spec then.
> "Everything optional" means feature support gets fragmented, and
> interoperability will cease to exist.
Hello -
Interoperability in this case means that the client is able to determine
whether the server implements a new feature, and then not use it if the
server hasn't implemented it.
There are protocol mechanisms in place to ensure that clients can
recognize that the server doesn't support any feature introduced by
NFSv4.2, so it is entirely safe to make them all optional.
As RFC 7862 points out, when a server lacks support for
FATTR4_CHANGE_ATTR_TYPE, the client is supposed to behave as if the
server returned TYPE_IS_UNDEFINED.
--
Chuck Lever
next prev parent reply other threads:[~2025-01-13 13:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-11 20:08 BUG: Linux 6.12 nfsd does not support FATTR4_WORD2_CHANGE_ATTR_TYPE in NFSv4.2 mode!! Takeshi Nishimura
2025-01-11 21:17 ` Rick Macklem
2025-01-13 4:06 ` Takeshi Nishimura
2025-01-13 12:34 ` Jeff Layton
2025-01-13 13:54 ` Chuck Lever [this message]
2025-01-13 12:27 ` Jeff Layton
2025-01-12 18:30 ` 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=dcbb68c5-efbe-4edb-8ba9-677bda178efd@oracle.com \
--to=chuck.lever@oracle.com \
--cc=linux-nfs@vger.kernel.org \
--cc=takeshi.nishimura.linux@gmail.com \
/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