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-9091 Lustre <p>The latest kernels have introduced a function that is very similar to what was done in <a href="https://jira.whamcloud.com/browse/LU-7334" title="Refactor lprocfs helpers" class="issue-link" data-issue-key="LU-7334"><del>LU-7334</del></a>. The difference is the unit labels i.e M instead of MB. Upstream will be wanting us to use this standard function.</p> LU-9091

Replace lprocfs_str_with_units_to_s64 with string_get_size()

Improvement Minor Closed Fixed James A Simmons James A Simmons patch Wed, 8 Feb 2017 23:29:34 +0000 Fri, 13 Nov 2020 23:30:31 +0000 Tue, 31 Mar 2020 11:39:42 +0000 Lustre 2.14.0 0 3 <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/30539" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30539</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> obd: replace lprocfs_str_to_s64<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 8e442396d9a2c6686b2e3aa3db889f03358e522a</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/31380" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/31380</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> tests: use upstream units for proc settings<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e8cc6cf7f2cdd3bf3a96a28b89eb60764d7b5573</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/35658" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35658</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> sysfs: use string helper like functions for sysfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 50f7e2f2a60b0f2d6ccb6043de6c2f3a7e542e7e</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35658/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35658/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> sysfs: use string helper like functions for sysfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d9e0c9f346d0f0cccfa37737807d6f9d08255ab7</p> <p>Landed for 2.14</p> <p>Another patch is needed as well as a final cleanup patch.</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/37274" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37274</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> obd: move remaining sysfs that needs units to sysfs_memparse()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5a13c7eccc3df9f34d2088cdaa0a9439b4286c05</p> <p><a href="https://review.whamcloud.com/#/c/34849" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/34849</a></p> <p><a href="https://review.whamcloud.com/#/c/37275" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/37275</a></p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/37622" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37622</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> sysfs: add memparse format support for defunit<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 63b1825928c5f7d6ed8bc906d5983c71a9d74c54</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37620/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37620/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> obdclass: allow bare KMGTPE param suffix<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d105cc4a6448b3c842790d0e1f0380b94d087f6f</p> <p>Seems to have all landed for 2.14</p> <p>One last cleanup patch to land - <a href="https://review.whamcloud.com/#/c/37275/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/37275/</a></p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37275/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37275/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9091" title="Replace lprocfs_str_with_units_to_s64 with string_get_size()" class="issue-link" data-issue-key="LU-9091"><del>LU-9091</del></a> procfs: remove old string parsing routines<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 48242f98c3add59d9e084d73057e64265230a19f</p> <p>Finally?</p> <p>Yep its finished!!!</p> Related LU-8066 LU-7334 LU-10517 LU-11157 LU-12511 Development Rank 1|hzz327: Rank (Obsolete) 9223372036854775807