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-15720 Lustre <p>performance regressions in stripe directory on 2.15.0 (commit;4d93fd7) were found against b2_14(commit:d4b9557).<br/> Here is configuration.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>4 x MDS (1 x MDT per MDS) 4 x OSS (2 x OSS per OSS) 40 x client [root@ec01 ~]# mkdir -p /exafs/d0/d1/d2/mdt_stripe/ [root@ec01 ~]# lfs setdirstripe -c 4 -D /exafs/d0/d1/d2/mdt_stripe/ [root@ec01 ~]# salloc -p 40n -N 40 --ntasks-per-node=16 mpirun --allow-run-as-root -oversubscribe -mca btl_openib_if_include mlx5_1:1 -x UCX_NET_DEVICES=mlx5_1:1 /work/tools/bin/mdtest -n 2000 -F -i 3 -p 10 -v -d /exafs/d0/d1/d2/mdt_stripe/ </pre> </div></div> <p>Here is test resutls.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>server: version=2.15.0_RC2_22_g4d93fd7 client: version=2.15.0_RC2_22_g4d93fd7 SUMMARY rate: (of 3 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 103733.203 76276.410 93728.713 15168.101 File stat 693152.731 656461.448 671671.960 19132.425 File read 259081.462 247951.008 253393.168 5569.308 File removal 145137.390 142142.699 143590.068 1499.846 Tree creation 48.035 1.922 17.475 26.467 Tree removal 35.643 15.861 24.045 10.323 </pre> </div></div> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>server: version=2.14.0_21_gd4b9557 client: version=2.14.0_21_gd4b9557 SUMMARY rate: (of 3 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 138939.425 81336.388 117014.695 31167.261 File stat 1678888.952 1580356.340 1645190.276 56162.463 File read 569731.788 528830.155 546121.363 21170.387 File removal 191837.291 186597.900 188595.661 2832.527 Tree creation 120.108 0.986 51.078 61.778 Tree removal 40.863 33.203 37.987 4.171 </pre> </div></div> <p>As far as I observed this, it seems to be server side regression since because performance with lustre-2.15 clients + lustre-2.14 was ok below.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>server: version=2.14.0_21_gd4b9557 client: version=2.15.0_RC2_22_g4d93fd7 SUMMARY rate: (of 3 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 132009.360 74074.615 106514.108 29585.056 File stat 1570754.679 1457120.401 1532703.082 65457.038 File read 563710.286 540228.432 553871.772 12194.544 File removal 189557.092 186065.253 187536.946 1809.374 Tree creation 54.678 1.883 19.576 30.399 Tree removal 42.065 41.677 41.875 0.194 </pre> </div></div> <p>it seems that the following patch where regressions started.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> LU-14459 lmv: change default hash type to crush Change the default hash type to CRUSH to minimize the number of directory entries that need to be migrated. </pre> </div></div> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>server: version=2.14.51_197_gf269497 client: version=2.15.0_RC2_22_g4d93fd7 SUMMARY rate: (of 3 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 148072.690 87600.145 127000.919 34149.618 File stat 1523849.471 1388808.972 1441253.182 72393.681 File read 562840.721 505515.837 538333.864 29552.364 File removal 197259.873 191117.823 194934.244 3331.372 Tree creation 111.869 1.707 39.426 62.755 Tree removal 44.113 30.518 36.562 6.922 </pre> </div></div> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>server: version=2.14.2.14.51_198_gbb60caa client: version=2.15.0_RC2_22_g4d93fd7 SUMMARY rate: (of 3 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 86531.781 63506.794 72790.003 12142.761 File stat 808075.643 746570.771 784071.104 32898.551 File read 260064.500 249212.881 256291.924 6135.058 File removal 159592.539 155603.788 157752.556 2012.224 Tree creation 120.060 1.138 41.069 68.410 Tree removal 37.780 37.263 37.450 0.287 V-1: Entering PrintTimestamp... </pre> </div></div> <p>I just found MDT load balancing seems to be not working well after patch. It's unbalanced file distribution across MDTs at create. For instance, here is just file creation test in a stripe directory.</p> <p>Before patch (commit:f269497)</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>mpirunp -np 640 mdtest -n 2000 -F -C -i 1 -p 10 -v -d /exafs/d0/d1/d2/mdt_stripe/ [root@ec01 ~]# lfs df -i | grep MDT exafs-MDT0000_UUID 83050496 320298 82730198 1% /exafs[MDT:0] exafs-MDT0001_UUID 83050496 320283 82730213 1% /exafs[MDT:1] exafs-MDT0002_UUID 83050496 320334 82730162 1% /exafs[MDT:2] exafs-MDT0003_UUID 83050496 320293 82730203 1% /exafs[MDT:3] </pre> </div></div> <p>After patch (commit:bb60caa)</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>[root@ec01 ~]# lfs df -i | grep MDT exafs-MDT0000_UUID 83050496 192404 82858092 1% /exafs[MDT:0] exafs-MDT0001_UUID 83050496 190698 82859798 1% /exafs[MDT:1] exafs-MDT0002_UUID 83050496 177266 82873230 1% /exafs[MDT:2] exafs-MDT0003_UUID 83050496 720852 82329644 1% /exafs[MDT:3] </pre> </div></div> <p>That's why mdtest's numbers was slower since one of MDS/MDT (MDT3 in this case) is more working longer than others. Eventually, mdtest's elapsed time is longer than balanced case.</p> LU-15720

