Re: [PATCH v4 0/2] cgroup: allow management of subtrees by new cgroup namespaces

From: Tejun Heo
Date: Fri May 20 2016 - 12:04:12 EST


Hello, James.

On Fri, May 20, 2016 at 11:30:58AM -0400, James Bottomley wrote:
> Given it's merge window time, I haven't yet had time to look at the
> patch, but I can tell you why it (or something like it) is necessary:
> unprivileged containers need to be able to set up cgroups as well as
> namespaces, so we do need a way for the user ns owner to modify cgroups
> in their default configuration otherwise cgroups just won't fit into
> the unprivileged model. Whether this should be through the cgroup ns

That can be allowed by delegating a cgroup sub-hierarchy to the
unpriviledged user. Sub-cgroups created by that user will be owned by
that user. IOW, if the owner of the cgroup hierarchy doesn't
explicitly delegate the subtree, the unpriv user can't have the
subtree. This is true for regular use cases and shouldn't be
different for namespaces.

> is up for debate, as is how we should actually allow this to happen and
> what we should present to the user ns owner, but we do need a way to do
> this.
>
> Delegation can't be through chmodding in this case because the user ns
> owner can't chmod something owned by init_user_ns root.

I still don't see why the existing mechanisms (including userns if
delegation from inside a ns is necessary) aren't enough. And even if
we need something, I'd much prefer not to add a behavior which wildly
deviates from the usual rules.

Thanks.

--
tejun