diff options
| author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-07-24 18:28:09 -0400 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-08-30 19:03:42 -0400 |
| commit | 74de6960c99d8df0d09fb29a7b014cb9c5571e2b (patch) | |
| tree | 2399404f1a3a043609c1f998c4c5f836e1100244 /kernel/rcu | |
| parent | 7e28c5af4ef6b539334aa5de40feca0c041c94df (diff) | |
rcu: Provide functions for determining if call_rcu() has been invoked
This commit adds rcu_head_init() and rcu_head_after_call_rcu() functions
to help RCU users detect when another CPU has passed the specified
rcu_head structure and function to call_rcu(). The rcu_head_init()
should be invoked before making the structure visible to RCU readers,
and then the rcu_head_after_call_rcu() may be invoked from within
an RCU read-side critical section on an rcu_head structure that
was obtained during a traversal of the data structure in question.
The rcu_head_after_call_rcu() function will return true if the rcu_head
structure has already been passed (with the specified function) to
call_rcu(), otherwise it will return false.
If rcu_head_init() has not been invoked on the rcu_head structure
or if the rcu_head (AKA callback) has already been invoked, then
rcu_head_after_call_rcu() will do WARN_ON_ONCE().
Reported-by: NeilBrown <neilb@suse.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
[ paulmck: Apply neilb naming feedback. ]
Diffstat (limited to 'kernel/rcu')
| -rw-r--r-- | kernel/rcu/rcu.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/kernel/rcu/rcu.h b/kernel/rcu/rcu.h index 5dec94509a7e..4c56c1d98fb3 100644 --- a/kernel/rcu/rcu.h +++ b/kernel/rcu/rcu.h | |||
| @@ -224,6 +224,7 @@ void kfree(const void *); | |||
| 224 | */ | 224 | */ |
| 225 | static inline bool __rcu_reclaim(const char *rn, struct rcu_head *head) | 225 | static inline bool __rcu_reclaim(const char *rn, struct rcu_head *head) |
| 226 | { | 226 | { |
| 227 | rcu_callback_t f; | ||
| 227 | unsigned long offset = (unsigned long)head->func; | 228 | unsigned long offset = (unsigned long)head->func; |
| 228 | 229 | ||
| 229 | rcu_lock_acquire(&rcu_callback_map); | 230 | rcu_lock_acquire(&rcu_callback_map); |
| @@ -234,7 +235,9 @@ static inline bool __rcu_reclaim(const char *rn, struct rcu_head *head) | |||
| 234 | return true; | 235 | return true; |
| 235 | } else { | 236 | } else { |
| 236 | RCU_TRACE(trace_rcu_invoke_callback(rn, head);) | 237 | RCU_TRACE(trace_rcu_invoke_callback(rn, head);) |
| 237 | head->func(head); | 238 | f = head->func; |
| 239 | WRITE_ONCE(head->func, (rcu_callback_t)0L); | ||
| 240 | f(head); | ||
| 238 | rcu_lock_release(&rcu_callback_map); | 241 | rcu_lock_release(&rcu_callback_map); |
| 239 | return false; | 242 | return false; |
| 240 | } | 243 | } |
