[LU-10990] Get rid of per-osc max_dirty_mb setting Created: 02/May/18 Updated: 23/Jul/18 Resolved: 18/Jul/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.12.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Oleg Drokin | Assignee: | Oleg Drokin |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
max_dirty_mb is an ages old kludge we had to ensure faster flushing that avoided various timeouts, limits on memory consumption and limits on data lost should the ost run out of space. All of those now have proper mechanisms: we have grants to ensure nothing runs out of space, we have per-node dirty and clean pages limits and we have lock prolongation for active IO. As such I believe we no longer need the max_dirty_mb |
| Comments |
| Comment by Gerrit Updater [ 02/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: https://review.whamcloud.com/32252 |
| Comment by Gerrit Updater [ 04/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: https://review.whamcloud.com/32288 |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32288/ |
| Comment by Peter Jones [ 18/Jul/18 ] |
|
Landed for 2.12 |