Skip to content

Commit

Permalink
selftests: forwarding: fix learning_test when h1 supports IFF_UNICAST…
Browse files Browse the repository at this point in the history
…_FLT

[ Upstream commit 1a635d3 ]

The first host interface has by default no interest in receiving packets
MAC DA de:ad:be:ef:13:37, so it might drop them before they hit the tc
filter and this might confuse the selftest.

Enable promiscuous mode such that the filter properly counts received
packets.

Fixes: d4deb01 ("selftests: forwarding: Add a test for FDB learning")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
Tested-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
vladimiroltean authored and gregkh committed Jul 12, 2022
1 parent 859b889 commit 9906c22
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions tools/testing/selftests/net/forwarding/lib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1063,6 +1063,7 @@ learning_test()
# FDB entry was installed.
bridge link set dev $br_port1 flood off

ip link set $host1_if promisc on
tc qdisc add dev $host1_if ingress
tc filter add dev $host1_if ingress protocol ip pref 1 handle 101 \
flower dst_mac $mac action drop
Expand Down Expand Up @@ -1112,6 +1113,7 @@ learning_test()

tc filter del dev $host1_if ingress protocol ip pref 1 handle 101 flower
tc qdisc del dev $host1_if ingress
ip link set $host1_if promisc off

bridge link set dev $br_port1 flood on

Expand Down

0 comments on commit 9906c22

Please sign in to comment.