diff options
author | Minchan Kim <minchan.kim@gmail.com> | 2011-03-22 19:32:53 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-03-22 20:44:03 -0400 |
commit | 3f58a82943337fb6e79acfa5346719a97d3c0b98 (patch) | |
tree | 667441ac13c845edac93c937d0baba03a2021ea8 /mm/swap.c | |
parent | 315601809d124d046abd6c3ffa346d0dbd7aa29d (diff) |
memcg: move memcg reclaimable page into tail of inactive list
The rotate_reclaimable_page function moves just written out pages, which
the VM wanted to reclaim, to the end of the inactive list. That way the
VM will find those pages first next time it needs to free memory.
This patch applies the rule in memcg. It can help to prevent unnecessary
working page eviction of memcg.
Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
Acked-by: Balbir Singh <balbir@linux.vnet.ibm.com>
Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Reviewed-by: Rik van Riel <riel@redhat.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/swap.c')
-rw-r--r-- | mm/swap.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -200,8 +200,9 @@ static void pagevec_move_tail(struct pagevec *pvec) | |||
200 | spin_lock(&zone->lru_lock); | 200 | spin_lock(&zone->lru_lock); |
201 | } | 201 | } |
202 | if (PageLRU(page) && !PageActive(page) && !PageUnevictable(page)) { | 202 | if (PageLRU(page) && !PageActive(page) && !PageUnevictable(page)) { |
203 | int lru = page_lru_base_type(page); | 203 | enum lru_list lru = page_lru_base_type(page); |
204 | list_move_tail(&page->lru, &zone->lru[lru].list); | 204 | list_move_tail(&page->lru, &zone->lru[lru].list); |
205 | mem_cgroup_rotate_reclaimable_page(page); | ||
205 | pgmoved++; | 206 | pgmoved++; |
206 | } | 207 | } |
207 | } | 208 | } |