If an earlier test run left p0/p1 as the interface altname,
'ip link set $VF name pN' can return RTNETLINK "File exists".
Remove the altname before renaming so ADD_VF is repeatable.
Fixes: 289e9f6baa7c ("tests: Add a simple DPDK rte_flow test framework.")
Signed-off-by: Eli Britstein <[email protected]>
---
tests/system-dpdk-offloads-macros.at | 2 ++
utilities/checkpatch_dict.txt | 1 +
2 files changed, 3 insertions(+)
diff --git a/tests/system-dpdk-offloads-macros.at
b/tests/system-dpdk-offloads-macros.at
index bb95fae6e..194d8bca8 100644
--- a/tests/system-dpdk-offloads-macros.at
+++ b/tests/system-dpdk-offloads-macros.at
@@ -129,6 +129,8 @@ m4_define([ADD_VF],
AT_CHECK([test -n "$VF"])
AT_CHECK([ip link set $VF down])
+ dnl If a prior run left $1 as an altname, rename fails with "File
exists".
+ AT_CHECK([ip link property del dev $VF altname $1 2>/dev/null || true])
AT_CHECK([ip link set $VF name $1])
AT_CHECK([ip link set $1 netns $2])
AT_CHECK([ovs-vsctl add-port $3 ovs-$1 -- \
diff --git a/utilities/checkpatch_dict.txt b/utilities/checkpatch_dict.txt
index c1f43e5af..000ea75e9 100644
--- a/utilities/checkpatch_dict.txt
+++ b/utilities/checkpatch_dict.txt
@@ -3,6 +3,7 @@ addresssanitizer
afxdp
alg
alloc
+altname
amd64
api
apis
--
2.34.1
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev