tjh.dev / kernel
Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

hrtimer: Drop unnecessary pointer indirection in hrtimer_expire_entry event

This pointer indirection is a remnant from when ktime_t was a struct,
today it is pointless.

Drop the pointer indirection.

Signed-off-by: Thomas Weißschuh (Schneider Electric) <thomas.weissschuh@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@kernel.org>
Link: https://patch.msgid.link/20260311-hrtimer-cleanups-v1-9-095357392669@linutronix.de

authored by

Thomas Weißschuh (Schneider Electric) and committed by
Thomas Gleixner bd803783 b94c076d

+5 -6
+3 -4
include/trace/events/timer.h
··· 254 254 /** 255 255 * hrtimer_expire_entry - called immediately before the hrtimer callback 256 256 * @hrtimer: pointer to struct hrtimer 257 - * @now: pointer to variable which contains current time of the 258 - * timers base. 257 + * @now: variable which contains current time of the timers base. 259 258 * 260 259 * Allows to determine the timer latency. 261 260 */ 262 261 TRACE_EVENT(hrtimer_expire_entry, 263 262 264 - TP_PROTO(struct hrtimer *hrtimer, ktime_t *now), 263 + TP_PROTO(struct hrtimer *hrtimer, ktime_t now), 265 264 266 265 TP_ARGS(hrtimer, now), 267 266 ··· 272 273 273 274 TP_fast_assign( 274 275 __entry->hrtimer = hrtimer; 275 - __entry->now = *now; 276 + __entry->now = now; 276 277 __entry->function = ACCESS_PRIVATE(hrtimer, function); 277 278 ), 278 279
+2 -2
kernel/time/hrtimer.c
··· 1884 1884 * __run_hrtimer() invocations. 1885 1885 */ 1886 1886 static void __run_hrtimer(struct hrtimer_cpu_base *cpu_base, struct hrtimer_clock_base *base, 1887 - struct hrtimer *timer, ktime_t *now, unsigned long flags) 1887 + struct hrtimer *timer, ktime_t now, unsigned long flags) 1888 1888 __must_hold(&cpu_base->lock) 1889 1889 { 1890 1890 enum hrtimer_restart (*fn)(struct hrtimer *); ··· 1989 1989 if (basenow < hrtimer_get_softexpires(timer)) 1990 1990 break; 1991 1991 1992 - __run_hrtimer(cpu_base, base, timer, &basenow, flags); 1992 + __run_hrtimer(cpu_base, base, timer, basenow, flags); 1993 1993 if (active_mask == HRTIMER_ACTIVE_SOFT) 1994 1994 hrtimer_sync_wait_running(cpu_base, flags); 1995 1995 }