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-10276 Lustre <p><tt>MDS_INODELOCK_XATTR</tt> has been protecting <em>all</em> EAs of an inode since its introduction for client-side EA caching. Because some EAs are used internally, the set of procedures contending for <tt>MDS_INODELOCK_XATTR</tt> has went beyond <tt>SETXATTR, GETXATTR</tt>. A few examples:</p> <ul> <li><tt>LINK</tt>, <tt>UNLINK</tt>, and <tt>RENAME</tt> take <tt>MDS_INODELOCK_XATTR</tt> for <tt>XATTR_NAME_LINK</tt>.</li> <li><tt>HSM_STATE_SET</tt>, <tt>mdt_hsm_release()</tt>, <tt>mdt_add_dirty_flag()</tt> take <tt>MDS_INODELOCK_XATTR</tt> for <tt>XATTR_NAME_HSM</tt>.</li> </ul> <p>Some of these may have an unnecessary negative effect on client-side EA caching efficiency.</p> <p>We might want to consider removing internal EAs from the scope of <tt>MDS_INODELOCK_XATTR</tt>. Potential questions include:</p> <ul> <li>Clients wouldn't be able to do <tt>listxattr()</tt> completely from EA caches. Would this affect any applications we care about?</li> <li>How to handle older clients who assume <tt>MDS_INODELOCK_XATTR</tt> protects all EAs?</li> </ul> LU-10276

Revise what MDS_INODELOCK_XATTR protects

Improvement Minor Open Unresolved WC Triage Li Wei Fri, 6 Mar 2015 03:10:02 +0000 Sat, 28 Oct 2023 07:46:51 +0000 0 5 <p>It seems possible to store "<tt>xe_value = ERR_PTR(-EREMOTE)</tt>" into the xattr cache to indicate that the xattr exists but the value is not in the cache. That would at least allow <tt>listxattr()</tt> from the client cache to work without having to cache all of the values.</p> <p>I've also filed <a href="https://jira.whamcloud.com/browse/LU-17238" title="negative xattr cache on client" class="issue-link" data-issue-key="LU-17238">LU-17238</a> to implement a proper negative xattr cache that would store "<tt>xe_value = ERR_PTR(-ENODATA)</tt>" to indicate that the requested name does not exist.</p> Related LU-2869 LU-17238 Development Rank 1|hzx7tz: Rank (Obsolete) 17754