diff options
| author | Thomas Huth <thuth@redhat.com> | 2016-02-17 19:23:19 +0100 |
|---|---|---|
| committer | David Gibson <david@gibson.dropbear.id.au> | 2016-02-18 11:08:43 +1100 |
| commit | 8a9c1b77e9df5b4a9fcc1ffe08b4fcff7b0c791f (patch) | |
| tree | 6da243bccc358153c601ccfb3337297ac9452993 | |
| parent | 1c81003accd70e79f1663ad4429435b4d3300d62 (diff) | |
| download | qemu-8a9c1b77e9df5b4a9fcc1ffe08b4fcff7b0c791f.zip qemu-8a9c1b77e9df5b4a9fcc1ffe08b4fcff7b0c791f.tar.gz qemu-8a9c1b77e9df5b4a9fcc1ffe08b4fcff7b0c791f.tar.bz2 | |
hw/ppc/spapr: Halt CPU when powering off via RTAS call
The LoPAPR specification defines the following for the RTAS
power-off call: "On successful operation, does not return".
However, the implementation in QEMU currently returns and runs
the guest CPU again for some more cycles. This caused some
trouble with the new ppc implementation of the kvm-unit-tests
recently. So let's make sure that the QEMU implementation
follows the spec, thus stop the CPU to make sure that the
RTAS call does not return to the guest anymore.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Tested-by: Andrew Jones <drjones@redhat.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
| -rw-r--r-- | hw/ppc/spapr_rtas.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/ppc/spapr_rtas.c b/hw/ppc/spapr_rtas.c index 07ad672..b7c5ebd 100644 --- a/hw/ppc/spapr_rtas.c +++ b/hw/ppc/spapr_rtas.c @@ -113,6 +113,7 @@ static void rtas_power_off(PowerPCCPU *cpu, sPAPRMachineState *spapr, return; } qemu_system_shutdown_request(); + cpu_stop_current(); rtas_st(rets, 0, RTAS_OUT_SUCCESS); } |