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-17195 Lustre <p>Add an option to dump debug log on checksum failure. Similar to dump_on_eviction/timeout</p> LU-17195

Add option to dump log on checksum error

Improvement Minor Closed Fixed Chris Horn Chris Horn Fri, 13 Oct 2023 16:03:36 +0000 Mon, 16 Oct 2023 20:56:38 +0000 Mon, 16 Oct 2023 20:56:38 +0000 0 3 <p>"Chris Horn &lt;chris.horn@hpe.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/c/fs/lustre-release/+/52691" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/52691</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-17195" title="Add option to dump log on checksum error" class="issue-link" data-issue-key="LU-17195"><del>LU-17195</del></a> obd: Add dump_on_checksum parameter<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e59ff6b4fb98e3e5b3f673bf09c5598b02ef343b</p> <p>Chris, how does this differ from the <tt>checksum_dump</tt> functionality added by patch <a href="https://review.whamcloud.com/23960" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/23960</a> "<tt><a href="https://jira.whamcloud.com/browse/LU-8376" title="Enhance debugging infos available for Lustre checksum errors" class="issue-link" data-issue-key="LU-8376"><del>LU-8376</del></a> ost: enhance end to end bulk cksum error report</tt>"? That will already dump the checksum data to a file when there is an error detected.</p> <p>AFAICT, the <a href="https://jira.whamcloud.com/browse/LU-8376" title="Enhance debugging infos available for Lustre checksum errors" class="issue-link" data-issue-key="LU-8376"><del>LU-8376</del></a> patch dumps the page content from the bulk xfer that failed the checksum. It does this on both client and server so that the content can be compared after the fact to see what has changed. My patch just dumps the lustre debug log to /tmp in the same manner as dump_on_eviction and dump_on_timeout. So I think these two patches are complimentary.</p> <p>I was mistaken. The existing checksum_dump feature already dumps the debug log, so this patch is redundant.</p> Related LU-8376 Development Rank 1|i03ybz: Rank (Obsolete) 9223372036854775807 Severity