From 4e99325b462ba18075768582621af74a6b79d2a5 Mon Sep 17 00:00:00 2001 From: Peter Zijlstra Date: Sun, 8 Jul 2007 01:13:06 -0700 Subject: [PATCH] mm: double mark_page_accessed() in read_cache_page_async() Fix a post-2.6.21 regression. read_cache_page_async() has two invocations of mark_page_accessed() which will launch pages right onto the active list. Remove the first one, keeping the latter one. This avoids marking unwanted pages active (in the retry loop). Signed-off-by: Peter Zijlstra Acked-by: Nick Piggin Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- mm/filemap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/filemap.c b/mm/filemap.c index edb1b0b5cc8..d1d9814f99d 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -1786,7 +1786,6 @@ retry: page = __read_cache_page(mapping, index, filler, data); if (IS_ERR(page)) return page; - mark_page_accessed(page); if (PageUptodate(page)) goto out; -- 2.41.0