Skip to content

Commit

Permalink
selftests: net: bridge: fix typo in vlan_filtering dependency test
Browse files Browse the repository at this point in the history
Prior patch:
]# TESTS=vlmc_filtering_test ./bridge_vlan_mcast.sh
TEST: Vlan multicast snooping enable                                [ OK ]
Device "bridge" does not exist.
TEST: Disable multicast vlan snooping when vlan filtering is disabled   [FAIL]
        Vlan filtering is disabled but multicast vlan snooping is still enabled

After patch:
# TESTS=vlmc_filtering_test ./bridge_vlan_mcast.sh
TEST: Vlan multicast snooping enable                                [ OK ]
TEST: Disable multicast vlan snooping when vlan filtering is disabled   [ OK ]

Fixes: f5a9dd5 ("selftests: net: bridge: add test for vlan_filtering dependency")
Cc: Nikolay Aleksandrov <nikolay@nvidia.com>
Signed-off-by: Ivan Vecera <ivecera@redhat.com>
Acked-by: Nikolay Aleksandrov <nikolay@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Ivan Vecera authored and David S. Miller committed Nov 29, 2021
1 parent fe42e88 commit 754d71b
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -527,7 +527,7 @@ vlmc_filtering_test()
{
RET=0
ip link set dev br0 type bridge vlan_filtering 0
ip -j -d link show dev bridge | \
ip -j -d link show dev br0 | \
jq -e "select(.[0].linkinfo.info_data.mcast_vlan_snooping == 1)" &>/dev/null
check_fail $? "Vlan filtering is disabled but multicast vlan snooping is still enabled"
log_test "Disable multicast vlan snooping when vlan filtering is disabled"
Expand Down

0 comments on commit 754d71b

Please sign in to comment.