Re: [PATCHSET cgroup/for-3.8] cgroup_freezer: allow migrationregardless of freezer state and update locking
From: Matt Helsley
Date: Thu Oct 18 2012 - 21:29:41 EST
On Thu, Oct 18, 2012 at 05:01:53PM -0700, Tejun Heo wrote:
> I probably have chosen the wrong word. I mean that it's a hierarchy
> management feature implemented at the wrong layer. If we want to
> provide cgroup migration locking, it should be implemented at the
> cgroup core layer as a controller independent feature. It's kinda
> interesting the incorrect layering here almost directly caused messy
> locking problem too. I hope we don't need it with (the imaginary)
> proper userland arbitration but even if we do implementing it in
> cgroup proper as a separate feature would be a lot less messy.
Yeah, that would be a nice cleanup too. I guess the ultra-careful way to
remove this feature would be something like:
Add an internal migration restriction (which may or may not be
exported as a userspace interface in a subsequent
Make the cgroup freezer use it.
Make the cgroup freezer WARN_ONCE() when the subsystem is first
mounted. Indicates that the behavior is going to
... time passes ...
Remove the use of the migration "lock" from the cgroup freezer
and the WARN_ONCE().
Which would also make the feature more obvious.
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/