[PATCH 2/2] clocksource: Fix opencoded cpumask_next_wrap()

From: Yury Norov
Date: Sat Jun 07 2025 - 10:11:45 EST


From: "Yury Norov [NVIDIA]" <yury.norov@xxxxxxxxx>

cpumask_next_wrap() is more verbose and efficient comparing to
cpumask_next() followed by cpumask_first().

Acked-by: John Stultz <jstultz@xxxxxxxxxx>
Signed-off-by: "Yury Norov [NVIDIA]" <yury.norov@xxxxxxxxx>
---
kernel/time/clocksource.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/kernel/time/clocksource.c b/kernel/time/clocksource.c
index a2f2e9f4d37b..e400fe150f9d 100644
--- a/kernel/time/clocksource.c
+++ b/kernel/time/clocksource.c
@@ -587,9 +587,7 @@ static void clocksource_watchdog(struct timer_list *unused)
* Cycle through CPUs to check if the CPUs stay synchronized
* to each other.
*/
- next_cpu = cpumask_next(raw_smp_processor_id(), cpu_online_mask);
- if (next_cpu >= nr_cpu_ids)
- next_cpu = cpumask_first(cpu_online_mask);
+ next_cpu = cpumask_next_wrap(raw_smp_processor_id(), cpu_online_mask);

/*
* Arm timer if not already pending: could race with concurrent
--
2.43.0