Commit 0a0dd05d authored Mar 31, 2009 by KOSAKI Motohiro Committed by Linus Torvalds Apr 01, 2009

mm: don't call mark_page_accessed() in do_swap_page()



commit bf3f3bc5 (mm: don't
mark_page_accessed in fault path) only remove the mark_page_accessed() in
filemap_fault().

Therefore, swap-backed pages and file-backed pages have inconsistent
behavior.  mark_page_accessed() should be removed from do_swap_page().

Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Nick Piggin <npiggin@suse.de>
Cc: Hugh Dickins <hugh@veritas.com>
Cc: Rik van Riel <riel@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent ee99c71c

mm/memory.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -2435,8 +2435,6 @@ static int do_swap_page(struct mm_struct mm, struct vm_area_struct vma,
		count_vm_event(PGMAJFAULT);
		}

		mark_page_accessed(page);

		lock_page(page);
		delayacct_clear_flag(DELAYACCT_PF_SWAPIN);

Admin message