login
Header Space

 
 

Re: [patch 21/21] cull non-reclaimable anon pages from the LRU at fault time

Previous thread: [patch 12/21] No Reclaim LRU Infrastructure by Rik van Riel on Thursday, February 28, 2008 - 3:29 pm. (5 messages)

Next thread: [patch 20/21] account mlocked pages by Rik van Riel on Thursday, February 28, 2008 - 3:29 pm. (1 message)
To: <linux-kernel@...>
Cc: KOSAKI Motohiro <kosaki.motohiro@...>, Lee Schermerhorn <Lee.Schermerhorn@...>, <linux-mm@...>
Date: Thursday, February 28, 2008 - 3:29 pm

V2 -&gt; V3:
+ rebase to 23-mm1 atop RvR's split lru series.

V1 -&gt; V2:
+  no changes

Optional part of "noreclaim infrastructure"

In the fault paths that install new anonymous pages, check whether
the page is reclaimable or not using lru_cache_add_active_or_noreclaim().
If the page is reclaimable, just add it to the active lru list [via
the pagevec cache], else add it to the noreclaim list.  

This "proactive" culling in the fault path mimics the handling of
mlocked pages in Nick Piggin's series to keep mlocked pages off
the lru lists.

Notes:

1) This patch is optional--e.g., if one is concerned about the
   additional test in the fault path.  We can defer the moving of
   nonreclaimable pages until when vmscan [shrink_*_list()]
   encounters them.  Vmscan will only need to handle such pages
   once.

2) I moved the call to page_add_new_anon_rmap() to before the test
   for page_reclaimable() and thus before the calls to
   lru_cache_add_{active|noreclaim}(), so that page_reclaimable()
   could recognize the page as anon, thus obviating, I think, the
   vma arg to page_reclaimable() for this purpose.  Still needed for
   culling mlocked pages in fault path [later patch].
   TBD:   I think this reordering is OK, but the previous order may
   have existed to close some obscure race?

3) With this and other patches above installed, any anon pages
   created before swap is added--e.g., init's anonymous memory--
   will be declared non-reclaimable and placed on the noreclaim
   LRU list.  Need to add mechanism to bring such pages back when
   swap becomes available.

Signed-off-by:  Lee Schermerhorn &lt;lee.schermerhorn@hp.com&gt;
Signed-off-by:  Rik van Riel &lt;riel@redhat.com&gt;

Index: linux-2.6.25-rc2-mm1/mm/memory.c
===================================================================
--- linux-2.6.25-rc2-mm1.orig/mm/memory.c	2008-02-28 00:27:06.000000000 -0500
+++ linux-2.6.25-rc2-mm1/mm/memory.c	2008-02-28 12:49:23.000000000 -0500
@@ -1678,7 +1678,7 @@ gotten...
To: <unlisted-recipients@...>, <@...>
Cc: <linux-kernel@...>, KOSAKI Motohiro <kosaki.motohiro@...>, <linux-mm@...>
Date: Thursday, February 28, 2008 - 4:19 pm

On Thu, 2008-02-28 at 14:29 -0500, Rik van Riel wrote:

&lt;delete prev note 3, as it referred to patches that are no longer in the
series [altho' I continue to maintain them separately, "just in case"].
replace with this note about vma arg to page_reclaimable()&gt;

3) The 'vma' argument to page_reclaimable() is require to notice that
   we're faulting a page into an mlock()ed vma w/o having to scan the
   page's rmap in the fault path.   Culling mlock()ed anon pages is

--
To: Lee Schermerhorn <Lee.Schermerhorn@...>
Cc: <linux-kernel@...>, KOSAKI Motohiro <kosaki.motohiro@...>, <linux-mm@...>
Date: Thursday, February 28, 2008 - 6:27 pm

On Thu, 28 Feb 2008 15:19:33 -0500

Thanks.  I have merged your new description.

-- 
All rights reversed.
--
Previous thread: [patch 12/21] No Reclaim LRU Infrastructure by Rik van Riel on Thursday, February 28, 2008 - 3:29 pm. (5 messages)

Next thread: [patch 20/21] account mlocked pages by Rik van Riel on Thursday, February 28, 2008 - 3:29 pm. (1 message)
speck-geostationary