]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commitdiff
sched: silence PROVE_RCU in sched_fork()
authorPeter Zijlstra <peterz@infradead.org>
Tue, 22 Jun 2010 09:44:53 +0000 (11:44 +0200)
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>
Wed, 23 Jun 2010 22:14:09 +0000 (15:14 -0700)
Because cgroup_fork() is ran before sched_fork() [ from copy_process() ]
and the child's pid is not yet visible the child is pinned to its
cgroup. Therefore we can silence this warning.

A nicer solution would be moving cgroup_fork() to right after
dup_task_struct() and exclude PF_STARTING from task_subsys_state().

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Reviewed-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
kernel/sched.c

index f8b8996228dd7d8de9ee49a845a193921f1b8a5d..a2d215d132f630ec8ed7dcf03af24b3c8524d966 100644 (file)
@@ -2494,7 +2494,16 @@ void sched_fork(struct task_struct *p, int clone_flags)
        if (p->sched_class->task_fork)
                p->sched_class->task_fork(p);
 
+       /*
+        * The child is not yet in the pid-hash so no cgroup attach races,
+        * and the cgroup is pinned to this child due to cgroup_fork()
+        * is ran before sched_fork().
+        *
+        * Silence PROVE_RCU.
+        */
+       rcu_read_lock();
        set_task_cpu(p, cpu);
+       rcu_read_unlock();
 
 #if defined(CONFIG_SCHEDSTATS) || defined(CONFIG_TASK_DELAY_ACCT)
        if (likely(sched_info_on()))