xen-devel

[Top] [All Lists]

[Xen-devel] [PATCH] xen/powernow: allow using lowest-power P-state

from [Jan Beulich]

[Permanent Link][Original]

To:	<xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject:	[Xen-devel] [PATCH] xen/powernow: allow using lowest-power P-state
From:	"Jan Beulich" <jbeulich@xxxxxxxxxx>
Date:	Mon, 08 Sep 2008 13:55:33 +0100
Cc:	mark.langsdorf@xxxxxxx
Delivery-date:	Mon, 08 Sep 2008 05:55:30 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
List-help:	<mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id:	Xen developer discussion <xen-devel.lists.xensource.com>
List-post:	<mailto:xen-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

The MSR field used here indicates the highest numbered valid P-state,
not the first invalid one.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>

Index: 2008-09-01/xen/arch/x86/acpi/cpufreq/powernow.c
===================================================================
--- 2008-09-01.orig/xen/arch/x86/acpi/cpufreq/powernow.c        2008-07-18 
16:19:34.000000000 +0200
+++ 2008-09-01/xen/arch/x86/acpi/cpufreq/powernow.c     2008-09-08 
14:46:12.000000000 +0200
@@ -197,8 +197,8 @@ static int powernow_cpufreq_cpu_init(str
 
     data->max_freq = perf->states[0].core_frequency * 1000;
     /* table init */
-    for (i=0; i<perf->state_count && i<max_hw_pstate; i++) {
-        if (i>0 && perf->states[i].core_frequency >=
+    for (i = 0; i < perf->state_count && i <= max_hw_pstate; i++) {
+        if (i > 0 && perf->states[i].core_frequency >=
             data->freq_table[valid_states-1].frequency / 1000)
             continue;
 




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-devel] [PATCH] xen/powernow: allow using lowest-power P-state, Jan Beulich <=

Previous by Date:	[Xen-devel] [PATCH] x86: fix cpufreq_dom_mask, Jan Beulich
Next by Date:	[Xen-devel] [PATCH] linux: make Xen cpufreq handling also work without _PSD objects, Jan Beulich
Previous by Thread:	[Xen-devel] [PATCH] x86: fix cpufreq_dom_mask, Jan Beulich
Next by Thread:	[Xen-devel] [PATCH] linux: make Xen cpufreq handling also work without _PSD objects, Jan Beulich
Indexes:	[Date] [Thread] [Top] [All Lists]