All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
@ 2013-03-20 16:17 dirk.brandewie
  2013-03-20 18:28 ` Rafael J. Wysocki
  2013-03-21  3:20 ` Viresh Kumar
  0 siblings, 2 replies; 8+ messages in thread
From: dirk.brandewie @ 2013-03-20 16:17 UTC (permalink / raw)
  To: linux-kernel, cpufreq; +Cc: Dirk Brandewie, Dirk Brandewie

From: Dirk Brandewie <dirk.brandewie@gmail.com>

Some VMs seem to try to implement some MSRs but not all the registers
the driver needs.  Check to make sure all the MSR that we need are
available. If any of the required MSRs are not available refuse to
load.

Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
---
 drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
 1 files changed, 26 insertions(+), 0 deletions(-)

diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index f6dd1e7..cd9c5f4 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
 
 static int __initdata no_load;
 
+static int intel_pstate_msrs_not_valid(void)
+{
+	/* Check that all the msr's we are using are valid. */
+	u64 aperf, mperf, tmp;
+
+	rdmsrl(MSR_IA32_APERF, aperf);
+	rdmsrl(MSR_IA32_MPERF, mperf);
+
+	if (!intel_pstate_min_pstate() ||
+		!intel_pstate_max_pstate() ||
+		!intel_pstate_turbo_pstate())
+		return -ENODEV;
+
+	rdmsrl(MSR_IA32_APERF, tmp);
+	if (!(tmp - aperf))
+		return -ENODEV;
+
+	rdmsrl(MSR_IA32_MPERF, tmp);
+	if (!(tmp - mperf))
+		return -ENODEV;
+
+	return 0;
+}
 static int __init intel_pstate_init(void)
 {
 	int cpu, rc = 0;
@@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
 	if (!id)
 		return -ENODEV;
 
+	if (intel_pstate_msrs_not_valid())
+		return -ENODEV;
+
 	pr_info("Intel P-state driver initializing.\n");
 
 	all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());
