Re: [PATCH v14 16/17] tests/qtest: netdev: test stream and dgram backends

[Laurent Vivier <lvivier@xxxxxxxxxx>]

On 11/3/22 12:07, Philippe Mathieu-Daudé wrote:
> On 3/11/22 10:33, Laurent Vivier wrote:
> > On 10/28/22 07:04, Jason Wang wrote:
> > > 在 2022/10/21 17:09, Laurent Vivier 写道:
> > > > Signed-off-by: Laurent Vivier <lvivier@xxxxxxxxxx>
> > > > Acked-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
> > > > ---
> > >
> > > I got this:
> > >
> > > 63/63 ERROR:../tests/qtest/netdev-socket.c:139:test_stream_inet_ipv6: assertion failed 
> > > (resp == expect): ("st0: index=0,type=stream,connection error\r\n" == "st0: 
> > > index=0,type=stream,tcp:::1:40389\r\n") ERROR
> > > 63/63 qemu:qtest+qtest-x86_64 / qtest-x86_64/netdev-socket                  ERROR 5.29s 
> > > killed by signal 6 SIGABRT
> > >  >>> QTEST_QEMU_IMG=./qemu-img QTEST_QEMU_BINARY=./qemu-system-x86_64 
> > > MALLOC_PERTURB_=96 
> > > QTEST_QEMU_STORAGE_DAEMON_BINARY=./storage-daemon/qemu-storage-daemon 
> > > G_TEST_DBUS_DAEMON=/home/devel/git/qemu/tests/dbus-vmstate-daemon.sh 
> > > /home/devel/git/qemu/build/tests/qtest/netdev-socket --tap -k
> > > ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― ✀ 
> > > ―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――
> > > stderr:
> > > **
> > > ERROR:../tests/qtest/netdev-socket.c:139:test_stream_inet_ipv6: assertion failed (resp 
> > > == expect): ("st0: index=0,type=stream,connection error\r\n" == "st0: 
> > > index=0,type=stream,tcp:::1:40389\r\n")
> > > (test program exited with status code -6)
> > I'm not able to reproduce the problem.
> >
> > Is this 100% reproducible?
> > Is IPv6 enabled on your test machine?
> If IPv6 is not available, this test should be skipped, not failing.
I agree.
But I'm not sure it's the real cause of the problem (perhaps a firewall 
problem?).

I think I should update my inet_get_free_port_socket() to get a free port from AF_INET6 
and not only from AF_INET.
Thanks,
Laurent