public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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 17:07:47 +0530	[thread overview]
Message-ID: <200901271707.48770.knikanth@suse.de> (raw)
In-Reply-To: <alpine.DEB.2.00.0901270316040.25608@chino.kir.corp.google.com>

On Tuesday 27 January 2009 16:51:26 David Rientjes wrote:
> On Tue, 27 Jan 2009, Nikanth Karthikesan wrote:
> > > 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.
>
> That's certainly idealistic, but cannot be done in an inexpensive way that
> would scale with the large systems that clients of cpusets typically use.

If we kill only the tasks for which cpuset_mems_allowed_intersects() is true 
on the first pass and even then if we do not get out of oom, we could go over 
again with this expensive check. Using this scheme, could kill more no of 
tasks than required, if a task with lots of memory has moved to a different 
cpuset. But it should be rare and not that severe compared to killing a 
totally innocent task like the current scheme does?!

Thanks
Nikanth

  reply	other threads:[~2009-01-27 11:40 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
2009-01-27 11:21                                 ` David Rientjes
2009-01-27 11:37                                   ` Nikanth Karthikesan [this message]
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=200901271707.48770.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