[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6] tests/qtest: netdev: test stream and dgram backends
From: |
Laurent Vivier |
Subject: |
Re: [PATCH v6] tests/qtest: netdev: test stream and dgram backends |
Date: |
Tue, 17 Jan 2023 18:24:45 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 |
On 1/17/23 17:45, Thomas Huth wrote:
On 17/01/2023 14.15, Laurent Vivier wrote:
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Thomas Huth <thuth@redhat.com>
---
Notes:
v6:
- call socket_init() otherwise socket_check_protocol_support() fails
- if socket_check_protocol_support() fails then calls g_abort() to
report a problem.
Last problem fixed, next one occurring:
Thank you Thomas
218/556 qemu:qtest+qtest-aarch64 / qtest-aarch64/netdev-socket
ERROR 0.77s exit status 3
------------------------------------- 8< -------------------------------------
stderr:
qemu-system-aarch64: -netdev
dgram,id=st0,remote.type=inet,remote.host=230.0.0.1,remote.port=1234: can't bind
ip=230.0.0.1 to socket: Unknown error
Broken pipe
It seems to be the test_dgram_mcast() test.
Perhaps windows doesn't support multicast datagram?
../tests/qtest/libqtest.c:195: kill_qemu() tried to terminate QEMU process but encountered
exit status 1 (expected 0)
TAP parsing error: Too few tests run (expected 6, got 4)
(test program exited with status code 3)
------------------------------------------------------------------------------
See: https://cirrus-ci.com/task/6491942456918016
I'm sorry I've not been able to find the cirrus-ci option in github, so I
didn't test.
... maybe it would be better to simply limit the test to
CONFIG_POSIX in the meson.build file?
I agree but first I'd like to only disable the multicast test. If it fails, I'll limit the
test to POSIX.
I'm sorry for the inconvenient.
Thanks,
Laurent