[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH for-2.12 v3 0/3] disable the decrementer interrupt w
From: |
Cédric Le Goater |
Subject: |
[Qemu-devel] [PATCH for-2.12 v3 0/3] disable the decrementer interrupt when a CPU is unplugged |
Date: |
Mon, 20 Nov 2017 11:03:44 +0100 |
Hello,
When a CPU is stopped with the 'stop-self' RTAS call, its state
'halted' is switched to 1 and, in this case, the MSR is not taken into
account anymore in the cpu_has_work() routine. Only the pending
hardware interrupts are checked with their LPCR:PECE* enablement bit.
If the DECR timer fires after 'stop-self' is called and before the CPU
'stop' state is reached, the nearly-dead CPU will have some work to do
and the guest will crash. This case happens very frequently with the
not yet upstream P9 XIVE exploitation mode. In XICS mode, the DECR is
occasionally fired but after 'stop' state, so no work is to be done
and the guest survives.
I suspect there is a race between the QEMU mainloop triggering the
timers and the TCG CPU thread but I could not quite identify the root
cause. To be safe, let's disable the decrementer interrupt in the LPCR
when the CPU is halted and reenable it when the CPU is restarted.
Reseting the MSR is now pointless, so remove this dubious workaround.
Thanks,
C.
Changes in v3:
- removed the ppc_cpu_pvr_match() routine testing the CPU family.
- introduced a cpu_ppc_papr_pece_bits() helper to gather the PECE
bits depending on the CPU family.
- enabled Power-saving mode Exit Cause exceptions only on the boot CPU.
Changes in v2:
- used a new routine ppc_cpu_pvr_match() to discriminate CPU versions
- removed the LPCR:PECE* enablement bit when the CPU is initialized
if it is a secondary
- included Nikunj's fix to reboot SMP TCG guests
Cédric Le Goater (3):
spapr/rtas: disable the decrementer interrupt when a CPU is unplugged
spapr/rtas: fix reboot of a a SMP TCG guest
spapr/rtas: do not reset the MSR in stop-self command
hw/ppc/spapr_cpu_core.c | 7 +++++++
hw/ppc/spapr_rtas.c | 19 +++++++++----------
target/ppc/cpu.h | 1 +
target/ppc/translate_init.c | 33 +++++++++++++++++++++++++--------
4 files changed, 42 insertions(+), 18 deletions(-)
--
2.13.6
- [Qemu-devel] [PATCH for-2.12 v3 0/3] disable the decrementer interrupt when a CPU is unplugged,
Cédric Le Goater <=