From: Peter Zijlstra Date: Sun, 8 Jul 2007 08:13:06 +0000 (-0700) Subject: mm: double mark_page_accessed() in read_cache_page_async() X-Git-Tag: v2.6.22~3 X-Git-Url: http://pilppa.com/gitweb/?a=commitdiff_plain;h=4e99325b462ba18075768582621af74a6b79d2a5;p=linux-2.6-omap-h63xx.git 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 --- 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;