imbalanced file creation in 'crush' striped directory

Bug Blocker Resolved Fixed Andreas Dilger Andreas Dilger Tue, 5 Apr 2022 18:13:00 +0000 Fri, 30 Sep 2022 21:36:03 +0000 Mon, 11 Jul 2022 12:53:51 +0000 Lustre 2.15.0 Lustre 2.16.0 0 4 <p>two things about the "temp" filenames:</p> <ul class="alternate" type="square"> <li>files <em>should</em> still be created on same MDT as the "non-temp" filename. However, if the filename is like "<tt>foo.12345678</tt>" then the hashing code will only ever use "<tt>foo</tt>" to determine the "proper" MDT index.</li> <li>the "temp filename" code should <b>not</b> consider suffixes with <b>only</b> numbers as a temp filename. That is specifically to avoid putting all "<tt>foo.nnnnnnnn</tt>" filenames on the same MDT. However, if there is a mix of numbers and letters (e.g. hex suffix?) then it might be doing the wrong thing.</li> </ul> <p>The version of mdtest that I'm using locally only has numbers in the suffix:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre># ls mdtest-easy/test-dir.0-0/mdtest_tree.0.0 file.mdtest.1.127 file.mdtest.1.128 file.mdtest.1.129 file.mdtest.1.13 file.mdtest.1.130 file.mdtest.1.131 file.mdtest.1.132 </pre> </div></div> <p>However, it might be getting confused by the extra '<tt>.</tt>' in the name if there are more files, like "<tt>file.mdtest.1.345678</tt>" or "<tt>file.mdtest.12.45678</tt>"? This would <b>incorrectly</b> fail the "<tt>(digit &gt;= suffixlen -1)</tt>" check because the second '<tt>.</tt>' is not counted in <tt>digit</tt> or <tt>upper</tt> or <tt>lower</tt>. There should probably be an additional check that there aren't non-alphanumeric characters in the suffix:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> <span class="code-keyword">if</span> ((digit &gt;= suffixlen - 1 &amp;&amp; !isdigit(name[namelen - suffixlen])) || upper == suffixlen || lower == suffixlen) <span class="code-keyword">return</span> <span class="code-keyword">false</span>; <span class="code-keyword">if</span> (type == LMV_HASH_TYPE_CRUSH2 &amp;&amp; digit + upper + lower != suffixlen) <span class="code-keyword">return</span> <span class="code-keyword">false</span>; </pre> </div></div> <p>Unfortunately, this changes the hash function subtly, so a new "<tt>LMV_HASH_TYPE_CRUSH2</tt>" hash type is needed for the new behavior. Otherwise, clients may <em>think</em> they know which MDT a particular filename is on but it would be wrong.</p> <p>I'm having a hard time to convince myself this is code correct. The comment in the commit message says:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>LU-13481 dne: improve temp file name check Previously if all but two characters in file name suffix are digit, it's not treated as temp file, as is too strict if suffix length is short, e.g. 6. Change it to allow one character, and this non-digit character should not be the starting character. </pre> </div></div> <p>Besides the problem with ".-_" characters in the suffix (which would make count of digits/upper/lower too small and fail the suffixlen check), it doesn't look like the isdigit() check is correct:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> <span class="code-keyword">if</span> ((digit &gt;= suffixlen - 1 &amp;&amp; !isdigit(name[namelen - suffixlen])) || upper == suffixlen || lower == suffixlen) <span class="code-keyword">return</span> <span class="code-keyword">false</span>; </pre> </div></div> <p>If "digit &gt;= suffixlen -1" (say name = "foo.12345678", digit = 8, suffixlen = 8) this check will fail (and return "true" for the temp filename check) because "1" is a digit. I think this is supposed to be just "isdigit(name[])" (no '!').<br/> Definitely "foo.12345678" should <b>not</b> be considered a temp file, since this is a common case (eg file.YYYYMMDD). The chance of a 6-number temp file is 1/40k, and an 8-number temp file being hit randomly is less than 1/2M. </p> <p>The original code even considered 6/6, 5/6 and 4/6 numbers to not be temp files (ie. "digit &gt;= suffixlen - 2") , but 4/6 numbers was too easily hit by mktemp. It was <b>supposed</b> to keep 8/8 and 7/8 as non-temp files as long as 7/8 was like "file.f1234567". The problem is that the 8/8 case also fails because the first char <b>is</b> a digit, so "<tt>!isdigit(name<span class="error">&#91;namelen-suffixlen&#93;</span>)</tt>" fails, and it doesn't matter if the "<tt>(digit &gt;= suffixlen - 1)</tt>" part is true or not because the "false" check is not met, and "true" is returned. </p> <p>The proper check should be something like:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> <span class="code-keyword">if</span> (digit == suffixlen || upper == suffixlen || lower == suffixlen || (digit == suffixlen - 1 &amp;&amp; !isdigit(name[namelen - suffixlen]))) <span class="code-keyword">return</span> <span class="code-keyword">false</span>; </pre> </div></div> <p>"Andreas Dilger &lt;adilger@whamcloud.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/47015" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47015</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15720" title="imbalanced file creation in &#39;crush&#39; striped directory" class="issue-link" data-issue-key="LU-15720"><del>LU-15720</del></a> dne: add crush2 hash type<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 13296e6231e07ca615d1e709b7c579b4878e1f16</p> <p>Test Configurations</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>1 x MDS(1xMDT, 12 CPU cores, 142GB RAM) 4 x OSS(2xOST/OSS) 40 x client(16 CPU cores, 96GB RAM) IB-HDR100 network [root@ec01 ~]# mkdir -p /exafs/d0/d1/d2/mdt_stripe/ [root@ec01 ~]# lfs setdirstripe -c 4 -D /exafs/d0/d1/d2/mdt_stripe/ </pre> </div></div> <p>2.15.0_RC2_39_g42a6d1f (<a href="https://jira.whamcloud.com/browse/LU-15702" title="(lov_lock.c:206:lov_lock_sub_init()) ASSERTION( (!(result == 0) || (nr == lovlck-&gt;lls_nr)) ) failed: " class="issue-link" data-issue-key="LU-15702"><del>LU-15702</del></a> lov: remove lo_trunc_stripeno)</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>[root@ec01 ~]# mpirun -np 640 mdtest -n 2000 -F -i 1 -p 30 -v -d /exafs/d0/d1/d2/mdt_stripe/ -C SUMMARY rate: (of 1 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 111117.930 111117.930 111117.930 0.000 File stat 0.000 0.000 0.000 0.000 File read 0.000 0.000 0.000 0.000 File removal 0.000 0.000 0.000 0.000 Tree creation 135.910 135.910 135.910 0.000 Tree removal 0.000 0.000 0.000 0.000 [root@ec01 ~]# lfs df -i | grep MDT exafs-MDT0000_UUID 83050496 192396 82858100 1% /exafs[MDT:0] exafs-MDT0001_UUID 83050496 720843 82329653 1% /exafs[MDT:1] exafs-MDT0002_UUID 83050496 177209 82873287 1% /exafs[MDT:2] exafs-MDT0003_UUID 83050496 190695 82859801 1% /exafs[MDT:3] </pre> </div></div> <p>2.15.0_RC2_40_g0090b6f <a href="https://jira.whamcloud.com/browse/LU-15692" title="performance regressions for files in stripe directory" class="issue-link" data-issue-key="LU-15692"><del>LU-15692</del></a> lmv: change default hash back to fnv_1a_64</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>[root@ec01 ~]# mpirun -np 640 mdtest -n 2000 -F -i 1 -p 30 -v -d /exafs/d0/d1/d2/mdt_stripe/ -C SUMMARY rate: (of 1 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 150766.352 150766.352 150766.352 0.000 File stat 0.000 0.000 0.000 0.000 File read 0.000 0.000 0.000 0.000 File removal 0.000 0.000 0.000 0.000 Tree creation 153.942 153.942 153.942 0.000 Tree removal 0.000 0.000 0.000 0.000 [root@ec01 ~]# lfs df -i | grep MDT exafs-MDT0000_UUID 83050496 320296 82730200 1% /exafs[MDT:0] exafs-MDT0001_UUID 83050496 320282 82730214 1% /exafs[MDT:1] exafs-MDT0002_UUID 83050496 320285 82730211 1% /exafs[MDT:2] exafs-MDT0003_UUID 83050496 320283 82730213 1% /exafs[MDT:3] </pre> </div></div> <p>2.15.0_RC3 + <a href="https://review.whamcloud.com/47015" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47015</a></p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>[root@ec01 ~]# mpirun -np 640 mdtest -n 2000 -F -i 1 -p 30 -v -d /exafs/d0/d1/d2/mdt_stripe/ -C SUMMARY rate: (of 1 iterations) Operation Max Min Mean Std Dev --------- --- --- ---- ------- File creation 149746.028 149746.028 149746.028 0.000 File stat 0.000 0.000 0.000 0.000 File read 0.000 0.000 0.000 0.000 File removal 0.000 0.000 0.000 0.000 Tree creation 14.232 14.232 14.232 0.000 Tree removal 0.000 0.000 0.000 0.000 [root@ec01 ~]# lfs df -i | grep MDT exafs-MDT0000_UUID 83050496 320296 82730200 1% /exafs[MDT:0] exafs-MDT0001_UUID 83050496 320289 82730207 1% /exafs[MDT:1] exafs-MDT0002_UUID 83050496 320289 82730207 1% /exafs[MDT:2] exafs-MDT0003_UUID 83050496 320290 82730206 1% /exafs[MDT:3] </pre> </div></div> <p>patch <a href="https://review.whamcloud.com/47015" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47015</a> is working fine for rule of mdtest's files and no performance degradations found. </p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/47015/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47015/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15720" title="imbalanced file creation in &#39;crush&#39; striped directory" class="issue-link" data-issue-key="LU-15720"><del>LU-15720</del></a> dne: add crush2 hash type<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 1ac4b9598ad6e2f94c4c672b4733186364255c6a</p> <p>Landed for 2.16</p> Cloners LU-15692 Related LU-15479 LU-15546 LU-13481 LU-11025 LU-14459 LU-16198 Development Rank 1|i02mjj: Rank (Obsolete) 9223372036854775807 Severity