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

From: Viresh Kumar
Date: Thu Dec 07 2017 - 04:34:02 EST


On 07-12-17, 09:26, Meelis Roos wrote:
> > On 06-12-17, 20:21, Meelis Roos wrote:
> > > 30000 was not reliable.
> > >
> > > I created root cron job
> > > @reboot sleep 120; /sbin/reboot
> > >
> > > and by the evening it was dead again.
> > >
> > > Will try 50000 tomorrow.
> >
> > Lets make it similar to what it was before my original patch modified
> > it, to avoid all corner cases.
> >
> > Please test against 200 ms, 200000 value here.
>
> Sorry, I confused 200000 vs 20000, will test 200000.
>
> But 200000 was the value before.

It was value of a different variable (transition_latency) at that
time. Just set transition_delay_us in my recent patch as 200,000 and
apply that over mainline.

I will resend the patch in the mean time as well.

> Shall I test 200000 with or without
> the other limiting patch?

--
viresh