[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 00/54] tests/qtest: Enable running qtest on Windows
From: |
Thomas Huth |
Subject: |
Re: [PATCH v4 00/54] tests/qtest: Enable running qtest on Windows |
Date: |
Wed, 28 Sep 2022 12:31:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0 |
On 27/09/2022 13.05, Bin Meng wrote:
In preparation to adding virtio-9p support on Windows, this series
enables running qtest on Windows, so that we can run the virtio-9p
tests on Windows to make sure it does not break accidently.
Thanks for your patches - I've picked many of them for my pull request that
I sent out earlier today, so you don't have to carry them along anymore once
the PR got merged.
For the patches that are not directly related to tests/ ... could you maybe
ask the corresponding maintainers to pick those up? I'm not sure whether
they should go through my testing branch, too...
Anyway, there seems to be one more issue: The migration test sometimes seems
to be failing on aarch64 with all your patches applied:
87/470 qemu:qtest+qtest-aarch64 / qtest-aarch64/migration-test
ERROR 261.71s killed by signal 6 SIGABRT
>>> MALLOC_PERTURB_=171 QTEST_QEMU_BINARY=./qemu-system-aarch64
QTEST_QEMU_IMG=./qemu-img
G_TEST_DBUS_DAEMON=/home/thuth/devel/qemu/tests/dbus-vmstate-daemon.sh
QTEST_QEMU_STORAGE_DAEMON_BINARY=./storage-daemon/qemu-storage-daemon
/home/thuth/tmp/qemu-build/tests/qtest/migration-test --tap -k
―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――
stderr:
**
ERROR:../../devel/qemu/tests/qtest/migration-helpers.c:205:wait_for_migration_status:
assertion failed: (g_test_timer_elapsed() < MIGRATION_STATUS_WAIT_TIMEOUT)
../../devel/qemu/tests/qtest/libqtest.c:201: kill_qemu() tried to terminate
QEMU process but encountered exit status 1 (expected 0)
(test program exited with status code -6)
―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――
Not sure whether it's really related to your patches or whether it's
something else that has been merged recently, I'm having problems to
reproduce it reliably, but it's definitely something we should keep an eye on...
Thomas
- [PATCH v4 00/54] tests/qtest: Enable running qtest on Windows, Bin Meng, 2022/09/27
- [PATCH v4 05/54] tests/qtest: ahci-test: Avoid using hardcoded /tmp, Bin Meng, 2022/09/27
- [PATCH v4 09/54] tests/qtest: fdc-test: Avoid using hardcoded /tmp, Bin Meng, 2022/09/27
- [PATCH v4 13/54] tests/qtest: ide-test: Avoid using hardcoded /tmp, Bin Meng, 2022/09/27
- [PATCH v4 25/54] block/vvfat: Unify the mkdir() call, Bin Meng, 2022/09/27
- [PATCH v4 19/54] tests/qtest: virtio-blk-test: Avoid using hardcoded /tmp, Bin Meng, 2022/09/27
- [PATCH v4 37/54] tests/qtest: {ahci, ide}-test: Use relative path for temporary files for win32, Bin Meng, 2022/09/27
- [PATCH v4 40/54] tests/qtest: ide-test: Open file in binary mode, Bin Meng, 2022/09/27
- Re: [PATCH v4 00/54] tests/qtest: Enable running qtest on Windows,
Thomas Huth <=