Whamcloud Community JIRA https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 [LU-12477] Remove obsolete config checks https://jira.whamcloud.com/browse/LU-12477 Lustre <p>As of 2.12/2.13, we no longer support many older kernels, including anything in the 2.x vintage.</p> <p>This is a ticket to track removing some of the config checks for these versions.</p> LU-12477 Remove obsolete config checks Improvement Minor Resolved Fixed James A Simmons Patrick Farrell Thu, 27 Jun 2019 17:06:03 +0000 Tue, 1 Aug 2023 22:04:06 +0000 Tue, 2 Jun 2020 23:28:43 +0000 Lustre 2.14.0 0 7 <p>I got inspired to do a few of these this morning, I have zero intention of trying to get them all. <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>Right now, I have:<br/> <a href="https://review.whamcloud.com/35342" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35342</a><br/> <a href="https://review.whamcloud.com/35343" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35343</a><br/> <a href="https://review.whamcloud.com/35344" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35344</a></p> <p>For this.</p> <p>These cover, I believe:<br/> All the obsolete libcfs checks</p> <p>The 'easy for Patrick to understand' obsolete lnet checks (so nothing in the OFED checking)</p> <p>And the last one is just scraping the surface of the core Lustre checks, of which there are <b>a lot</b>.</p> <p>There are many, many obsolete checks not covered here.  This would be a good newbie project...</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/35461" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35461</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> llite: use generic_error_remove_page()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 23eb08f241e4d177f8e892fdbe748711854f4865</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/35462" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35462</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: use delete_from_page_cache() for directory pages.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 539fe85d2e2da65988873d199d5031d50ed35346</p> <p>Applied the two above patches will let use remove lustre_patchless_compat.h and it autoconf test.</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35461/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35461/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> llite: use generic_error_remove_page()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f114d4dac6f842212e248f2ed4a50ce3249109d8</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35462/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35462/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: use delete_from_page_cache() for directory pages.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 43d5806ffe6d6017aea0892de78dc27e54861de1</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/35539" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35539</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: eliminate lustre_patchless_compat.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 91097dd068e61bd57c154a86b300636b9e61d863</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/35551" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35551</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: remove 2.6 kernel support<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 9dc477ccf810a9da7df96ea13e51852c4cb38e2e</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35551/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35551/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: remove &lt; 2.6.39 kernel support<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 5595f7b429ded89d9925ec98696da1a1727034e7</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35539/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35539/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: eliminate lustre_patchless_compat.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: ab38b346d9176dce1b02cf631db31998e0170832</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35343/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35343/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lnet: Remove obsolete config options<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 25929ea0d2955afc6fb1150e57fd8e527ffc6b4f</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/37085" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37085</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: remove obsolete config checks<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b3db43823a10e5a1e0e26229f6b7d77bfbb6ea6c</p> <p>James, what do you think about splitting the configure checks into common and server-only sections? That would probably remove a bunch of checks from the client build process, which is one of the slower parts of the build today.</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35342/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35342/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> libcfs: Remove obsolete config checks<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c4c17fa4a3f5d9c3df44e19ab3385c8de655cdef</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/37352" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37352</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: remove obsolete ext4 patches<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 33c0a1768c4070e604863ecf439f3151e7322084</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37085/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37085/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: remove obsolete config checks<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8e88bbfef5795b3bf9541c8cb5b2dd8a3bb11d9f</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/37725" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37725</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: restore time_after32()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: d04651d36cbfedf9b3c75ffaea13f9cee3fb3bb9</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37725/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37725/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: restore time_after32()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 33141a149023ac5c3631da911cbc67116f915464</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37352/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37352/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: remove obsolete ext4 patches<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: fc87b01f96e8485a3713d0e446551e985e0e4aa0</p> <p>Looks like this has all landed for 2.14</p> <p>A cleanup of the kernel patches are left. Also the old dev rdonly work can be removed.</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/38068" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38068</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: remove dev_read_only handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 45bfcd1a1af8d1fcb6e5d527abc863b2ac51ae4d</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/38070" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38070</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: remove dev_read_only handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: dea498289bf70fb1a3b59d03e5bb03816ce18ae3</p> <p>Hi James,</p> <p>A small change in "lustre/Changelog" is also required.</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> # git diff diff --git a/lustre/ChangeLog b/lustre/ChangeLog index dfb5fa8..a20c9d4 100644 --- a/lustre/ChangeLog +++ b/lustre/ChangeLog @@ -5,7 +5,6 @@ TBD Whamcloud * Server primary kernels built and tested during release cycle: 3.10.0-1062.9.1.el7 (RHEL7.7) * Other server kernels known to build and work at some point (others may also work): - 3.10.0-862.14.4.el7 (RHEL7.5) 3.10.0-957.27.2.el7 (RHEL7.6) 4.18.0-80.11.2.el8 (RHEL8.0) 4.18.0-147.3.1.el8 (RHEL8.1) </pre> </div></div> <p>I ran into the compatibility issue first hand as I was still using 3.10.0-862.14.4.el7 (RHEL7.5) as my dev system. Of course after a bump to 3.10.0-957.27.2.el7 (7.6) things are fine now.</p> <p>Thanks</p> <p> </p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/38139" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38139</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: cleanup left over mballoc prefetch patch<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 09f74f159bc60aa71d8d0fbe963737b4925231af</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38070/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38070/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: remove dev_read_only handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 20553c985a518ebc1d5da2a1ddf44c4eec3db044</p> <p>Shaun Tancheff (shaun.tancheff@hpe.com) uploaded a new patch: <a href="https://review.whamcloud.com/38268" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38268</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: drop support for sles12<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2bc4770ed8c79da2d6c6e4dc2c01df23c62371c5</p> <p>Shaun Tancheff (shaun.tancheff@hpe.com) uploaded a new patch: <a href="https://review.whamcloud.com/38269" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38269</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> kernel: drop support for unused patches<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5b247e40989e69166c497998fe1b2f35b187d6ed</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/38438" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38438</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: finish removing HAVE_CLEAR_RDONLY_ON_PUT<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: c5285d6c35b2fb1bfaf3be319621f7b462e4ce89</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38139/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38139/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: last cleanups<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: ce4d3aaac7c25d0b00f610e0fbd8f4142df6a80f</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38438/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38438/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: finish removing HAVE_CLEAR_RDONLY_ON_PUT<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 50b6be5d1d5e2d7a7954df82fbda7af93471550e</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38268/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38268/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> ldiskfs: drop SUSE kernel 4.4 and earlier<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 89075044b3a86454cc542e6a5a2994d44f667ac9</p> <p>Is this work complete or is there more to come still?</p> <p>From my perspective there is nothing further.</p> <p>ok - thanks <a href="https://jira.whamcloud.com/secure/ViewProfile.jspa?name=stancheff" class="user-hover" rel="stancheff">stancheff</a></p> <p>I concur. Our patchset is so much smaller <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/40831" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/40831</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> libcfs: Further reduce complexity for shrinkers.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 549435cd15bf1f1c2d038da8843ce1229074b60f</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/40883" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/40883</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: check return status of register_shrinker()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b388d26f5580b7ecbd46beb107d01c0698b7503f</p> <p>Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/41086" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41086</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> llite: remove unused ll_teardown_mmaps()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 216e7cae0fd736971aaa95974f72b76a2b546014</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41086/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41086/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> llite: remove unused ll_teardown_mmaps()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 647c96562b27cef99e5986911c4db83dcfee0620</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/40831/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/40831/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> libcfs: Further reduce complexity for shrinkers.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 782725b11021b2f1c3467863216836ee94bcd874</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/40883/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/40883/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> lustre: check return status of register_shrinker()<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 812b2ccf0284df42e8a88a6b7c4c3874dd721c71</p> <p>Sebastien Buisson (sbuisson@ddn.com) uploaded a new patch: <a href="https://review.whamcloud.com/43623" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/43623</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12477" title="Remove obsolete config checks" class="issue-link" data-issue-key="LU-12477"><del>LU-12477</del></a> libcfs: Remove obsolete config checks<br/> Project: fs/lustre-release<br/> Branch: b2_12<br/> Current Patch Set: 1<br/> Commit: 2828793cd9d4938ca95b509ad73c7342dca7043b</p> Duplicate Related LU-14453 LU-14823 LU-12561 LU-13127 Development Rank 1|i00itj: Rank (Obsolete) 9223372036854775807