]> git.itanic.dy.fi Git - linux-stable/commit
selftests: net: Fix bridge backup port test flakiness
authorIdo Schimmel <idosch@nvidia.com>
Thu, 8 Feb 2024 12:31:10 +0000 (14:31 +0200)
committerJakub Kicinski <kuba@kernel.org>
Fri, 9 Feb 2024 19:29:52 +0000 (11:29 -0800)
commit38ee0cb2a2e2ade077442085638eb181b0562971
treec9117539e17de2bb9a635a1069b0eecdbceb6ad2
parent02d9009f4e8c27dcf10c3e39bc0666436686a219
selftests: net: Fix bridge backup port test flakiness

The test toggles the carrier of a bridge port in order to test the
bridge backup port feature.

Due to the linkwatch delayed work the carrier change is not always
reflected fast enough to the bridge driver and packets are not forwarded
as the test expects, resulting in failures [1].

Fix by busy waiting on the bridge port state until it changes to the
desired state following the carrier change.

[1]
 # Backup port
 # -----------
 [...]
 # TEST: swp1 carrier off                                              [ OK ]
 # TEST: No forwarding out of swp1                                     [FAIL]
 [  641.995910] br0: port 1(swp1) entered disabled state
 # TEST: No forwarding out of vx0                                      [ OK ]

Fixes: b408453053fb ("selftests: net: Add bridge backup port and backup nexthop ID test")
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reviewed-by: Petr Machata <petrm@nvidia.com>
Acked-by: Paolo Abeni <pabeni@redhat.com>
Acked-by: Nikolay Aleksandrov <razor@blackwall.org>
Link: https://lore.kernel.org/r/20240208123110.1063930-1-idosch@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/test_bridge_backup_port.sh