On 14/05/2024 03:42, Hangbin Liu wrote: > On Mon, May 13, 2024 at 01:52:57PM +0300, Nikolay Aleksandrov wrote: >> $MZ $h1 -c 1 -b $ALL_MAC -B $ALL_GROUP -t ip "proto=2,p=$MZPKT_ALLOW2" >> -q >> - sleep 3 >> + sleep 5 >> bridge -j -d -s mdb show dev br0 \ >> | jq -e ".[].mdb[] | \ >> select(.grp == \"$TEST_GROUP\" and \ >> diff --git a/tools/testing/selftests/net/forwarding/bridge_mld.sh >> b/tools/testing/selftests/net/forwarding/bridge_mld.sh >> index e2b9ff773c6b..f84ab2e65754 100755 >> --- a/tools/testing/selftests/net/forwarding/bridge_mld.sh >> +++ b/tools/testing/selftests/net/forwarding/bridge_mld.sh >> >> $MZ $h1 -c 1 $MZPKT_ALLOW2 -q >> - sleep 3 >> + sleep 5 >> bridge -j -d -s mdb show dev br0 \ >> | jq -e ".[].mdb[] | \ >> select(.grp == \"$TEST_GROUP\" and \ > > Maybe use a slow_wait to check the result? > > Thanks > Hangbin
What would it improve? The wait is exact, we know how many seconds exactly so a plain sleep is enough and easier to backport if this is applied to -net.
