On Mon, Nov 17, 2025 at 03:37:04PM +0100, Jan Beulich wrote: > It's only ever handle_hpet_broadcast() that's used. While we now don't > enable IRQs right away, still play safe and convert the function pointer > to a boolean, to make sure no calls occur too early. > > Signed-off-by: Jan Beulich <[email protected]>
Acked-by: Roger Pau Monné <[email protected]> > --- > v2: Re-base over changes earlier in the series. > > --- a/xen/arch/x86/hpet.c > +++ b/xen/arch/x86/hpet.c > @@ -40,7 +40,7 @@ struct hpet_event_channel > s_time_t next_event; > cpumask_var_t cpumask; > spinlock_t lock; > - void (*event_handler)(struct hpet_event_channel *ch); > + bool event_handler; It would be nice to also get rid of this field, but I don't see any other input that we could use to ensure the channel is ready to receive interrupts. Thanks, Roger.
