<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 21, 2020 at 10:14 AM Murphy Zhou <<a href="mailto:jencce.kernel@gmail.com">jencce.kernel@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Li,<br>
<br>
On Thu, Aug 20, 2020 at 02:16:07PM +0800, Li Wang wrote:<br>
> Hi Murphy,<br>
> <br>
> On Thu, Aug 20, 2020 at 9:52 AM Murphy Zhou <<a href="mailto:jencce.kernel@gmail.com" target="_blank">jencce.kernel@gmail.com</a>> wrote:<br>
> <br>
> > Instead of TBROK, probably the underlying filesystem not support this flag.<br>
> ><br>
> <br>
> Which filesystem do you mean here? What kind of error it was given?<br>
> Maybe we should confirm the errno is EINVAL or ENOTSUP before TCONF.<br>
<br>
NFS, with latest upstream kernel.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It sounds like a new issue appears in the mainline kernel testing.</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Like:<br>
<br>
tst_test.c:1247: INFO: Timeout per run is 0h 15m 00s<br>
setxattr03.c:142: BROK: Set setxattr03immutable immutable failed: ENOTTY (25)<br>
<span class="gmail_default" style="font-size:small"></span>setxattr03.c:154: WARN: Unset setxattr03immutable immutable failed: ENOTTY (25)<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px">I noticed the </span><span class="gmail_default"></span>setxattr03<span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px"> skipped on kernel v5.8-rc6, which means the newer kernel</span></div><div class="gmail_default" style="font-size:small">starts<span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px"> to support xattr in </span>NFSv4.2<span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px">? if so, shouldn't we check why it returns ENOTTY</span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px">but not ENOTSUP in failed setxflags?</span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px"><br></span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-family:"Liberation Sans","Helvetica Neue",Helvetica,Arial,sans-serif;font-size:14px">(btw, I assumed you guys did NOT change the mount way with </span>user_xattr option)</div>   </div><div><span class="gmail_default" style="font-size:small">    </span>Kernel Version: 5.8.0-rc6+<br>    setxattr03.c:135: CONF: No xattr support in fs or fs mounted without user_xattr option<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The reason not checking errno is that `chattr +i` is a very basic operation,<br>
if it fails, most probably it's not supported. NFS returning ENOTTY<br>
makes checking errno more tricky. So I did not check it and made it as<br>
simple as the checking in xfstests.<br>
<br>
Thanks very much for reviewing!<br>
<br>
> <br>
> -- <br>
> Regards,<br>
> Li Wang<br>
<br>
-- <br>
Murphy<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>