diff options
author | Ingo Molnar <mingo@kernel.org> | 2017-11-06 03:43:13 +0300 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-11-06 03:43:13 +0300 |
commit | 19c5787a5ff88a959e5854ab6ddef7f5777c5837 (patch) | |
tree | 4aad2b3ff9e6e02ae45398470495dfe88cf7f6b9 /arch/x86 | |
parent | 1e4c4f610f774df6088d7c065b2dd4d22adba698 (diff) | |
parent | 1943dc07b45e347c52c1bfdd4a37e04a86e399aa (diff) | |
download | linux-19c5787a5ff88a959e5854ab6ddef7f5777c5837.tar.xz |
Merge branch 'x86/fpu' into x86/asm, to pick up fix
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/x86')
-rw-r--r-- | arch/x86/kernel/cpu/cpuid-deps.c | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/arch/x86/kernel/cpu/cpuid-deps.c b/arch/x86/kernel/cpu/cpuid-deps.c index c21f22d836ad..904b0a3c4e53 100644 --- a/arch/x86/kernel/cpu/cpuid-deps.c +++ b/arch/x86/kernel/cpu/cpuid-deps.c @@ -62,23 +62,19 @@ const static struct cpuid_dep cpuid_deps[] = { {} }; -static inline void __clear_cpu_cap(struct cpuinfo_x86 *c, unsigned int bit) -{ - clear_bit32(bit, c->x86_capability); -} - -static inline void __setup_clear_cpu_cap(unsigned int bit) -{ - clear_cpu_cap(&boot_cpu_data, bit); - set_bit32(bit, cpu_caps_cleared); -} - static inline void clear_feature(struct cpuinfo_x86 *c, unsigned int feature) { - if (!c) - __setup_clear_cpu_cap(feature); - else - __clear_cpu_cap(c, feature); + /* + * Note: This could use the non atomic __*_bit() variants, but the + * rest of the cpufeature code uses atomics as well, so keep it for + * consistency. Cleanup all of it separately. + */ + if (!c) { + clear_cpu_cap(&boot_cpu_data, feature); + set_bit(feature, (unsigned long *)cpu_caps_cleared); + } else { + clear_bit(feature, (unsigned long *)c->x86_capability); + } } /* Take the capabilities and the BUG bits into account */ |