diff options
| author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2010-02-25 17:06:47 -0500 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2010-02-26 02:20:46 -0500 |
| commit | d9f1bb6ad7fc53c406706f47858dd5ff030b14a3 (patch) | |
| tree | 4af519d68b396e372e6a5afbb5e968de3bacd123 /include/linux | |
| parent | 056ba4a9bea5f32781a36b797c562fb731e5eaa6 (diff) | |
rcu: Make rcu_read_lock_sched_held() take boot time into account
Before the scheduler starts, all tasks are non-preemptible by
definition. So, during that time, rcu_read_lock_sched_held()
needs to always return "true". This patch makes that be so.
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: laijs@cn.fujitsu.com
Cc: dipankar@in.ibm.com
Cc: mathieu.desnoyers@polymtl.ca
Cc: josh@joshtriplett.org
Cc: dvhltc@us.ibm.com
Cc: niv@us.ibm.com
Cc: peterz@infradead.org
Cc: rostedt@goodmis.org
Cc: Valdis.Kletnieks@vt.edu
Cc: dhowells@redhat.com
LKML-Reference: <1267135607-7056-2-git-send-email-paulmck@linux.vnet.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/rcupdate.h | 4 | ||||
| -rw-r--r-- | include/linux/rcutiny.h | 4 | ||||
| -rw-r--r-- | include/linux/rcutree.h | 1 |
3 files changed, 3 insertions, 6 deletions
diff --git a/include/linux/rcupdate.h b/include/linux/rcupdate.h index 1a4de31bd7b4..fcea332a8424 100644 --- a/include/linux/rcupdate.h +++ b/include/linux/rcupdate.h | |||
| @@ -62,6 +62,8 @@ extern int sched_expedited_torture_stats(char *page); | |||
| 62 | 62 | ||
| 63 | /* Internal to kernel */ | 63 | /* Internal to kernel */ |
| 64 | extern void rcu_init(void); | 64 | extern void rcu_init(void); |
| 65 | extern int rcu_scheduler_active; | ||
| 66 | extern void rcu_scheduler_starting(void); | ||
| 65 | 67 | ||
| 66 | #if defined(CONFIG_TREE_RCU) || defined(CONFIG_TREE_PREEMPT_RCU) | 68 | #if defined(CONFIG_TREE_RCU) || defined(CONFIG_TREE_PREEMPT_RCU) |
| 67 | #include <linux/rcutree.h> | 69 | #include <linux/rcutree.h> |
| @@ -140,7 +142,7 @@ static inline int rcu_read_lock_sched_held(void) | |||
| 140 | 142 | ||
| 141 | if (debug_locks) | 143 | if (debug_locks) |
| 142 | lockdep_opinion = lock_is_held(&rcu_sched_lock_map); | 144 | lockdep_opinion = lock_is_held(&rcu_sched_lock_map); |
| 143 | return lockdep_opinion || preempt_count() != 0; | 145 | return lockdep_opinion || preempt_count() != 0 || !rcu_scheduler_active; |
| 144 | } | 146 | } |
| 145 | 147 | ||
| 146 | #else /* #ifdef CONFIG_DEBUG_LOCK_ALLOC */ | 148 | #else /* #ifdef CONFIG_DEBUG_LOCK_ALLOC */ |
diff --git a/include/linux/rcutiny.h b/include/linux/rcutiny.h index 2b70d4e37383..a5195875480a 100644 --- a/include/linux/rcutiny.h +++ b/include/linux/rcutiny.h | |||
| @@ -105,10 +105,6 @@ static inline void rcu_exit_nohz(void) | |||
| 105 | 105 | ||
| 106 | #endif /* #else #ifdef CONFIG_NO_HZ */ | 106 | #endif /* #else #ifdef CONFIG_NO_HZ */ |
| 107 | 107 | ||
| 108 | static inline void rcu_scheduler_starting(void) | ||
| 109 | { | ||
| 110 | } | ||
| 111 | |||
| 112 | static inline void exit_rcu(void) | 108 | static inline void exit_rcu(void) |
| 113 | { | 109 | { |
| 114 | } | 110 | } |
diff --git a/include/linux/rcutree.h b/include/linux/rcutree.h index 704a010f686c..42cc3a04779e 100644 --- a/include/linux/rcutree.h +++ b/include/linux/rcutree.h | |||
| @@ -35,7 +35,6 @@ struct notifier_block; | |||
| 35 | extern void rcu_sched_qs(int cpu); | 35 | extern void rcu_sched_qs(int cpu); |
| 36 | extern void rcu_bh_qs(int cpu); | 36 | extern void rcu_bh_qs(int cpu); |
| 37 | extern int rcu_needs_cpu(int cpu); | 37 | extern int rcu_needs_cpu(int cpu); |
| 38 | extern void rcu_scheduler_starting(void); | ||
| 39 | extern int rcu_expedited_torture_stats(char *page); | 38 | extern int rcu_expedited_torture_stats(char *page); |
| 40 | 39 | ||
| 41 | #ifdef CONFIG_TREE_PREEMPT_RCU | 40 | #ifdef CONFIG_TREE_PREEMPT_RCU |
