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

Reply via email to