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

diff --git a/include/trace/events/timer.h b/include/trace/events/timer.h
index a54613f59e55bf89b6908012b32ff6a283eb4746..07cbb9836b91ef1e96c8b25db06b5659e5a5de56 100644 (file)
--- a/include/trace/events/timer.h
+++ b/include/trace/events/timer.h
@@ -254,14 +254,13 @@ TRACE_EVENT(hrtimer_start,
 /**
  * hrtimer_expire_entry - called immediately before the hrtimer callback
  * @hrtimer:   pointer to struct hrtimer
- * @now:       pointer to variable which contains current time of the
- *             timers base.
+ * @now:       variable which contains current time of the timers base.
  *
  * Allows to determine the timer latency.
  */
 TRACE_EVENT(hrtimer_expire_entry,
 
-       TP_PROTO(struct hrtimer *hrtimer, ktime_t *now),
+       TP_PROTO(struct hrtimer *hrtimer, ktime_t now),
 
        TP_ARGS(hrtimer, now),
 
@@ -273,7 +272,7 @@ TRACE_EVENT(hrtimer_expire_entry,
 
        TP_fast_assign(
                __entry->hrtimer        = hrtimer;
-               __entry->now            = *now;
+               __entry->now            = now;
                __entry->function       = ACCESS_PRIVATE(hrtimer, function);
        ),
 

diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c
index c7e7d3a0d6bbbc5e83f944c894f1fae5991d6ae0..a71e2caa14027c9d252da1a7d6d86d8b9c65211e 100644 (file)
--- a/kernel/time/hrtimer.c
+++ b/kernel/time/hrtimer.c
@@ -1884,7 +1884,7 @@ EXPORT_SYMBOL_GPL(hrtimer_active);
  * __run_hrtimer() invocations.
  */
 static void __run_hrtimer(struct hrtimer_cpu_base *cpu_base, struct hrtimer_clock_base *base,
-                         struct hrtimer *timer, ktime_t *now, unsigned long flags)
+                         struct hrtimer *timer, ktime_t now, unsigned long flags)
        __must_hold(&cpu_base->lock)
 {
        enum hrtimer_restart (*fn)(struct hrtimer *);
@@ -1989,7 +1989,7 @@ static void __hrtimer_run_queues(struct hrtimer_cpu_base *cpu_base, ktime_t now,
                        if (basenow < hrtimer_get_softexpires(timer))
                                break;
 
-                       __run_hrtimer(cpu_base, base, timer, &basenow, flags);
+                       __run_hrtimer(cpu_base, base, timer, basenow, flags);
                        if (active_mask == HRTIMER_ACTIVE_SOFT)
                                hrtimer_sync_wait_running(cpu_base, flags);
                }