Commit 5b8e7418 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

selftests: net: local_termination: add one more test for VLAN-aware bridges

The current bridge() test is for packet reception on a VLAN-unaware
bridge. Some things are different enough with VLAN-aware bridges that
it's worth renaming this test into vlan_unaware_bridge(), and add a new
vlan_aware_bridge() test.

The two will share the same implementation: bridge() becomes a common
function, which receives $vlan_filtering as an argument. Rename it to
test_bridge() at the same time, because just bridge() pollutes the
global namespace and we cannot invoke the binary with the same name from
the iproute2 package currently.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent df7cf5cc
#!/bin/bash #!/bin/bash
# SPDX-License-Identifier: GPL-2.0 # SPDX-License-Identifier: GPL-2.0
ALL_TESTS="standalone bridge" ALL_TESTS="standalone vlan_unaware_bridge vlan_aware_bridge"
NUM_NETIFS=2 NUM_NETIFS=2
PING_COUNT=1 PING_COUNT=1
REQUIRE_MTOOLS=yes REQUIRE_MTOOLS=yes
...@@ -233,7 +233,9 @@ h2_destroy() ...@@ -233,7 +233,9 @@ h2_destroy()
bridge_create() bridge_create()
{ {
ip link add br0 type bridge local vlan_filtering=$1
ip link add br0 type bridge vlan_filtering $vlan_filtering
ip link set br0 address $BRIDGE_ADDR ip link set br0 address $BRIDGE_ADDR
ip link set br0 up ip link set br0 up
...@@ -277,10 +279,12 @@ standalone() ...@@ -277,10 +279,12 @@ standalone()
h1_destroy h1_destroy
} }
bridge() test_bridge()
{ {
local vlan_filtering=$1
h1_create h1_create
bridge_create bridge_create $vlan_filtering
macvlan_create br0 macvlan_create br0
run_test $h1 br0 run_test $h1 br0
...@@ -290,6 +294,16 @@ bridge() ...@@ -290,6 +294,16 @@ bridge()
h1_destroy h1_destroy
} }
vlan_unaware_bridge()
{
test_bridge 0
}
vlan_aware_bridge()
{
test_bridge 1
}
cleanup() cleanup()
{ {
pre_cleanup pre_cleanup
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment