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-14736 Lustre <p>leak-finder.pl is checking for leaks, it's not an <b>error</b> in the script if it finds some.</p> <p>The main effect is to make it more annoying to redirect the script output.  (It also prints some basic status information to stderr, for which there is no good reason at all.)</p> LU-14736

leak-finder.pl should not print to STDERR

Improvement Minor Resolved Fixed Andreas Dilger Patrick Farrell Sat, 5 Jun 2021 21:08:44 +0000 Wed, 22 Mar 2023 01:52:11 +0000 Wed, 22 Mar 2023 01:52:11 +0000 Lustre 2.16.0 0 4 <p>Patrick Farrell (farr0186@gmail.com) uploaded a new patch: <a href="https://review.whamcloud.com/43934" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/43934</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> utils: Change leak_check to use stdout<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 983cadac1137cad6e0376e61fb80ecfa2b52fc9e</p> <p>Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/43983" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/43983</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> utils: update leak-finder.pl for new format<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: ae84ad355bb350924fb1963256a360c6f117ab2e</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/43934/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/43934/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> utils: Change leak_finder to use stdout<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 975b944b815412baef344f58a8325423b5d95e1e</p> <p>I improved the patch at <a href="https://review.whamcloud.com/43983" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/43983</a> a little by adding two new options:</p> <ul> <li>--by_func: sort the leak logs by function name in ASC order;</li> <li>--summary: print a summary report by the number of total leak bytes of each function in ASC order. The output is like <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>------------------------------------------------------------------------------------------------------------- function | each_leak_bytes | leak_times | total_leak_bytes -----------------------------------------------------------|-----------------|------------|------------------ cfs_hash_buckets_realloc:893:(new_bkts) | 8 | 1 | 8 ptlrpcd_init:789:(ptlrpcds) | 16 | 1 | 16 lnet_ni_add_interface:426:(ni-&gt;ni_interfaces[niface]) | 6 | 3 | 18 class_register_type:224:(type-&gt;typ_name) | 4 | 5 | 20 lprocfs_alloc_stats:1282:(stats-&gt;ls_cnt_header) | 24 | 1 | 24 ... </pre> </div></div> <p> </p></li> </ul> <p>These options are helpful for me when I worked on a recent memory leak issue.</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/43983/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/43983/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> utils: update leak-finder.pl for new format<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 5b998d803ffbd02ed306195d231e0a3478658574</p> <p>"Andreas Dilger &lt;adilger@whamcloud.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/c/fs/lustre-release/+/49357" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/49357</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> tests: improve leak-finder compatibility<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 1a2c31849f443c75f1b549a56f6ed03abe908d2a</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/49357/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/49357/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-14736" title="leak-finder.pl should not print to STDERR" class="issue-link" data-issue-key="LU-14736"><del>LU-14736</del></a> tests: improve leak-finder compatibility<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8ab8483c8f876480d5979e4b54e2e6f33cd9b7a7</p> <p>Landed for 2.16</p> Related Development Rank 1|i01waf: Rank (Obsolete) 9223372036854775807