From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45469) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z5uLs-0007FN-Po for qemu-devel@nongnu.org; Fri, 19 Jun 2015 07:20:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z5uLn-000337-J5 for qemu-devel@nongnu.org; Fri, 19 Jun 2015 07:20:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:50669) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z5uLn-00031x-90 for qemu-devel@nongnu.org; Fri, 19 Jun 2015 07:20:47 -0400 From: =?UTF-8?q?Radim=20Kr=C4=8Dm=C3=A1=C5=99?= Date: Fri, 19 Jun 2015 13:20:40 +0200 Message-Id: <1434712840-14373-1-git-send-email-rkrcmar@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH v2] target-i386: emulate CPUID level of real hardware List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: pbonzini@redhat.com, bsd@redhat.com, ehabkost@redhat.com, rth@twiddle.net W10 insider has a bug where it ignores CPUID level and interprets CPUID.(EAX=3D07H, ECX=3D0H) incorrectly, because CPUID in fact returned CPUID.(EAX=3D04H, ECX=3D0H); this resulted in execution of unsupported instructions. While it's a Windows bug, there is no reason to emulate incorrect level. I have used http://instlatx64.atw.hu/ as a source of CPUID and checked that it matches Penryn Xeon X5472, Westmere Xeon W3520, SandyBridge i5-2540M, and Haswell i5-4670T. kvm64 and qemu64 were bumped to 0xD to allow all available features for them (and to avoid the same Windows bug). Signed-off-by: Radim Kr=C4=8Dm=C3=A1=C5=99 --- v2: * maintained compatibility for old (< 2.4) machine types [Paolo] * dropped automatic increase to 0xd, for compatibility [Eduardo] I devote this code to copy&paste. hw/i386/pc_piix.c | 18 ++++++++++++++++ hw/i386/pc_q35.c | 18 ++++++++++++++++ target-i386/cpu.c | 63 ++++++++++++++++++++++++++++++++++++++-----------= ------ target-i386/cpu.h | 3 +++ 4 files changed, 83 insertions(+), 19 deletions(-) diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c index e142f75649d8..c335e230277b 100644 --- a/hw/i386/pc_piix.c +++ b/hw/i386/pc_piix.c @@ -307,6 +307,24 @@ static void pc_init1(MachineState *machine) static void pc_compat_2_3(MachineState *machine) { savevm_skip_section_footers(); + + x86_cpu_compat_set_level("qemu64", 4); + x86_cpu_compat_set_level("kvm64", 5); + x86_cpu_compat_set_level("pentium3", 2); + x86_cpu_compat_set_level("n270", 5); + x86_cpu_compat_set_level("Conroe", 4); + x86_cpu_compat_set_level("Penryn", 4); + x86_cpu_compat_set_level("Nehalem", 4); + + x86_cpu_compat_set_xlevel("n270", 0x8000000a); + x86_cpu_compat_set_xlevel("Penryn", 0x8000000a); + x86_cpu_compat_set_xlevel("Nehalem", 0x8000000a); + x86_cpu_compat_set_xlevel("Westmere", 0x8000000a); + x86_cpu_compat_set_xlevel("SandyBridge", 0x8000000a); + x86_cpu_compat_set_xlevel("Haswell", 0x8000000a); + x86_cpu_compat_set_xlevel("Haswell-noTSX", 0x8000000a); + x86_cpu_compat_set_xlevel("Broadwell", 0x8000000a); + x86_cpu_compat_set_xlevel("Broadwell-noTSX", 0x8000000a); } =20 static void pc_compat_2_2(MachineState *machine) diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c index 082cd93bb2d7..99bfd6d6f288 100644 --- a/hw/i386/pc_q35.c +++ b/hw/i386/pc_q35.c @@ -291,6 +291,24 @@ static void pc_q35_init(MachineState *machine) static void pc_compat_2_3(MachineState *machine) { savevm_skip_section_footers(); + + x86_cpu_compat_set_level("qemu64", 4); + x86_cpu_compat_set_level("kvm64", 5); + x86_cpu_compat_set_level("pentium3", 2); + x86_cpu_compat_set_level("n270", 5); + x86_cpu_compat_set_level("Conroe", 4); + x86_cpu_compat_set_level("Penryn", 4); + x86_cpu_compat_set_level("Nehalem", 4); + + x86_cpu_compat_set_xlevel("n270", 0x8000000a); + x86_cpu_compat_set_xlevel("Penryn", 0x8000000a); + x86_cpu_compat_set_xlevel("Nehalem", 0x8000000a); + x86_cpu_compat_set_xlevel("Westmere", 0x8000000a); + x86_cpu_compat_set_xlevel("SandyBridge", 0x8000000a); + x86_cpu_compat_set_xlevel("Haswell", 0x8000000a); + x86_cpu_compat_set_xlevel("Haswell-noTSX", 0x8000000a); + x86_cpu_compat_set_xlevel("Broadwell", 0x8000000a); + x86_cpu_compat_set_xlevel("Broadwell-noTSX", 0x8000000a); } =20 static void pc_compat_2_2(MachineState *machine) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 4e7cdaaaa57e..7dabf7de25ea 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -677,7 +677,7 @@ struct X86CPUDefinition { static X86CPUDefinition builtin_x86_defs[] =3D { { .name =3D "qemu64", - .level =3D 4, + .level =3D 0xd, .vendor =3D CPUID_VENDOR_AMD, .family =3D 6, .model =3D 6, @@ -753,7 +753,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { }, { .name =3D "kvm64", - .level =3D 5, + .level =3D 0xd, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 15, .model =3D 6, @@ -864,7 +864,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { }, { .name =3D "pentium3", - .level =3D 2, + .level =3D 3, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 6, .model =3D 7, @@ -889,8 +889,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { }, { .name =3D "n270", - /* original is on level 10 */ - .level =3D 5, + .level =3D 10, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 6, .model =3D 28, @@ -910,12 +909,12 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT2_NX, .features[FEAT_8000_0001_ECX] =3D CPUID_EXT3_LAHF_LM, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel(R) Atom(TM) CPU N270 @ 1.60GHz", }, { .name =3D "Conroe", - .level =3D 4, + .level =3D 10, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 6, .model =3D 15, @@ -932,12 +931,12 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT2_LM | CPUID_EXT2_NX | CPUID_EXT2_SYSCALL, .features[FEAT_8000_0001_ECX] =3D CPUID_EXT3_LAHF_LM, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Celeron_4x0 (Conroe/Merom Class Core 2)", }, { .name =3D "Penryn", - .level =3D 4, + .level =3D 10, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 6, .model =3D 23, @@ -955,12 +954,12 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT2_LM | CPUID_EXT2_NX | CPUID_EXT2_SYSCALL, .features[FEAT_8000_0001_ECX] =3D CPUID_EXT3_LAHF_LM, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core 2 Duo P9xxx (Penryn Class Core 2)", }, { .name =3D "Nehalem", - .level =3D 4, + .level =3D 11, .vendor =3D CPUID_VENDOR_INTEL, .family =3D 6, .model =3D 26, @@ -978,7 +977,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT2_LM | CPUID_EXT2_SYSCALL | CPUID_EXT2_NX, .features[FEAT_8000_0001_ECX] =3D CPUID_EXT3_LAHF_LM, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core i7 9xx (Nehalem Class Core i7)", }, { @@ -1002,7 +1001,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT2_LM | CPUID_EXT2_SYSCALL | CPUID_EXT2_NX, .features[FEAT_8000_0001_ECX] =3D CPUID_EXT3_LAHF_LM, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Westmere E56xx/L56xx/X56xx (Nehalem-C)", }, { @@ -1031,7 +1030,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT3_LAHF_LM, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Xeon E312xx (Sandy Bridge)", }, { @@ -1063,7 +1062,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_EXT3_LAHF_LM, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Xeon E3-12xx v2 (Ivy Bridge)", }, { @@ -1097,7 +1096,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_7_0_EBX_BMI2 | CPUID_7_0_EBX_ERMS | CPUID_7_0_EBX_INVP= CID, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core Processor (Haswell, no TSX)", }, { .name =3D "Haswell", @@ -1131,7 +1130,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_7_0_EBX_RTM, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core Processor (Haswell)", }, { @@ -1167,7 +1166,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_7_0_EBX_SMAP, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core Processor (Broadwell, no TSX)", }, { @@ -1203,7 +1202,7 @@ static X86CPUDefinition builtin_x86_defs[] =3D { CPUID_7_0_EBX_SMAP, .features[FEAT_XSAVE] =3D CPUID_XSAVE_XSAVEOPT, - .xlevel =3D 0x8000000A, + .xlevel =3D 0x80000008, .model_id =3D "Intel Core Processor (Broadwell)", }, { @@ -1387,6 +1386,32 @@ void x86_cpu_compat_set_features(const char *cpu_m= odel, FeatureWord w, } } =20 +void x86_cpu_compat_set_level(const char *cpu_model, uint32_t level) +{ + X86CPUDefinition *def; + int i; + + for (i =3D 0; i < ARRAY_SIZE(builtin_x86_defs); i++) { + def =3D &builtin_x86_defs[i]; + if (!cpu_model || !strcmp(cpu_model, def->name)) { + def->level =3D level; + } + } +} + +void x86_cpu_compat_set_xlevel(const char *cpu_model, uint32_t xlevel) +{ + X86CPUDefinition *def; + int i; + + for (i =3D 0; i < ARRAY_SIZE(builtin_x86_defs); i++) { + def =3D &builtin_x86_defs[i]; + if (!cpu_model || !strcmp(cpu_model, def->name)) { + def->xlevel =3D xlevel; + } + } +} + static uint32_t x86_cpu_get_supported_feature_word(FeatureWord w, bool migratable_only)= ; =20 diff --git a/target-i386/cpu.h b/target-i386/cpu.h index 603aaf0924e3..b1f16ee6195a 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -1331,6 +1331,9 @@ void cpu_report_tpr_access(CPUX86State *env, TPRAcc= ess access); void x86_cpu_compat_set_features(const char *cpu_model, FeatureWord w, uint32_t feat_add, uint32_t feat_remove= ); =20 +void x86_cpu_compat_set_level(const char *cpu_model, uint32_t level); +void x86_cpu_compat_set_xlevel(const char *cpu_model, uint32_t xlevel); + void x86_cpu_compat_kvm_no_autoenable(FeatureWord w, uint32_t features); void x86_cpu_compat_kvm_no_autodisable(FeatureWord w, uint32_t features)= ; =20 --=20 2.4.4