[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 04/62] Revert "spapr: support memory unplug for q
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PULL 04/62] Revert "spapr: support memory unplug for qtest" |
Date: |
Tue, 12 Mar 2019 10:09:38 +0100 |
On Tue, 12 Mar 2019 19:54:04 +1100
David Gibson <address@hidden> wrote:
> From: Greg Kurz <address@hidden>
>
> Commit b8165118f52c broke CPU hotplug tests for old machine types:
>
> $ QTEST_QEMU_BINARY=ppc64-softmmu/qemu-system-ppc64 ./tests/cpu-plug-test
> -m=slow
> /ppc64/cpu-plug/pseries-3.1/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.12-sxxm/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-3.0/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.10/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.11/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.12/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.9/device-add/2x3x1&maxcpus=6: OK
> /ppc64/cpu-plug/pseries-2.7/device-add/2x3x1&maxcpus=6: **
> ERROR:/home/thuth/devel/qemu/hw/ppc/spapr_events.c:313:rtas_event_log_to_source:
> assertion failed: (source->enabled)
> Broken pipe
> /home/thuth/devel/qemu/tests/libqtest.c:143: kill_qemu() detected QEMU death
> from signal 6 (Aborted) (core dumped)
> Aborted (core dumped)
>
> The approach of faking the availability of OV5_HP_EVT causes the
> code to assume the hotplug event source is enabled, which is wrong
> for older machines.
>
> We've now fixed CAS under qtest with a different approach. Therefore,
> this reverts commit b8165118f52ce5ee88565d3cec83d30374efdc96.
>
Cool.
> A subsequent patch will address the problem of CAS under qtest from
> a different angle.
>
This sentence should have been removed... no big deal I guess.
> Reported-by: Thomas Huth <address@hidden>
> Signed-off-by: Greg Kurz <address@hidden>
> Message-Id: <address@hidden>
> Tested-by: Michael Roth <address@hidden>
> Reviewed-by: Michael Roth <address@hidden>
> Signed-off-by: David Gibson <address@hidden>
> ---
> hw/ppc/spapr_ovec.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/hw/ppc/spapr_ovec.c b/hw/ppc/spapr_ovec.c
> index 12510b236a..318bf33de4 100644
> --- a/hw/ppc/spapr_ovec.c
> +++ b/hw/ppc/spapr_ovec.c
> @@ -16,7 +16,6 @@
> #include "qemu/bitmap.h"
> #include "exec/address-spaces.h"
> #include "qemu/error-report.h"
> -#include "sysemu/qtest.h"
> #include "trace.h"
> #include <libfdt.h>
>
> @@ -132,11 +131,6 @@ bool spapr_ovec_test(sPAPROptionVector *ov, long bitnr)
> g_assert(ov);
> g_assert(bitnr < OV_MAXBITS);
>
> - /* support memory unplug for qtest */
> - if (qtest_enabled() && bitnr == OV5_HP_EVT) {
> - return true;
> - }
> -
> return test_bit(bitnr, ov->bitmap) ? true : false;
> }
>
- [Qemu-devel] [PULL 00/62] ppc-for-4.0 queue 20190312, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 04/62] Revert "spapr: support memory unplug for qtest", David Gibson, 2019/03/12
- Re: [Qemu-devel] [PULL 04/62] Revert "spapr: support memory unplug for qtest",
Greg Kurz <=
- [Qemu-devel] [PULL 01/62] vfio/spapr: Fix indirect levels calculation, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 03/62] spapr: Simulate CAS for qtest, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 02/62] vfio/spapr: Rename local systempagesize variable, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 05/62] target/ppc/spapr: Add SPAPR_CAP_LARGE_DECREMENTER, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 07/62] target/ppc: Implement large decrementer support for KVM, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 14/62] target/ppc: Move handling of hardware breakpoints to a separate function, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 09/62] target/ppc/spapr: Add workaround option to SPAPR_CAP_IBS, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 10/62] target/ppc/spapr: Add SPAPR_CAP_CCF_ASSIST, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 17/62] target/ppc/spapr: Clear partition table entry when allocating hash table, David Gibson, 2019/03/12
- [Qemu-devel] [PULL 08/62] target/ppc/spapr: Enable the large decrementer for pseries-4.0, David Gibson, 2019/03/12