diff options
author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2012-11-29 03:46:05 -0500 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2013-02-07 18:10:39 -0500 |
commit | 3bc97a782cc8c112f64a25143452b06206364cc8 (patch) | |
tree | b09b1d1be6158948d9bb0f848dcd3d66f3b17737 | |
parent | ab4d2986e44c589aa1b647d7da5e21c2707babea (diff) |
srcu: Remove checks preventing offline CPUs from calling srcu_read_lock()
SRCU has its own statemachine and no longer relies on normal RCU.
Its read-side critical section can now be used by an offline CPU, so this
commit removes the check and the comments, reverting the SRCU portion
of c0d6d01b (rcu: Check for illegal use of RCU from offlined CPUs).
It also makes the code match the comments in whatisRCU.txt:
g. Do you need read-side critical sections that are respected
even though they are in the middle of the idle loop, during
user-mode execution, or on an offlined CPU? If so, SRCU is the
only choice that will work for you.
[ paulmck: There is at least one remaining issue, namely use of lockdep
with tracing enabled. ]
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
-rw-r--r-- | include/linux/srcu.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/include/linux/srcu.h b/include/linux/srcu.h index 6eb691b08358..ae3ee39eb699 100644 --- a/include/linux/srcu.h +++ b/include/linux/srcu.h | |||
@@ -163,9 +163,6 @@ void srcu_barrier(struct srcu_struct *sp); | |||
163 | * power mode. This way we can notice an extended quiescent state to | 163 | * power mode. This way we can notice an extended quiescent state to |
164 | * other CPUs that started a grace period. Otherwise we would delay any | 164 | * other CPUs that started a grace period. Otherwise we would delay any |
165 | * grace period as long as we run in the idle task. | 165 | * grace period as long as we run in the idle task. |
166 | * | ||
167 | * Similarly, we avoid claiming an SRCU read lock held if the current | ||
168 | * CPU is offline. | ||
169 | */ | 166 | */ |
170 | static inline int srcu_read_lock_held(struct srcu_struct *sp) | 167 | static inline int srcu_read_lock_held(struct srcu_struct *sp) |
171 | { | 168 | { |
@@ -173,8 +170,6 @@ static inline int srcu_read_lock_held(struct srcu_struct *sp) | |||
173 | return 1; | 170 | return 1; |
174 | if (rcu_is_cpu_idle()) | 171 | if (rcu_is_cpu_idle()) |
175 | return 0; | 172 | return 0; |
176 | if (!rcu_lockdep_current_cpu_online()) | ||
177 | return 0; | ||
178 | return lock_is_held(&sp->dep_map); | 173 | return lock_is_held(&sp->dep_map); |
179 | } | 174 | } |
180 | 175 | ||