https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-16085 Lustre <p>This issue was created by maloo for sarah &lt;sarah@whamcloud.com&gt;</p> <p>This issue relates to the following test suite run: <a href="https://testing.whamcloud.com/test_sets/d93ea6cd-9f2e-4488-84c9-da7169aaac3c" class="external-link" target="_blank" rel="nofollow noopener">https://testing.whamcloud.com/test_sets/d93ea6cd-9f2e-4488-84c9-da7169aaac3c</a></p> <p>test_106c failed with the following error:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>supported stx_attributes: got '2030', expected '30' </pre> </div></div> <p>It is similar as <a href="https://jira.whamcloud.com/browse/LU-13628" title="sanityn test_106a test_106c: supported stx_attributes: got &#39;&#39;?&#39;&#39;, expected &#39;&#39;30&#39;&#39;" class="issue-link" data-issue-key="LU-13628"><del>LU-13628</del></a> but this is with Ubuntu2204 client</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>== sanityn test 106c: Verify statx attributes mask ======= 11:30:40 (1660044640) sanityn test_106c: @@@@@@ FAIL: supported stx_attributes: got '2030', expected '30' </pre> </div></div> <p>VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV<br/> sanityn test_106c - supported stx_attributes: got '2030', expected '30'</p> LU-16085

Ubuntu 22.04 sanityn test_106c: supported stx_attributes: got '2030', expected '30'

Bug Minor Resolved Fixed Sebastien Buisson Maloo Tue, 9 Aug 2022 19:10:14 +0000 Fri, 23 Sep 2022 16:34:45 +0000 Thu, 1 Sep 2022 15:11:38 +0000 Lustre 2.15.1 Lustre 2.16.0 Lustre 2.15.2 0 4 <p>It looks like this is a new addition to the <tt>statx()</tt> call since commit v5.7-rc4-9-g80340fe3605c (Ubuntu 22.04 using 5.15.0-39) that adds:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> #define STATX_ATTR_MOUNT_ROOT 0x00002000 <span class="code-comment">/* Root of a mount */</span> </pre> </div></div> <p>That said, I'm not sure why <tt>STATX_ATTR_MOUNT_ROOT</tt> would be set on a <b>file</b>?</p> <p>so the test should be changed to expect <b>at least</b> the 0x30 flag:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> # Mask supported in stx_attributes by Lustre is # STATX_ATTR_IMMUTABLE(0x10) | STATX_ATTR_APPEND(0x20) : (0x30). (( 0x$mask &amp; 0x30 == 0x30 )) || error <span class="code-quote">"supported stx_attributes: got <span class="code-quote">'0x$mask'</span>, expected <span class="code-quote">'0x30'</span> at least"</span> </pre> </div></div> <p>It looks like we should add <tt>STATX_ATTR_ENCRYPTED</tt> to the list also. This should be set in the <tt>stat-&gt;attributes_mask</tt> in <tt>ll_getattr()</tt> if those flags are even <em>possible</em> for the filesystem, and then in the returned <tt>stat-&gt;attributes</tt> if the flags are actually <em>set</em> on the inode. It isn't totally clear what to do on a client where the server does not support fscrypt, but I'd guess we leave out the <tt>STATX_ATTR_ENCRYPTED</tt> flag in that case? The same will need to be done for <tt>STATX_ATTR_COMPRESSED</tt> in the future.</p> <p>The <tt>ll_getattr()</tt> code <b>also</b> can return (via <tt>ll_inode_to_ext_flags()</tt>) the <tt>LUSTRE_SYNC_FL</tt> and <tt>LUSTRE_DIRSYNC_FL</tt>, so I'm wondering if those should also be returned in <tt>stat-&gt;attributes_mask</tt>, even though they are not defined for <tt>statx</tt>, or if <tt>stat-&gt;attributes</tt> should be masked out by the defined <tt>STATX_ATTR_&#42;</tt> values in <tt>attributes_mask</tt> to avoid future conflicts?</p> <p>"Sebastien Buisson &lt;sbuisson@ddn.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/48208" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/48208</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16085" title="Ubuntu 22.04 sanityn test_106c: supported stx_attributes: got &#39;2030&#39;, expected &#39;30&#39;" class="issue-link" data-issue-key="LU-16085"><del>LU-16085</del></a> llite: fix stat attributes_mask<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5369d4ecd1e081f48098d153a70b58da807d192a</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/48208/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/48208/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16085" title="Ubuntu 22.04 sanityn test_106c: supported stx_attributes: got &#39;2030&#39;, expected &#39;30&#39;" class="issue-link" data-issue-key="LU-16085"><del>LU-16085</del></a> llite: fix stat attributes_mask<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 0e48653c27eacad29dbff1589da771ad4f5d1014</p> <p>"Sebastien Buisson &lt;sbuisson@ddn.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/48404" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/48404</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16085" title="Ubuntu 22.04 sanityn test_106c: supported stx_attributes: got &#39;2030&#39;, expected &#39;30&#39;" class="issue-link" data-issue-key="LU-16085"><del>LU-16085</del></a> llite: fix stat attributes_mask<br/> Project: fs/lustre-release<br/> Branch: b2_15<br/> Current Patch Set: 1<br/> Commit: 85ad0d58b0a0e101e89ab36344267e0392f74e23</p> <p>Landed for 2.16</p> Related LU-10026 LU-12275 LU-13717 LU-12480 Development Rank 1|i02wof: Rank (Obsolete) 9223372036854775807 Severity