Message ID | 55bcfe11-251c-606e-6f0f-a03880efa390@suse.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | x86: drop cpu_has_ffxsr | expand |
On 24/04/2020 11:24, Jan Beulich wrote: > It's definition is bogus when it comes to Hygon CPUs, but since we don't > use it anywhere drop it rather than correcting it. > > Signed-off-by: Jan Beulich <jbeulich@suse.com> I had wondered about that too. Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
--- a/xen/include/asm-x86/cpufeature.h +++ b/xen/include/asm-x86/cpufeature.h @@ -66,8 +66,6 @@ /* CPUID level 0x80000001.edx */ #define cpu_has_nx boot_cpu_has(X86_FEATURE_NX) -#define cpu_has_ffxsr ((boot_cpu_data.x86_vendor == X86_VENDOR_AMD) \ - && boot_cpu_has(X86_FEATURE_FFXSR)) #define cpu_has_page1gb boot_cpu_has(X86_FEATURE_PAGE1GB) #define cpu_has_rdtscp boot_cpu_has(X86_FEATURE_RDTSCP) #define cpu_has_3dnow_ext boot_cpu_has(X86_FEATURE_3DNOWEXT)
It's definition is bogus when it comes to Hygon CPUs, but since we don't use it anywhere drop it rather than correcting it. Signed-off-by: Jan Beulich <jbeulich@suse.com>