[LU-13503] allow setting larger max_mod_rpcs_in_flight dynamically Created: 02/May/20 Updated: 08/Jan/23 Resolved: 20/Nov/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.14.0, Lustre 2.12.6 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Andreas Dilger | Assignee: | Jian Yu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||||||
| Description |
|
Currently, an error is reported if mdc.*.max_mod_rpcs_in_flight is set larger or equal to mdc.*.max_rpcs_in_flight. However, it makes more sense to allow the user to set max_mod_rpcs_in_flight directly, and just increase max_rpcs_in_flight appropriately, rather than returning an error and telling the user to do that instead. Also, during mount config parameter processing, there may be a race between MDC setup and config log handling, so allow max_mod_rpcs_in_flight to be temporarily increased, and then it will be reduced later to the server value when the OCD information is handled. |
| Comments |
| Comment by Gerrit Updater [ 02/May/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/38455 |
| Comment by Andreas Dilger [ 28/May/20 ] |
|
A separate patch is needed to allow using "lctl set_param [-P] mdt.*.max_mod_rpcs_in_flight=N" via the normal Lustre parameter interface. It doesn't make sense that this is only available as a module parameter. |
| Comment by James A Simmons [ 05/Jun/20 ] |
|
Note this sysfs file has issues with import locking. That should be fixed in another patch. |
| Comment by Gerrit Updater [ 10/Jun/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38455/ |
| Comment by Peter Jones [ 10/Jun/20 ] |
|
Landed for 2.14 |
| Comment by Gerrit Updater [ 10/Jun/20 ] |
|
Jian Yu (yujian@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/38893 |
| Comment by Jian Yu [ 10/Jun/20 ] |
|
There are two more patches needed for this ticket. |
| Comment by Gerrit Updater [ 11/Jul/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38893/ |
| Comment by Jian Yu [ 20/Nov/20 ] |
|
The issue in this ticket was resolved. The remaining two separate issues are tracked in LU-14144 and LU-14145. |