diff options
author | Christoph Lameter <cl@linux.com> | 2010-12-08 10:22:55 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2010-12-17 09:07:19 -0500 |
commit | 909ea96468096b07fbb41aaf69be060d92bd9271 (patch) | |
tree | a7e015edd96b5f674874fe78cdd889769e130a2a /kernel/rcutree.c | |
parent | 780f36d8b3fa9572f731d4fb85067b2e45e6f993 (diff) |
core: Replace __get_cpu_var with __this_cpu_read if not used for an address.
__get_cpu_var() can be replaced with this_cpu_read and will then use a
single read instruction with implied address calculation to access the
correct per cpu instance.
However, the address of a per cpu variable passed to __this_cpu_read()
cannot be determined (since it's an implied address conversion through
segment prefixes). Therefore apply this only to uses of __get_cpu_var
where the address of the variable is not used.
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Hugh Dickins <hughd@google.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Acked-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/rcutree.c')
-rw-r--r-- | kernel/rcutree.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/rcutree.c b/kernel/rcutree.c index ccdc04c47981..aeebf772d6a2 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c | |||
@@ -367,8 +367,8 @@ void rcu_irq_exit(void) | |||
367 | WARN_ON_ONCE(rdtp->dynticks & 0x1); | 367 | WARN_ON_ONCE(rdtp->dynticks & 0x1); |
368 | 368 | ||
369 | /* If the interrupt queued a callback, get out of dyntick mode. */ | 369 | /* If the interrupt queued a callback, get out of dyntick mode. */ |
370 | if (__get_cpu_var(rcu_sched_data).nxtlist || | 370 | if (__this_cpu_read(rcu_sched_data.nxtlist) || |
371 | __get_cpu_var(rcu_bh_data).nxtlist) | 371 | __this_cpu_read(rcu_bh_data.nxtlist)) |
372 | set_need_resched(); | 372 | set_need_resched(); |
373 | } | 373 | } |
374 | 374 | ||