On 12/06/2018 16:23, Peter Zijlstra wrote: > On Tue, Jun 12, 2018 at 02:00:11PM +0200, Daniel Lezcano wrote: >> +static void idle_injection_last_man(struct idle_injection_device *ii_dev) >> +{ >> + unsigned int run_duration_ms; >> + >> + run_duration_ms = READ_ONCE(ii_dev->run_duration_ms); >> + if (run_duration_ms) { >> + hrtimer_start(&ii_dev->timer, ms_to_ktime(run_duration_ms), >> + HRTIMER_MODE_REL_PINNED); > > What's the point of PINNED here? AFAICT this gets called from a > different CPU every time.
Right, I will remove it. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog