[Xen-devel] [PATCH] Fix bug: unify cpu_dev attr as __cpuinitdata

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

Fix bug: unify cpu_dev attr as __cpuinitdata

Currently different x86 cpu define different attr for cpu_dev.
Some cpu define as __initdata, this would be risk under cpu hotplug.
This patch fix the bug, unify them as __cpuinitdata, as what AMD cpu define now.

Signed-off-by: Liu, Jinsong <jinsong.liu@xxxxxxxxx>
               Shan, Haitao <haitao.shan@xxxxxxxxx>


diff -r c9a131669548 xen/arch/x86/cpu/centaur.c
--- a/xen/arch/x86/cpu/centaur.c        Sat Jul 02 03:13:41 2011 +0800
+++ b/xen/arch/x86/cpu/centaur.c        Mon Jul 04 11:18:40 2011 +0800
@@ -85,7 +85,7 @@ static unsigned int centaur_size_cache(s
        return size;
 }
 
-static struct cpu_dev centaur_cpu_dev __initdata = {
+static struct cpu_dev centaur_cpu_dev __cpuinitdata = {
        .c_vendor       = "Centaur",
        .c_ident        = { "CentaurHauls" },
        .c_init         = init_centaur,
diff -r c9a131669548 xen/arch/x86/cpu/cyrix.c
--- a/xen/arch/x86/cpu/cyrix.c  Sat Jul 02 03:13:41 2011 +0800
+++ b/xen/arch/x86/cpu/cyrix.c  Mon Jul 04 11:18:40 2011 +0800
@@ -284,7 +284,7 @@ static inline int test_cyrix_52div(void)
        return (unsigned char) (test >> 8) == 0x02;
 }
 
-static struct cpu_dev cyrix_cpu_dev __initdata = {
+static struct cpu_dev cyrix_cpu_dev __cpuinitdata = {
        .c_vendor       = "Cyrix",
        .c_ident        = { "CyrixInstead" },
        .c_init         = init_cyrix,
@@ -299,7 +299,7 @@ int __init cyrix_init_cpu(void)
 
 //early_arch_initcall(cyrix_init_cpu);
 
-static struct cpu_dev nsc_cpu_dev __initdata = {
+static struct cpu_dev nsc_cpu_dev __cpuinitdata = {
        .c_vendor       = "NSC",
        .c_ident        = { "Geode by NSC" },
        .c_init         = init_cyrix,
diff -r c9a131669548 xen/arch/x86/cpu/intel.c
--- a/xen/arch/x86/cpu/intel.c  Sat Jul 02 03:13:41 2011 +0800
+++ b/xen/arch/x86/cpu/intel.c  Mon Jul 04 11:18:40 2011 +0800
@@ -271,7 +271,7 @@ static unsigned int intel_size_cache(str
        return size;
 }
 
-static struct cpu_dev intel_cpu_dev __devinitdata = {
+static struct cpu_dev intel_cpu_dev __cpuinitdata = {
        .c_vendor       = "Intel",
        .c_ident        = { "GenuineIntel" },
        .c_models = {
diff -r c9a131669548 xen/arch/x86/cpu/transmeta.c
--- a/xen/arch/x86/cpu/transmeta.c      Sat Jul 02 03:13:41 2011 +0800
+++ b/xen/arch/x86/cpu/transmeta.c      Mon Jul 04 11:18:40 2011 +0800
@@ -92,7 +92,7 @@ static void transmeta_identify(struct cp
        }
 }
 
-static struct cpu_dev transmeta_cpu_dev __initdata = {
+static struct cpu_dev transmeta_cpu_dev __cpuinitdata = {
        .c_vendor       = "Transmeta",
        .c_ident        = { "GenuineTMx86", "TransmetaCPU" },
        .c_init         = init_transmeta,

Attachment: cpu_init_fix.patch
Description: cpu_init_fix.patch

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