On Tue, Jun 25, 2013 at 10:02:38AM +0800, Li Zefan wrote: > > @@ -5046,8 +5049,8 @@ static const struct file_operations proc > > void cgroup_fork(struct task_struct *child) > > { > > task_lock(current); > > + get_css_set(task_css_set(current)); > > child->cgroups = current->cgroups; > > While we use RCU_INIT_POINTER() in cgroup_exit(), we don't need to use it here? Yeap, because both are RCU pointers. There's no cross (sparse) address space assignment going on. Thanks. -- tejun _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/containers