Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Ladislav Michl
On Thu, Jan 10, 2019 at 08:50:14AM +0100, Vincent Guittot wrote: > On Thu, 10 Jan 2019 at 08:46, Ladislav Michl wrote: > > > > On Wed, Jan 09, 2019 at 11:06:34PM +0100, Rafael J. Wysocki wrote: > > > On Wed, Jan 9, 2019 at 7:05 PM Vincent Guittot > > > wrote: > > > > > > > > On Wed, 9 Jan 2019

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Thu, 10 Jan 2019 at 08:46, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 11:06:34PM +0100, Rafael J. Wysocki wrote: > > On Wed, Jan 9, 2019 at 7:05 PM Vincent Guittot > > wrote: > > > > > > On Wed, 9 Jan 2019 at 18:26, Ladislav Michl wrote: > > > > > > > > On Wed, Jan 09, 2019 at

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Ladislav Michl
On Wed, Jan 09, 2019 at 11:06:34PM +0100, Rafael J. Wysocki wrote: > On Wed, Jan 9, 2019 at 7:05 PM Vincent Guittot > wrote: > > > > On Wed, 9 Jan 2019 at 18:26, Ladislav Michl wrote: > > > > > > On Wed, Jan 09, 2019 at 05:32:31PM +0100, Vincent Guittot wrote: > > > > On Wed, 9 Jan 2019 at

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Rafael J. Wysocki
On Wed, Jan 9, 2019 at 7:05 PM Vincent Guittot wrote: > > On Wed, 9 Jan 2019 at 18:26, Ladislav Michl wrote: > > > > On Wed, Jan 09, 2019 at 05:32:31PM +0100, Vincent Guittot wrote: > > > On Wed, 9 Jan 2019 at 17:07, Ladislav Michl wrote: > > > > > > > > On Wed, Jan 09, 2019 at 03:12:25PM

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 18:26, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 05:32:31PM +0100, Vincent Guittot wrote: > > On Wed, 9 Jan 2019 at 17:07, Ladislav Michl wrote: > > > > > > On Wed, Jan 09, 2019 at 03:12:25PM +0100, Vincent Guittot wrote: > > > > Please keep all thread list when

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Ladislav Michl
On Wed, Jan 09, 2019 at 05:32:31PM +0100, Vincent Guittot wrote: > On Wed, 9 Jan 2019 at 17:07, Ladislav Michl wrote: > > > > On Wed, Jan 09, 2019 at 03:12:25PM +0100, Vincent Guittot wrote: > > > Please keep all thread list when replying :-) > > > > Ahh, sorry for that... > > [snip] > > > On

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Tony Lindgren
* Vincent Guittot [190109 16:56]: > On Wed, 9 Jan 2019 at 17:50, Tony Lindgren wrote: > > > > * Vincent Guittot [190109 16:48]: > > > On Wed, 9 Jan 2019 at 17:28, Tony Lindgren wrote: > > > > > > > > * Rafael J. Wysocki [190109 09:44]: > > > > > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 17:50, Tony Lindgren wrote: > > * Vincent Guittot [190109 16:48]: > > On Wed, 9 Jan 2019 at 17:28, Tony Lindgren wrote: > > > > > > * Rafael J. Wysocki [190109 09:44]: > > > > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > > > > > > > > > * Vincent Guittot

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Tony Lindgren
* Vincent Guittot [190109 16:48]: > On Wed, 9 Jan 2019 at 17:28, Tony Lindgren wrote: > > > > * Rafael J. Wysocki [190109 09:44]: > > > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > > > > > > > * Vincent Guittot [190109 01:42]: > > > > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800),

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 17:28, Tony Lindgren wrote: > > * Rafael J. Wysocki [190109 09:44]: > > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > > > > > * Vincent Guittot [190109 01:42]: > > > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > > > > Lowering the

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 17:07, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 03:12:25PM +0100, Vincent Guittot wrote: > > Please keep all thread list when replying :-) > > Ahh, sorry for that... > [snip] > > On Wed, 9 Jan 2019 at 14:33, Ladislav Michl wrote: > > > I agree, but it doea all the

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Tony Lindgren
* Rafael J. Wysocki [190109 09:44]: > On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > > > * Vincent Guittot [190109 01:42]: > > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > > > Lowering the autosuspend_delay_ms to 2100 ms makes things work again. > > > >

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Ladislav Michl
On Wed, Jan 09, 2019 at 03:12:25PM +0100, Vincent Guittot wrote: > Please keep all thread list when replying :-) Ahh, sorry for that... [snip] > On Wed, 9 Jan 2019 at 14:33, Ladislav Michl wrote: > > I agree, but it doea all the magic correctly, so you won't need > > to touch that code is

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
Please keep all thread list when replying :-) On Wed, 9 Jan 2019 at 14:33, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 02:24:37PM +0100, Vincent Guittot wrote: > > On Wed, 9 Jan 2019 at 12:58, Ladislav Michl wrote: > > > > > > On Wed, Jan 09, 2019 at 12:27:57PM +0100, Vincent Guittot

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 12:58, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 12:27:57PM +0100, Vincent Guittot wrote: > > On Wed, 9 Jan 2019 at 12:17, Ladislav Michl wrote: > > > > > > On Wed, Jan 09, 2019 at 02:42:18AM +0100, Vincent Guittot wrote: > > > > Le Tuesday 08 Jan 2019 à 13:37:43

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Vincent Guittot
On Wed, 9 Jan 2019 at 12:17, Ladislav Michl wrote: > > On Wed, Jan 09, 2019 at 02:42:18AM +0100, Vincent Guittot wrote: > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > > * Vincent Guittot [190108 16:42]: > > > > On Tue, 8 Jan 2019 at 16:53, Tony Lindgren wrote: > > >

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Ladislav Michl
On Wed, Jan 09, 2019 at 02:42:18AM +0100, Vincent Guittot wrote: > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > * Vincent Guittot [190108 16:42]: > > > On Tue, 8 Jan 2019 at 16:53, Tony Lindgren wrote: > > > > Hmm so could it be that we now rely on timers that that may

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-09 Thread Rafael J. Wysocki
On Wed, Jan 9, 2019 at 2:51 AM Tony Lindgren wrote: > > * Vincent Guittot [190109 01:42]: > > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > > Lowering the autosuspend_delay_ms to 2100 ms makes things work again. > > > Anything higher than 2200 ms seems to somehow time

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Tony Lindgren
* Vincent Guittot [190109 01:42]: > Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > > Lowering the autosuspend_delay_ms to 2100 ms makes things work again. > > Anything higher than 2200 ms seems to somehow time out immediately > > now :) > > This is quite close to the max ns

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Vincent Guittot
Le Tuesday 08 Jan 2019 à 13:37:43 (-0800), Tony Lindgren a écrit : > * Vincent Guittot [190108 16:42]: > > On Tue, 8 Jan 2019 at 16:53, Tony Lindgren wrote: > > > Hmm so could it be that we now rely on timers that that may > > > not be capable of waking up the system from idle states with > > >

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Tony Lindgren
* Vincent Guittot [190108 16:42]: > On Tue, 8 Jan 2019 at 16:53, Tony Lindgren wrote: > > Hmm so could it be that we now rely on timers that that may > > not be capable of waking up the system from idle states with > > hrtimer? > > With nohz and hrtimer enabled, timer relies on hrtimer to

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Vincent Guittot
On Tue, 8 Jan 2019 at 16:53, Tony Lindgren wrote: > > * Vincent Guittot [190108 08:00]: > > Hi Tony, > > > > On Tue, 8 Jan 2019 at 00:38, Tony Lindgren wrote: > > > > > > Hi all, > > > > > > Looks like commit 8234f6734c5d ("PM-runtime: Switch autosuspend > > > over to using hrtimers") caused a

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Tony Lindgren
* Vincent Guittot [190108 08:00]: > Hi Tony, > > On Tue, 8 Jan 2019 at 00:38, Tony Lindgren wrote: > > > > Hi all, > > > > Looks like commit 8234f6734c5d ("PM-runtime: Switch autosuspend > > over to using hrtimers") caused a regression on at least > > omap5-uevm where 8250 UART rx wake no

Re: Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-08 Thread Vincent Guittot
Hi Tony, On Tue, 8 Jan 2019 at 00:38, Tony Lindgren wrote: > > Hi all, > > Looks like commit 8234f6734c5d ("PM-runtime: Switch autosuspend > over to using hrtimers") caused a regression on at least > omap5-uevm where 8250 UART rx wake no longer works. I have not > noticed this happening on

Regression in v5.0-rc1 with autosuspend hrtimers

2019-01-07 Thread Tony Lindgren
Hi all, Looks like commit 8234f6734c5d ("PM-runtime: Switch autosuspend over to using hrtimers") caused a regression on at least omap5-uevm where 8250 UART rx wake no longer works. I have not noticed this happening on others so far. The devices I've tested all are using 8250 with dedicated