Re: [mm] [PATCH 4/4] Memory cgroup hierarchy feature selector

From: KAMEZAWA Hiroyuki
Date: Sun Nov 02 2008 - 01:31:20 EST


On Sun, 02 Nov 2008 11:33:51 +0530
Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> wrote:

> KAMEZAWA Hiroyuki wrote:
> > On Sun, 02 Nov 2008 00:19:02 +0530
> > Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> wrote:
> >
> >> Don't enable multiple hierarchy support by default. This patch introduces
> >> a features element that can be set to enable the nested depth hierarchy
> >> feature. This feature can only be enabled when there is just one cgroup
> >> (the root cgroup).
> >>
> > Why the flag is for the whole system ?
> > flag-per-subtree is of no use ?
>
> Flag per subtree might not be useful, since we charge all the way up to root,
Ah, what I said is "How about enabling/disabling hierarhcy support per subtree ?"
Sorry for bad text.

like this.
- you can set hierarchy mode of a cgroup turned on/off when...
* you don't have any tasks under it && it doesn't have any child cgroup.

Thanks,
-Kame

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/