selftests/net: so_txtime: fix clang issues for target arch PowerPC
authorTanner Love <tannerlove@google.com>
Mon, 27 Jul 2020 16:25:30 +0000 (12:25 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 27 Jul 2020 19:56:58 +0000 (12:56 -0700)
On powerpcle, int64_t maps to long long. Clang 9 threw:
warning: absolute value function 'labs' given an argument of type \
'long long' but has parameter of type 'long' which may cause \
truncation of value [-Wabsolute-value]
        if (labs(tstop - texpect) > cfg_variance_us)

Tested: make -C tools/testing/selftests TARGETS="net" run_tests

Fixes: af5136f95045 ("selftests/net: SO_TXTIME with ETF and FQ")
Signed-off-by: Tanner Love <tannerlove@google.com>
Acked-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/so_txtime.c

index ceaad78..3155fbb 100644 (file)
@@ -121,7 +121,7 @@ static bool do_recv_one(int fdr, struct timed_send *ts)
        if (rbuf[0] != ts->data)
                error(1, 0, "payload mismatch. expected %c", ts->data);
 
-       if (labs(tstop - texpect) > cfg_variance_us)
+       if (llabs(tstop - texpect) > cfg_variance_us)
                error(1, 0, "exceeds variance (%d us)", cfg_variance_us);
 
        return false;