[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu
From: |
Jin Dongming |
Subject: |
[Qemu-devel] [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version |
Date: |
Thu, 14 Oct 2010 17:55:28 +0900 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.1; ja; rv:1.9.2.7) Gecko/20100713 Thunderbird/3.1.1 |
There is no reason why SRAO event received by the main thread
is the only one that being broadcasted.
According to the x86 ASDM vol.3A 15.10.4.1,
MCE signal is broadcast on processor version 06H_EH or later.
This change is required to handle SRAR in the guest.
Signed-off-by: Hidetoshi Seto <address@hidden>
Tested-by: Jin Dongming <address@hidden>
---
qemu-kvm.c | 63 +++++++++++++++++++++++++++++------------------------------
1 files changed, 31 insertions(+), 32 deletions(-)
diff --git a/qemu-kvm.c b/qemu-kvm.c
index d2b2459..846f0b6 100644
--- a/qemu-kvm.c
+++ b/qemu-kvm.c
@@ -1149,6 +1149,34 @@ static int kvm_mce_in_progress(CPUState *env)
return !!(msr_mcg_status.data & MCG_STATUS_MCIP);
}
+static void kvm_mce_inj_broadcast(CPUState *env, struct kvm_x86_mce *mce)
+{
+ struct kvm_x86_mce mce_sub = {
+ .bank = 1,
+ .status = MCI_STATUS_VAL | MCI_STATUS_UC,
+ .mcg_status = MCG_STATUS_MCIP | MCG_STATUS_RIPV,
+ .addr = 0,
+ .misc = 0,
+ };
+ CPUState *cenv;
+ int family, model, cpuver = env->cpuid_version;
+
+ family = (cpuver >> 8) & 0xf;
+ model = ((cpuver >> 12) & 0xf0) + ((cpuver >> 4) & 0xf);
+
+ kvm_inject_x86_mce_on(env, mce, 1);
+
+ /* Broadcast MCA signal for processor version 06H_EH and above */
+ if ((family == 6 && model >= 14) || family > 6) {
+ for (cenv = first_cpu; cenv != NULL; cenv = cenv->next_cpu) {
+ if (cenv == env) {
+ continue;
+ }
+ kvm_inject_x86_mce_on(cenv, &mce_sub, 1);
+ }
+ }
+}
+
static void kvm_do_set_mce(CPUState *env, struct kvm_x86_mce *mce,
int abort_on_error)
{
@@ -1175,7 +1203,7 @@ static void kvm_mce_inj_srar_dataload(CPUState *env,
target_phys_addr_t paddr)
.misc = (MCM_ADDR_PHYS << 6) | 0xc,
};
- kvm_do_set_mce(env, &mce, 1);
+ kvm_mce_inj_broadcast(env, &mce);
}
static void kvm_mce_inj_srao_memscrub(CPUState *env, target_phys_addr_t paddr)
@@ -1190,32 +1218,7 @@ static void kvm_mce_inj_srao_memscrub(CPUState *env,
target_phys_addr_t paddr)
.misc = (MCM_ADDR_PHYS << 6) | 0xc,
};
- kvm_do_set_mce(env, &mce, 1);
-}
-
-static void kvm_mce_inj_srao_broadcast(target_phys_addr_t paddr)
-{
- struct kvm_x86_mce mce_srao_memscrub = {
- .bank = 9,
- .status = MCI_STATUS_VAL | MCI_STATUS_UC | MCI_STATUS_EN
- | MCI_STATUS_MISCV | MCI_STATUS_ADDRV | MCI_STATUS_S
- | 0xc0,
- .mcg_status = MCG_STATUS_MCIP | MCG_STATUS_RIPV,
- .addr = paddr,
- .misc = (MCM_ADDR_PHYS << 6) | 0xc,
- };
- struct kvm_x86_mce mce_dummy = {
- .bank = 1,
- .status = MCI_STATUS_VAL | MCI_STATUS_UC,
- .mcg_status = MCG_STATUS_MCIP | MCG_STATUS_RIPV,
- .addr = 0,
- .misc = 0,
- };
- CPUState *cenv;
-
- kvm_inject_x86_mce_on(first_cpu, &mce_srao_memscrub, 1);
- for (cenv = first_cpu->next_cpu; cenv != NULL; cenv = cenv->next_cpu)
- kvm_inject_x86_mce_on(cenv, &mce_dummy, 1);
+ kvm_mce_inj_broadcast(env, &mce);
}
#endif
@@ -1255,11 +1258,7 @@ static void kvm_handle_sigbus(CPUState *env, int code,
void *vaddr)
kvm_mce_inj_srar_dataload(target_env, paddr);
} else {
/* Fake an Intel architectural Memory scrubbing UCR */
- if (env) {
- kvm_mce_inj_srao_memscrub(target_env, paddr);
- } else {
- kvm_mce_inj_srao_broadcast(paddr);
- }
+ kvm_mce_inj_srao_memscrub(target_env, paddr);
}
return;
}
--
1.7.1.1
- [Qemu-devel] [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version,
Jin Dongming <=
- [Qemu-devel] Re: [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version, Marcelo Tosatti, 2010/10/14
- Re: [Qemu-devel] Re: [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version, Hidetoshi Seto, 2010/10/14
- Re: [Qemu-devel] Re: [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version, Huang Ying, 2010/10/15
- Re: [Qemu-devel] Re: [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version, Marcelo Tosatti, 2010/10/15
- Re: [Qemu-devel] Re: [PATCH 11/11] kvm, x86: broadcast mce depending on the cpu version, Hidetoshi Seto, 2010/10/18
- [Qemu-devel] [PATCH uq/master 1/2] kvm, x86: ignore SRAO only when MCG_SER_P is available, Hidetoshi Seto, 2010/10/18
- [Qemu-devel] [PATCH uq/master 2/2] kvm, x86: broadcast mce depending on the cpu version, Hidetoshi Seto, 2010/10/18