Re: [RFC 0/5] forced comounts for cgroups.

From: Peter Zijlstra
Date: Wed Sep 05 2012 - 06:05:02 EST


On Wed, 2012-09-05 at 02:32 -0700, Tejun Heo wrote:
> Hey, again.
>
> On Wed, Sep 05, 2012 at 11:06:33AM +0200, Peter Zijlstra wrote:
> > Doing all this runtime is just going to make the mess even bigger,
> > because now we have to deal with even more stupid cases.
> >
> > So either we go and try to contain this mess as proposed by Glauber or
> > we go delete controllers.. I've had it with this crap.
>
> cpuacct is rather unique tho. I think it's gonna be silly whether the
> hierarchy is unified or not.
>
> 1. If they always can live on the exact same hierarchy, there's no
> point in having the two separate. Just merge them.
>
> 2. If they need differing levels of granularity, they either need to
> do it completely separately as they do now or have some form of
> dynamic optimization if absolutely necesary.
>
> So, I think that choice is rather separate from other issues. If
> cpuacct is gonna be kept, I'd just keep it separate and warn that it
> incurs extra overhead for the current users if for nothing else.
> Otherwise, kill it or merge it into cpu.

Quite, hence my 'proposal' to remove cpuacct.

There was some whining last time Glauber proposed this, but the one
whining never convinced and has gone away from Linux, so lets just do
this.

Lets make cpuacct print a deprecated msg to dmesg for a few releases and
make cpu do all this.

The co-mounting stuff would have been nice for cpusets as well, knowing
all your tasks are affine to a subset of cpus allows for a few
optimizations (smaller cpumask iterations), but I guess we'll have to do
that dynamically, we'll just have to see how ugly that is.


--
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/