[LU-8708] Grant shrinking disabled all the time Created: 13/Oct/16 Updated: 04/Feb/20 Resolved: 01/Sep/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.12.0 |
| Type: | Improvement | Priority: | Critical |
| Reporter: | Oleg Drokin | Assignee: | Zhenyu Xu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Severity: | 3 | ||||||||||||||||||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||||||||||||||||||
| Description |
|
It appears that OBD_CONNECT_GRANT_SHRINK is not set by client at all. It was disabled all the way back in 1.8.1 time due to bz 19507 "for debugging" and never reenabled. Commit a3c09ef128651a8f041171f2d6099b3ba8d8f2c6. I guess that was an omission and we should reenable it back? |
| Comments |
| Comment by Malcolm Cowe (Inactive) [ 14/Oct/16 ] |
|
Possibly unrelated, but back in the 1.8.x releases, there were issues with grant leaks, where the workaround was to increase the grant interval before it was ultimately fixed (BZ 22755. The following thread summarises the original issue: https://groups.google.com/forum/#!topic/lustre-discuss-list/4szexbm2HV0 |
| Comment by Andreas Dilger [ 14/Oct/16 ] |
|
This was disabled in https://bugzilla.lustre.org/show_bug.cgi?id=19507 |
| Comment by Joseph Gmitter (Inactive) [ 14/Oct/16 ] |
|
Hi Bobijam, Can you please look into this issue for 2.10? Thanks. |
| Comment by Andreas Dilger [ 14/Oct/16 ] |
|
It looks like https://bugzilla.lustre.org/attachment.cgi?id=24058 was never landed and may be a good starting point. |
| Comment by Gerrit Updater [ 17/Oct/16 ] |
|
Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/23202 |
| Comment by Gerrit Updater [ 17/Oct/16 ] |
|
Bobi Jam (bobijam@hotmail.com) uploaded a new patch: http://review.whamcloud.com/23203 |
| Comment by Nathan Rutman [ 26/Oct/17 ] |
|
Is this fixed for 2.11 ("Fix Version")? Last activity was a year ago... |
| Comment by Andreas Dilger [ 28/Oct/17 ] |
|
Nathan, neither of these patches have landed. |
| Comment by Gerrit Updater [ 24/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/23202/ |
| Comment by Oleg Drokin [ 28/Aug/18 ] |
|
This patch potentially leads to |
| Comment by Gerrit Updater [ 01/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/23203/ |
| Comment by Peter Jones [ 01/Sep/18 ] |
|
Landed for 2.12 |