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-13486 Lustre <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> Lustre: DEBUG MARKER: == sanity test 32c: stat d32c/ext2-mountpoint/../d2/test_dir ========================================= 12:58:57 (1587715137) striped dir -i0 -c2 -H fnv_1a_64 /mnt/lustre/d32c.sanity/ext2-mountpoint EXT2-fs (loop4): warning: mounting ext3 filesystem as ext2 striped dir -i0 -c2 -H all_char /mnt/lustre/d32c.sanity/d2/test_dir /mnt/lustre/d32c.sanity/ext2-mountpoint/../d2/test_dir has type dir OK umount: /mnt/lustre/d32c.sanity/ext2-mountpoint: not mounted. losetup: option requires an argument -- <span class="code-quote">'d'</span> Try <span class="code-quote">'losetup --help'</span> <span class="code-keyword">for</span> more information. sanity test_32c: @@@@@@ FAIL: test_32c failed with 32 </pre> </div></div> <p>this is because any access to ext2-mountpoint denty causes denty invalidation and umount</p> <p>introduced with: a36d9c3911 2020-04-02 | <a href="https://jira.whamcloud.com/browse/LU-9868" title="dcache/namei fixes for lustre" class="issue-link" data-issue-key="LU-9868">LU-9868</a> llite: Get rid of ll_dcompare </p> linux-4.18.0-32.el8 LU-13486

tests sanity/32* fail

Bug Minor Resolved Fixed Andreas Dilger Alex Zhuravlev Fri, 24 Apr 2020 08:07:52 +0000 Wed, 17 Feb 2021 22:39:28 +0000 Thu, 14 May 2020 23:09:16 +0000 Upstream Lustre 2.14.0 0 5 <p>James or Neil, could you please take a look at this? It looks like a regression from patch <a href="https://review.whamcloud.com/24175" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/24175</a> "<tt><a href="https://jira.whamcloud.com/browse/LU-9868" title="dcache/namei fixes for lustre" class="issue-link" data-issue-key="LU-9868">LU-9868</a> llite: Get rid of ll_dcompare</tt>".</p> <p>The problem here is the parent of ext2-mountpoint gets invalidated.  e.g. 'd32c.sanity' in the root lustre directory.</p> <p>When that gets invalidated, it isn't a mountpoint so ll_revalidate_dentry can return 0, and then the whole directory tree from there down is invalidated.</p> <p>ll_revalidate_dentry needs to actually test if the dentry is valid or not.  It cannot just return "Maybe it is invalid, maybe not".</p> <p>I suspect that if it finds lld_invalid to be true, it needs to talk to the MDS -  do a 'lookup' and get the relevant lock.</p> <p>But probably we need to revert that patch for now.</p> <p> </p> <p>Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/38498" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38498</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13486" title="tests sanity/32* fail" class="issue-link" data-issue-key="LU-13486"><del>LU-13486</del></a> llite: restore ll_dcompare()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 291725afbdb6afc0f1fcbe2c655543ed62ffbc00</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38498/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38498/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13486" title="tests sanity/32* fail" class="issue-link" data-issue-key="LU-13486"><del>LU-13486</del></a> llite: restore ll_dcompare()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 00fa5e8ef45592fbf2a783a1985dc9fd989f419b</p> <p>Landed for 2.14</p> Related LU-9868 Development Rank 1|i00ysf: Rank (Obsolete) 9223372036854775807 Severity