[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] x86: remove assumptions about the layout of x86_capabilities
commit d8ba3a9e444c6943199130eea32904bc245a6d27 Author: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> AuthorDate: Mon Oct 26 14:02:30 2015 +0100 Commit: Jan Beulich <jbeulich@xxxxxxxx> CommitDate: Mon Oct 26 14:02:30 2015 +0100 x86: remove assumptions about the layout of x86_capabilities Future work will rearange it, invalidating these assumptions. No functional change. Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx> --- xen/arch/x86/cpu/amd.c | 2 +- xen/arch/x86/cpu/centaur.c | 6 ++---- xen/arch/x86/cpu/common.c | 18 ++++++++++-------- xen/arch/x86/efi/efi-boot.h | 3 ++- xen/arch/x86/mpparse.c | 6 ++++-- xen/include/asm-x86/cpufeature.h | 1 + 6 files changed, 20 insertions(+), 16 deletions(-) diff --git a/xen/arch/x86/cpu/amd.c b/xen/arch/x86/cpu/amd.c index fd57370..74a0152 100644 --- a/xen/arch/x86/cpu/amd.c +++ b/xen/arch/x86/cpu/amd.c @@ -444,7 +444,7 @@ static void __devinit init_amd(struct cpuinfo_x86 *c) /* Bit 31 in normal CPUID used for nonstandard 3DNow ID; 3DNow is IDd by bit 31 in extended CPUID (1*32+31) anyway */ - __clear_bit(0*32+31, c->x86_capability); + __clear_bit(X86_FEATURE_3DNOW_ALT, c->x86_capability); if (c->x86 == 0xf && c->x86_model < 0x14 && cpu_has(c, X86_FEATURE_LAHF_LM)) { diff --git a/xen/arch/x86/cpu/centaur.c b/xen/arch/x86/cpu/centaur.c index 66b1995..c0ac117 100644 --- a/xen/arch/x86/cpu/centaur.c +++ b/xen/arch/x86/cpu/centaur.c @@ -39,10 +39,8 @@ static void init_c3(struct cpuinfo_x86 *c) printk(KERN_INFO "CPU: Enabled h/w RNG\n"); } - /* store Centaur Extended Feature Flags as - * word 5 of the CPU capability bit array - */ - c->x86_capability[5] = cpuid_edx(0xC0000001); + c->x86_capability[cpufeat_word(X86_FEATURE_XSTORE)] + = cpuid_edx(0xC0000001); } if (c->x86 == 0x6 && c->x86_model >= 0xf) { diff --git a/xen/arch/x86/cpu/common.c b/xen/arch/x86/cpu/common.c index a5caa84..653b052 100644 --- a/xen/arch/x86/cpu/common.c +++ b/xen/arch/x86/cpu/common.c @@ -200,13 +200,13 @@ static void __init early_cpu_detect(void) if (c->x86 >= 0x6) c->x86_model += ((tfms >> 16) & 0xF) << 4; c->x86_mask = tfms & 15; - cap0 &= ~cleared_caps[0]; - cap4 &= ~cleared_caps[4]; + cap0 &= ~cleared_caps[cpufeat_word(X86_FEATURE_FPU)]; + cap4 &= ~cleared_caps[cpufeat_word(X86_FEATURE_XMM3)]; if (cap0 & cpufeat_mask(X86_FEATURE_CLFLSH)) c->x86_cache_alignment = ((misc >> 8) & 0xff) * 8; /* Leaf 0x1 capabilities filled in early for Xen. */ - c->x86_capability[0] = cap0; - c->x86_capability[4] = cap4; + c->x86_capability[cpufeat_word(X86_FEATURE_FPU)] = cap0; + c->x86_capability[cpufeat_word(X86_FEATURE_XMM3)] = cap4; } static void __cpuinit generic_identify(struct cpuinfo_x86 *c) @@ -225,8 +225,8 @@ static void __cpuinit generic_identify(struct cpuinfo_x86 *c) /* Intel-defined flags: level 0x00000001 */ cpuid(0x00000001, &tfms, &ebx, &excap, &capability); - c->x86_capability[0] = capability; - c->x86_capability[4] = excap; + c->x86_capability[cpufeat_word(X86_FEATURE_FPU)] = capability; + c->x86_capability[cpufeat_word(X86_FEATURE_XMM3)] = excap; c->x86 = (tfms >> 8) & 15; c->x86_model = (tfms >> 4) & 15; if (c->x86 == 0xf) @@ -247,8 +247,10 @@ static void __cpuinit generic_identify(struct cpuinfo_x86 *c) c->extended_cpuid_level = cpuid_eax(0x80000000); if ( (c->extended_cpuid_level & 0xffff0000) == 0x80000000 ) { if ( c->extended_cpuid_level >= 0x80000001 ) { - c->x86_capability[1] = cpuid_edx(0x80000001); - c->x86_capability[6] = cpuid_ecx(0x80000001); + c->x86_capability[cpufeat_word(X86_FEATURE_SYSCALL)] + = cpuid_edx(0x80000001); + c->x86_capability[cpufeat_word(X86_FEATURE_LAHF_LM)] + = cpuid_ecx(0x80000001); } if ( c->extended_cpuid_level >= 0x80000004 ) get_model_name(c); /* Default name */ diff --git a/xen/arch/x86/efi/efi-boot.h b/xen/arch/x86/efi/efi-boot.h index d8ca862..6dbb14d 100644 --- a/xen/arch/x86/efi/efi-boot.h +++ b/xen/arch/x86/efi/efi-boot.h @@ -608,7 +608,8 @@ static void __init efi_arch_cpu(void) if ( cpuid_eax(0x80000000) > 0x80000000 ) { cpuid_ext_features = cpuid_edx(0x80000001); - boot_cpu_data.x86_capability[1] = cpuid_ext_features; + boot_cpu_data.x86_capability[cpufeat_word(X86_FEATURE_SYSCALL)] + = cpuid_ext_features; } } diff --git a/xen/arch/x86/mpparse.c b/xen/arch/x86/mpparse.c index 8609f4a..dc74e37 100644 --- a/xen/arch/x86/mpparse.c +++ b/xen/arch/x86/mpparse.c @@ -495,7 +495,8 @@ static inline void __init construct_default_ISA_mptable(int mpc_default_type) processor.mpc_cpufeature = (boot_cpu_data.x86 << 8) | (boot_cpu_data.x86_model << 4) | boot_cpu_data.x86_mask; - processor.mpc_featureflag = boot_cpu_data.x86_capability[0]; + processor.mpc_featureflag = + boot_cpu_data.x86_capability[cpufeat_word(X86_FEATURE_FPU)]; processor.mpc_reserved[0] = 0; processor.mpc_reserved[1] = 0; for (i = 0; i < 2; i++) { @@ -799,7 +800,8 @@ int __devinit mp_register_lapic ( processor.mpc_cpuflag |= (boot_cpu ? CPU_BOOTPROCESSOR : 0); processor.mpc_cpufeature = (boot_cpu_data.x86 << 8) | (boot_cpu_data.x86_model << 4) | boot_cpu_data.x86_mask; - processor.mpc_featureflag = boot_cpu_data.x86_capability[0]; + processor.mpc_featureflag + = boot_cpu_data.x86_capability[cpufeat_word(X86_FEATURE_FPU)]; processor.mpc_reserved[0] = 0; processor.mpc_reserved[1] = 0; diff --git a/xen/include/asm-x86/cpufeature.h b/xen/include/asm-x86/cpufeature.h index e823954..af127cf 100644 --- a/xen/include/asm-x86/cpufeature.h +++ b/xen/include/asm-x86/cpufeature.h @@ -45,6 +45,7 @@ #define X86_FEATURE_ACC (0*32+29) /* Automatic clock control */ #define X86_FEATURE_IA64 (0*32+30) /* IA-64 processor */ #define X86_FEATURE_PBE (0*32+31) /* Pending Break Enable */ +#define X86_FEATURE_3DNOW_ALT (0*32+31) /* AMD nonstandard 3DNow (Aliases PBE) */ /* AMD-defined CPU features, CPUID level 0x80000001, word 1 */ /* Don't duplicate feature flags which are redundant with Intel! */ -- generated by git-patchbot for /home/xen/git/xen.git#master _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxx http://lists.xensource.com/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |