diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2013-08-18 14:59:25 -0400 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2013-09-23 12:18:17 -0400 |
commit | 15f5191b6acbbb38029b06284e8fd20275e7cfe8 (patch) | |
tree | cadd66927a556a5584be2fc453440d88130560a0 /kernel/rcutiny.c | |
parent | 69a79bb12a81024d718e73c52e886907a3777b34 (diff) |
rcu: Avoid sparse warnings in rcu_nocb_wake trace event
The event-tracing macros do not like bool tracing arguments, so this
commit makes them be of type char. This change has the knock-on effect
of making it illegal to pass a pointer into one of these arguments, so
also change rcutiny's first call to trace_rcu_batch_end() to convert
from pointer to boolean, prefixing with "!!".
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/rcutiny.c')
-rw-r--r-- | kernel/rcutiny.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/rcutiny.c b/kernel/rcutiny.c index 9ed6075dc562..80b6e273f1c5 100644 --- a/kernel/rcutiny.c +++ b/kernel/rcutiny.c | |||
@@ -273,7 +273,7 @@ static void __rcu_process_callbacks(struct rcu_ctrlblk *rcp) | |||
273 | if (&rcp->rcucblist == rcp->donetail) { | 273 | if (&rcp->rcucblist == rcp->donetail) { |
274 | RCU_TRACE(trace_rcu_batch_start(rcp->name, 0, 0, -1)); | 274 | RCU_TRACE(trace_rcu_batch_start(rcp->name, 0, 0, -1)); |
275 | RCU_TRACE(trace_rcu_batch_end(rcp->name, 0, | 275 | RCU_TRACE(trace_rcu_batch_end(rcp->name, 0, |
276 | ACCESS_ONCE(rcp->rcucblist), | 276 | !!ACCESS_ONCE(rcp->rcucblist), |
277 | need_resched(), | 277 | need_resched(), |
278 | is_idle_task(current), | 278 | is_idle_task(current), |
279 | false)); | 279 | false)); |