On Mon, Nov 21 2022 at 16:35, Steven Rostedt wrote: > On Tue, 15 Nov 2022 21:28:49 +0100 (CET) > Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote: >> @@ -1532,6 +1573,12 @@ static void expire_timers(struct timer_b >> >> fn = timer->function; >> >> + if (WARN_ON_ONCE(!fn)) { >> + /* Should never happen. Emphasis on should! */ >> + base->running_timer = NULL; >> + return; > > Why return and not continue? > > Wont this drop the other timers in the queue? Duh. Yes. Thanks for catching that!