[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 16/32] tests/qtest: remove clock_steps from virtio tests
From: |
Alex Bennée |
Subject: |
[PATCH v4 16/32] tests/qtest: remove clock_steps from virtio tests |
Date: |
Wed, 8 Jan 2025 12:10:38 +0000 |
In the qtest environment time will not step forward if the system is
paused (timers disabled) or we have no timer events to fire. As a
result VirtIO events are responded to directly and we don't need to
step time forward.
We still do timeout processing to handle the fact the target QEMU may
not be ready to respond right away. This will usually be due to a slow
CI system or if QEMU is running under something like rr.
Future qtest patches will assert that time actually changes when a
step is requested.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
---
v2
- s/with/when/
- drop clock_step entirely
---
tests/qtest/libqos/virtio.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/tests/qtest/libqos/virtio.c b/tests/qtest/libqos/virtio.c
index a21b6eee9c..2e7979652f 100644
--- a/tests/qtest/libqos/virtio.c
+++ b/tests/qtest/libqos/virtio.c
@@ -170,7 +170,6 @@ void qvirtio_wait_queue_isr(QTestState *qts, QVirtioDevice
*d,
gint64 start_time = g_get_monotonic_time();
for (;;) {
- qtest_clock_step(qts, 100);
if (d->bus->get_queue_isr_status(d, vq)) {
return;
}
@@ -192,7 +191,6 @@ uint8_t qvirtio_wait_status_byte_no_isr(QTestState *qts,
QVirtioDevice *d,
uint8_t val;
while ((val = qtest_readb(qts, addr)) == 0xff) {
- qtest_clock_step(qts, 100);
g_assert(!d->bus->get_queue_isr_status(d, vq));
g_assert(g_get_monotonic_time() - start_time <= timeout_us);
}
@@ -219,14 +217,12 @@ void qvirtio_wait_used_elem(QTestState *qts,
QVirtioDevice *d,
for (;;) {
uint32_t got_desc_idx;
- qtest_clock_step(qts, 100);
if (d->bus->get_queue_isr_status(d, vq) &&
qvirtqueue_get_buf(qts, vq, &got_desc_idx, len)) {
g_assert_cmpint(got_desc_idx, ==, desc_idx);
return;
}
-
g_assert(g_get_monotonic_time() - start_time <= timeout_us);
}
}
--
2.39.5
- [PATCH v4 02/32] tests/functional: update the i386 tuxrun tests, (continued)
- [PATCH v4 02/32] tests/functional: update the i386 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 30/32] pc-bios: ensure keymaps dependencies set vnc tests, Alex Bennée, 2025/01/08
- [PATCH v4 11/32] tests/functional: update the riscv64 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 22/32] tests/functional: bail aarch64_virt tests early if missing TCG, Alex Bennée, 2025/01/08
- [PATCH v4 14/32] tests/functional: update the x86_64 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 15/32] tests/functional/aarch64: add tests for FEAT_RME, Alex Bennée, 2025/01/08
- [PATCH v4 10/32] tests/functional: update the riscv32 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 05/32] tests/functional: update the mips32el tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 16/32] tests/qtest: remove clock_steps from virtio tests,
Alex Bennée <=
- [PATCH v4 21/32] tests/functional: remove unused kernel_command_line, Alex Bennée, 2025/01/08
- [PATCH v4 09/32] tests/functional: update the ppc64 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 23/32] tests/functional: extend test_aarch64_virt with vulkan test, Alex Bennée, 2025/01/08
- [PATCH v4 03/32] tests/functional: add a m68k tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 12/32] tests/functional: update the s390x tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 06/32] tests/functional: update the mips64 tuxrun tests, Alex Bennée, 2025/01/08
- [PATCH v4 13/32] tests/functional: update the sparc64 tuxrun tests, Alex Bennée, 2025/01/08