diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2024-09-02 01:59:42 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2024-10-02 12:58:46 +0200 |
commit | 89d26d34ce8bb95552570e729d3a7e8cb17f7369 (patch) | |
tree | 37809986e7c11b2c31b5fad8ed8f19dff2fb8dd6 /tests | |
parent | ab891454ebe82f7e359be721007652556f9f8356 (diff) | |
download | qemu-89d26d34ce8bb95552570e729d3a7e8cb17f7369.zip qemu-89d26d34ce8bb95552570e729d3a7e8cb17f7369.tar.gz qemu-89d26d34ce8bb95552570e729d3a7e8cb17f7369.tar.bz2 |
tests/unit: remove block layer code from test-nested-aio-poll
GCC is reporting a NULL pointer dereference when compiling aio_wait_kick()
with LTO.
The issue is that test-nested-aio-poll.c does not call qemu_init_main_loop().
It doesn't _need_ to because it never calls AIO_WAIT_WHILE(), but it seems
that LTO does not do enough dead-code elimination to catch that.
Fortunately aio_wait_kick() is only used in few places, and only in block
layer or system emulation code; and this test only needs the core event loop
functionality. It does not even need iothreads. So remove everything that
calls aio_wait_kick(), which is nice for coverage compared to adding the call
to qemu_init_main_loop().
Reported-by: Richard W.M. Jones <rjones@redhat.com>
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2434
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/unit/meson.build | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/unit/meson.build b/tests/unit/meson.build index 21d1013..d5248ae 100644 --- a/tests/unit/meson.build +++ b/tests/unit/meson.build @@ -116,7 +116,7 @@ if have_block if host_os != 'windows' tests += { 'test-image-locking': [testblock], - 'test-nested-aio-poll': [testblock], + 'test-nested-aio-poll': [], } endif if config_host_data.get('CONFIG_REPLICATION') |