Re: [PATCH v6] tests/qtest: netdev: test stream and dgram backends

[Laurent Vivier <lvivier@redhat.com>]

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