diff options
author | Yafang Shao <laoar.shao@gmail.com> | 2017-11-15 20:33:45 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-11-15 21:21:03 -0500 |
commit | 0f6d24f878568fac579a1962d0bf7cb9f01e0ceb (patch) | |
tree | 5fa7529b42621ab7b1a86ec8d5f3dc01cb9d7b10 /mm/page-writeback.c | |
parent | 66e8b438bd5c75498cfe915c4219049eaebcb869 (diff) |
mm/page-writeback.c: print a warning if the vm dirtiness settings are illogical
The vm direct limit setting must be set greater than vm background limit
setting. Otherwise print a warning to help the operator to figure out
that the vm dirtiness settings is in illogical state.
Link: http://lkml.kernel.org/r/1506592464-30962-1-git-send-email-laoar.shao@gmail.com
Signed-off-by: Yafang Shao <laoar.shao@gmail.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Michal Hocko <mhocko@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page-writeback.c')
-rw-r--r-- | mm/page-writeback.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index 76a43c17761b..768fe4e37e6a 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -433,8 +433,11 @@ static void domain_dirty_limits(struct dirty_throttle_control *dtc) | |||
433 | else | 433 | else |
434 | bg_thresh = (bg_ratio * available_memory) / PAGE_SIZE; | 434 | bg_thresh = (bg_ratio * available_memory) / PAGE_SIZE; |
435 | 435 | ||
436 | if (bg_thresh >= thresh) | 436 | if (unlikely(bg_thresh >= thresh)) { |
437 | pr_warn("vm direct limit must be set greater than background limit.\n"); | ||
437 | bg_thresh = thresh / 2; | 438 | bg_thresh = thresh / 2; |
439 | } | ||
440 | |||
438 | tsk = current; | 441 | tsk = current; |
439 | if (tsk->flags & PF_LESS_THROTTLE || rt_task(tsk)) { | 442 | if (tsk->flags & PF_LESS_THROTTLE || rt_task(tsk)) { |
440 | bg_thresh += bg_thresh / 4 + global_wb_domain.dirty_limit / 32; | 443 | bg_thresh += bg_thresh / 4 + global_wb_domain.dirty_limit / 32; |