[LU-13386] allow over reservation to cover current read Created: 25/Mar/20 Updated: 10/Apr/20 Resolved: 07/Apr/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.14.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Wang Shilong (Inactive) | Assignee: | Wang Shilong (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
ll_ra_count_get() is designed to allow over reservation for performance reasons, this could happen that multiple threads consuming a small readahead window limit, it will fallback to small 4K read which hurts performances. |
| Comments |
| Comment by Gerrit Updater [ 25/Mar/20 ] |
|
Wang Shilong (wshilong@ddn.com) uploaded a new patch: https://review.whamcloud.com/38047 |
| Comment by Gerrit Updater [ 07/Apr/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38047/ |
| Comment by Nathan Dauchy (Inactive) [ 07/Apr/20 ] |
|
Please also backport to b2_12. Thanks! |
| Comment by Peter Jones [ 07/Apr/20 ] |
|
Don't worry dauchy we do have a process to track this kind of thing (though admittedly it usually takes longer than 30 mins |
| Comment by Wang Shilong (Inactive) [ 08/Apr/20 ] |
|
I checked that b2_12 did not backport:
So we don't need the patch to b2_12 branch unless above patch ported to b2_12 firstly. |