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 | |
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')
-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 |