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-9520 Lustre <p>LLAPI uses a fixed-sized (4096 bytes) buffer to read changelog records. It should be possible for users to set a different value.</p> LU-9520

Variable buffer size in llapi changelog reader

Improvement Minor Closed Won't Fix Quentin Bouget Henri Doreau Wed, 17 May 2017 11:29:51 +0000 Wed, 6 Apr 2022 14:04:04 +0000 Wed, 6 Apr 2022 14:04:04 +0000 0 5 <p>Henri Doreau (henri.doreau@cea.fr) uploaded a new patch: <a href="https://review.whamcloud.com/27159" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/27159</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9520" title="Variable buffer size in llapi changelog reader" class="issue-link" data-issue-key="LU-9520"><del>LU-9520</del></a> utils: resizeable changelog read buffer<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 60b26820f09394feb5bd01096914ba48b605ea06</p> <p>Thanks for the patch Henri</p> <p>In addition to adding an API to change the buffe size, it is worthwhile to ask if the current size is too small for best performance. Should the size be increased by default to something larger (16KB, 64KB, 1MB, ...)? It doesn't make sense to leave the current default value performing poorly for most users. </p> <p>Quentin</p> <p>Is this still a patch CEA are interested in?</p> <p>Peter</p> <p>No, thank you. But I agree with Andreas, the buffer is probably too small for most use cases: a changelog record is at least 64 bytes long, so 4K bytes only hold 64 changelog records at best.</p> <p>I will start working on changelog processing in a few weeks, if this becomes an issue I will reopen the ticket (assuming it is closed by then).</p> <p>ok thanks! If you intend to work on a different approach eventually then I think that it is ok to reuse this ticket</p> <p>Ok, thank you.</p> Development Rank 1|hzzcyf: Rank (Obsolete) 9223372036854775807