On Tue, Oct 16, 2012 at 03:28:39PM -0700, Tejun Heo wrote: > Hello, > > This patchset updates cgroup_freezer so that > > * Unfreezable kernel tasks don't prevent a cgroup from transitioning > into FROZEN from FREEZING. There's nothing userland can do with or > about such tasks. Seems like a non-problem. Do you have a testcase showing how kernel threads prevent cgroups that should be freezable from being frozen? It used to be that you couldn't move kernel threads out of the root cgroup and the root cgroup was not freezable. So this was never a problem before. Is there some change here that I'm unaware of? > > * Tasks can be moved in and out of a frozen cgroup. Tasks are made to > conform to the state of the new cgroup during migration. This > behavior makes a lot more sense and removes the use of > ->can_attach() which makes co-mounting difficult. One nice aspect of freezing the set of tasks in the cgroup as well as the tasks themselves was you had a fixed set of tasks to work with (from userspace or otherwise). With this change that will no longer be true. This is a userspace-visible behavior change and userspace code may have relied on this feature. Will this work for the CRIU folks? With this patch one of the tasks being checkpointed could become thawed simply by some other process writing the pid into a different cgroup's tasks file. In contrast, with the current code they'd have to explicitly thaw its cgroup first. Cheers, -Matt Helsley -- 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
