From: Kienan Stewart Date: Mon, 22 Jan 2024 16:33:39 +0000 (-0500) Subject: Fix: timer_start changed in linux 6.8.0-rc1 X-Git-Url: http://git.liburcu.org/?p=lttng-modules.git;a=commitdiff_plain;h=64d3007be5b53a3cef2a51c481a17a4f8f183df2 Fix: timer_start changed in linux 6.8.0-rc1 See upstream commit commit dbcdcb62b59db2cf6a24113873b90da15c6f0b19 Author: Anna-Maria Behnsen Date: Fri Dec 1 10:26:26 2023 +0100 tracing/timers: Enhance timer_start tracepoint For starting a timer, the timer is enqueued into a bucket of the timer wheel. The bucket expiry is the defacto expiry of the timer but it is not equal the timer expiry because of increasing granularity when bucket is in a higher level of the wheel. To be able to figure out in a trace whether a timer expired in time or not, the bucket expiry time is required as well. Add bucket expiry time to the timer_start tracepoint and thereby simplify the arguments. Change-Id: I4868092765745b1efd0c48f13c0b837f2007dcb6 Signed-off-by: Kienan Stewart Signed-off-by: Mathieu Desnoyers --- diff --git a/include/instrumentation/events/timer.h b/include/instrumentation/events/timer.h index 843a3f67..193650bb 100644 --- a/include/instrumentation/events/timer.h +++ b/include/instrumentation/events/timer.h @@ -77,6 +77,29 @@ LTTNG_TRACEPOINT_EVENT_INSTANCE(timer_class, timer_init, TP_ARGS(timer) ) +#if (LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(6,8,0)) +/** + * timer_start - called when the timer is started + * @timer: pointer to struct timer_list + * @expires: the timers expiry time + * @flags: the timers expiry time + */ +LTTNG_TRACEPOINT_EVENT(timer_start, + + TP_PROTO(struct timer_list *timer, unsigned long bucket_expiry), + + TP_ARGS(timer, bucket_expiry), + + TP_FIELDS( + ctf_integer_hex(void *, timer, timer) + ctf_integer_hex(void *, function, timer->function) + ctf_integer(unsigned long, expires, timer->expires) + ctf_integer(unsigned long, bucket_expiry, bucket_expiry) + ctf_integer(unsigned long, now, jiffies) + ctf_integer(unsigned int, flags, timer->flags) + ) +) +#else /** * timer_start - called when the timer is started * @timer: pointer to struct timer_list @@ -98,6 +121,7 @@ LTTNG_TRACEPOINT_EVENT(timer_start, ctf_integer(unsigned int, flags, flags) ) ) +#endif #if (LTTNG_LINUX_VERSION_CODE >= LTTNG_KERNEL_VERSION(5,2,0) || \ LTTNG_RHEL_KERNEL_RANGE(4,18,0,193,0,0, 4,19,0,0,0,0))