Commit e3ee5e66 authored by Peter Zijlstra's avatar Peter Zijlstra Committed by Ingo Molnar
Browse files

time/tick-broadcast: Remove RCU_NONIDLE() usage 



No callers left that have already disabled RCU.

Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: default avatarIngo Molnar <mingo@kernel.org>
Tested-by: default avatarTony Lindgren <tony@atomide.com>
Tested-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Acked-by: default avatarMark Rutland <mark.rutland@arm.com>
Acked-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: default avatarFrederic Weisbecker <frederic@kernel.org>
Link: https://lore.kernel.org/r/20230112195540.927904612@infradead.org
parent 880970b5

kernel/time/tick-broadcast-hrtimer.c

+12 −17
Original line number Diff line number Diff line
@@ -56,12 +56,7 @@ static int bc_set_next(ktime_t expires, struct clock_event_device *bc) 
	 * hrtimer callback function is currently running, then

	 * hrtimer_start() cannot move it and the timer stays on the CPU on

	 * which it is assigned at the moment.

	 *

	 * As this can be called from idle code, the hrtimer_start()

	 * invocation has to be wrapped with RCU_NONIDLE() as

	 * hrtimer_start() can call into tracing.

	 */

	RCU_NONIDLE( {

	hrtimer_start(&bctimer, expires, HRTIMER_MODE_ABS_PINNED_HARD);

	/*

	 * The core tick broadcast mode expects bc->bound_on to be set
@@ -74,7 +69,7 @@ static int bc_set_next(ktime_t expires, struct clock_event_device *bc) 
	 * base lock.

	 */

	bc->bound_on = bctimer.base->cpu_base->cpu;

	} );



	return 0;

}