diff options
author | Minchan Kim <minchan.kim@gmail.com> | 2011-05-24 17:12:31 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-05-25 08:39:27 -0700 |
commit | 821ed6bbed3cf41c4050a431eeb822b33868d36a (patch) | |
tree | 50f6808a20263aaaf546a69218e1c1312ce7b51e | |
parent | 2cbea1d3ab11946885d37a2461072ee4d687cb4e (diff) | |
download | kernel_samsung_tuna-821ed6bbed3cf41c4050a431eeb822b33868d36a.zip kernel_samsung_tuna-821ed6bbed3cf41c4050a431eeb822b33868d36a.tar.gz kernel_samsung_tuna-821ed6bbed3cf41c4050a431eeb822b33868d36a.tar.bz2 |
mm: filter unevictable page out in deactivate_page()
It's pointless that deactive_page's operates on unevictable pages. This
patch removes unnecessary overhead which might be a bit problem in case
that there are many unevictable page in system(ex, mprotect workload)
[akpm@linux-foundation.org: tidy up comment]
Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
Reviewed-by: Rik van Riel<riel@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | mm/swap.c | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -476,6 +476,13 @@ static void drain_cpu_pagevecs(int cpu) */ void deactivate_page(struct page *page) { + /* + * In a workload with many unevictable page such as mprotect, unevictable + * page deactivation for accelerating reclaim is pointless. + */ + if (PageUnevictable(page)) + return; + if (likely(get_page_unless_zero(page))) { struct pagevec *pvec = &get_cpu_var(lru_deactivate_pvecs); |