|
 |
9ae3a8 |
From eb87f1106d038247356ecd3071e6fa5654386ff5 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
9ae3a8 |
Date: Fri, 9 Sep 2016 19:08:33 +0200
|
|
|
9ae3a8 |
Subject: [PATCH 1/2] target-i386: Add support for FEAT_7_0_ECX
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
9ae3a8 |
Message-id: <1473448114-1430-2-git-send-email-ehabkost@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 72276
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.3 qemu-kvm PATCH v2 1/2] target-i386: Add support for FEAT_7_0_ECX
|
|
|
9ae3a8 |
Bugzilla: 1372459
|
|
|
9ae3a8 |
RH-Acked-by: Bandan Das <bsd@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
From: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1372459
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Done from scratch by copying what is done for FEAT_7_0_EBX. Compare
|
|
|
9ae3a8 |
to upstream commit f74eefe ("target-i386: Add PKU and and OSPKE support",
|
|
|
9ae3a8 |
2016-01-21), but without actually adding PKU and OSPKE. Because all
|
|
|
9ae3a8 |
these features are "-cpu host"-only, they can be added without modifying
|
|
|
9ae3a8 |
machine types.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
[ehabkost: v2: removed the non-upstream cpuid_level < 7 check]
|
|
|
9ae3a8 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
target-i386/cpu.c | 29 ++++++++++++++++++++++++++++-
|
|
|
9ae3a8 |
target-i386/cpu.h | 1 +
|
|
|
9ae3a8 |
2 files changed, 29 insertions(+), 1 deletion(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
|
|
|
9ae3a8 |
index 1001c47..d9c214c 100644
|
|
|
9ae3a8 |
--- a/target-i386/cpu.c
|
|
|
9ae3a8 |
+++ b/target-i386/cpu.c
|
|
|
9ae3a8 |
@@ -150,6 +150,17 @@ static const char *cpuid_7_0_ebx_feature_name[] = {
|
|
|
9ae3a8 |
NULL, NULL, "avx512pf", "avx512er", "avx512cd", NULL, NULL, NULL,
|
|
|
9ae3a8 |
};
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
+static const char *cpuid_7_0_ecx_feature_name[] = {
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+ NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
+};
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
static const char *cpuid_xsave_feature_name[] = {
|
|
|
9ae3a8 |
"xsaveopt", "xsavec", "xgetbv1", NULL,
|
|
|
9ae3a8 |
NULL, NULL, NULL, NULL,
|
|
|
9ae3a8 |
@@ -204,6 +215,12 @@ static FeatureWordInfo feature_word_info[FEATURE_WORDS] = {
|
|
|
9ae3a8 |
.cpuid_needs_ecx = true, .cpuid_ecx = 0,
|
|
|
9ae3a8 |
.cpuid_reg = R_EBX,
|
|
|
9ae3a8 |
},
|
|
|
9ae3a8 |
+ [FEAT_7_0_ECX] = {
|
|
|
9ae3a8 |
+ .feat_names = cpuid_7_0_ecx_feature_name,
|
|
|
9ae3a8 |
+ .cpuid_eax = 7,
|
|
|
9ae3a8 |
+ .cpuid_needs_ecx = true, .cpuid_ecx = 0,
|
|
|
9ae3a8 |
+ .cpuid_reg = R_ECX,
|
|
|
9ae3a8 |
+ },
|
|
|
9ae3a8 |
[FEAT_XSAVE] = {
|
|
|
9ae3a8 |
.feat_names = cpuid_xsave_feature_name,
|
|
|
9ae3a8 |
.cpuid_eax = 0xd,
|
|
|
9ae3a8 |
@@ -462,6 +479,7 @@ typedef struct x86_def_t {
|
|
|
9ae3a8 |
CPUID_7_0_EBX_FSGSBASE, CPUID_7_0_EBX_HLE, CPUID_7_0_EBX_AVX2,
|
|
|
9ae3a8 |
CPUID_7_0_EBX_ERMS, CPUID_7_0_EBX_INVPCID, CPUID_7_0_EBX_RTM,
|
|
|
9ae3a8 |
CPUID_7_0_EBX_RDSEED */
|
|
|
9ae3a8 |
+#define TCG_7_0_ECX_FEATURES 0
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
/* built-in CPU model definitions
|
|
|
9ae3a8 |
*/
|
|
|
9ae3a8 |
@@ -1198,8 +1216,11 @@ static void kvm_cpu_fill_host(x86_def_t *x86_cpu_def)
|
|
|
9ae3a8 |
if (x86_cpu_def->level >= 7) {
|
|
|
9ae3a8 |
x86_cpu_def->features[FEAT_7_0_EBX] =
|
|
|
9ae3a8 |
kvm_arch_get_supported_cpuid(s, 0x7, 0, R_EBX);
|
|
|
9ae3a8 |
+ x86_cpu_def->features[FEAT_7_0_ECX] =
|
|
|
9ae3a8 |
+ kvm_arch_get_supported_cpuid(s, 0x7, 0, R_ECX);
|
|
|
9ae3a8 |
} else {
|
|
|
9ae3a8 |
x86_cpu_def->features[FEAT_7_0_EBX] = 0;
|
|
|
9ae3a8 |
+ x86_cpu_def->features[FEAT_7_0_ECX] = 0;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
x86_cpu_def->features[FEAT_XSAVE] =
|
|
|
9ae3a8 |
kvm_arch_get_supported_cpuid(s, 0xd, 1, R_EAX);
|
|
|
9ae3a8 |
@@ -1283,6 +1304,9 @@ static int kvm_check_features_against_host(X86CPU *cpu)
|
|
|
9ae3a8 |
{&env->features[FEAT_7_0_EBX],
|
|
|
9ae3a8 |
&host_def.features[FEAT_7_0_EBX],
|
|
|
9ae3a8 |
FEAT_7_0_EBX },
|
|
|
9ae3a8 |
+ {&env->features[FEAT_7_0_ECX],
|
|
|
9ae3a8 |
+ &host_def.features[FEAT_7_0_ECX],
|
|
|
9ae3a8 |
+ FEAT_7_0_ECX },
|
|
|
9ae3a8 |
{&env->features[FEAT_XSAVE],
|
|
|
9ae3a8 |
&host_def.features[FEAT_XSAVE],
|
|
|
9ae3a8 |
FEAT_XSAVE },
|
|
|
9ae3a8 |
@@ -1824,6 +1848,7 @@ static void cpu_x86_parse_featurestr(X86CPU *cpu, char *features, Error **errp)
|
|
|
9ae3a8 |
env->features[FEAT_KVM] |= plus_features[FEAT_KVM];
|
|
|
9ae3a8 |
env->features[FEAT_SVM] |= plus_features[FEAT_SVM];
|
|
|
9ae3a8 |
env->features[FEAT_7_0_EBX] |= plus_features[FEAT_7_0_EBX];
|
|
|
9ae3a8 |
+ env->features[FEAT_7_0_ECX] |= plus_features[FEAT_7_0_ECX];
|
|
|
9ae3a8 |
env->features[FEAT_XSAVE] |= plus_features[FEAT_XSAVE];
|
|
|
9ae3a8 |
env->features[FEAT_1_EDX] &= ~minus_features[FEAT_1_EDX];
|
|
|
9ae3a8 |
env->features[FEAT_1_ECX] &= ~minus_features[FEAT_1_ECX];
|
|
|
9ae3a8 |
@@ -1833,6 +1858,7 @@ static void cpu_x86_parse_featurestr(X86CPU *cpu, char *features, Error **errp)
|
|
|
9ae3a8 |
env->features[FEAT_KVM] &= ~minus_features[FEAT_KVM];
|
|
|
9ae3a8 |
env->features[FEAT_SVM] &= ~minus_features[FEAT_SVM];
|
|
|
9ae3a8 |
env->features[FEAT_7_0_EBX] &= ~minus_features[FEAT_7_0_EBX];
|
|
|
9ae3a8 |
+ env->features[FEAT_7_0_ECX] &= ~minus_features[FEAT_7_0_ECX];
|
|
|
9ae3a8 |
env->features[FEAT_XSAVE] &= ~minus_features[FEAT_XSAVE];
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
out:
|
|
|
9ae3a8 |
@@ -1969,6 +1995,7 @@ static void cpu_x86_register(X86CPU *cpu, const char *name, Error **errp)
|
|
|
9ae3a8 |
env->features[FEAT_SVM] = def->features[FEAT_SVM];
|
|
|
9ae3a8 |
env->features[FEAT_C000_0001_EDX] = def->features[FEAT_C000_0001_EDX];
|
|
|
9ae3a8 |
env->features[FEAT_7_0_EBX] = def->features[FEAT_7_0_EBX];
|
|
|
9ae3a8 |
+ env->features[FEAT_7_0_ECX] = def->features[FEAT_7_0_ECX];
|
|
|
9ae3a8 |
env->features[FEAT_XSAVE] = def->features[FEAT_XSAVE];
|
|
|
9ae3a8 |
env->cpuid_xlevel2 = def->xlevel2;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -2206,7 +2233,7 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count,
|
|
|
9ae3a8 |
if (count == 0) {
|
|
|
9ae3a8 |
*eax = 0; /* Maximum ECX value for sub-leaves */
|
|
|
9ae3a8 |
*ebx = env->features[FEAT_7_0_EBX]; /* Feature flags */
|
|
|
9ae3a8 |
- *ecx = 0; /* Reserved */
|
|
|
9ae3a8 |
+ *ecx = env->features[FEAT_7_0_ECX]; /* Feature flags */
|
|
|
9ae3a8 |
*edx = 0; /* Reserved */
|
|
|
9ae3a8 |
} else {
|
|
|
9ae3a8 |
*eax = 0;
|
|
|
9ae3a8 |
diff --git a/target-i386/cpu.h b/target-i386/cpu.h
|
|
|
9ae3a8 |
index da7e060..5c62ee3 100644
|
|
|
9ae3a8 |
--- a/target-i386/cpu.h
|
|
|
9ae3a8 |
+++ b/target-i386/cpu.h
|
|
|
9ae3a8 |
@@ -400,6 +400,7 @@ typedef enum FeatureWord {
|
|
|
9ae3a8 |
FEAT_1_EDX, /* CPUID[1].EDX */
|
|
|
9ae3a8 |
FEAT_1_ECX, /* CPUID[1].ECX */
|
|
|
9ae3a8 |
FEAT_7_0_EBX, /* CPUID[EAX=7,ECX=0].EBX */
|
|
|
9ae3a8 |
+ FEAT_7_0_ECX, /* CPUID[EAX=7,ECX=0].ECX */
|
|
|
9ae3a8 |
FEAT_8000_0001_EDX, /* CPUID[8000_0001].EDX */
|
|
|
9ae3a8 |
FEAT_8000_0001_ECX, /* CPUID[8000_0001].ECX */
|
|
|
9ae3a8 |
FEAT_C000_0001_EDX, /* CPUID[C000_0001].EDX */
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.8.3.1
|
|
|
9ae3a8 |
|