Messages in this thread | | | Date | Wed, 12 May 2021 16:59:32 +0200 | From | Peter Zijlstra <> | Subject | Re: [patch 2/8] hrtimer: Force clock_was_set() handling for the HIGHRES=n, NOHZ=y case |
| |
On Tue, Apr 27, 2021 at 10:25:39AM +0200, Thomas Gleixner wrote: > void clock_was_set(void) > { > + if (!hrtimer_hres_active() && !tick_nohz_active) > + goto out_timerfd; > + > /* Retrigger the CPU local events everywhere */ > on_each_cpu(retrigger_next_event, NULL, 1); > + > +out_timerfd: > timerfd_clock_was_set(); > }
Isn't that simpler when written like:
if (hrtimer_hres_active() || tick_nohz_active()) on_each_cpu(retrigger_next_event, NULL, 1);
timerfd_clock_was_set();
?
| |