selftests: mlxsw: qos_mc_aware: Specify arping timeout as an integer
authorAmit Cohen <amitc@mellanox.com>
Thu, 21 May 2020 12:11:45 +0000 (15:11 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 22 May 2020 23:08:14 +0000 (16:08 -0700)
Starting from iputils s20190709 (used in Fedora 31), arping does not
support timeout being specified as a decimal:

$ arping -c 1 -I swp1 -b 192.0.2.66 -q -w 0.1
arping: invalid argument: '0.1'

Previously, such timeouts were rounded to an integer.

Fix this by specifying the timeout as an integer.

Fixes: a5ee171d087e ("selftests: mlxsw: qos_mc_aware: Add a test for UC awareness")
Signed-off-by: Amit Cohen <amitc@mellanox.com>
Reviewed-by: Petr Machata <petrm@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/drivers/net/mlxsw/qos_mc_aware.sh

index 24dd8ed..b025dae 100755 (executable)
@@ -300,7 +300,7 @@ test_uc_aware()
        local i
 
        for ((i = 0; i < attempts; ++i)); do
-               if $ARPING -c 1 -I $h1 -b 192.0.2.66 -q -w 0.1; then
+               if $ARPING -c 1 -I $h1 -b 192.0.2.66 -q -w 1; then
                        ((passes++))
                fi