diff options
author | Paul E. McKenney <paul.mckenney@linaro.org> | 2012-01-23 15:41:26 -0500 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2012-02-21 12:06:03 -0500 |
commit | c0d6d01bffdce19fa19baad6cb8cc3eed7bfd6f5 (patch) | |
tree | ea4c20ddc87b94d7bef2402fe6dbe3186be263c2 /kernel/rcutree.c | |
parent | 24cd7fd0eaa0d9f5e197ff77a83b006a86696068 (diff) |
rcu: Check for illegal use of RCU from offlined CPUs
Although it is legal to use RCU during early boot, it is anything
but legal to use RCU at runtime from an offlined CPU. After all, RCU
explicitly ignores offlined CPUs. This commit therefore adds checks
for runtime use of RCU from offlined CPUs.
These checks are not perfect, in particular, they can be subverted
through use of things like rcu_dereference_raw(). Note that it is not
possible to put checks in rcu_read_lock() and friends due to the fact
that these primitives are used in code that might be used under either
RCU or lock-based protection, which means that checking rcu_read_lock()
gets you fat piles of false positives.
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/rcutree.c')
-rw-r--r-- | kernel/rcutree.c | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/kernel/rcutree.c b/kernel/rcutree.c index dccd2f78db4e..bcf7db2f2fd2 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c | |||
@@ -591,6 +591,35 @@ int rcu_is_cpu_idle(void) | |||
591 | } | 591 | } |
592 | EXPORT_SYMBOL(rcu_is_cpu_idle); | 592 | EXPORT_SYMBOL(rcu_is_cpu_idle); |
593 | 593 | ||
594 | #ifdef CONFIG_HOTPLUG_CPU | ||
595 | |||
596 | /* | ||
597 | * Is the current CPU online? Disable preemption to avoid false positives | ||
598 | * that could otherwise happen due to the current CPU number being sampled, | ||
599 | * this task being preempted, its old CPU being taken offline, resuming | ||
600 | * on some other CPU, then determining that its old CPU is now offline. | ||
601 | * It is OK to use RCU on an offline processor during initial boot, hence | ||
602 | * the check for rcu_scheduler_fully_active. | ||
603 | * | ||
604 | * Disable checking if in an NMI handler because we cannot safely report | ||
605 | * errors from NMI handlers anyway. | ||
606 | */ | ||
607 | bool rcu_lockdep_current_cpu_online(void) | ||
608 | { | ||
609 | bool ret; | ||
610 | |||
611 | if (in_nmi()) | ||
612 | return 1; | ||
613 | preempt_disable(); | ||
614 | ret = cpu_online(smp_processor_id()) || | ||
615 | !rcu_scheduler_fully_active; | ||
616 | preempt_enable(); | ||
617 | return ret; | ||
618 | } | ||
619 | EXPORT_SYMBOL_GPL(rcu_lockdep_current_cpu_online); | ||
620 | |||
621 | #endif /* #ifdef CONFIG_HOTPLUG_CPU */ | ||
622 | |||
594 | #endif /* #ifdef CONFIG_PROVE_RCU */ | 623 | #endif /* #ifdef CONFIG_PROVE_RCU */ |
595 | 624 | ||
596 | /** | 625 | /** |