diff options
author | Tejun Heo <tj@kernel.org> | 2013-01-07 11:51:07 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2013-01-07 11:51:07 -0500 |
commit | 0772324ae669f787b42fdb9fc5ac2c3d1c0df509 (patch) | |
tree | 6a7aacbd773a1b3867c6bd2d62c796b79a3579fd /kernel/cpuset.c | |
parent | 01c889cf4f2e16c9a21376f8583f4a2872d7d1da (diff) |
cpuset: remove fast exit path from remove_tasks_in_empty_cpuset()
The function isn't that hot, the overhead of missing the fast exit is
low, the test itself depends heavily on cgroup internals, and it's
gonna be a hindrance when trying to decouple cpuset locking from
cgroup core. Remove the fast exit path.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
Diffstat (limited to 'kernel/cpuset.c')
-rw-r--r-- | kernel/cpuset.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/kernel/cpuset.c b/kernel/cpuset.c index 854b8bfbc15c..5372b6f5e5b3 100644 --- a/kernel/cpuset.c +++ b/kernel/cpuset.c | |||
@@ -1968,14 +1968,6 @@ static void remove_tasks_in_empty_cpuset(struct cpuset *cs) | |||
1968 | struct cpuset *parent; | 1968 | struct cpuset *parent; |
1969 | 1969 | ||
1970 | /* | 1970 | /* |
1971 | * The cgroup's css_sets list is in use if there are tasks | ||
1972 | * in the cpuset; the list is empty if there are none; | ||
1973 | * the cs->css.refcnt seems always 0. | ||
1974 | */ | ||
1975 | if (list_empty(&cs->css.cgroup->css_sets)) | ||
1976 | return; | ||
1977 | |||
1978 | /* | ||
1979 | * Find its next-highest non-empty parent, (top cpuset | 1971 | * Find its next-highest non-empty parent, (top cpuset |
1980 | * has online cpus, so can't be empty). | 1972 | * has online cpus, so can't be empty). |
1981 | */ | 1973 | */ |