Re: [PATCH 4/5] intel_pstate: Remove C0 tracking

Rafael J. Wysocki
2014-05-12T23:28:45Z

On Monday, May 12, 2014 07:39:35 AM Dirk Brandewie wrote:
> On 05/12/2014 05:16 AM, Rafael J. Wysocki wrote:
> > On Monday, May 12, 2014 05:27:25 AM Stratos Karafotis wrote:
> >> Hi,
> >>
> >
> > [cut]
> >
> >>
> >> With this patch, my CPU (Core i7-3770 @ 3.90GHz) seems to never use lowest
> >> frequencies. Even on an idle system I get always ~2GHz. Normally,
> >> on an idle system it used to be 1.6GHz.
> >> On very small loads (mp3 decoding) the CPU goes up to 2.7G GHz (it used to
> >> be 1.6GHz)
> >>
> >> Reverting, this patch on my local build, the problem is resolved.
> >
> > Dirk, seriously, I can't regard this as a fix that can go into -rc6.
> >
>
> Ok I will resubmit after more testing
>
> > Which of the other patch in the series are must-go for 3.15? [1-2/5] I guess?
> > And do we need [2/5] it in -stable too?
>
> 1/5 is for stable it fixes a random MCE on baytrail.
> 2/5 is for stable it should have went with the stop_cpu patch

Which -stable are these two for? All of them?

> 5/5 can go too since it is just adding CPU IDs

Well, I'm a bit reluctant to push that for -rc6. There were cases where new
IDs broke stuff that worked without them just fine.

--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.