x86: Stop CPUID from claiming we support xsave.
authorGabe Black <gabeblack@google.com>
Wed, 14 Aug 2019 22:35:48 +0000 (15:35 -0700)
committerGabe Black <gabeblack@google.com>
Fri, 16 Aug 2019 01:05:01 +0000 (01:05 +0000)
xsave is a fairly complex feature which we don't support in gem5, but
we do report that we support it through CPUID. It looks like I confused
it with FXSAVE which is an instruction related to SSE. This change
turns that bit back off again.

Change-Id: I00fc79168c5f7095b5241e870a4c8782e4385425
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/20169
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Reviewed-by: Pouya Fotouhi <pfotouhi@ucdavis.edu>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/x86/cpuid.cc

index 867087eb098a015be0da83739f29a533cd7d2c34..e6a898a08c270654d64fec9065531da56ef5a89c 100644 (file)
@@ -162,7 +162,7 @@ namespace X86ISA {
                 break;
               case FamilyModelStepping:
                 result = CpuidResult(0x00020f51, 0x00000805,
-                                     0xe7dbfbff, 0x04000209);
+                                     0xe7dbfbff, 0x00000209);
                 break;
               case ExtendedFeatures:
                 result = CpuidResult(0x00000000, 0x01800000,