[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] scripts/device-crash-test: vmcoreinfo in error_
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH] scripts/device-crash-test: vmcoreinfo in error_whitelist for ppc64 |
Date: |
Fri, 3 Nov 2017 12:49:43 +0100 |
Hi Daniel
On Fri, Oct 27, 2017 at 8:17 PM, Daniel Henrique Barboza
<address@hidden> wrote:
> The device vmcoreinfo depends on the fw_cfg device to work, a device
> that isn't available in most ppc64 machines. This makes device-crash-test
> spams the following error when running with qemu-system-ppc64:
>
> "INFO: log: qemu-system-ppc64: -device vmcoreinfo: vmcoreinfo device
> requires fw_cfg with DMA"
>
> To fix it, this patch adds the following ppc64 machines into the
> error_whitelist for the vmcoreinfo device:
>
> - pseries.*
> - 40p
> - bamboo
> - g3beige
> - mac99
> - mpc8544ds
> - powernv
> - ppce500
> - prep
> - virtex-ml507
>
> After this patch, the only ppc64 machine that reports error on the
> vmcoreinfo device is the 'none' type.
>
> Signed-off-by: Daniel Henrique Barboza <address@hidden>
> ---
> scripts/device-crash-test | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/scripts/device-crash-test b/scripts/device-crash-test
> index 043b24a4aa..fb8a43d679 100755
> --- a/scripts/device-crash-test
> +++ b/scripts/device-crash-test
> @@ -163,6 +163,18 @@ ERROR_WHITELIST = [
> # "spapr-cpu-core needs a pseries machine"
> {'machine':'(?!pseries).*', 'device':'.*-spapr-cpu-core',
> 'expected':True},
>
> + # vmcoreinfo requires a fw_cfg device and will fail in these ppc64
> machines
> + {'machine':'pseries.*', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'40p', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'bamboo', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'g3beige', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'mac99', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'mpc8544ds', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'powernv', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'ppce500', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'prep', 'device':'vmcoreinfo', 'expected':True},
> + {'machine':'virtex-ml507', 'device':'vmcoreinfo', 'expected':True},
You could try to make it fit on one line with
r"(pseries.*|40p|bamboo|...)". Declaring a ppc64_machines before would
probably help too.
Any idea for an alternative vmcoreinfo device approach on ppc64?
> +
> # KVM-specific devices shouldn't be tried without accel=kvm:
> {'accel':'(?!kvm).*', 'device':'kvmclock', 'expected':True},
> {'accel':'(?!kvm).*', 'device':'kvm-pci-assign', 'expected':True},
> --
> 2.13.6
>
>
--
Marc-André Lureau
- Re: [Qemu-devel] [PATCH] scripts/device-crash-test: vmcoreinfo in error_whitelist for ppc64,
Marc-André Lureau <=