lkml.org 
[lkml]   [2009]   [Jun]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: Found the commit that causes the OOMs
From
HI, David.

First of all, Thanks for your effort to find out cause.

Unfortunately, I don't have followed your problem.
I guess you met OOM problem with no swap device. right ?

My patch shouldn't have affect yours.
The patch's motivation is following as.

"If our system have no swap device, we can't reclaim anon pages.
So, anon pages's moving in anon lru list is unnecessary."

If we don't call shrink_active_list in shrink_zone's tail,
it can affect reclaim_stat->recent_[rotated|scanned].

Then it can affect number of pages for scanning in anon lru list.
But, Look at shrink_zone.

If we don't have swap device, we never scan anon lru list forcely.
(anon lru's percent is always zero)

Nonetheless, OOM happen.

Hmm..
Could I show your oops and show_mem information, please ?

Rik, Kosaki, What do you think ?

On Sat, Jun 27, 2009 at 4:12 PM, David Howells<dhowells@redhat.com> wrote:
>
> I've managed to bisect things to find the commit that causes the OOMs.  It's:
>
>        commit 69c854817566db82c362797b4a6521d0b00fe1d8
>        Author: MinChan Kim <minchan.kim@gmail.com>
>        Date:   Tue Jun 16 15:32:44 2009 -0700
>
>            vmscan: prevent shrinking of active anon lru list in case of no swap space V3
>
>            shrink_zone() can deactivate active anon pages even if we don't have a
>            swap device.  Many embedded products don't have a swap device.  So the
>            deactivation of anon pages is unnecessary.
>
>            This patch prevents unnecessary deactivation of anon lru pages.  But, it
>            don't prevent aging of anon pages to swap out.
>
>            Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
>            Acked-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
>            Cc: Johannes Weiner <hannes@cmpxchg.org>
>            Acked-by: Rik van Riel <riel@redhat.com>
>            Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
>            Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
>
> This exhibits the problem.  The previous commit:
>
>        commit 35282a2de4e5e4e173ab61aa9d7015886021a821
>        Author: Brice Goglin <Brice.Goglin@ens-lyon.org>
>        Date:   Tue Jun 16 15:32:43 2009 -0700
>
>            migration: only migrate_prep() once per move_pages()
>
> survives 16 iterations of the LTP syscall testsuite without exhibiting the
> problem.
>
> David
>



--
Kinds regards,
Minchan Kim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2009-06-27 14:11    [W:0.177 / U:7.708 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site