diff options
| author | Paul E. McKenney <paul.mckenney@linaro.org> | 2011-12-09 14:43:47 -0800 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2011-12-11 10:33:19 -0800 |
| commit | 4f89b336fd1edf0c88875d0b7fcdc288c7de903d (patch) | |
| tree | 8cadb7d4c326ad01ce7f0da55db8f03af17917e6 /kernel/rcutiny_plugin.h | |
| parent | 70321d447aa1a7cc2d60db16234f43c5a65630e7 (diff) | |
| download | op-kernel-dev-4f89b336fd1edf0c88875d0b7fcdc288c7de903d.zip op-kernel-dev-4f89b336fd1edf0c88875d0b7fcdc288c7de903d.tar.gz | |
rcu: Apply ACCESS_ONCE() to rcu_boost() return value
Both TINY_RCU's and TREE_RCU's implementations of rcu_boost() access
the ->boost_tasks and ->exp_tasks fields without preventing concurrent
changes to these fields. This commit therefore applies ACCESS_ONCE in
order to prevent compiler mischief.
Signed-off-by: Paul E. McKenney <paul.mckenney@linaro.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/rcutiny_plugin.h')
| -rw-r--r-- | kernel/rcutiny_plugin.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/rcutiny_plugin.h b/kernel/rcutiny_plugin.h index dfa97cb..9cb1ae4 100644 --- a/kernel/rcutiny_plugin.h +++ b/kernel/rcutiny_plugin.h @@ -312,8 +312,8 @@ static int rcu_boost(void) rt_mutex_lock(&mtx); rt_mutex_unlock(&mtx); /* Keep lockdep happy. */ - return rcu_preempt_ctrlblk.boost_tasks != NULL || - rcu_preempt_ctrlblk.exp_tasks != NULL; + return ACCESS_ONCE(rcu_preempt_ctrlblk.boost_tasks) != NULL || + ACCESS_ONCE(rcu_preempt_ctrlblk.exp_tasks) != NULL; } /* |
