Re: [patch 0/2] cpusets, cpu_cgroup: disallow attaching kthreadd

From: Tejun Heo
Date: Thu Apr 05 2012 - 19:13:15 EST


Hey, David.

On Thu, Apr 05, 2012 at 04:05:43PM -0700, David Rientjes wrote:
> The cgroups of interest here, cpusets and cpu, which have restrictions for
> PF_THREAD_BOUND threads is disjoint from memcg and cpuacct, which I'm
> saying it's useful for. There's no requirement to mount them all. But if
> you do try to do memory accounting in the presence of cpusets, though, it
> should be restricted.
>
> We could still move to a single hierarchy, though, if we introduced a
> change to allow PF_THREAD_BOUND to attach to both cpusets and cpu iff
> its bound cpu is allowed and then do -EINVAL if its changed while still
> attached.

Yeah, making the affected controllers to deal with them somehow
definitely is an option and will probably have to be chosen for some
other cases too (e.g. rt tasks). I still feel lukewarm about the
ability to put kthreadd to !root cgroup tho. I mean, for any kind of
sane accounting, forked kthreads would have to be further classified
and where kthreadd lives and fork happens wouldn't matter all that
much. It feels like we're fussing over stuff which isn't that
significant either way.

Thanks.

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