diff options
author | Badhri Jagan Sridharan <badhri@google.com> | 2015-05-07 19:20:34 -0400 |
---|---|---|
committer | John Stultz <john.stultz@linaro.org> | 2015-05-22 13:36:06 -0400 |
commit | 4e413e8526aa53393d0b3d9ecbdb0436203586ee (patch) | |
tree | ae164d7a2a9d5db9c6c13ee49caf84d1e6aea979 | |
parent | 57d05a93ada77c4f8a6112cbc867a2948dce7991 (diff) |
tracing: timer: Add deferrable flag to timer_start
The timer_start event now shows whether the timer is
deferrable in case of a low-res timer. The debug_activate
function now includes a deferrable flag while calling
the trace_timer_start event.
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@kernel.org>
Acked-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Badhri Jagan Sridharan <Badhri@google.com>
[jstultz: Fixed minor whitespace and grammer tweaks
pointed out by Ingo]
Signed-off-by: John Stultz <john.stultz@linaro.org>
-rw-r--r-- | include/trace/events/timer.h | 13 | ||||
-rw-r--r-- | kernel/time/timer.c | 2 |
2 files changed, 10 insertions, 5 deletions
diff --git a/include/trace/events/timer.h b/include/trace/events/timer.h index 68c2c2000f02..d7abef1fe6e0 100644 --- a/include/trace/events/timer.h +++ b/include/trace/events/timer.h | |||
@@ -43,15 +43,18 @@ DEFINE_EVENT(timer_class, timer_init, | |||
43 | */ | 43 | */ |
44 | TRACE_EVENT(timer_start, | 44 | TRACE_EVENT(timer_start, |
45 | 45 | ||
46 | TP_PROTO(struct timer_list *timer, unsigned long expires), | 46 | TP_PROTO(struct timer_list *timer, |
47 | unsigned long expires, | ||
48 | unsigned int deferrable), | ||
47 | 49 | ||
48 | TP_ARGS(timer, expires), | 50 | TP_ARGS(timer, expires, deferrable), |
49 | 51 | ||
50 | TP_STRUCT__entry( | 52 | TP_STRUCT__entry( |
51 | __field( void *, timer ) | 53 | __field( void *, timer ) |
52 | __field( void *, function ) | 54 | __field( void *, function ) |
53 | __field( unsigned long, expires ) | 55 | __field( unsigned long, expires ) |
54 | __field( unsigned long, now ) | 56 | __field( unsigned long, now ) |
57 | __field( unsigned int, deferrable ) | ||
55 | ), | 58 | ), |
56 | 59 | ||
57 | TP_fast_assign( | 60 | TP_fast_assign( |
@@ -59,11 +62,13 @@ TRACE_EVENT(timer_start, | |||
59 | __entry->function = timer->function; | 62 | __entry->function = timer->function; |
60 | __entry->expires = expires; | 63 | __entry->expires = expires; |
61 | __entry->now = jiffies; | 64 | __entry->now = jiffies; |
65 | __entry->deferrable = deferrable; | ||
62 | ), | 66 | ), |
63 | 67 | ||
64 | TP_printk("timer=%p function=%pf expires=%lu [timeout=%ld]", | 68 | TP_printk("timer=%p function=%pf expires=%lu [timeout=%ld] defer=%c", |
65 | __entry->timer, __entry->function, __entry->expires, | 69 | __entry->timer, __entry->function, __entry->expires, |
66 | (long)__entry->expires - __entry->now) | 70 | (long)__entry->expires - __entry->now, |
71 | __entry->deferrable > 0 ? 'y':'n') | ||
67 | ); | 72 | ); |
68 | 73 | ||
69 | /** | 74 | /** |
diff --git a/kernel/time/timer.c b/kernel/time/timer.c index d4af7c56c95d..7775d454a204 100644 --- a/kernel/time/timer.c +++ b/kernel/time/timer.c | |||
@@ -650,7 +650,7 @@ static inline void | |||
650 | debug_activate(struct timer_list *timer, unsigned long expires) | 650 | debug_activate(struct timer_list *timer, unsigned long expires) |
651 | { | 651 | { |
652 | debug_timer_activate(timer); | 652 | debug_timer_activate(timer); |
653 | trace_timer_start(timer, expires); | 653 | trace_timer_start(timer, expires, tbase_get_deferrable(timer->base)); |
654 | } | 654 | } |
655 | 655 | ||
656 | static inline void debug_deactivate(struct timer_list *timer) | 656 | static inline void debug_deactivate(struct timer_list *timer) |