diff options
author | Bjoern B. Brandenburg <bbb@cs.unc.edu> | 2011-11-24 13:42:59 -0500 |
---|---|---|
committer | Bjoern B. Brandenburg <bbb@cs.unc.edu> | 2011-11-24 13:42:59 -0500 |
commit | d1d6e4c300d858c47b834be145f30973bc2921bf (patch) | |
tree | 60f2ee8c7250f6c25246eca7b247132d0e19dcc2 /litmus | |
parent | a7a7f71529d9a6aae02ab3cb64451e036ce9d028 (diff) |
Add option to turn off preemption state tracing
Preemption state tracing is only useful when debugging preemption-
and IPI-related races. Since it creates a lot of clutter in the logs,
this patch turns it off unless explicitly requested.
Diffstat (limited to 'litmus')
-rw-r--r-- | litmus/Kconfig | 14 | ||||
-rw-r--r-- | litmus/preempt.c | 2 |
2 files changed, 16 insertions, 0 deletions
diff --git a/litmus/Kconfig b/litmus/Kconfig index d7fde6f97e14..94b48e199577 100644 --- a/litmus/Kconfig +++ b/litmus/Kconfig | |||
@@ -199,6 +199,20 @@ config SCHED_DEBUG_TRACE_CALLER | |||
199 | 199 | ||
200 | If unsure, say No. | 200 | If unsure, say No. |
201 | 201 | ||
202 | config PREEMPT_STATE_TRACE | ||
203 | bool "Trace preemption state machine transitions" | ||
204 | depends on SCHED_DEBUG_TRACE | ||
205 | default n | ||
206 | help | ||
207 | With this option enabled, each CPU will log when it transitions | ||
208 | states in the preemption state machine. This state machine is | ||
209 | used to determine how to react to IPIs (avoid races with in-flight IPIs). | ||
210 | |||
211 | Warning: this creates a lot of information in the debug trace. Only | ||
212 | recommended when you are debugging preemption-related races. | ||
213 | |||
214 | If unsure, say No. | ||
215 | |||
202 | endmenu | 216 | endmenu |
203 | 217 | ||
204 | endmenu | 218 | endmenu |
diff --git a/litmus/preempt.c b/litmus/preempt.c index ebe2e3461895..5704d0bf4c0b 100644 --- a/litmus/preempt.c +++ b/litmus/preempt.c | |||
@@ -30,8 +30,10 @@ void sched_state_will_schedule(struct task_struct* tsk) | |||
30 | /* Litmus tasks should never be subject to a remote | 30 | /* Litmus tasks should never be subject to a remote |
31 | * set_tsk_need_resched(). */ | 31 | * set_tsk_need_resched(). */ |
32 | BUG_ON(is_realtime(tsk)); | 32 | BUG_ON(is_realtime(tsk)); |
33 | #ifdef CONFIG_PREEMPT_STATE_TRACE | ||
33 | TRACE_TASK(tsk, "set_tsk_need_resched() ret:%p\n", | 34 | TRACE_TASK(tsk, "set_tsk_need_resched() ret:%p\n", |
34 | __builtin_return_address(0)); | 35 | __builtin_return_address(0)); |
36 | #endif | ||
35 | } | 37 | } |
36 | 38 | ||
37 | /* Called by the IPI handler after another CPU called smp_send_resched(). */ | 39 | /* Called by the IPI handler after another CPU called smp_send_resched(). */ |