434531619f
Hardware clears the MSR[ME] bit when delivering a machine check
interrupt, so that is what QEMU does.
The spapr environment runs in supervisor mode though, and receives
machine check interrupts after they are processed by the hypervisor,
and MSR[ME] must always be enabled in supervisor mode (otherwise it
could checkstop the system). So MSR[ME] must not be cleared when
delivering machine checks to the supervisor.
The fix to prevent supervisor mode from modifying MSR[ME] also
prevented it from re-enabling the incorrectly cleared MSR[ME] bit
when returning from handling the interrupt. Before that fix, the
problem was not very noticable with well-behaved code. So the
Fixes tag is not strictly correct, but practically they go together.
Found by kvm-unit-tests machine check tests (not yet upstream).
Fixes:
|
||
|---|---|---|
| .. | ||
| translate | ||
| arch_dump.c | ||
| compat.c | ||
| cpu_init.c | ||
| cpu-models.c | ||
| cpu-models.h | ||
| cpu-param.h | ||
| cpu-qom.h | ||
| cpu.c | ||
| cpu.h | ||
| dfp_helper.c | ||
| excp_helper.c | ||
| fpu_helper.c | ||
| gdbstub.c | ||
| helper_regs.c | ||
| helper_regs.h | ||
| helper.h | ||
| insn32.decode | ||
| insn64.decode | ||
| int_helper.c | ||
| internal.h | ||
| Kconfig | ||
| kvm_ppc.h | ||
| kvm.c | ||
| machine.c | ||
| mem_helper.c | ||
| meson.build | ||
| misc_helper.c | ||
| mmu_common.c | ||
| mmu_helper.c | ||
| mmu-book3s-v3.c | ||
| mmu-book3s-v3.h | ||
| mmu-books.h | ||
| mmu-hash32.c | ||
| mmu-hash32.h | ||
| mmu-hash64.c | ||
| mmu-hash64.h | ||
| mmu-radix64.c | ||
| mmu-radix64.h | ||
| power8-pmu-regs.c.inc | ||
| power8-pmu.c | ||
| power8-pmu.h | ||
| ppc-qmp-cmds.c | ||
| spr_common.h | ||
| tcg-stub.c | ||
| timebase_helper.c | ||
| trace-events | ||
| trace.h | ||
| translate.c | ||
| user_only_helper.c | ||