diff options
| author | Johannes Weiner <[email protected]> | 2014-05-06 12:50:07 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2014-05-06 13:04:59 -0700 |
| commit | 623762517e2370be3b3f95f4fe08d6c063a49b06 (patch) | |
| tree | 94d42e42a3b1eb693ba387b6b297351ef6b03ed2 /lib/memory-notifier-error-inject.c | |
| parent | 6b6751f7feba68d8f5c72b72cc69a1c5a625529c (diff) | |
revert "mm: vmscan: do not swap anon pages just because free+file is low"
This reverts commit 0bf1457f0cfc ("mm: vmscan: do not swap anon pages
just because free+file is low") because it introduced a regression in
mostly-anonymous workloads, where reclaim would become ineffective and
trap every allocating task in direct reclaim.
The problem is that there is a runaway feedback loop in the scan balance
between file and anon, where the balance tips heavily towards a tiny
thrashing file LRU and anonymous pages are no longer being looked at.
The commit in question removed the safe guard that would detect such
situations and respond with forced anonymous reclaim.
This commit was part of a series to fix premature swapping in loads with
relatively little cache, and while it made a small difference, the cure
is obviously worse than the disease. Revert it.
Signed-off-by: Johannes Weiner <[email protected]>
Reported-by: Christian Borntraeger <[email protected]>
Acked-by: Christian Borntraeger <[email protected]>
Acked-by: Rafael Aquini <[email protected]>
Cc: Rik van Riel <[email protected]>
Cc: <[email protected]> [3.12+]
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'lib/memory-notifier-error-inject.c')
0 files changed, 0 insertions, 0 deletions