Skip to content

Commit f312368

Browse files
pmachataksacilotto
authored andcommitted
selftests: net: mirror_gre_vlan_bridge_1q: Make an FDB entry static
BugLink: https://bugs.launchpad.net/bugs/1929615 [ Upstream commit c8d0260 ] 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 6ab4c31 ("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: 9c7c8a8 ("selftests: forwarding: mirror_gre_vlan_bridge_1q: Add more tests") Signed-off-by: Petr Machata <[email protected]> Reviewed-by: Ido Schimmel <[email protected]> Signed-off-by: David S. Miller <[email protected]> Signed-off-by: Sasha Levin <[email protected]> Signed-off-by: Kamal Mostafa <[email protected]> Signed-off-by: Kleber Sacilotto de Souza <[email protected]>
1 parent cec8ebb commit f312368

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/testing/selftests/net/forwarding/mirror_gre_vlan_bridge_1q.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,7 @@ test_span_gre_fdb_roaming()
271271

272272
while ((RET == 0)); do
273273
bridge fdb del dev $swp3 $h3mac vlan 555 master 2>/dev/null
274-
bridge fdb add dev $swp2 $h3mac vlan 555 master
274+
bridge fdb add dev $swp2 $h3mac vlan 555 master static
275275
sleep 1
276276
fail_test_span_gre_dir $tundev ingress
277277

0 commit comments

Comments
 (0)