[LU-13107] clean up lctl commands

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0, Lustre 2.15.0
• Assignee: Andreas Dilger
• Reporter: Andreas Dilger

Description

Remove an obsolete lctl command ("lov_getconfig"), as well as the ioctl that it was calling. It was only recently used by sanity test_44a, which can instead use "lfs getstripe -d $DIR" now that LU-11656 is landed.

The lctl command list should also be reordered so that the "testing (DANGEROUS)" commands are listed at the end, and LFSCK and llog commands are before that, to avoid confusion.

There are also a number of very ancient jt_ptl_ and jt_obd_ function definitions and ancient ioctl numbers defined that are no longer used and can be removed.

Issue Links

• is related to: LU-16634 Null pointer dereference in lustre_set_wire_obdo
• is related to: LU-17489 fix 'lfs getname' ioctl breakage
• is related to: LU-6202 clean up ioctl handling

Comments

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48886/
Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ccee6b92ec4d338f54df0ebb2e8533b870265e4b

[Gerrit Updater]

"Andreas Dilger <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48886
Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 44c558d9b540c9f2d106a72f76559c044537346c

[Peter Jones]

Seems to have all landed for 2.15

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37108/
Subject: LU-13107 utils: clean up lctl command usage
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b0efebdaef52d8ac9b02857166ceb00079612ebc

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37107/
Subject: LU-13107 uapi: remove obsolete ioctls
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 0f38a0b9db4f7c6c101ee55fe7b94d49a32eabe9

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37106/
Subject: LU-13107 utils: remove lctl lov_getconfig command
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d2cb789537485eec9bfc5bc7d4c85025b0488478

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37108
Subject: LU-13107 utils: clean up lctl command usage
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8d6b295ffb2e421335b99b31ebd7027f6e8fd848

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37107
Subject: LU-13107 uapi: remove obsolete ioctls
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8a050e48b2bef44cc2739cc1c705a7383113383a

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37106
Subject: LU-13107 utils: remove lctl lov_getconfig command
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 65eda0b10346dd88b81e37ec2741b589b82142c0