From: Andrew Morton <akpm@linux-foundation.org>
Date: Fri, 28 Sep 2012 00:19:08 +0000 (+1000)
Subject: mm-fix-nonuniform-page-status-when-writing-new-file-with-small-buffer-fix-fix
X-Git-Tag: next-20121004~1^2~355
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=7815688a4af17361d694c5844a8e81f90e80c1ce;p=karo-tx-linux.git

mm-fix-nonuniform-page-status-when-writing-new-file-with-small-buffer-fix-fix

grab better comment from the v3 patch

Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: KOSAKI Motohiro <kosaki.motohiro@gmail.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Robin Dong <sanbai@taobao.com>
Cc: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

diff --git a/mm/swap.c b/mm/swap.c
index 7a555e227d61..f76c76c7501b 100644
--- a/mm/swap.c
+++ b/mm/swap.c
@@ -447,8 +447,12 @@ void mark_page_accessed(struct page *page)
 EXPORT_SYMBOL(mark_page_accessed);
 
 /*
- * Check the pagevec space before adding a new page into it, to prevent
- * nonuniform page status in mark_page_accessed() after __lru_cache_add().
+ * Order of operations is important: flush the pagevec when it's already
+ * full, not when adding the last page, to make sure that last page is
+ * not added to the LRU directly when passed to this function. Because
+ * mark_page_accessed() (called after this when writing) only activates
+ * pages that are on the LRU, linear writes in subpage chunks would see
+ * every PAGEVEC_SIZE page activated, which is unexpected.
  */
 void __lru_cache_add(struct page *page, enum lru_list lru)
 {