public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Vladislav D. Buzov" <vbuzov@embeddedalley.com>
To: Paul Menage <menage@google.com>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux Containers Mailing List 
	<containers@lists.linux-foundation.org>,
	Dan Malek <dan@embeddedalley.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Balbir Singh <balbir@linux.vnet.ibm.com>
Subject: Re: [PATCH 1/1] Memory usage limit notification addition to memcg
Date: Mon, 13 Jul 2009 18:00:53 -0700	[thread overview]
Message-ID: <4A5BD8C5.1060902@embeddedalley.com> (raw)
In-Reply-To: <6599ad830907131515h3c9622b5v309cf8f13d272bab@mail.gmail.com>

Paul Menage wrote:
> On Tue, Jul 7, 2009 at 5:56 PM, KAMEZAWA
> Hiroyuki<kamezawa.hiroyu@jp.fujitsu.com> wrote:
>   
>> I know people likes to wait for file descriptor to get notification in these days.
>> Can't we have "event" file descriptor in cgroup layer and make it reusable for
>> other purposes ?
>>     
>
> I agree - rather than having to add a separate "wait for value to
> cross X threshold" file for each numeric usage value that people might
> be concerned about, it would be better to have a generic way to do it
> for any file. Given that this is a userspace API, it would be better
> to work out at least the generic API first, even if the initial
> implementation isn't generic.
>  "OOM"
>   
We all agree on that. Our original intention was to create the memory
limit notification capability, not a generic mechanism for the cgroup
events. So, we started out with something that fits the current model.

> - independent thresholds for different waiters
>   
The threshold is one of the Memory Controller cgroup attributes, as well
as memory usage limit. It is not a user task attribute and should not be
controlled by user tasks. It's administrator or some control task job to
set memory usage parameters within the cgroup. Once the threshold is
reached, all tasks within the cgroup should be notified and do their
best to free the memory.


  reply	other threads:[~2009-07-14  1:01 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-13 22:08 [PATCH] Memory usage limit notification addition to memcg Dan Malek
2009-04-13 23:08 ` Andrew Morton
2009-04-13 23:45   ` Dan Malek
2009-04-13 23:54     ` Andrew Morton
2009-04-14  0:52       ` Dan Malek
2009-04-15  0:35 ` KAMEZAWA Hiroyuki
2009-04-15  2:34   ` Dan Malek
2009-04-15  2:58     ` KAMEZAWA Hiroyuki
2009-04-15  7:32       ` Dan Malek
2009-04-15  7:33         ` KAMEZAWA Hiroyuki
2009-04-15  8:24   ` Balbir Singh
2009-04-15 17:35     ` Dan Malek
2009-04-16  3:15       ` Balbir Singh
2009-07-07 20:25 ` [PATCH 0/1] " Vladislav Buzov
2009-07-07 20:25   ` [PATCH 1/1] " Vladislav Buzov
2009-07-08  0:56     ` KAMEZAWA Hiroyuki
2009-07-09  1:43       ` Vladislav D. Buzov
2009-07-13  0:52         ` KAMEZAWA Hiroyuki
2009-07-13 21:21           ` Vladislav D. Buzov
2009-07-13 22:15       ` Paul Menage
2009-07-14  1:00         ` Vladislav D. Buzov [this message]
2009-07-14  1:03           ` KAMEZAWA Hiroyuki
2009-07-14  1:43         ` KOSAKI Motohiro
2009-07-14 19:13           ` Dan Malek
2009-07-16 17:15             ` Balbir Singh
2009-07-16 18:16               ` Dan Malek
2009-07-17  2:33                 ` Balbir Singh
2009-07-15  6:02         ` Balbir Singh
2009-07-08  3:52     ` Balbir Singh
2009-07-10 18:01       ` Vladislav D. Buzov
2009-07-14  0:16   ` [PATCH 0/2] Memory usage limit notification feature (v3) Vladislav Buzov
2009-07-14  0:16     ` [PATCH 1/2] Resource usage threshold notification addition to res_counter (v3) Vladislav Buzov
2009-07-14  0:16       ` [PATCH 2/2] Memory usage limit notification addition to memcg (v3) Vladislav Buzov
2009-07-14  0:30       ` [PATCH 1/2] Resource usage threshold notification addition to res_counter (v3) KAMEZAWA Hiroyuki
2009-07-14  1:29         ` Vladislav D. Buzov
2009-07-14  1:45           ` KAMEZAWA Hiroyuki
2009-07-14  0:36       ` Paul Menage
2009-07-14  0:47         ` KAMEZAWA Hiroyuki
2009-07-14  0:20     ` [PATCH 0/2] Memory usage limit notification feature (v3) Paul Menage
2009-07-14  0:31       ` KOSAKI Motohiro

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=4A5BD8C5.1060902@embeddedalley.com \
    --to=vbuzov@embeddedalley.com \
    --cc=akpm@linux-foundation.org \
    --cc=balbir@linux.vnet.ibm.com \
    --cc=containers@lists.linux-foundation.org \
    --cc=dan@embeddedalley.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=menage@google.com \
    /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