On Monday, January 06, 2014 10:59:16 AM dirk.brandewie@xxxxxxxxx wrote: > From: Dirk Brandewie <dirk.j.brandewie@xxxxxxxxx> > > KVM environments do not support APERF/MPERF MSRs. intel_pstate cannot > operate without these registers. > > The previous validity checks in intel_pstate_msrs_not_valid() are > insufficent in nested KVMs. > > Fixes: > https://bugzilla.redhat.com/show_bug.cgi?id=1046317 > > Signed-off-by: Dirk Brandewie <dirk.j.brandewie@xxxxxxxxx> OK, I'll queue it up as a fix for 3.13. Minor nit: "Fixes:" is for the IDs of commits fixed by your patch. To reference a bug entry please use "Buglink" or "References" (which is preferred, because I will change anything else to that anyway :-)). Thanks! > Cc: All applicable <stable@xxxxxxxxxxxxxxx> > --- > drivers/cpufreq/intel_pstate.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c > index 0f63f5d..fe91dad 100644 > --- a/drivers/cpufreq/intel_pstate.c > +++ b/drivers/cpufreq/intel_pstate.c > @@ -619,7 +619,8 @@ static void intel_pstate_timer_func(unsigned long __data) > } > > #define ICPU(model, policy) \ > - { X86_VENDOR_INTEL, 6, model, X86_FEATURE_ANY, (unsigned long)&policy } > + { X86_VENDOR_INTEL, 6, model, X86_FEATURE_APERFMPERF,\ > + (unsigned long)&policy } > > static const struct x86_cpu_id intel_pstate_cpu_ids[] = { > ICPU(0x2a, core_params), > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html