Messages in this thread | | | Date | Tue, 15 Dec 2009 13:57:38 +0900 | From | KAMEZAWA Hiroyuki <> | Subject | Re: [BUGFIX][PATCH] oom-kill: fix NUMA consraint check with nodemask v4.2 |
| |
On Tue, 15 Dec 2009 13:35:46 +0900 KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> wrote:
> On Mon, 14 Dec 2009 20:30:37 -0800 (PST) > David Rientjes <rientjes@google.com> wrote: > > > 2 ideas which I can think of now are.. > > > 1) add sysctl_oom_calc_on_committed_memory > > > If this is set, use vm-size instead of rss. > > > > > > > I would agree only if the oom killer used total_vm as a the default, it is > > long-standing and allows for the aforementioned capability that you lose > > with rss. I have no problem with the added sysctl to use rss as the > > baseline when enabled. > > > I'll prepare a patch for adds > > sysctl_oom_kill_based_on_rss (default=0) > Hmm..
But for usual desktop users, using rss as default,as memory-eater-should-die. sounds reasoable to me. Hmm...maybe some automatic detection logic is required.
As my 1st version shows,
CONSTRAINT_CPUSET -> use vm_size CONSTRAINT_LOWMEM -> use lowmem_rss CONSTRAINT_NONE -> use rss seems like a landing point for all stake holders. No ?
Thanks, -Kame
| |