[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <186278b5-b2d5-408e-8d89-8cdff6efe41a@blackwall.org>
Date: Tue, 14 May 2024 10:46:15 +0300
From: Nikolay Aleksandrov <razor@...ckwall.org>
To: Hangbin Liu <liuhangbin@...il.com>
Cc: netdev@...r.kernel.org, kuba@...nel.org, roopa@...dia.com,
bridge@...ts.linux.dev, edumazet@...gle.com, pabeni@...hat.com
Subject: Re: [PATCH net] selftests: net: bridge: increase IGMP/MLD exclude
timeout membership interval
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.
Powered by blists - more mailing lists