On Tue, May 14, 2024 at 10:46:15AM +0300, Nikolay Aleksandrov wrote: > >> 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. I just afraid it fails on debug kernel with this exact time. If you think the time is enough. I'm OK with it. Thanks Hangbin
