On Wed, 31 Jul 2019, Peter Zijlstra wrote:

> On Wed, Jul 31, 2019 at 12:33:55AM +0200, Thomas Gleixner wrote:
> > +static struct k_itimer *timer_wait_running(struct k_itimer *timer,
> > +                                      unsigned long *flags)
> > +{
> > +   const struct k_clock *kc = READ_ONCE(timer->kclock);
> > +   timer_t timer_id = READ_ONCE(timer->it_id);
> > +
> > +   /* Prevent kfree(timer) after dropping the lock */
> > +   rcu_read_lock();
> > +   unlock_timer(timer, *flags);
> > +
> > +   if (kc->timer_arm == common_hrtimer_arm)
> > +           hrtimer_cancel_wait_running(&timer->it.real.timer);
> > +   else if (kc == &alarm_clock)
> > +           hrtimer_cancel_wait_running(&timer->it.alarm.alarmtimer.timer);
> 
>       else WARN();

Yup. Working on it ...

Reply via email to