From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail144.messagelabs.com (mail144.messagelabs.com [216.82.254.51]) by kanga.kvack.org (Postfix) with SMTP id 260466B007B for ; Tue, 27 Oct 2009 20:15:57 -0400 (EDT) Received: from m4.gw.fujitsu.co.jp ([10.0.50.74]) by fgwmail5.fujitsu.co.jp (Fujitsu Gateway) with ESMTP id n9S0FsOU016413 for (envelope-from kamezawa.hiroyu@jp.fujitsu.com); Wed, 28 Oct 2009 09:15:54 +0900 Received: from smail (m4 [127.0.0.1]) by outgoing.m4.gw.fujitsu.co.jp (Postfix) with ESMTP id 1D74E45DE79 for ; Wed, 28 Oct 2009 09:15:54 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (s4.gw.fujitsu.co.jp [10.0.50.94]) by m4.gw.fujitsu.co.jp (Postfix) with ESMTP id EA49245DE70 for ; Wed, 28 Oct 2009 09:15:53 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id C95A8E18003 for ; Wed, 28 Oct 2009 09:15:53 +0900 (JST) Received: from ml13.s.css.fujitsu.com (ml13.s.css.fujitsu.com [10.249.87.103]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id 585751DB803E for ; Wed, 28 Oct 2009 09:15:53 +0900 (JST) Date: Wed, 28 Oct 2009 09:13:21 +0900 From: KAMEZAWA Hiroyuki Subject: Re: [RFC][PATCH] oom_kill: avoid depends on total_vm and use real RSS/swap value for oom_score (Re: Memory overcommit Message-Id: <20091028091321.b136d9d9.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <4AE730C2.30401@gmail.com> References: <20091014135119.e1baa07f.kamezawa.hiroyu@jp.fujitsu.com> <4ADE3121.6090407@gmail.com> <20091026105509.f08eb6a3.kamezawa.hiroyu@jp.fujitsu.com> <4AE5CB4E.4090504@gmail.com> <20091027122213.f3d582b2.kamezawa.hiroyu@jp.fujitsu.com> <2f11576a0910262310g7aea23c0n9bfc84c900879d45@mail.gmail.com> <20091027153429.b36866c4.minchan.kim@barrios-desktop> <20091027153626.c5a4b5be.kamezawa.hiroyu@jp.fujitsu.com> <28c262360910262355p3cac5c1bla4de9d42ea67fb4e@mail.gmail.com> <20091027164526.da6a23cb.kamezawa.hiroyu@jp.fujitsu.com> <4AE730C2.30401@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org To: vedran.furac@gmail.com Cc: Minchan Kim , KOSAKI Motohiro , linux-mm@kvack.org, linux-kernel@vger.kernel.org, "hugh.dickins@tiscali.co.uk" , "akpm@linux-foundation.org" , rientjes@google.com List-ID: On Tue, 27 Oct 2009 18:41:22 +0100 Vedran FuraA? wrote: > KAMEZAWA Hiroyuki wrote: > > > On Tue, 27 Oct 2009 15:55:26 +0900 > > Minchan Kim wrote: > > > >>>> Hmm. > >>>> I wonder why we consider VM size for OOM kiling. > >>>> How about RSS size? > >>>> > >>> Maybe the current code assumes "Tons of swap have been generated, already" if > >>> oom-kill is invoked. Then, just using mm->anon_rss will not be correct. > >>> > >>> Hm, should we count # of swap entries reference from mm ?.... > >> In Vedran case, he didn't use swap. So, Only considering vm is the problem. > >> I think it would be better to consider both RSS + # of swap entries as > >> Kosaki mentioned. > >> > > Then, maybe this kind of patch is necessary. > > This is on 2.6.31...then I may have to rebase this to mmotom. > > Added more CCs. > > > > Vedran, I'm glad if you can test this patch. > > Thanks for the patch! I'll test it during this week a report after that. > > > Instead of total_vm, we should use anon/file/swap usage of a process, I think. > > This patch adds mm->swap_usage and calculate oom_score based on > > anon_rss + file_rss + swap_usage. > > Isn't file_rss shared between processes? Sorry, I'm newbie. :) > It's shared. But in typical case, file_rss will very small at OOM. > % pmap $(pidof test) > 29049: ./test > 0000000000400000 4K r-x-- /home/vedranf/dev/tmp/test > 0000000000600000 4K rw--- /home/vedranf/dev/tmp/test > 00002ba362a80000 116K r-x-- /lib/ld-2.10.1.so > 00002ba362a9d000 12K rw--- [ anon ] > 00002ba362c9c000 4K r---- /lib/ld-2.10.1.so > 00002ba362c9d000 4K rw--- /lib/ld-2.10.1.so > 00002ba362c9e000 1320K r-x-- /lib/libc-2.10.1.so > 00002ba362de8000 2044K ----- /lib/libc-2.10.1.so > 00002ba362fe7000 16K r---- /lib/libc-2.10.1.so > 00002ba362feb000 4K rw--- /lib/libc-2.10.1.so > 00002ba362fec000 1024028K rw--- [ anon ] // <-- This > 00007ffff4618000 84K rw--- [ stack ] > 00007ffff47b7000 4K r-x-- [ anon ] > ffffffffff600000 4K r-x-- [ anon ] > total 1027648K > > I would just look at anon if that's OK (or possible). > > > Considering usual applications, this will be much better information than > > total_vm. > > Agreed. > > > score PID name > > 4033 3176 gnome-panel > > 4077 3113 xinit > > 4526 3190 python > > 4820 3161 gnome-settings- > > 4989 3289 gnome-terminal > > 7105 3271 tomboy > > 8427 3177 nautilus > > 17549 3140 gnome-session > > 128501 3299 bash > > 256106 3383 mmap > > > > This order is not bad, I think. > > Yes, this looks much better now. Bash is only having somewhat strangely > high score. > It gets half score of mmap....If mmap goes, bash's score will goes down dramatically. I'll read other's comments and tweak this patch more. Thanks, -Kame > Regards, > > Vedran > > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@kvack.org. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: email@kvack.org > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org