On 01.11.2019 22:51, William Tu wrote:
AF_XDP veth does not support TCP with namespaces. This patch skips them. Signed-off-by: William Tu <u9012...@gmail.com> --- v1: - Tested-at: https://travis-ci.org/williamtu/ovs-travis/builds/606194550 - make check-afxdp all pass on my machine, but cirrus, it is still unstable, observe 1 failed at: https://cirrus-ci.com/task/6597038589870080 --- tests/system-afxdp-macros.at | 7 +++++++ tests/system-kmod-macros.at | 8 ++++++++ tests/system-traffic.at | 18 +++++++++--------- tests/system-userspace-macros.at | 8 ++++++++ 4 files changed, 32 insertions(+), 9 deletions(-) diff --git a/tests/system-afxdp-macros.at b/tests/system-afxdp-macros.at index f0683c0a901b..27c220998355 100644 --- a/tests/system-afxdp-macros.at +++ b/tests/system-afxdp-macros.at @@ -37,3 +37,10 @@ m4_define([CONFIGURE_VETH_OFFLOADS], # m4_define([OVS_START_L7], [AT_SKIP_IF([:])]) + +# OVS_CHECK_NC() +# +# Check if comman nc is available. Always skip because +# AF_XDP veth does not support TCP with namespaces. +m4_define([OVS_CHECK_NC], + [AT_SKIP_IF([:])])
Most of the tests uses 'nc -u' generating UDP traffic and works fine. We should not skip them. There are only 2 tests that are failing now. Both uses 'nc' without '-u' option, i.e. generates TCP traffic. Only these tests should be skipped. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
