]> git.karo-electronics.de Git - linux-beck.git/commit
cgroup: move module ref handling into rebind_subsystems()
authorTejun Heo <tj@kernel.org>
Fri, 12 Jul 2013 20:38:17 +0000 (13:38 -0700)
committerTejun Heo <tj@kernel.org>
Tue, 16 Jul 2013 11:28:24 +0000 (05:28 -0600)
commit1d5be6b287c8efc879fbe578e2b7bc8f7a38f313
tree68117b91fc21c26754c23721ede5b0b90cf78a5d
parentf172e67cf9d842bc646d0f66792e38435a334b1e
cgroup: move module ref handling into rebind_subsystems()

Module ref handling in cgroup is rather weird.
parse_cgroupfs_options() grabs all the modules for the specified
subsystems.  A module ref is kept if the specified subsystem is newly
bound to the hierarchy.  If not, or the operation fails, the refs are
dropped.  This scatters module ref handling across multiple functions
making it difficult to track.  It also make the function nasty to use
for dynamic subsystem binding which is necessary for the planned
unified hierarchy.

There's nothing which requires the subsystem modules to be pinned
between parse_cgroupfs_options() and rebind_subsystems() in both mount
and remount paths.  parse_cgroupfs_options() can just parse and
rebind_subsystems() can handle pinning the subsystems that it wants to
bind, which is a natural part of its task - binding - anyway.

Move module ref handling into rebind_subsystems() which makes the code
a lot simpler - modules are gotten iff it's gonna be bound and put iff
unbound or binding fails.

v2: Li pointed out that if a controller module is unloaded between
    parsing and binding, rebind_subsystems() won't notice the missing
    controller as it only iterates through existing controllers.  Fix
    it by updating rebind_subsystems() to compare @added_mask to
    @pinned and fail with -ENOENT if they don't match.

Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
kernel/cgroup.c