From: Nikanth Karthikesan <knikanth@suse.de>
To: David Rientjes <rientjes@google.com>
Cc: "Evgeniy Polyakov" <zbr@ioremap.net>,
"Andrew Morton" <akpm@linux-foundation.org>,
"Alan Cox" <alan@lxorguk.ukuu.org.uk>,
linux-kernel@vger.kernel.org,
"Linus Torvalds" <torvalds@linux-foundation.org>,
"Chris Snook" <csnook@redhat.com>,
"Arve Hjønnevåg" <arve@android.com>,
"Paul Menage" <menage@google.com>,
containers@lists.linux-foundation.org,
"Balbir Singh" <balbir@linux.vnet.ibm.com>,
"KOSAKI Motohiro" <kosaki.motohiro@jp.fujitsu.com>
Subject: Re: [RFC] [PATCH] Cgroup based OOM killer controller
Date: Tue, 27 Jan 2009 16:38:21 +0530 [thread overview]
Message-ID: <200901271638.21720.knikanth@suse.de> (raw)
In-Reply-To: <alpine.DEB.2.00.0901270244380.23757@chino.kir.corp.google.com>
On Tuesday 27 January 2009 16:23:00 David Rientjes wrote:
> On Tue, 27 Jan 2009, Nikanth Karthikesan wrote:
> > > As previously stated, I think the heuristic to penalize tasks for not
> > > having an intersection with the set of allowable nodes of the oom
> > > triggering task could be made slightly more severe. That's irrelevant
> > > to your patch, though.
> >
> > But the heuristic makes it non-deterministic, unlike memcg case. And this
> > mandates special handling for cpuset constrained OOM conditions in this
> > patch.
>
> Dividing a badness score by 8 if a task's set of allowable nodes do not
> insect with the oom triggering task's set does not make an otherwise
> deterministic algorithm non-deterministic.
>
> I don't understand what you're arguing for here. Are you suggesting that
> we should not prefer tasks that intersect the set of allowable nodes?
> That makes no sense if the goal is to allow for future memory freeing.
>
No. Actually I am just wondering, will it be possible to check whether a
particular task has memory allocated or mmaped from this node to avoid killing
an innocent task. I compared with memcg, to say that memcg never kills a task
not related to the memcg constrained oom. Sorry if I was unclear, earlier. If
we do this, oom-controller will not require special handling for cpuset
constrained ooms.
Thanks
Nikanth
next prev parent reply other threads:[~2009-01-27 11:11 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-21 11:08 [RFC] [PATCH] Cgroup based OOM killer controller Nikanth Karthikesan
2009-01-21 13:17 ` Evgeniy Polyakov
2009-01-21 15:24 ` Nikanth Karthikesan
2009-01-21 20:49 ` David Rientjes
2009-01-22 2:53 ` KAMEZAWA Hiroyuki
2009-01-22 5:12 ` Nikanth Karthikesan
2009-01-22 5:12 ` Nikanth Karthikesan
2009-01-22 8:43 ` David Rientjes
2009-01-22 9:23 ` Nikanth Karthikesan
2009-01-22 9:39 ` David Rientjes
2009-01-22 10:10 ` Nikanth Karthikesan
2009-01-22 10:18 ` David Rientjes
2009-01-22 9:50 ` Evgeniy Polyakov
2009-01-22 10:00 ` David Rientjes
2009-01-22 10:14 ` Evgeniy Polyakov
2009-01-22 10:27 ` David Rientjes
2009-01-22 13:21 ` Evgeniy Polyakov
2009-01-22 20:28 ` David Rientjes
2009-01-22 21:06 ` Evgeniy Polyakov
2009-01-22 21:35 ` David Rientjes
2009-01-22 22:04 ` Evgeniy Polyakov
2009-01-22 22:28 ` David Rientjes
2009-01-22 22:53 ` Evgeniy Polyakov
2009-01-22 23:25 ` Evgeniy Polyakov
2009-01-27 23:55 ` Paul Menage
2009-01-23 9:45 ` Nikanth Karthikesan
2009-01-23 10:33 ` David Rientjes
2009-01-23 14:56 ` Nikanth Karthikesan
2009-01-23 20:44 ` David Rientjes
2009-01-27 10:20 ` Nikanth Karthikesan
2009-01-27 10:53 ` David Rientjes
2009-01-27 11:08 ` Nikanth Karthikesan [this message]
2009-01-27 11:21 ` David Rientjes
2009-01-27 11:37 ` Nikanth Karthikesan
2009-01-27 20:29 ` David Rientjes
2009-01-28 1:00 ` Paul Menage
2009-01-29 15:48 ` Nikanth Karthikesan
2009-01-22 3:28 ` KAMEZAWA Hiroyuki
2009-01-22 5:13 ` Nikanth Karthikesan
2009-01-22 5:27 ` KAMEZAWA Hiroyuki
2009-01-22 6:11 ` Nikanth Karthikesan
2009-01-22 5:39 ` Arve Hjønnevåg
2009-01-22 6:12 ` Nikanth Karthikesan
2009-01-22 6:29 ` Arve Hjønnevåg
2009-01-22 6:42 ` Nikanth Karthikesan
2009-01-26 19:54 ` Balbir Singh
2009-01-26 19:56 ` Alan Cox
2009-01-27 7:02 ` KOSAKI Motohiro
2009-01-27 7:26 ` Balbir Singh
2009-01-27 7:39 ` David Rientjes
2009-01-27 7:44 ` KOSAKI Motohiro
2009-01-27 7:51 ` David Rientjes
2009-01-27 9:31 ` Evgeniy Polyakov
2009-01-27 9:37 ` David Rientjes
2009-01-27 13:40 ` Evgeniy Polyakov
2009-01-27 20:37 ` David Rientjes
2009-01-27 21:51 ` Evgeniy Polyakov
2009-01-27 10:40 ` KOSAKI Motohiro
2009-01-27 13:45 ` Evgeniy Polyakov
2009-01-27 15:40 ` Balbir Singh
2009-01-27 21:54 ` Evgeniy Polyakov
2009-01-27 20:41 ` David Rientjes
2009-01-27 21:55 ` Evgeniy Polyakov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200901271638.21720.knikanth@suse.de \
--to=knikanth@suse.de \
--cc=akpm@linux-foundation.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=arve@android.com \
--cc=balbir@linux.vnet.ibm.com \
--cc=containers@lists.linux-foundation.org \
--cc=csnook@redhat.com \
--cc=kosaki.motohiro@jp.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=menage@google.com \
--cc=rientjes@google.com \
--cc=torvalds@linux-foundation.org \
--cc=zbr@ioremap.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox