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 at

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 05:32:31

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 17:07,

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 +0100,

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 repl

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 Wed,

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 wr

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 [1901

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), Ton

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 autos

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 m

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. > > > > Anyth

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 ktime_t

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 wrote

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 (-0

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 out

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 > > > h

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 gener

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 r

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 longer

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 others