Hello, On Thu, Feb 02, 2017 at 01:32:19PM -0800, Andy Lutomirski wrote: > > * Thread mode is explicitly enabled on a cgroup by writing "enable" > > into "cgroup.threads" file. The cgroup shouldn't have any child > > cgroups or enabled controllers. > > Why do you need to manually turn it on? That is, couldn't it be > automatic based on what controllers are enabled? This came up already but it's not like some controllers are inherently thread-only. Consider CPU, all in-context CPU cycle consumptions are tied to the thread; however, we also want to be able to account for CPU cycles consumed for, for example, memory reclaim or encryption during writeback. I played with an interface where thread mode is enabled automatically upto the common ancestor of the threads but not only was it complicated to implement but also the eventual behavior was very confusing as the resource domain can change without any active actions from the user. I think keeping things simple is the right choice here. > > * Once enabled, arbitrary sub-hierarchy can be created and threads can > > be put anywhere in the subtree by writing TIDs into "cgroup.threads" > > file. Process granularity and no-internal-process constraint don't > > apply in a threaded subtree. > > I'm a bit worried that this conflates two different things. There's > thread support, i.e. allowing individual threads to be placed into > cgroups. There's also more flexible sub-hierarchy support, i.e. > relaxing no-internal-process constraints. For the "cpuacct" > controller, for example, both of these make sense. But what if > someone writes a controller (directio, for example, just to make > something up) for which thread granularity makes sense but relaxing > no-internal-process constraints does not? If a controller can't possibly define how internal competition should be handled, which is unlikely - the problem is being consistent and sensible, defining something isn't difficult - the controller can simply error out those cases either on configuration or migration. Again, I'm very doubtful we'll need that but if we ever need that denying specific configurations is the best we can do anyway. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
