selftests: net: mirror_gre_vlan_bridge_1q: Make an FDB entry static
authorPetr Machata <petrm@nvidia.com>
Fri, 23 Apr 2021 12:19:43 +0000 (14:19 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 23 Apr 2021 21:01:28 +0000 (14:01 -0700)
commitc8d0260cdd96fdccdef0509c4160e28a1012a5d7
tree449cb25a877a4d3ba70a7fb125bfd83dde89af5b
parente40fa65c79b59ef662bf3f9e81cc2236fb9581e3
selftests: net: mirror_gre_vlan_bridge_1q: Make an FDB entry static

The FDB roaming test installs a destination MAC address on the wrong
interface of an FDB database and tests whether the mirroring fails, because
packets are sent to the wrong port. The test by mistake installs the FDB
entry as local. This worked previously, because drivers were notified of
local FDB entries in the same way as of static entries. However that has
been fixed in the commit 6ab4c3117aec ("net: bridge: don't notify switchdev
for local FDB addresses"), and local entries are not notified anymore. As a
result, the HW is not reconfigured for the FDB roam, and mirroring keeps
working, failing the test.

To fix the issue, mark the FDB entry as static.

Fixes: 9c7c8a82442c ("selftests: forwarding: mirror_gre_vlan_bridge_1q: Add more tests")
Signed-off-by: Petr Machata <petrm@nvidia.com>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/forwarding/mirror_gre_vlan_bridge_1q.sh