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-6027 Lustre <p>Current b2_5 and master have the following issues with EAs:</p> <ul> <li>Setting a new EA with an empty value, which is permitted according to attr(5), succeeds without actually storing the EA to the OSD. A subsequent getxattr() call will return ENODATA.</li> </ul> <ul> <li>Calling fgetxattr() on an orphan file may get ENOENT, if it causes an EA cache refill.</li> </ul> LU-6027

Issues with EAs of orphan files and EAs with empty values

Bug Minor Resolved Fixed Li Wei Li Wei Mon, 15 Dec 2014 01:39:55 +0000 Wed, 15 Jul 2015 14:18:22 +0000 Thu, 8 Jan 2015 13:49:42 +0000 Lustre 2.7.0 Lustre 2.5.4 Lustre 2.7.0 Lustre 2.5.4 0 7 <p>master</p> <p><a href="http://review.whamcloud.com/12880" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12880</a><br/> <a href="http://review.whamcloud.com/12884" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12884</a><br/> <a href="http://review.whamcloud.com/12900" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12900</a></p> <p>b2_5</p> <p><a href="http://review.whamcloud.com/12944" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12944</a><br/> <a href="http://review.whamcloud.com/12945" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12945</a><br/> <a href="http://review.whamcloud.com/12946" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12946</a></p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12880/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12880/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6027" title="Issues with EAs of orphan files and EAs with empty values" class="issue-link" data-issue-key="LU-6027"><del>LU-6027</del></a> osd-zfs: Preserve lu_buf when listing EAs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 75f0f0675d0eadc634c098d664b7af78547999c9</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12884/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12884/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6027" title="Issues with EAs of orphan files and EAs with empty values" class="issue-link" data-issue-key="LU-6027"><del>LU-6027</del></a> mdd: Don't list "trusted.link" for orphans<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 4991f179f6794d9d64d69fbb39338e738522881a</p> <p>The tests for this issue require attr/xattr.h, part of libattr-dev. Should they simply be added to the dependencies for the default build (with tests enabled)? Here is a proposed change for autoconf to return an error if attr/xattr.h is not present.</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">diff --git a/config/lustre-build.m4 b/config/lustre-build.m4 index da197ba..5fc614e 100644 --- a/config/lustre-build.m4 +++ b/config/lustre-build.m4 @@ -267,6 +267,13 @@ AC_ARG_ENABLE([tests], [disable building of Lustre tests]), [], [enable_tests=<span class="code-quote">"yes"</span>]) AC_MSG_RESULT([$enable_tests]) +AS_IF([test <span class="code-quote">"x$enable_tests"</span> = xyes],[ + AC_CHECK_HEADER([attr/xattr.h],[],[ + AC_MSG_ERROR([\ +lattr is needed <span class="code-keyword">for</span> tests, install libattr-devel (RHEL) or \ +libattr1-dev (Debian)]) +]) +]) ]) # LB_CONFIG_TESTS # </pre> </div></div> <p>Also libattr-dev should be added to lustre.spec.in and libattr-dev1 to debian/control{,.main}. There's really no option besides installing libattr1-dev for Ubuntu, since --disable-tests isn't properly implemented in debian/rules.</p> <p>Does &lt;sys/xattr.h&gt; alone work for these test or is libattr required.. A patch was created for xattr.h issues at <a href="https://jira.whamcloud.com/browse/LU-6027" title="Issues with EAs of orphan files and EAs with empty values" class="issue-link" data-issue-key="LU-6027"><del>LU-6027</del></a>.</p> <p>Yes, patch <a href="http://review.whamcloud.com/13197" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13197</a> fixes the issue.</p> <p>Patch landed to Master.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/12900/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/12900/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6027" title="Issues with EAs of orphan files and EAs with empty values" class="issue-link" data-issue-key="LU-6027"><del>LU-6027</del></a> mdt: Allow user EAs with empty values<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 165f65a7aa07d55d284d419cdaa04eed2d96ade3</p> Related LU-6073 LU-6184 Development Rank 1|hzx2i7: Rank (Obsolete) 16803 Severity