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
--
To unsubscribe from this list: send the line "unsubscribe cgroups" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
