Messages in this thread | | | Date | Wed, 21 Dec 2011 09:53:08 +0800 | From | Li Zefan <> | Subject | Re: [PATCH 1/2] cgroup: Remove unnecessary task_lock before fetching css_set on migration |
| |
Frederic Weisbecker wrote: > When we fetch the css_set of the tasks on cgroup migration, we don't need > anymore to synchronize against cgroup_exit() that could swap the old one > with init_css_set. Now that we are using threadgroup_lock() during > the migrations, we don't need to worry about it anymore. > > Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Good catch!
Reviewed-by: Li Zefan <lizf@cn.fujitsu.com>
> Cc: Tejun Heo <tj@kernel.org> > Cc: Li Zefan <lizf@cn.fujitsu.com> > Cc: Containers <containers@lists.linux-foundation.org> > Cc: Cgroups <cgroups@vger.kernel.org> > Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> > Cc: Oleg Nesterov <oleg@redhat.com> > Cc: Andrew Morton <akpm@linux-foundation.org> > Cc: Paul Menage <paul@paulmenage.org> > Cc: Mandeep Singh Baines <msb@chromium.org> > --- > kernel/cgroup.c | 21 +++++++++++---------- > 1 files changed, 11 insertions(+), 10 deletions(-)
| |