[LU-9325] Improve lustre kernel string handling

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0
• Affects Version/s: Lustre 2.10.0
• Labels: patch

Epic/Theme

upstream

• Assignee: James A Simmons
• Reporter: James A Simmons

Description

While pushing work upstream some of the feedback was related to how Lustre does its own direct parsing of string which is frowned on. Secondly we are still not using newer kernel string interfaces like kstrtoul instead of simple_string() and other examples exist as well that have been merged upstream.

Issue Links

• is related to: LU-18066 client mount options can be lost
• is related to: LU-12861 Add and use scnprintf() where using snprintf() is not handling return correctly
• is related to: LU-12511 Prepare lustre for adoption into the linux kernel
• is related to: LU-4423 Tracking of patches from upstream kernel to Lustre client

Comments

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40119
Subject: LU-9325 osd-ldisk: replace simple_strto* with kstr* functions
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8ddd1d616f35a9b9937631fa5e6d653d8e855886

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/39498/
Subject: LU-9325 fld: replace simple_strto* with kstr* functions
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f875fa73f215c3e1f71656dd8aa0cd30d931fa09

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/39498
Subject: LU-9325 fld: replace simple_strto* with kstr* functions
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3feaa7d148d9b64e9d1fb9c4afef7f13f59f52d5

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/38846/
Subject: LU-9325 mdt: replace simple_strtol() with kstrtol()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4115148e4ac5ccb3620a9f8d3e48da0a712943c5

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/38846
Subject: LU-9325 mdt: replace simple_strtol() with kstrtol()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 201128895c982522dc78b9f6c98329e56a7cb5a1

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/36551/
Subject: LU-9325 mds: replace simple_strtol use with target_name2index()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7d37148e36854a293fdc40d7d771769f0a6b64e0

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/36551
Subject: LU-9325 mds: replace simple_strtol use with target_name2index()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8625814aecc72beb592d6620b4382c007c119a04

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33959/
Subject: LU-9325 msg: replace obsolete simple_strtoul use
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 30239b4aecbe80e14f65316defd9fe162426147a

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33959
Subject: LU-9325 msg: replace obsolete simple_strtoul use
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f7ebdb6c5aa4a42e4a7ce4e1ec3daecdb1f86107

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33617
Subject: LU-9325 obdclass: make niduuid for lustre_stop_mgc() static
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: fdf3062c711fa27504be9029e03a4c84c9014ed5

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33385
Subject: LU-9325 libcfs: don't stomp on module string for cfs_str2num_check
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d9f60a170dd8f7786b52b8f09bfe04218c374cbe