diff options
| author | Tejun Heo <tj@kernel.org> | 2013-01-07 08:51:07 -0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-01-07 08:51:07 -0800 |
| commit | 0772324ae669f787b42fdb9fc5ac2c3d1c0df509 (patch) | |
| tree | 6a7aacbd773a1b3867c6bd2d62c796b79a3579fd /kernel/cpuset.c | |
| parent | 01c889cf4f2e16c9a21376f8583f4a2872d7d1da (diff) | |
| download | lwn-0772324ae669f787b42fdb9fc5ac2c3d1c0df509.tar.gz lwn-0772324ae669f787b42fdb9fc5ac2c3d1c0df509.zip | |
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) struct cpuset *parent; /* - * The cgroup's css_sets list is in use if there are tasks - * in the cpuset; the list is empty if there are none; - * the cs->css.refcnt seems always 0. - */ - if (list_empty(&cs->css.cgroup->css_sets)) - return; - - /* * Find its next-highest non-empty parent, (top cpuset * has online cpus, so can't be empty). */ |