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-8837 Lustre <p>With the syncing of the upstream client its becoming more clear what is the client code. Currently a lot of server code is included with the lustre client package. Its time to properly separate it out.</p> LU-8837

Cleanly separate server code from client code.

Improvement Minor Open Unresolved James A Simmons James A Simmons Tue, 15 Nov 2016 18:39:03 +0000 Tue, 6 Dec 2022 02:43:06 +0000 Lustre 2.10.0 Upstream 0 9 <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/23775" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/23775</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> build: reduce obdclass code built for client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 16409e689948d62f71433dd0f27c5bb4b2a7075e</p> <p>this first patch is just the tip of the iceberg.</p> <p>James</p> <p>Have you got a handle on this task or is it something that could/should be shared between multiple engineers?</p> <p>Peter</p> <p>I got it. Just need the reviews to make sure its done right in some spots.</p> <p>Note to self that server_name2svname() is really a server side function. Will submit patch once lustre_disk.h lands for <a href="https://jira.whamcloud.com/browse/LU-6401" title="Untangle lustre userland and kernel headers" class="issue-link" data-issue-key="LU-6401"><del>LU-6401</del></a> work.</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/28315" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28315</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> utils: make tools lightweight for lustre clients<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5b5e2ad6c1fc950a167bb2c6db2bdddb25b1de99</p> <p>This patch with the last patch for <a href="https://jira.whamcloud.com/browse/LU-6401" title="Untangle lustre userland and kernel headers" class="issue-link" data-issue-key="LU-6401"><del>LU-6401</del></a> moves lustre to being able to be built against the upstream client <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/biggrin.png" height="16" width="16" align="absmiddle" alt="" border="0"/> With the completion of this work lustre will have reached a important mile stone.</p> <p>James, all your work regarding upstream client is really nice and important! Thanks for that!</p> <p>I have the idea that Lustre build system, in the future, would handle server compilation simply like additional stuff to do, if requested by the admin.</p> <p>Client code will be a set of kernel modules (which could be taken from Linux kernel) and a set of CLI tools (from lustre source code for now).</p> <p>Server code will be <em>another</em> set of kernel modules and possibly CLI tools. This would simplify packing a lot.</p> <p> </p> <p>For now, do you know which lustre modules have a different code depending if server compilation is enabled or not?</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/28315/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28315/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> utils: make tools lightweight for lustre clients<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d62efba975d27d98f2d50f14d49dda53d2355d0c</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/32100" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/32100</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> obd: remove linux directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 49af44e389e3ff7c85ac81d0c6e8e3d2b204b371</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/32100/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/32100/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> obd: remove linux directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 6f8d0e2905286cadda1564ace6c2e6b034f0dfd6</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/39683" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/39683</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> build: use tgt_pool for lov layer<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: a3a32c954f2200aab775896f9c93172cc6c76b25</p> <p>James Simmons (jsimmons@infradead.org) uploaded a new patch: <a href="https://review.whamcloud.com/40821" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/40821</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> build: make OpenSFS code buildable against Lustre linux client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 3160ddf498af93c3a78e69d38eaa57171a905672</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41768" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41768</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> target: don't build any 'target' on client.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 4f997afbe27b06fe3a215c8b45352240a9890e3b</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41769" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41769</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: restrict sec_ctx.c to server-only<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: ac69e41035641aa8ac0ab525d5f136ef7cf86111</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41947" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41947</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: mark some functions as static<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2e6f199f904dab34d72ad691ed1dc404f2f7082e</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41948" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41948</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: only build service.c on the server<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: add6c3ec335112a6b667ec3bf5b5c6b90c374e3c</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41949" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41949</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lmv: don't use lqr_alloc spinlock in lmv<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 37d8d4022d092a1244f5b4d7814ffefe429b26ed</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41950" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41950</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lmv: don't include struct lu_qos_rr in client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 3b605f90c50a3824d64adad948b98ffbaf4bb9e1</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41951" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41951</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: move lu_tgt_pool out of obd_target.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: ff6318d60d235c7f699bf9a71e6be67ad3a83ea7</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41952" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41952</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: remove target declarations from obd.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 4fc25bc4084439c1072ec3990951edd23b87ac66</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41993" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41993</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lod: move lod-specifc pool config code into lod_dev<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 37c7e85c8b04710f9b3353a6b5bc6d27f4730f62</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41994" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41994</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: make uapi...lustre_disk.h unnecessary on client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 25cdb824871124bd2a4f2828c54a8fe1292b0ead</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/41995" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41995</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> mgc: move server-only code out of mgc_request.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 8efe2d0d913c57402cca0584dd3f981b83baa4b3</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41947/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41947/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: mark some functions as static<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f77e53d3656504c804fe3dd0a3fb72080229b648</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41949/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41949/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lmv: don't use lqr_alloc spinlock in lmv<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 3e14a71d87efde0a91ef04d3f37f0d16e3c2b450</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41993/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41993/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lod: move lod-specifc pool config code into lod_dev<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a12a61ab7f927dfc2213b213167be60369eaa275</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41950/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41950/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lmv: don't include struct lu_qos_rr in client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: b6882c8ae4f47e931cc7b49da1c6bffaaf31499b</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/41951/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41951/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: move lu_tgt_pool out of obd_target.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 5a8dc02609ace484079a66c13e9e35c04af7f06c</p> <p>"Jian Yu &lt;yujian@whamcloud.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/44829" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/44829</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> utils: move lustre_disk_data back to lustre_disk.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 045cb5aa53d92717d48faac1ec5462420a1983ef</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/44829/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/44829/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> utils: move lustre_disk_data back to lustre_disk.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: b5ddb7dab5fca41b493212002406034925770942</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/41994/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/41994/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: make uapi...lustre_disk.h unnecessary on client<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: e34950d29f1168a8a85f26e862eaec888b94350e</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/41767/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/41767/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: make ldlm and target file lists<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f32bdde0574daae522a61d00f9dda87c84c9dd05</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/41995/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/41995/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> mgc: move server-only code out of mgc_request.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 41610e6207ef5ef43093fb79d54200516b5b2838</p> <p>"Neil Brown &lt;neilb@suse.de&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/c/fs/lustre-release/+/49126" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/49126</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: fix problem with distclean<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 517eb826900435a14fdc5356af92cd60fc77ca82</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/49126/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/49126/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> ptlrpc: fix problem with distclean<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 4fe328b39be6188d7e470260943e7b6aba739422</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/41768/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/41768/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> target: don't build any 'target' on client.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a5d5bfbf9b6141954ec56e196aceaa233ef0228b</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/41952/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/41952/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-8837" title="Cleanly separate server code from client code." class="issue-link" data-issue-key="LU-8837">LU-8837</a> lustre: remove target declarations from obd.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 888962496078198250d8ad63e4ac928c45f31473</p> Related LU-14055 LU-14983 LU-14291 Development Rank 1|hzyvpz: Rank (Obsolete) 9223372036854775807