On 9.7.2013 17.54, Dirk Brandewie wrote: > On 06/28/2013 05:24 AM, Nell Hardcastle wrote: >> Enable the intel_pstate driver for Haswell CPUs and one Ivy Bridge >> model not already included (0x3E). I've been testing this change on >> the Core i5-4200u (model 0x45) and I referenced the list of models in >> tools/power/x86/turbostat/turbostat.c:has_nehalem_turbo_ratio_limit to >> determine the others. >> >> Fixes https://bugzilla.kernel.org/show_bug.cgi?id=60211 >> >> Signed-off-by: Nell Hardcastle <nell@xxxxxxxxxxxx> >> --- >> drivers/cpufreq/intel_pstate.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/drivers/cpufreq/intel_pstate.c >> b/drivers/cpufreq/intel_pstate.c >> index 07f2840..dbd1032 100644 >> --- a/drivers/cpufreq/intel_pstate.c >> +++ b/drivers/cpufreq/intel_pstate.c >> @@ -522,6 +522,11 @@ static const struct x86_cpu_id >> intel_pstate_cpu_ids[] = { >> ICPU(0x2a, default_policy), >> ICPU(0x2d, default_policy), >> ICPU(0x3a, default_policy), >> + ICPU(0x3c, default_policy), >> + ICPU(0x3e, default_policy), >> + ICPU(0x3f, default_policy), >> + ICPU(0x45, default_policy), >> + ICPU(0x46, default_policy), > > Nak for now. > > We have not completed our testing on these CPUs. I would like to > get my testing done before I start seeing issues from the field come > in. > > If this patch were to hangout in testing for a while it would not > hurt my feelings there it could only hurt those that knew what > they were getting into :-) > Hi, I want to let you know that I've tested intel_pstate with Xeon E3-1265Lv3 (model 0x3c) successfully. For this CPU model the patch seems to be fine. Turbo mode works as intended. Core frequencies scale according to number of active cores. Tested-by: Ilari Stenroth <ilari.stenroth@xxxxxxxxx> Regards, Ilari Stenroth -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
