Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-07 Thread Rafael J. Wysocki
On Thursday, December 7, 2017 1:51:04 PM CET Meelis Roos wrote: > > On 06-12-17, 20:21, Meelis Roos wrote: > > > 3 was not reliable. > > > > > > I created root cron job > > > @reboot sleep 120; /sbin/reboot > > > > > > and by the evening it was dead again. > > > > > > Will try 5 tomorrow

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-07 Thread Meelis Roos
> On 06-12-17, 20:21, Meelis Roos wrote: > > 3 was not reliable. > > > > I created root cron job > > @reboot sleep 120; /sbin/reboot > > > > and by the evening it was dead again. > > > > Will try 5 tomorrow. > > Lets make it similar to what it was before my original patch modified > it,

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-07 Thread Viresh Kumar
On 07-12-17, 09:26, Meelis Roos wrote: > > On 06-12-17, 20:21, Meelis Roos wrote: > > > 3 was not reliable. > > > > > > I created root cron job > > > @reboot sleep 120; /sbin/reboot > > > > > > and by the evening it was dead again. > > > > > > Will try 5 tomorrow. > > > > Lets make it s

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-06 Thread Meelis Roos
> On 06-12-17, 20:21, Meelis Roos wrote: > > 3 was not reliable. > > > > I created root cron job > > @reboot sleep 120; /sbin/reboot > > > > and by the evening it was dead again. > > > > Will try 5 tomorrow. > > Lets make it similar to what it was before my original patch modified > it,

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-06 Thread Meelis Roos
> > 3 was not reliable. > > > > I created root cron job > > @reboot sleep 120; /sbin/reboot > > > > and by the evening it was dead again. > > > > Will try 5 tomorrow. > > Lets make it similar to what it was before my original patch modified > it, to avoid all corner cases. > > Please t

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-06 Thread Viresh Kumar
On 06-12-17, 20:21, Meelis Roos wrote: > 3 was not reliable. > > I created root cron job > @reboot sleep 120; /sbin/reboot > > and by the evening it was dead again. > > Will try 5 tomorrow. Lets make it similar to what it was before my original patch modified it, to avoid all corner cas

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-06 Thread Meelis Roos
> >> > diff --git a/drivers/cpufreq/longhaul.c b/drivers/cpufreq/longhaul.c > >> > index c46a12df40dd..56eafcb07859 100644 > >> > --- a/drivers/cpufreq/longhaul.c > >> > +++ b/drivers/cpufreq/longhaul.c > >> > @@ -894,7 +894,7 @@ static int longhaul_cpu_init(struct cpufreq_policy > >> > *policy) >

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-05 Thread Rafael J. Wysocki
On Tue, Dec 5, 2017 at 9:54 AM, Meelis Roos wrote: >> > The commit e948bc8fbee0 ("cpufreq: Cap the default transition delay >> > value to 10 ms") caused a regression on EPIA-M min-ITX computer where >> > shutdown or reboot hangs occasionally with a print message like: >> > >> > longhaul: Warning:

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-05 Thread Meelis Roos
> > The commit e948bc8fbee0 ("cpufreq: Cap the default transition delay > > value to 10 ms") caused a regression on EPIA-M min-ITX computer where > > shutdown or reboot hangs occasionally with a print message like: > > > > longhaul: Warning: Timeout while waiting for idle PCI bus > > cpufreq: __ta

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-05 Thread Meelis Roos
> The commit e948bc8fbee0 ("cpufreq: Cap the default transition delay > value to 10 ms") caused a regression on EPIA-M min-ITX computer where > shutdown or reboot hangs occasionally with a print message like: > > longhaul: Warning: Timeout while waiting for idle PCI bus > cpufreq: __target_index:

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-12-04 Thread Rafael J. Wysocki
On Wednesday, November 29, 2017 7:59:27 AM CET Meelis Roos wrote: > > > diff --git a/drivers/cpufreq/longhaul.c b/drivers/cpufreq/longhaul.c > > > index c46a12df40dd..56eafcb07859 100644 > > > --- a/drivers/cpufreq/longhaul.c > > > +++ b/drivers/cpufreq/longhaul.c > > > @@ -894,7 +894,7 @@ static i

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-11-28 Thread Meelis Roos
> > diff --git a/drivers/cpufreq/longhaul.c b/drivers/cpufreq/longhaul.c > > index c46a12df40dd..56eafcb07859 100644 > > --- a/drivers/cpufreq/longhaul.c > > +++ b/drivers/cpufreq/longhaul.c > > @@ -894,7 +894,7 @@ static int longhaul_cpu_init(struct cpufreq_policy > > *policy) > > if ((lo

Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-11-28 Thread Rafael J. Wysocki
On Tue, Nov 28, 2017 at 4:11 AM, Viresh Kumar wrote: > The commit e948bc8fbee0 ("cpufreq: Cap the default transition delay > value to 10 ms") caused a regression on EPIA-M min-ITX computer where > shutdown or reboot hangs occasionally with a print message like: > > longhaul: Warning: Timeout while

[PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms

2017-11-27 Thread Viresh Kumar
The commit e948bc8fbee0 ("cpufreq: Cap the default transition delay value to 10 ms") caused a regression on EPIA-M min-ITX computer where shutdown or reboot hangs occasionally with a print message like: longhaul: Warning: Timeout while waiting for idle PCI bus cpufreq: __target_index: Failed to ch