[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PULL 00/31] Trivial patches for 2017-09-25
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-trivial] [PULL 00/31] Trivial patches for 2017-09-25 |
Date: |
Tue, 26 Sep 2017 08:57:00 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
26.09.2017 02:22, Peter Maydell wrote:
[]
>> trivial patches for 2017-09-25
>
> This fails 'make check' on most of my configs:
>
> GTESTER check-qtest-ppc64
> qemu-system-ppc64: -object
> filter-mirror,id=qtest-f0,netdev=qtest-bn0,queue=tx,outdev=mirror0:
> Device 'qtest-bn0' not found
> Broken pipe
> GTester: last random seed: R02Sb816ff80b7d08ef6a5328ff373d8cd65
>
> GLib-CRITICAL **: void g_hook_destroy_link(GHookList *, GHook *):
> assertion `hook != NULL' failed
> aborting...
For what it's worth, this time I just _forgot_ to run tests. What a shame..
It fails due to:
Author: Eduardo Otubo <address@hidden>
Date: Mon Aug 21 17:50:05 2017 +0200
filter-mirror: segfault when specifying non existent device
When using filter-mirror like the example below where the interface
'ndev0' does not exist on the host, QEMU crashes into segmentation
fault.
$ qemu-system-x86_64 -S -machine pc -netdev user,id=ndev0 -object
filter-mirror,id=test-object,netdev=ndev0
This happens because the function filter_mirror_setup() does not checks
if the device actually exists and still keep on processing calling
qemu_chr_find(). This patch fixes this issue.
Signed-off-by: Eduardo Otubo <address@hidden>
Reviewed-by: Zhang Chen <address@hidden>
Reverting this patch makes it run again. Cc'ing Eduardo.
I'll remove this patch, patch "hw/isa/pc87312: Mark the device with
user_creatable = false" (will be replaced with a better variant),
and update another patch stripping one more trailing whitespace,
and resend.
Thanks,
/mjt
- [Qemu-trivial] [PULL 23/31] trivial: Add missing "-m" parameter in docs/memory-hotplug.txt, (continued)
- [Qemu-trivial] [PULL 23/31] trivial: Add missing "-m" parameter in docs/memory-hotplug.txt, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 30/31] Drop gld linker usage on SunOS, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 20/31] hw/display/xenfb.c: Add trace_xenfb_key_event, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 22/31] chardev/baum: fix baum that releases brlapi twice, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 29/31] tests/boot-sector: Increase timeout to 600 seconds, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 19/31] aux-to-i2c-bridge: don't allow user to create one, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 31/31] hw/isa/pc87312: Mark the device with user_creatable = false, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 09/31] MAINTAINERS: add missing PCI entries, Michael Tokarev, 2017/09/24
- [Qemu-trivial] [PULL 27/31] nbd-client: Use correct macro parenthesization, Michael Tokarev, 2017/09/24
- Re: [Qemu-trivial] [Qemu-devel] [PULL 00/31] Trivial patches for 2017-09-25, Peter Maydell, 2017/09/25
- Re: [Qemu-trivial] [PULL 00/31] Trivial patches for 2017-09-25,
Michael Tokarev <=
- Message not available