[LU-14289] Restrict libcfs to compatibility code only

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.15.0
• Assignee: Neil Brown
• Reporter: Neil Brown

Description

Libcfs currently contains both compatibility code and support code such as abstract-data-types and logging code.  It would be best to keep these separate, particularly in view of the goal of landing lustre in the mainline kernel, where having a "libcfs" would not fit well.

Some of the support code duplicates functionality already provided by Linux, and this is slowly being deprecated and removed in favour of the Linux code.  There is still move to be done.

Other support code is not as generic as it might seem (e.g. heap.c) and would fit well inside the only module that uses it.

The goal of this improvement is to move everything other than compatibility code out of libcfs.  If there is anything remaining that would go in "linux/lib" upsteam, (there might be some, but not much), it should go into a new "lib" directory.

 

Issue Links

• is related to: LU-9859 libcfs simplification

Comments

[Peter Jones]

Seems like all patches landed for 2.15

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41992/
Subject: LU-14289 libcfs: discard cfs_array_alloc()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8ec9fe0d55b1f5a7854dabe240f43e6cd5cb22ad

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41375/
Subject: LU-14289 ptlrpc: rename cfs_binheap to simply binheap
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8587508f5ddd7b8e764abb4c6fc078ba0f0bf1ad

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41992
Subject: LU-14289 libcfs: discard cfs_array_alloc()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 39747fc6462bb8b43d7f6ba5901b2d95ab425ddb

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41129/
Subject: LU-14289 ptlrpc: move heap.c from libcfs to ptlrpc
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f7f0b104bcf3827ffe9025a2347f0147c2103809

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41375
Subject: LU-14289 ptlrpc: rename cfs_binheap to simply binheap
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9e393aad2263232dd290bb073e4c483a364ff191

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41129
Subject: LU-14289 ptlrpc: move heap.c from libcfs to ptlrpc
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: fdab01470d1b38459a53f1cc5c64bc807ba1d085