[Xen-devel] [PATCH] X86: cpufreq init cleanup

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Resend it, mail server seems have some issue yesterday :(

Thanks,
Jinsong

==========================

X86: cpufreq init cleanup

c/s 20325 change AMD cpufreq init logic.
Before that, AMD cpu start cpufreq init logic only when all cpus ready.
c/s 20325 change it to per cpu add, however, leave code un-elegant.

This patch do a little cleanup work.

Signed-off-by: Liu, Jinsong <jinsong.liu@xxxxxxxxx>

diff -r 15ad40a174dc xen/arch/x86/acpi/cpufreq/cpufreq.c
--- a/xen/arch/x86/acpi/cpufreq/cpufreq.c       Thu May 19 19:00:23 2011 +0800
+++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c       Mon May 30 20:30:11 2011 +0800
@@ -695,15 +695,11 @@ __initcall(cpufreq_driver_init);
 
 int cpufreq_cpu_init(unsigned int cpuid)
 {
-    static int cpu_count=0;
     int ret;
 
-    cpu_count++; 
-
     /* Currently we only handle Intel and AMD processor */
-    if ( boot_cpu_data.x86_vendor == X86_VENDOR_INTEL )
-        ret = cpufreq_add_cpu(cpuid);
-    else if ( boot_cpu_data.x86_vendor == X86_VENDOR_AMD )
+    if ( (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL ) ||
+         (boot_cpu_data.x86_vendor == X86_VENDOR_AMD ) )
         ret = cpufreq_add_cpu(cpuid);
     else
         ret = -EFAULT;

Attachment: cpufreq_fix.patch
Description: cpufreq_fix.patch

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