Re: [Devel] [PATCH rh7] ms/hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2018-09-25 Thread Kirill Tkhai
On 24.09.2018 15:07, Andrey Ryabinin wrote: > From: Peter Zijlstra > > Because we drop cpu_base->lock around calling hrtimer::function, it is > possible for hrtimer_start() to come in between and enqueue the timer. > > If hrtimer::function then returns HRTIMER_RESTART we'll hit the BUG_ON > beca

[Devel] [PATCH rh7] ms/hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2018-09-24 Thread Andrey Ryabinin
From: Peter Zijlstra Because we drop cpu_base->lock around calling hrtimer::function, it is possible for hrtimer_start() to come in between and enqueue the timer. If hrtimer::function then returns HRTIMER_RESTART we'll hit the BUG_ON because HRTIMER_STATE_ENQUEUED will be set. Since the above i