[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL v3 00/81] pci,pc,virtio: features, tests, fixes, cleanups
From: |
Igor Mammedov |
Subject: |
Re: [PULL v3 00/81] pci,pc,virtio: features, tests, fixes, cleanups |
Date: |
Tue, 8 Nov 2022 14:32:48 +0100 |
On Mon, 7 Nov 2022 07:30:03 -0500
"Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Mon, Nov 07, 2022 at 05:43:44AM -0500, Stefan Hajnoczi wrote:
> > Hi Michael and Igor,
> > Looks like the ACPI commits broken the virtio-vga module:
> >
> > >>> QTEST_QEMU_IMG=./qemu-img MALLOC_PERTURB_=60
> > >>> G_TEST_DBUS_DAEMON=/builds/qemu-project/qemu/tests/dbus-vmstate-daemon.sh
> > >>> QTEST_QEMU_BINARY=./qemu-system-or1k
> > >>> QTEST_QEMU_STORAGE_DAEMON_BINARY=./storage-daemon/qemu-storage-daemon
> > >>> /builds/qemu-project/qemu/build/tests/qtest/device-introspect-test
> > >>> --tap -k
> > ――――――――――――――――――――――――――――――――――――― ✀
> > ―――――――――――――――――――――――――――――――――――――
> > stderr:
> > failed to open module:
> > /builds/qemu-project/qemu/build/qemu-bundle/usr/local/lib64/qemu/hw-display-virtio-vga.so:
> > undefined symbol: aml_return
> > qemu-system-or1k: ../util/error.c:59: error_setv: Assertion `*errp ==
> > NULL' failed.
> > Broken pipe
> > ../tests/qtest/libqtest.c:188: kill_qemu() detected QEMU death from
> > signal 6 (Aborted) (core dumped)
> > TAP parsing error: Too few tests run (expected 6, got 0)
> > (test program exited with status code -6)
> > ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――
> > 154/274 qemu:qtest+qtest-or1k / qtest-or1k/machine-none-test OK 0.05s
> > 1 subtests passed
> > 155/274 qemu:qtest+qtest-or1k / qtest-or1k/qmp-test OK 0.19s 4 subtests
> > passed
> > 156/274 qemu:qtest+qtest-or1k / qtest-or1k/qmp-cmd-test ERROR 1.72s
> > killed by signal 6 SIGABRT
> > >>> QTEST_QEMU_IMG=./qemu-img
> > >>> G_TEST_DBUS_DAEMON=/builds/qemu-project/qemu/tests/dbus-vmstate-daemon.sh
> > >>> QTEST_QEMU_BINARY=./qemu-system-or1k
> > >>> QTEST_QEMU_STORAGE_DAEMON_BINARY=./storage-daemon/qemu-storage-daemon
> > >>> MALLOC_PERTURB_=53
> > >>> /builds/qemu-project/qemu/build/tests/qtest/qmp-cmd-test --tap -k
> > ――――――――――――――――――――――――――――――――――――― ✀
> > ―――――――――――――――――――――――――――――――――――――
> > stderr:
> > failed to open module:
> > /builds/qemu-project/qemu/build/qemu-bundle/usr/local/lib64/qemu/hw-display-virtio-vga.so:
> > undefined symbol: aml_return
> > qemu-system-or1k: ../util/error.c:59: error_setv: Assertion `*errp ==
> > NULL' failed.
> > Broken pipe
> > ../tests/qtest/libqtest.c:188: kill_qemu() detected QEMU death from
> > signal 6 (Aborted) (core dumped)
> > TAP parsing error: Too few tests run (expected 62, got 31)
> > (test program exited with status code -6)
> > ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――
> >
> > https://gitlab.com/qemu-project/qemu/-/jobs/3281425457
> >
> > Stefan
>
>
> Hmm it passed for me:
>
> https://gitlab.com/mstredhat/qemu/-/jobs/3279401710
I'm sorry, I was 'out of service' yesterday.
I see that Ani already fixed the issue, thanks!
>
> Igor you did make a change around VGA:
>
> commit 03d525c27ab0b268cf375d8823f05e91509222b8
> Author: Igor Mammedov <imammedo@redhat.com>
> Date: Mon Oct 17 12:21:36 2022 +0200
>
> acpi: pc: vga: use AcpiDevAmlIf interface to build VGA device descriptors
>
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> Message-Id: <20221017102146.2254096-2-imammedo@redhat.com>
> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> NB: we do not expect any functional change in
> any ACPI tables with this change. It's only a refactoring.
>
> Reviewed-by: Ani Sinha <ani@anisinha.ca>
>
>
> can you take a look pls?
> How bad is it if I drop that patch?
It can't be dropped without dropping whole series
which heavily depends on it.
- [PULL v3 62/81] msix: Assert that specified vector is in range, (continued)
- [PULL v3 62/81] msix: Assert that specified vector is in range, Michael S. Tsirkin, 2022/11/05
- [PULL v3 67/81] hmat acpi: Don't require initiator value in -numa, Michael S. Tsirkin, 2022/11/05
- [PULL v3 69/81] tests: acpi: q35: add test for hmat nodes without initiators, Michael S. Tsirkin, 2022/11/05
- [PULL v3 71/81] tests: Add HMAT AArch64/virt empty table files, Michael S. Tsirkin, 2022/11/05
- [PULL v3 75/81] vfio: move implement of vfio_get_xlat_addr() to memory.c, Michael S. Tsirkin, 2022/11/05
- [PULL v3 78/81] intel-iommu: convert VTD_PE_GET_FPD_ERR() to be a function, Michael S. Tsirkin, 2022/11/05
- [PULL v3 81/81] vhost-user: Support vhost_dev_start, Michael S. Tsirkin, 2022/11/05
- [PULL v3 77/81] intel-iommu: drop VTDBus, Michael S. Tsirkin, 2022/11/05
- Re: [PULL v3 00/81] pci,pc,virtio: features, tests, fixes, cleanups, Stefan Hajnoczi, 2022/11/07
- Re: [PULL v3 00/81] pci,pc,virtio: features, tests, fixes, cleanups, Michael S. Tsirkin, 2022/11/07
- Re: [PULL v3 00/81] pci,pc,virtio: features, tests, fixes, cleanups,
Igor Mammedov <=