selftests/net: give more time to udpgro bg processes to complete startup
authorAdrien Thierry <athierry@redhat.com>
Tue, 1 Nov 2022 18:48:08 +0000 (14:48 -0400)
committerDavid S. Miller <davem@davemloft.net>
Fri, 4 Nov 2022 10:10:59 +0000 (10:10 +0000)
In some conditions, background processes in udpgro don't have enough
time to set up the sockets. When foreground processes start, this
results in the test failing with "./udpgso_bench_tx: sendmsg: Connection
refused". For instance, this happens from time to time on a Qualcomm
SA8540P SoC running CentOS Stream 9.

To fix this, increase the time given to background processes to
complete the startup before foreground processes start.

Signed-off-by: Adrien Thierry <athierry@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/udpgro.sh
tools/testing/selftests/net/udpgro_bench.sh
tools/testing/selftests/net/udpgro_frglist.sh

index ebbd0b2..6a443ca 100755 (executable)
@@ -50,7 +50,7 @@ run_one() {
                echo "failed" &
 
        # Hack: let bg programs complete the startup
-       sleep 0.1
+       sleep 0.2
        ./udpgso_bench_tx ${tx_args}
        ret=$?
        wait $(jobs -p)
@@ -117,7 +117,7 @@ run_one_2sock() {
                echo "failed" &
 
        # Hack: let bg programs complete the startup
-       sleep 0.1
+       sleep 0.2
        ./udpgso_bench_tx ${tx_args} -p 12345
        sleep 0.1
        # first UDP GSO socket should be closed at this point
index fad2d1a..8a1109a 100755 (executable)
@@ -39,7 +39,7 @@ run_one() {
        ip netns exec "${PEER_NS}" ./udpgso_bench_rx -t ${rx_args} -r &
 
        # Hack: let bg programs complete the startup
-       sleep 0.1
+       sleep 0.2
        ./udpgso_bench_tx ${tx_args}
 }
 
index 832c738..7fe85ba 100755 (executable)
@@ -44,7 +44,7 @@ run_one() {
        ip netns exec "${PEER_NS}" ./udpgso_bench_rx ${rx_args} -r &
 
        # Hack: let bg programs complete the startup
-       sleep 0.1
+       sleep 0.2
        ./udpgso_bench_tx ${tx_args}
 }