[LU-9611] fix default stripe count/offset proc tunables

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.9.0, Lustre 2.10.0
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Type: Bug
• Priority: Major
• Reporter: Andreas Dilger
• Assignee: Andreas Dilger
• Resolution: Fixed
• Severity: 3

Description

Since patch http://review.whamcloud.com/16930 "LU-7334 lprocfs: Refactored string to value helpers" was landed, it incorrectly checks that lod.*.stripecount and lod.*.stripeoffset are not negative, but in fact they can both be "-1" to indicate that the filesystem-wide default value should be used. These parameters are also set internally if using lfs setstripe -c -1 $MOUNT or lfs setstripe -i -1 $MOUNT to set the system wide default, generating a console error on the MDS:

LustreError: 12558:0:(obd_config.c:1392:class_process_proc_param())  lov.: error writing proc entry 'stripecount': rc = -34
LustreError: 12558:0:(obd_config.c:1392:class_process_proc_param())  lov.: error writing proc entry 'stripeoffset': rc = -34

Issue Links

• is related to: LU-7334 Refactor lprocfs helpers

Comments

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/27473
Subject: LU-9611 lod: allow -1 for default stripe count/offset
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a7c6c37e491182bfd0519b7d737972bf4ea64404

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27473/
Subject: LU-9611 lod: allow -1 for default stripe count/offset
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a4771a6935a97376cc93dfaf023b0a12e9d07511

[Peter Jones]

Landed for 2.10

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/27946
Subject: LU-9611 lov: allow lov.*.stripe

{size,count}

=-1 param
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6774bd00931d05e78d42c1cfed31cce458fb9e95

[John Hammond]

The second change, https://review.whamcloud.com/#/c/27946/, is still unlanded.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27946/
Subject: LU-9611 lov: allow lov.*.stripe

{size,count}

=-1 param
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f93276d9a5b5df475c9f6007265692632d8e7222

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30407
Subject: LU-9611 lov: allow lov.*.stripe

{size,count}

=-1 param
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 3cf0e254f1afa1c87f0867b5c1d39f55a5bb8a0d

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/30407/
Subject: LU-9611 lov: allow lov.*.stripe

{size,count}

=-1 param
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: bb1663cb01744b68aa29d625cea8652ae0457257