-- 
1.7.7.6


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-20 16:17 [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid dirk.brandewie
@ 2013-03-20 18:28 ` Rafael J. Wysocki
  2013-03-20 18:28   ` Dirk Brandewie
  2013-03-21  3:20 ` Viresh Kumar
  1 sibling, 1 reply; 8+ messages in thread
From: Rafael J. Wysocki @ 2013-03-20 18:28 UTC (permalink / raw)
  To: dirk.brandewie; +Cc: linux-kernel, cpufreq, Dirk Brandewie

On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
> From: Dirk Brandewie <dirk.brandewie@gmail.com>
> 
> Some VMs seem to try to implement some MSRs but not all the registers
> the driver needs.  Check to make sure all the MSR that we need are
> available. If any of the required MSRs are not available refuse to
> load.
> 
> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>

Is this needed for v3.9?  Any pointers to bug reports etc.?

Rafael


> ---
>  drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
>  1 files changed, 26 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> index f6dd1e7..cd9c5f4 100644
> --- a/drivers/cpufreq/intel_pstate.c
> +++ b/drivers/cpufreq/intel_pstate.c
> @@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
>  
>  static int __initdata no_load;
>  
> +static int intel_pstate_msrs_not_valid(void)
> +{
> +	/* Check that all the msr's we are using are valid. */
> +	u64 aperf, mperf, tmp;
> +
> +	rdmsrl(MSR_IA32_APERF, aperf);
> +	rdmsrl(MSR_IA32_MPERF, mperf);
> +
> +	if (!intel_pstate_min_pstate() ||
> +		!intel_pstate_max_pstate() ||
> +		!intel_pstate_turbo_pstate())
> +		return -ENODEV;
> +
> +	rdmsrl(MSR_IA32_APERF, tmp);
> +	if (!(tmp - aperf))
> +		return -ENODEV;
> +
> +	rdmsrl(MSR_IA32_MPERF, tmp);
> +	if (!(tmp - mperf))
> +		return -ENODEV;
> +
> +	return 0;
> +}
>  static int __init intel_pstate_init(void)
>  {
>  	int cpu, rc = 0;
> @@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
>  	if (!id)
>  		return -ENODEV;
>  
> +	if (intel_pstate_msrs_not_valid())
> +		return -ENODEV;
> +
>  	pr_info("Intel P-state driver initializing.\n");
>  
>  	all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());
> 
-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-20 18:28 ` Rafael J. Wysocki
@ 2013-03-20 18:28   ` Dirk Brandewie
  2013-03-21  0:08     ` Rafael J. Wysocki
  0 siblings, 1 reply; 8+ messages in thread
From: Dirk Brandewie @ 2013-03-20 18:28 UTC (permalink / raw)
  To: Rafael J. Wysocki; +Cc: dirk.brandewie, linux-kernel, cpufreq, Dirk Brandewie

On 03/20/2013 11:28 AM, Rafael J. Wysocki wrote:
> On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
>> From: Dirk Brandewie <dirk.brandewie@gmail.com>
>>
>> Some VMs seem to try to implement some MSRs but not all the registers
>> the driver needs.  Check to make sure all the MSR that we need are
>> available. If any of the required MSRs are not available refuse to
>> load.
>>
>> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
>
> Is this needed for v3.9?  Any pointers to bug reports etc.?
>

Sorry I saw right after I sent the mail that the bug report was missing

     https://bugzilla.redhat.com/show_bug.cgi?id=922923
     Reported-by: Josh Stone <jistone@redhat.com>

Would you like me to spin the patch?

> Rafael
>
>
>> ---
>>   drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
>>   1 files changed, 26 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
>> index f6dd1e7..cd9c5f4 100644
>> --- a/drivers/cpufreq/intel_pstate.c
>> +++ b/drivers/cpufreq/intel_pstate.c
>> @@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
>>
>>   static int __initdata no_load;
>>
>> +static int intel_pstate_msrs_not_valid(void)
>> +{
>> +	/* Check that all the msr's we are using are valid. */
>> +	u64 aperf, mperf, tmp;
>> +
>> +	rdmsrl(MSR_IA32_APERF, aperf);
>> +	rdmsrl(MSR_IA32_MPERF, mperf);
>> +
>> +	if (!intel_pstate_min_pstate() ||
>> +		!intel_pstate_max_pstate() ||
>> +		!intel_pstate_turbo_pstate())
>> +		return -ENODEV;
>> +
>> +	rdmsrl(MSR_IA32_APERF, tmp);
>> +	if (!(tmp - aperf))
>> +		return -ENODEV;
>> +
>> +	rdmsrl(MSR_IA32_MPERF, tmp);
>> +	if (!(tmp - mperf))
>> +		return -ENODEV;
>> +
>> +	return 0;
>> +}
>>   static int __init intel_pstate_init(void)
>>   {
>>   	int cpu, rc = 0;
>> @@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
>>   	if (!id)
>>   		return -ENODEV;
>>
>> +	if (intel_pstate_msrs_not_valid())
>> +		return -ENODEV;
>> +
>>   	pr_info("Intel P-state driver initializing.\n");
>>
>>   	all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());
>>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-20 18:28   ` Dirk Brandewie
@ 2013-03-21  0:08     ` Rafael J. Wysocki
  2013-03-22  0:40       ` Rafael J. Wysocki
  0 siblings, 1 reply; 8+ messages in thread
From: Rafael J. Wysocki @ 2013-03-21  0:08 UTC (permalink / raw)
  To: Dirk Brandewie; +Cc: linux-kernel, cpufreq, Dirk Brandewie

On Wednesday, March 20, 2013 11:28:49 AM Dirk Brandewie wrote:
> On 03/20/2013 11:28 AM, Rafael J. Wysocki wrote:
> > On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
> >> From: Dirk Brandewie <dirk.brandewie@gmail.com>
> >>
> >> Some VMs seem to try to implement some MSRs but not all the registers
> >> the driver needs.  Check to make sure all the MSR that we need are
> >> available. If any of the required MSRs are not available refuse to
> >> load.
> >>
> >> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
> >
> > Is this needed for v3.9?  Any pointers to bug reports etc.?
> >
> 
> Sorry I saw right after I sent the mail that the bug report was missing
> 
>      https://bugzilla.redhat.com/show_bug.cgi?id=922923
>      Reported-by: Josh Stone <jistone@redhat.com>
> 
> Would you like me to spin the patch?

No, thanks, this information is sufficient.

Thanks,
Rafael


> >> ---
> >>   drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
> >>   1 files changed, 26 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> >> index f6dd1e7..cd9c5f4 100644
> >> --- a/drivers/cpufreq/intel_pstate.c
> >> +++ b/drivers/cpufreq/intel_pstate.c
> >> @@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
> >>
> >>   static int __initdata no_load;
> >>
> >> +static int intel_pstate_msrs_not_valid(void)
> >> +{
> >> +	/* Check that all the msr's we are using are valid. */
> >> +	u64 aperf, mperf, tmp;
> >> +
> >> +	rdmsrl(MSR_IA32_APERF, aperf);
> >> +	rdmsrl(MSR_IA32_MPERF, mperf);
> >> +
> >> +	if (!intel_pstate_min_pstate() ||
> >> +		!intel_pstate_max_pstate() ||
> >> +		!intel_pstate_turbo_pstate())
> >> +		return -ENODEV;
> >> +
> >> +	rdmsrl(MSR_IA32_APERF, tmp);
> >> +	if (!(tmp - aperf))
> >> +		return -ENODEV;
> >> +
> >> +	rdmsrl(MSR_IA32_MPERF, tmp);
> >> +	if (!(tmp - mperf))
> >> +		return -ENODEV;
> >> +
> >> +	return 0;
> >> +}
> >>   static int __init intel_pstate_init(void)
> >>   {
> >>   	int cpu, rc = 0;
> >> @@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
> >>   	if (!id)
> >>   		return -ENODEV;
> >>
> >> +	if (intel_pstate_msrs_not_valid())
> >> +		return -ENODEV;
> >> +
> >>   	pr_info("Intel P-state driver initializing.\n");
> >>
> >>   	all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());
> >>
> 
-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-20 16:17 [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid dirk.brandewie
  2013-03-20 18:28 ` Rafael J. Wysocki
@ 2013-03-21  3:20 ` Viresh Kumar
  1 sibling, 0 replies; 8+ messages in thread
From: Viresh Kumar @ 2013-03-21  3:20 UTC (permalink / raw)
  To: dirk.brandewie; +Cc: linux-kernel, cpufreq, Dirk Brandewie

On Wed, Mar 20, 2013 at 9:47 PM,  <dirk.brandewie@gmail.com> wrote:
> From: Dirk Brandewie <dirk.brandewie@gmail.com>
>
> Some VMs seem to try to implement some MSRs but not all the registers
> the driver needs.  Check to make sure all the MSR that we need are
> available. If any of the required MSRs are not available refuse to
> load.
>
> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
> ---
>  drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
>  1 files changed, 26 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> index f6dd1e7..cd9c5f4 100644
> --- a/drivers/cpufreq/intel_pstate.c
> +++ b/drivers/cpufreq/intel_pstate.c
> @@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
>
>  static int __initdata no_load;
>
> +static int intel_pstate_msrs_not_valid(void)
> +{
> +       /* Check that all the msr's we are using are valid. */
> +       u64 aperf, mperf, tmp;
> +
> +       rdmsrl(MSR_IA32_APERF, aperf);
> +       rdmsrl(MSR_IA32_MPERF, mperf);
> +
> +       if (!intel_pstate_min_pstate() ||
> +               !intel_pstate_max_pstate() ||
> +               !intel_pstate_turbo_pstate())
> +               return -ENODEV;
> +
> +       rdmsrl(MSR_IA32_APERF, tmp);
> +       if (!(tmp - aperf))
> +               return -ENODEV;
> +
> +       rdmsrl(MSR_IA32_MPERF, tmp);
> +       if (!(tmp - mperf))
> +               return -ENODEV;
> +
> +       return 0;
> +}

Add blank line here.

>  static int __init intel_pstate_init(void)
>  {
>         int cpu, rc = 0;
> @@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
>         if (!id)
>                 return -ENODEV;
>
> +       if (intel_pstate_msrs_not_valid())
> +               return -ENODEV;
> +
>         pr_info("Intel P-state driver initializing.\n");
>
>         all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-21  0:08     ` Rafael J. Wysocki
@ 2013-03-22  0:40       ` Rafael J. Wysocki
  2013-03-22 15:06         ` Dave Jones
  0 siblings, 1 reply; 8+ messages in thread
From: Rafael J. Wysocki @ 2013-03-22  0:40 UTC (permalink / raw)
  To: Dirk Brandewie; +Cc: linux-kernel, cpufreq, Dirk Brandewie

On Thursday, March 21, 2013 01:08:03 AM Rafael J. Wysocki wrote:
> On Wednesday, March 20, 2013 11:28:49 AM Dirk Brandewie wrote:
> > On 03/20/2013 11:28 AM, Rafael J. Wysocki wrote:
> > > On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
> > >> From: Dirk Brandewie <dirk.brandewie@gmail.com>
> > >>
> > >> Some VMs seem to try to implement some MSRs but not all the registers
> > >> the driver needs.  Check to make sure all the MSR that we need are
> > >> available. If any of the required MSRs are not available refuse to
> > >> load.
> > >>
> > >> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
> > >
> > > Is this needed for v3.9?  Any pointers to bug reports etc.?
> > >
> > 
> > Sorry I saw right after I sent the mail that the bug report was missing
> > 
> >      https://bugzilla.redhat.com/show_bug.cgi?id=922923
> >      Reported-by: Josh Stone <jistone@redhat.com>
> > 
> > Would you like me to spin the patch?
> 
> No, thanks, this information is sufficient.

Applied to linux-pm.git/bleeding-edge and will be moved to linux-next after
build testing.

Thanks,
Rafael


> > >> ---
> > >>   drivers/cpufreq/intel_pstate.c |   26 ++++++++++++++++++++++++++
> > >>   1 files changed, 26 insertions(+), 0 deletions(-)
> > >>
> > >> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> > >> index f6dd1e7..cd9c5f4 100644
> > >> --- a/drivers/cpufreq/intel_pstate.c
> > >> +++ b/drivers/cpufreq/intel_pstate.c
> > >> @@ -752,6 +752,29 @@ static struct cpufreq_driver intel_pstate_driver = {
> > >>
> > >>   static int __initdata no_load;
> > >>
> > >> +static int intel_pstate_msrs_not_valid(void)
> > >> +{
> > >> +	/* Check that all the msr's we are using are valid. */
> > >> +	u64 aperf, mperf, tmp;
> > >> +
> > >> +	rdmsrl(MSR_IA32_APERF, aperf);
> > >> +	rdmsrl(MSR_IA32_MPERF, mperf);
> > >> +
> > >> +	if (!intel_pstate_min_pstate() ||
> > >> +		!intel_pstate_max_pstate() ||
> > >> +		!intel_pstate_turbo_pstate())
> > >> +		return -ENODEV;
> > >> +
> > >> +	rdmsrl(MSR_IA32_APERF, tmp);
> > >> +	if (!(tmp - aperf))
> > >> +		return -ENODEV;
> > >> +
> > >> +	rdmsrl(MSR_IA32_MPERF, tmp);
> > >> +	if (!(tmp - mperf))
> > >> +		return -ENODEV;
> > >> +
> > >> +	return 0;
> > >> +}
> > >>   static int __init intel_pstate_init(void)
> > >>   {
> > >>   	int cpu, rc = 0;
> > >> @@ -764,6 +787,9 @@ static int __init intel_pstate_init(void)
> > >>   	if (!id)
> > >>   		return -ENODEV;
> > >>
> > >> +	if (intel_pstate_msrs_not_valid())
> > >> +		return -ENODEV;
> > >> +
> > >>   	pr_info("Intel P-state driver initializing.\n");
> > >>
> > >>   	all_cpu_data = vmalloc(sizeof(void *) * num_possible_cpus());
> > >>
> > 
> 
-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-22  0:40       ` Rafael J. Wysocki
@ 2013-03-22 15:06         ` Dave Jones
  2013-03-23  0:06           ` Rafael J. Wysocki
  0 siblings, 1 reply; 8+ messages in thread
From: Dave Jones @ 2013-03-22 15:06 UTC (permalink / raw)
  To: Rafael J. Wysocki; +Cc: Dirk Brandewie, linux-kernel, cpufreq, Dirk Brandewie

On Fri, Mar 22, 2013 at 01:40:49AM +0100, Rafael J. Wysocki wrote:
 > On Thursday, March 21, 2013 01:08:03 AM Rafael J. Wysocki wrote:
 > > On Wednesday, March 20, 2013 11:28:49 AM Dirk Brandewie wrote:
 > > > On 03/20/2013 11:28 AM, Rafael J. Wysocki wrote:
 > > > > On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
 > > > >> From: Dirk Brandewie <dirk.brandewie@gmail.com>
 > > > >>
 > > > >> Some VMs seem to try to implement some MSRs but not all the registers
 > > > >> the driver needs.  Check to make sure all the MSR that we need are
 > > > >> available. If any of the required MSRs are not available refuse to
 > > > >> load.
 > > > >>
 > > > >> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
 > > > >
 > > > > Is this needed for v3.9?  Any pointers to bug reports etc.?
 > > > >
 > > > 
 > > > Sorry I saw right after I sent the mail that the bug report was missing
 > > > 
 > > >      https://bugzilla.redhat.com/show_bug.cgi?id=922923
 > > >      Reported-by: Josh Stone <jistone@redhat.com>
 > > > 
 > > > Would you like me to spin the patch?
 > > 
 > > No, thanks, this information is sufficient.
 > 
 > Applied to linux-pm.git/bleeding-edge and will be moved to linux-next after
 > build testing.

'next' seems to be imply you're targetting this for 3.10 ?
In case it wasn't clear, this is needed for 3.9.

	Dave


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid
  2013-03-22 15:06         ` Dave Jones
@ 2013-03-23  0:06           ` Rafael J. Wysocki
  0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2013-03-23  0:06 UTC (permalink / raw)
  To: Dave Jones; +Cc: Dirk Brandewie, linux-kernel, cpufreq, Dirk Brandewie

On Friday, March 22, 2013 11:06:47 AM Dave Jones wrote:
> On Fri, Mar 22, 2013 at 01:40:49AM +0100, Rafael J. Wysocki wrote:
>  > On Thursday, March 21, 2013 01:08:03 AM Rafael J. Wysocki wrote:
>  > > On Wednesday, March 20, 2013 11:28:49 AM Dirk Brandewie wrote:
>  > > > On 03/20/2013 11:28 AM, Rafael J. Wysocki wrote:
>  > > > > On Wednesday, March 20, 2013 09:17:24 AM dirk.brandewie@gmail.com wrote:
>  > > > >> From: Dirk Brandewie <dirk.brandewie@gmail.com>
>  > > > >>
>  > > > >> Some VMs seem to try to implement some MSRs but not all the registers
>  > > > >> the driver needs.  Check to make sure all the MSR that we need are
>  > > > >> available. If any of the required MSRs are not available refuse to
>  > > > >> load.
>  > > > >>
>  > > > >> Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
>  > > > >
>  > > > > Is this needed for v3.9?  Any pointers to bug reports etc.?
>  > > > >
>  > > > 
>  > > > Sorry I saw right after I sent the mail that the bug report was missing
>  > > > 
>  > > >      https://bugzilla.redhat.com/show_bug.cgi?id=922923
>  > > >      Reported-by: Josh Stone <jistone@redhat.com>
>  > > > 
>  > > > Would you like me to spin the patch?
>  > > 
>  > > No, thanks, this information is sufficient.
>  > 
>  > Applied to linux-pm.git/bleeding-edge and will be moved to linux-next after
>  > build testing.
> 
> 'next' seems to be imply you're targetting this for 3.10 ?
> In case it wasn't clear, this is needed for 3.9.

I'm going to push this for 3.9 after it has spent a couple of days in
linux-next just in case.  So 3.9-rc5 is the target.

Thanks,
Rafael


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

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2013-03-22 23:59 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-20 16:17 [PATCH] cpufreq/intel_pstate: Add function to check that all MSR's are valid dirk.brandewie
2013-03-20 18:28 ` Rafael J. Wysocki
2013-03-20 18:28   ` Dirk Brandewie
2013-03-21  0:08     ` Rafael J. Wysocki
2013-03-22  0:40       ` Rafael J. Wysocki
2013-03-22 15:06         ` Dave Jones
2013-03-23  0:06           ` Rafael J. Wysocki
2013-03-21  3:20 ` Viresh Kumar

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.