diff options
| author | Li Zhong <zhong@linux.vnet.ibm.com> | 2012-11-28 17:15:21 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2012-11-28 06:02:39 -0800 |
| commit | fddfb02ad0d0d3b479c2a26a8ae7e6411b34706b (patch) | |
| tree | f3d8aa9e526486f4d07fc6f0a3619ce98ffeb3b9 /kernel/cgroup.c | |
| parent | 811d8d6ff59cbc7d618dfa2cd339ba6c3691a7eb (diff) | |
| download | lwn-fddfb02ad0d0d3b479c2a26a8ae7e6411b34706b.tar.gz lwn-fddfb02ad0d0d3b479c2a26a8ae7e6411b34706b.zip | |
cgroup: move list add after list head initilization
2243076ad1 ("cgroup: initialize cgrp->allcg_node in
init_cgroup_housekeeping()") initializes cgrp->allcg_node in
init_cgroup_housekeeping(). Then in init_cgroup_root(), we should
call init_cgroup_housekeeping() before adding it to &root->allcg_list;
otherwise, we are initializing an entry already in a list.
Signed-off-by: Li Zhong <zhong@linux.vnet.ibm.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/cgroup.c')
| -rw-r--r-- | kernel/cgroup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 55a0a770a5a2..c02b05560d10 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c @@ -1401,8 +1401,8 @@ static void init_cgroup_root(struct cgroupfs_root *root) root->number_of_cgroups = 1; cgrp->root = root; cgrp->top_cgroup = cgrp; - list_add_tail(&cgrp->allcg_node, &root->allcg_list); init_cgroup_housekeeping(cgrp); + list_add_tail(&cgrp->allcg_node, &root->allcg_list); } static bool init_root_id(struct cgroupfs_root *root) |