diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2013-09-24 18:27:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-24 20:00:26 -0400 |
commit | b1aff7fcf86c88472b0a70f15d89d7a4adba07bb (patch) | |
tree | 00a1b6194f6abe8a93a1bfb180365800a2873b65 /mm/vmscan.c | |
parent | 694fbc0fe78518d06efa63910bf4ecee660e7852 (diff) |
revert "vmscan, memcg: do softlimit reclaim also for targeted reclaim"
Revert commit a5b7c87f9207 ("vmscan, memcg: do softlimit reclaim also
for targeted reclaim")
I merged this prematurely - Michal and Johannes still disagree about the
overall design direction and the future remains unclear.
Cc: Michal Hocko <mhocko@suse.cz>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/vmscan.c')
-rw-r--r-- | mm/vmscan.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index 17a7134de4d7..0e081cada4ba 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c | |||
@@ -142,7 +142,7 @@ static bool global_reclaim(struct scan_control *sc) | |||
142 | 142 | ||
143 | static bool mem_cgroup_should_soft_reclaim(struct scan_control *sc) | 143 | static bool mem_cgroup_should_soft_reclaim(struct scan_control *sc) |
144 | { | 144 | { |
145 | return !mem_cgroup_disabled(); | 145 | return !mem_cgroup_disabled() && global_reclaim(sc); |
146 | } | 146 | } |
147 | #else | 147 | #else |
148 | static bool global_reclaim(struct scan_control *sc) | 148 | static bool global_reclaim(struct scan_control *sc) |
@@ -2195,7 +2195,7 @@ __shrink_zone(struct zone *zone, struct scan_control *sc, bool soft_reclaim) | |||
2195 | struct lruvec *lruvec; | 2195 | struct lruvec *lruvec; |
2196 | 2196 | ||
2197 | if (soft_reclaim && | 2197 | if (soft_reclaim && |
2198 | !mem_cgroup_soft_reclaim_eligible(memcg, root)) { | 2198 | !mem_cgroup_soft_reclaim_eligible(memcg)) { |
2199 | memcg = mem_cgroup_iter(root, memcg, &reclaim); | 2199 | memcg = mem_cgroup_iter(root, memcg, &reclaim); |
2200 | continue; | 2200 | continue; |
2201 | } | 2201 | } |