Re: [PATCH] PM-runtime: fix deadlock when canceling hrtimer

2019-02-21 Thread Vincent Guittot
On Thu, 21 Feb 2019 at 10:35, Rafael J. Wysocki wrote: > > On Thu, Feb 21, 2019 at 8:59 AM Vincent Guittot > wrote: > > > > When rpm_resume() desactivates the autosuspend timer, it should only try > > to cancel hrtimer but not wait for the handler to finish because both > > rpm_resume() and

Re: [PATCH] PM-runtime: fix deadlock when canceling hrtimer

2019-02-21 Thread Rafael J. Wysocki
On Thu, Feb 21, 2019 at 8:59 AM Vincent Guittot wrote: > > When rpm_resume() desactivates the autosuspend timer, it should only try > to cancel hrtimer but not wait for the handler to finish because both > rpm_resume() and pm_suspend_timer_fn() are taking the power.lock. > We can have the

[PATCH] PM-runtime: fix deadlock when canceling hrtimer

2019-02-20 Thread Vincent Guittot
When rpm_resume() desactivates the autosuspend timer, it should only try to cancel hrtimer but not wait for the handler to finish because both rpm_resume() and pm_suspend_timer_fn() are taking the power.lock. We can have the deadlock sequence: CPU0 CPU1 rpm_resume()