|
 |
9ae3a8 |
From 9875f9aaef996083064bf6a7afa1bfa30ff7d953 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
Date: Wed, 13 Dec 2017 15:43:40 -0200
|
|
|
9ae3a8 |
Subject: [PATCH 2/3] target-i386: add support for SPEC_CTRL MSR
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
9ae3a8 |
Message-id: <20171213174341.20684-3-ehabkost@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: n/a
|
|
|
9ae3a8 |
O-Subject: [CONFIDENTIAL][RHEL-7.5 qemu-kvm PATCH v2 2/3] target-i386: add
|
|
|
9ae3a8 |
support for SPEC_CTRL MSR
|
|
|
9ae3a8 |
Bugzilla: CVE-2017-5715
|
|
|
9ae3a8 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
target-i386/cpu.h | 4 ++++
|
|
|
9ae3a8 |
target-i386/kvm.c | 14 ++++++++++++++
|
|
|
9ae3a8 |
target-i386/machine.c | 21 +++++++++++++++++++++
|
|
|
9ae3a8 |
3 files changed, 39 insertions(+)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/target-i386/cpu.h b/target-i386/cpu.h
|
|
|
9ae3a8 |
index 1e5c980..7d815cd 100644
|
|
|
9ae3a8 |
--- a/target-i386/cpu.h
|
|
|
9ae3a8 |
+++ b/target-i386/cpu.h
|
|
|
9ae3a8 |
@@ -304,6 +304,7 @@
|
|
|
9ae3a8 |
#define MSR_IA32_APICBASE_ENABLE (1<<11)
|
|
|
9ae3a8 |
#define MSR_IA32_APICBASE_BASE (0xfffff<<12)
|
|
|
9ae3a8 |
#define MSR_TSC_ADJUST 0x0000003b
|
|
|
9ae3a8 |
+#define MSR_IA32_SPEC_CTRL 0x48
|
|
|
9ae3a8 |
#define MSR_IA32_TSCDEADLINE 0x6e0
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
#define MSR_P6_PERFCTR0 0xc1
|
|
|
9ae3a8 |
@@ -963,6 +964,7 @@ typedef struct CPUX86State {
|
|
|
9ae3a8 |
uint64_t msr_fixed_counters[MAX_FIXED_COUNTERS];
|
|
|
9ae3a8 |
uint64_t msr_gp_counters[MAX_GP_COUNTERS];
|
|
|
9ae3a8 |
uint64_t msr_gp_evtsel[MAX_GP_COUNTERS];
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
uint64_t msr_hv_hypercall;
|
|
|
9ae3a8 |
uint64_t msr_hv_guest_os_id;
|
|
|
9ae3a8 |
uint64_t msr_hv_vapic;
|
|
|
9ae3a8 |
@@ -1037,6 +1039,8 @@ typedef struct CPUX86State {
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
uint32_t pkru;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
+ uint64_t spec_ctrl;
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
TPRAccess tpr_access_type;
|
|
|
9ae3a8 |
} CPUX86State;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/target-i386/kvm.c b/target-i386/kvm.c
|
|
|
9ae3a8 |
index 0dc0e79..24d17ad 100644
|
|
|
9ae3a8 |
--- a/target-i386/kvm.c
|
|
|
9ae3a8 |
+++ b/target-i386/kvm.c
|
|
|
9ae3a8 |
@@ -77,6 +77,7 @@ static bool has_msr_hv_vapic;
|
|
|
9ae3a8 |
static bool has_msr_hv_tsc;
|
|
|
9ae3a8 |
static bool has_msr_mtrr;
|
|
|
9ae3a8 |
static bool has_msr_xss;
|
|
|
9ae3a8 |
+static bool has_msr_spec_ctrl;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
static bool has_msr_architectural_pmu;
|
|
|
9ae3a8 |
static uint32_t num_architectural_pmu_counters;
|
|
|
9ae3a8 |
@@ -800,6 +801,10 @@ static int kvm_get_supported_msrs(KVMState *s)
|
|
|
9ae3a8 |
has_msr_xss = true;
|
|
|
9ae3a8 |
continue;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
+ if (kvm_msr_list->indices[i] == MSR_IA32_SPEC_CTRL) {
|
|
|
9ae3a8 |
+ has_msr_spec_ctrl = true;
|
|
|
9ae3a8 |
+ continue;
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -1187,6 +1192,9 @@ static int kvm_put_msrs(X86CPU *cpu, int level)
|
|
|
9ae3a8 |
if (has_msr_xss) {
|
|
|
9ae3a8 |
kvm_msr_entry_set(&msrs[n++], MSR_IA32_XSS, env->xss);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
+ if (has_msr_spec_ctrl) {
|
|
|
9ae3a8 |
+ kvm_msr_entry_set(&msrs[n++], MSR_IA32_SPEC_CTRL, env->spec_ctrl);
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
#ifdef TARGET_X86_64
|
|
|
9ae3a8 |
if (lm_capable_kernel) {
|
|
|
9ae3a8 |
kvm_msr_entry_set(&msrs[n++], MSR_CSTAR, env->cstar);
|
|
|
9ae3a8 |
@@ -1544,6 +1552,9 @@ static int kvm_get_msrs(X86CPU *cpu)
|
|
|
9ae3a8 |
if (has_msr_xss) {
|
|
|
9ae3a8 |
msrs[n++].index = MSR_IA32_XSS;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
+ if (has_msr_spec_ctrl) {
|
|
|
9ae3a8 |
+ msrs[n++].index = MSR_IA32_SPEC_CTRL;
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
if (!env->tsc_valid) {
|
|
|
9ae3a8 |
@@ -1786,6 +1797,9 @@ static int kvm_get_msrs(X86CPU *cpu)
|
|
|
9ae3a8 |
env->mtrr_var[MSR_MTRRphysIndex(index)].base = msrs[i].data;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
break;
|
|
|
9ae3a8 |
+ case MSR_IA32_SPEC_CTRL:
|
|
|
9ae3a8 |
+ env->spec_ctrl = msrs[i].data;
|
|
|
9ae3a8 |
+ break;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/target-i386/machine.c b/target-i386/machine.c
|
|
|
9ae3a8 |
index ba34088..d883c86 100644
|
|
|
9ae3a8 |
--- a/target-i386/machine.c
|
|
|
9ae3a8 |
+++ b/target-i386/machine.c
|
|
|
9ae3a8 |
@@ -742,6 +742,24 @@ static const VMStateDescription vmstate_pkru = {
|
|
|
9ae3a8 |
};
|
|
|
9ae3a8 |
#endif
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
+static bool spec_ctrl_needed(void *opaque)
|
|
|
9ae3a8 |
+{
|
|
|
9ae3a8 |
+ X86CPU *cpu = opaque;
|
|
|
9ae3a8 |
+ CPUX86State *env = &cpu->env;
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ return env->spec_ctrl != 0;
|
|
|
9ae3a8 |
+}
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+static const VMStateDescription vmstate_spec_ctrl = {
|
|
|
9ae3a8 |
+ .name = "cpu/spec_ctrl",
|
|
|
9ae3a8 |
+ .version_id = 1,
|
|
|
9ae3a8 |
+ .minimum_version_id = 1,
|
|
|
9ae3a8 |
+ .fields = (VMStateField[]){
|
|
|
9ae3a8 |
+ VMSTATE_UINT64(env.spec_ctrl, X86CPU),
|
|
|
9ae3a8 |
+ VMSTATE_END_OF_LIST()
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
+};
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
const VMStateDescription vmstate_x86_cpu = {
|
|
|
9ae3a8 |
.name = "cpu",
|
|
|
9ae3a8 |
.version_id = 12,
|
|
|
9ae3a8 |
@@ -896,6 +914,9 @@ const VMStateDescription vmstate_x86_cpu = {
|
|
|
9ae3a8 |
.vmsd = &vmstate_pkru,
|
|
|
9ae3a8 |
.needed = pkru_needed,
|
|
|
9ae3a8 |
#endif
|
|
|
9ae3a8 |
+ }, {
|
|
|
9ae3a8 |
+ .vmsd = &vmstate_spec_ctrl,
|
|
|
9ae3a8 |
+ .needed = spec_ctrl_needed,
|
|
|
9ae3a8 |
} , {
|
|
|
9ae3a8 |
/* empty */
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.8.3.1
|
|
|
9ae3a8 |
|