Skip to content

Commit cdbde7e

Browse files
idoschkuba-moo
authored andcommitted
selftests: mlxsw: Add a test for invalid locked bridge port configurations
Test that locked bridge port configurations that are not supported by mlxsw are rejected. Signed-off-by: Ido Schimmel <idosch@nvidia.com> Reviewed-by: Petr Machata <petrm@nvidia.com> Signed-off-by: Petr Machata <petrm@nvidia.com> Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
1 parent fb39843 commit cdbde7e

1 file changed

Lines changed: 31 additions & 0 deletions

File tree

tools/testing/selftests/drivers/net/mlxsw/rtnetlink.sh

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ ALL_TESTS="
3434
nexthop_obj_bucket_offload_test
3535
nexthop_obj_blackhole_offload_test
3636
nexthop_obj_route_offload_test
37+
bridge_locked_port_test
3738
devlink_reload_test
3839
"
3940
NUM_NETIFS=2
@@ -917,6 +918,36 @@ nexthop_obj_route_offload_test()
917918
simple_if_fini $swp1 192.0.2.1/24 2001:db8:1::1/64
918919
}
919920

921+
bridge_locked_port_test()
922+
{
923+
RET=0
924+
925+
ip link add name br1 up type bridge vlan_filtering 0
926+
927+
ip link add link $swp1 name $swp1.10 type vlan id 10
928+
ip link set dev $swp1.10 master br1
929+
930+
bridge link set dev $swp1.10 locked on
931+
check_fail $? "managed to set locked flag on a VLAN upper"
932+
933+
ip link set dev $swp1.10 nomaster
934+
ip link set dev $swp1 master br1
935+
936+
bridge link set dev $swp1 locked on
937+
check_fail $? "managed to set locked flag on a bridge port that has a VLAN upper"
938+
939+
ip link del dev $swp1.10
940+
bridge link set dev $swp1 locked on
941+
942+
ip link add link $swp1 name $swp1.10 type vlan id 10
943+
check_fail $? "managed to configure a VLAN upper on a locked port"
944+
945+
log_test "bridge locked port"
946+
947+
ip link del dev $swp1.10 &> /dev/null
948+
ip link del dev br1
949+
}
950+
920951
devlink_reload_test()
921952
{
922953
# Test that after executing all the above configuration tests, a

0 commit comments

Comments
 (0)