Skip to content

Commit 040a6cb

Browse files
pmachatakuba-moo
authored andcommitted
selftests: forwarding: lib: Add an autodefer variant of forwarding_enable()
Most forwarding tests invoke forwarding_enable() to enable the router and forwarding_restore() to restore the original configuration. Add a helper, adf_forwarding_enable(), which is like forwarding_enable(), but takes care of scheduling the cleanup automatically. Convert the tests that currently use defer to schedule the cleanup. Signed-off-by: Petr Machata <petrm@nvidia.com> Reviewed-by: Simon Horman <horms@kernel.org> Link: https://patch.msgid.link/78b752c40069cde21c44dcf4c7b966a76a0eef2c.1758821127.git.petrm@nvidia.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
1 parent f53748d commit 040a6cb

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

tools/testing/selftests/net/forwarding/bridge_fdb_local_vlan_0.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,7 @@ setup_prepare()
9595
h3=${NETIFS[p6]}
9696

9797
adf_vrf_prepare
98-
99-
forwarding_enable
100-
defer forwarding_restore
98+
adf_forwarding_enable
10199

102100
host_create "$h1" 192.0.2.1/28 2001:db8:1::1/64
103101
host_create "$h2" 192.0.2.2/28 2001:db8:1::2/64

tools/testing/selftests/net/forwarding/lib.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1023,6 +1023,12 @@ forwarding_restore()
10231023
sysctl_restore net.ipv4.conf.all.forwarding
10241024
}
10251025

1026+
adf_forwarding_enable()
1027+
{
1028+
forwarding_enable
1029+
defer forwarding_restore
1030+
}
1031+
10261032
declare -A MTU_ORIG
10271033
mtu_set()
10281034
{

tools/testing/selftests/net/forwarding/vxlan_bridge_1q_mc_ul.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -370,9 +370,7 @@ setup_prepare()
370370
h3=${NETIFS[p6]}
371371

372372
adf_vrf_prepare
373-
374-
forwarding_enable
375-
defer forwarding_restore
373+
adf_forwarding_enable
376374

377375
adf_ip_link_add "v1$h2" type veth peer name "v2$h2"
378376
adf_ip_link_add "v1$h3" type veth peer name "v2$h3"

tools/testing/selftests/net/forwarding/vxlan_reserved.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,7 @@ setup_prepare()
8787
rp2=${NETIFS[p4]}
8888

8989
adf_vrf_prepare
90-
91-
forwarding_enable
92-
defer forwarding_restore
90+
adf_forwarding_enable
9391

9492
h1_create
9593
switch_create

0 commit comments

Comments
 (0)