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-9127 Lustre <p>tgt_cb_last_committed() prints a D_HA message even if last_committed was not updated.</p> <p>The problem is the cpu overhead associated with a bunch of similar (and mostly useless) messages and potential Lustre debug log overflow.</p> <p>A trivial patch will be uploaded shortly.</p> LU-9127

too many debug messages from tgt_cb_last_committed()

Bug Minor Resolved Fixed WC Triage Andrew Perepechko patch Wed, 15 Feb 2017 09:01:52 +0000 Wed, 5 Aug 2020 21:11:41 +0000 Sat, 18 Mar 2017 13:47:53 +0000 Lustre 2.7.0 Lustre 2.10.0 0 6 <p>Andrew Perepechko (andrew.perepechko@seagate.com) uploaded a new patch: <a href="https://review.whamcloud.com/25469" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/25469</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9127" title="too many debug messages from tgt_cb_last_committed()" class="issue-link" data-issue-key="LU-9127"><del>LU-9127</del></a> target: tgt_cb_last_committed is too noisy<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5432848f7eb666babf4193320499540f48bc23a0</p> <p>Is this different from patch <a href="http://review.whamcloud.com/14611:" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14611:</a></p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> LU-6527 ext4: journal_commit_callback optimization Don't take spinlock in tgt_cb_last_committed, if exp_last_committed was updated with higher trasno. Also change list_add_tail to list_add. It gives advantages to ldiskfs in tgt_cb_last_committed. In the beginning of list will be placed thandles with the highest transaction numbers. So at the first iterations we will have the highest transno. It will save from extra call of ptlrpc_commit_replies. Signed-off-by: Sergey Cheremencev &lt;sergey.cheremencev@seagate.com&gt; Xyratex-bug-id: MRP-2575 </pre> </div></div> <p>That was landed to master for 2.7.61 but and the backport for b2_7_fe is <a href="https://review.whamcloud.com/19904" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/19904</a></p> <p>Ah, I see it now...</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/25469/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/25469/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9127" title="too many debug messages from tgt_cb_last_committed()" class="issue-link" data-issue-key="LU-9127"><del>LU-9127</del></a> target: tgt_cb_last_committed is too noisy<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 11eda8742fa4505bc62c6113b067a75687f171cc</p> <p>Landed for 2.10</p> Duplicate Development Rank 1|hzz3x3: Rank (Obsolete) 9223372036854775807 Severity