[#LU-8837] Cleanly separate server code from client code.

[LU-8837] Cleanly separate server code from client code. Created: 15/Nov/16 Updated: 06/Dec/22
Status:	Open
Project:	Lustre
Component/s:	None
Affects Version/s:	Lustre 2.10.0
Fix Version/s:	Upstream

Type:

Improvement

Priority:

Minor

Reporter:

James A Simmons

Assignee:

James A Simmons

Resolution:

Unresolved

Votes:

Labels:

None

Issue Links:

Related
is related to	~~LU-14055~~	Write performance regression caused b...	Resolved
is related to	LU-14983	actually use CONFIGS/mountdata in ser...	Open
is related to	LU-14291	Improve use of HAVE_SERVER_SUPPORT an...	Reopened

Rank (Obsolete):

9223372036854775807

Description

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.

Comments

Comment by Gerrit Updater [ 15/Nov/16 ]

James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23775
Subject: LU-8837 build: reduce obdclass code built for client
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 16409e689948d62f71433dd0f27c5bb4b2a7075e

Comment by James A Simmons [ 15/Nov/16 ]

this first patch is just the tip of the iceberg.

Comment by Peter Jones [ 16/Nov/16 ]

James

Have you got a handle on this task or is it something that could/should be shared between multiple engineers?

Peter

Comment by James A Simmons [ 16/Nov/16 ]

I got it. Just need the reviews to make sure its done right in some spots.

Comment by James A Simmons [ 14/Feb/17 ]

Note to self that server_name2svname() is really a server side function. Will submit patch once lustre_disk.h lands for ~~LU-6401~~ work.

Comment by Gerrit Updater [ 02/Aug/17 ]

James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28315
Subject: LU-8837 utils: make tools lightweight for lustre clients
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5b5e2ad6c1fc950a167bb2c6db2bdddb25b1de99

Comment by James A Simmons [ 02/Aug/17 ]

This patch with the last patch for ~~LU-6401~~ moves lustre to being able to be built against the upstream client With the completion of this work lustre will have reached a important mile stone.

Comment by Aurelien Degremont (Inactive) [ 20/Sep/17 ]

James, all your work regarding upstream client is really nice and important! Thanks for that!

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.

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).

Server code will be another set of kernel modules and possibly CLI tools. This would simplify packing a lot.

For now, do you know which lustre modules have a different code depending if server compilation is enabled or not?

Comment by Gerrit Updater [ 22/Nov/17 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28315/
Subject: LU-8837 utils: make tools lightweight for lustre clients
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d62efba975d27d98f2d50f14d49dda53d2355d0c

Comment by Gerrit Updater [ 20/Apr/18 ]

James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32100
Subject: LU-8837 obd: remove linux directory
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 49af44e389e3ff7c85ac81d0c6e8e3d2b204b371

Comment by Gerrit Updater [ 06/May/18 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/32100/
Subject: LU-8837 obd: remove linux directory
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6f8d0e2905286cadda1564ace6c2e6b034f0dfd6

Comment by Gerrit Updater [ 17/Aug/20 ]

James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/39683
Subject: LU-8837 build: use tgt_pool for lov layer
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a3a32c954f2200aab775896f9c93172cc6c76b25

Comment by Gerrit Updater [ 01/Dec/20 ]

James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/40821
Subject: LU-8837 build: make OpenSFS code buildable against Lustre linux client
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3160ddf498af93c3a78e69d38eaa57171a905672

Comment by Gerrit Updater [ 26/Feb/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/41768
Subject: LU-8837 target: don't build any 'target' on client.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4f997afbe27b06fe3a215c8b45352240a9890e3b

Comment by Gerrit Updater [ 26/Feb/21 ]

Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/41769
Subject: LU-8837 ptlrpc: restrict sec_ctx.c to server-only
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: ac69e41035641aa8ac0ab525d5f136ef7cf86111