[LU-5275] clean up technical debt for proc_dir_entry changes

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.7.0
• Affects Version/s: Lustre 2.7.0
• Labels: patch
• Severity: 3
• Assignee: John Hammond
• Reporter: Jodi Levi (Inactive)

Description

This is to address the last comment made in LU-3319 to cleanup the technical debt.

Issue Links

• is related to: LU-5983 builds broken by LU-5275 fix
• is related to: LU-4416 support for 3.12 linux kernel
• is related to: LU-3319 Adapt to 3.10 upstream kernel proc_dir_entry change

Comments

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13341
Subject: LU-5275 libcfs: merge params_tree.h into lprocfs_status.h
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b22696a65873fdb69caf2a652f3bfe266bfcb489

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/13225/
Subject: LU-5275 lprocfs: reduce scope of params_tree.h
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f9920b4924edce1bd341622eee4281fdcd41845a

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13330
Subject: LU-5275 lprocfs: sync names to upstream kernel lustre client
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a0ab151983e0655f95f7c5b53b09054b5159578a

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13299
Subject: LU-5275 lprocfs: replace LPROCFS with CONFIG_PROC_FS
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1cd9b42ae4bfa063ddeb1d9c153b86afb1373264

[James A Simmons]

I have done exhaustive testing and looked very closely at the what states lustre procfs handling is in for the case of client and server being on the same node. I loaded lod then lov and vice versa as well as unload the modules in the two possible orders. I detected no leaks in either case. In order to fill out out proc trees we have to preform mounting/unmounting of the MDT and the client. I tried both orders of mounting and
umount and saw no issues with remove_proc_subtree. Tired various combos of mounting/unmounting and module loading/unloading. No memory leaks/disappearing directories or handling symlinks occurred. Please note this is all done on a 3.12 kernel. Also looking at detail of the procfs initialization and removal don't show a issue with remove_proc_subtree.

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13225
Subject: LU-5275 lprocfs: reduce scope of params_tree.h
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d4921339526c9076bf7a72660c86dac883b9eed4

[Jodi Levi (Inactive)]

Patch landed to master.

[Johann Lombardi (Inactive)]

For the record, i have reverted http://review.whamcloud.com/#/c/12298 since it broken the el6.6 build.

[Gerrit Updater]

Johann Lombardi ([e-mail]) merged in patch http://review.whamcloud.com/12953/
Subject: Revert "LU-5275 lprocfs: remove last of non seq data structs and functions."
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b1e595c09e1b07a6840142b3ae015b8a5a8affeb

[Gerrit Updater]

Johann Lombardi ([e-mail]) uploaded a new patch: http://review.whamcloud.com/12953
Subject: Revert "LU-5275 lprocfs: remove last of non seq data structs and functions."
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 80e25c90a3d45ee0ec55b0a83282168b0583303a

[James A Simmons]

Update patch http://review.whamcloud.com/#/c/12235. This should be the last patch in the series.