diff options
| author | Tejun Heo <tj@kernel.org> | 2013-06-28 21:08:27 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-07-16 05:28:24 -0600 |
| commit | a698b4488ab98deef6c3beeba3e27fea17650132 (patch) | |
| tree | b165584741f936387a9bbb66e879f52fbcb0ea71 /kernel/cgroup.c | |
| parent | 1d5be6b287c8efc879fbe578e2b7bc8f7a38f313 (diff) | |
| download | op-kernel-dev-a698b4488ab98deef6c3beeba3e27fea17650132.zip op-kernel-dev-a698b4488ab98deef6c3beeba3e27fea17650132.tar.gz | |
cgroup: remove gratuituous BUG_ON()s from rebind_subsystems()
rebind_subsystems() performs santiy checks even on subsystems which
aren't specified to be added or removed and the checks aren't all that
useful given that these are in a very cold path while the violations
they check would trip up in much hotter paths.
Let's remove these from rebind_subsystems().
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
Diffstat (limited to 'kernel/cgroup.c')
| -rw-r--r-- | kernel/cgroup.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 2a8cf1a..345fac8 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c @@ -1077,15 +1077,6 @@ static int rebind_subsystems(struct cgroupfs_root *root, /* subsystem is now free - drop reference on module */ module_put(ss->module); root->subsys_mask &= ~bit; - } else if (bit & root->subsys_mask) { - /* Subsystem state should already exist */ - BUG_ON(!cgrp->subsys[i]); -#ifdef CONFIG_MODULE_UNLOAD - BUG_ON(ss->module && !module_refcount(ss->module)); -#endif - } else { - /* Subsystem state shouldn't exist */ - BUG_ON(cgrp->subsys[i]); } } |
