[LU-1214] PTLRPC related modules cleanup

• Type: Task
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.4.0, Lustre 2.7.0
• Labels: llite patch
• Environment: Linux client.
• Assignee: Keith Mannthey (Inactive)
• Reporter: Xuezhao Liu

Description

Clean up Lustre client to facilitate pushing it into upstream Linux kernel. Need to split some server code dependencies from client to avoidmaking extensive server code changes.

There is a discussion thread at lustre-dev mail-list:
http://lists.lustre.org/pipermail/lustre-devel/2012-February/003990.html

Issue Links

• is related to: LU-6206 conf-sanity test_53a: Insane OST thread counts
• is related to: LU-6206 conf-sanity test_53a: Insane OST thread counts
• Trackbacks: Lustre Community Development in Progress Features are being developed for future Lustre releases both at Whamcloud and by other organizations in the Lustre community. These will be eligible for inclusion in future Lustre releases as per our processes

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/13647/
Subject: Revert "LU-1214 ptlrpc: start minimum service threads"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 65612d9a63139fd2a349a7eb59b32c89892249e4

[Gerrit Updater]

Oleg Drokin ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13647
Subject: Revert "LU-1214 ptlrpc: start minimum service threads"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 280975f11e1e3e265dba05e3f1e89870739ec98f

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/2876/
Subject: LU-1214 ptlrpc: start minimum service threads
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 43f96aa9cc3cec66d9b9e0a03e5fc23e094525e7

[Keith Mannthey (Inactive)]

Patches were landed.

[Keith Mannthey (Inactive)]

http://review.whamcloud.com/#change,4489
Was abandoned in favor of: http://review.whamcloud.com/#change,5190 that has been merged.

I feel this LU is complete. Please reopen if I am mistaken or have missed a patch.

[Xuezhao Liu]

There is one small patch in review:
http://review.whamcloud.com/#change,4489

[Keith Mannthey (Inactive)]

Any update? I saw on Nov 7 2364 was abandoned.

Is there a separate LU to track the master change? Or can you link the master change here?

[Xuezhao Liu]

That small patch (2364) was submitted to Orion due to some code dependency of llog. Seems some llog code changes are landing on Master branch, I will rebase it to Master at appropiate time when that code dependency resolved. Thanks.

[Keith Mannthey (Inactive)]

Is this change still needed for the Orion or Master branch? What needs to happen with this code?

http://review.whamcloud.com/#change,2364

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el6,ofa #497
LU-1214 ptlrpc: splits server-side connection/bulkIO/recovery (Revision 18d78c77953017e5a76cd10bc74a0d078217a626)

Result = SUCCESS
Oleg Drokin : 18d78c77953017e5a76cd10bc74a0d078217a626
Files :

• lustre/include/lustre_lib.h
• lustre/include/lustre_net.h
• lustre/ldlm/ldlm_lib.c
• lustre/ptlrpc/sec.c
• lustre/obdecho/echo_client.c
• lustre/ptlrpc/events.c
• lustre/ptlrpc/niobuf.c
• lustre/ptlrpc/client.c
• lustre/include/lustre_sec.h
• lustre/ldlm/ldlm_lockd.c
• lustre/obdecho/autoMakefile.am
• lustre/ptlrpc/ptlrpc_module.c
• lustre/obdecho/echo.c
• lustre/obdecho/Makefile.in
• lustre/obdecho/echo_internal.h
• lustre/ptlrpc/ptlrpc_internal.h