Re: [PATCH] mm: double mark_page_accessed() in read_cache_page_async()

2007-07-06 Thread Andrew Morton
On Fri, 06 Jul 2007 18:03:54 +0200 Peter Zijlstra <[EMAIL PROTECTED]> wrote: > read_cache_page_async() seems to have 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

[PATCH] mm: double mark_page_accessed() in read_cache_page_async()

2007-07-06 Thread Peter Zijlstra
read_cache_page_async() seems to have 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 <[EMAIL PROTECTED]> --- mm/