[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups
|
From: |
Igor Mammedov |
|
Subject: |
Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups |
|
Date: |
Wed, 16 Nov 2022 11:42:29 +0100 |
On Tue, 15 Nov 2022 15:01:06 +0100
Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
> Hi,
>
> On 7/11/22 23:47, Michael S. Tsirkin wrote:
>
> > ----------------------------------------------------------------
> > pci,pc,virtio: features, tests, fixes, cleanups
> >
> > lots of acpi rework
> > first version of biosbits infrastructure
> > ASID support in vhost-vdpa
> > core_count2 support in smbios
> > PCIe DOE emulation
> > virtio vq reset
> > HMAT support
> > part of infrastructure for viommu support in vhost-vdpa
> > VTD PASID support
> > fixes, tests all over the place
> Apparently unrelated to these fixes, but going from 6295a58ad1 to
> v7.2.0-rc0 triggered rebuilding ACPI files and I now get:
>
> 45/510 qemu:qtest+qtest-i386 / qtest-i386/bios-tables-test
> ERROR 14.73s killed by signal 6 SIGABRT
> 74/510 qemu:qtest+qtest-x86_64 / qtest-x86_64/bios-tables-test
> ERROR 12.56s killed by signal 6 SIGABRT
>
> Running manually:
>
> $ QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/bios-tables-test
> ...
> # starting QEMU: exec ./qemu-system-x86_64 -qtest
> unix:/tmp/qtest-239233.sock -qtest-log /dev/null -chardev
> socket,path=/tmp/qtest-239233.qmp,id=char0 -mon
> chardev=char0,mode=control -display none -machine pc -accel kvm -accel
> tcg -net none -machine smm=off -drive
> id=hd0,if=none,file=tests/acpi-test-disk-QmvOOR,format=raw -device
> ide-hd,drive=hd0 -accel qtest
> Could not access KVM kernel module: Permission denied
> qemu-system-x86_64: -accel kvm: failed to initialize kvm: Permission denied
> acpi-test: Warning! DSDT binary file mismatch. Actual
> [aml:/tmp/aml-Y06RV1], Expected [aml:tests/data/acpi/pc/DSDT.nosmm].
> See source file tests/qtest/bios-tables-test.c for instructions on how
> to update expected files.
> to see ASL diff between mismatched files install IASL, rebuild QEMU from
> scratch and re-run tests with V=1 environment variable set**
> ERROR:../../tests/qtest/bios-tables-test.c:533:test_acpi_asl: assertion
> failed: (all_tables_match)
> Bail out! ERROR:../../tests/qtest/bios-tables-test.c:533:test_acpi_asl:
> assertion failed: (all_tables_match)
> Aborted
>
> I blew/recreated my build directory and can reproduce.
it works for me with KVM and without it on RHEL 8.7 x86 host.
It looks like stale file (though you are saying that you've recreated
directory)
>
> $ uname -sm
> Linux x86_64
>
> Any clue?
just to be sure can you make fresh tree clone and build that?
if that won't work, make sure you have IASL installed and run test
manually with V=1 env var, that should print diff on failure
- [PULL v4 77/83] hw/smbios: add core_count2 to smbios table type 4, (continued)
- [PULL v4 77/83] hw/smbios: add core_count2 to smbios table type 4, Michael S. Tsirkin, 2022/11/07
- [PULL v4 78/83] bios-tables-test: teach test to use smbios 3.0 tables, Michael S. Tsirkin, 2022/11/07
- [PULL v4 79/83] tests/acpi: allow changes for core_count2 test, Michael S. Tsirkin, 2022/11/07
- [PULL v4 80/83] bios-tables-test: add test for number of cores > 255, Michael S. Tsirkin, 2022/11/07
- [PULL v4 81/83] tests/acpi: update tables for new core count test, Michael S. Tsirkin, 2022/11/07
- [PULL v4 82/83] hw/virtio: introduce virtio_device_should_start, Michael S. Tsirkin, 2022/11/07
- [PULL v4 83/83] checkpatch: better pattern for inline comments, Michael S. Tsirkin, 2022/11/07
- Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups, Michael S. Tsirkin, 2022/11/08
- Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups, Philippe Mathieu-Daudé, 2022/11/15
- Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups,
Igor Mammedov <=