lkml.org 
[lkml]   [2021]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 211/917] selftests: net: fib_nexthops: Wait before checking reported idle time
    Date
    From: Petr Machata <petrm@nvidia.com>

    [ Upstream commit b69c99463d414cc263411462d52f25205657e9af ]

    The purpose of this test is to verify that after a short activity passes,
    the reported time is reasonable: not zero (which could be reported by
    mistake), and not something outrageous (which would be indicative of an
    issue in used units).

    However, the idle time is reported in units of clock_t, or hundredths of
    second. If the initial sequence of commands is very quick, it is possible
    that the idle time is reported as just flat-out zero. When this test was
    recently enabled in our nightly regression, we started seeing spurious
    failures for exactly this reason.

    Therefore buffer the delay leading up to the test with a sleep, to make
    sure there is no legitimate way of reporting 0.

    Signed-off-by: Petr Machata <petrm@nvidia.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/testing/selftests/net/fib_nexthops.sh | 1 +
    1 file changed, 1 insertion(+)

    diff --git a/tools/testing/selftests/net/fib_nexthops.sh b/tools/testing/selftests/net/fib_nexthops.sh
    index 0d293391e9a44..b5a69ad191b07 100755
    --- a/tools/testing/selftests/net/fib_nexthops.sh
    +++ b/tools/testing/selftests/net/fib_nexthops.sh
    @@ -2078,6 +2078,7 @@ basic_res()
    "id 101 index 0 nhid 2 id 101 index 1 nhid 2 id 101 index 2 nhid 1 id 101 index 3 nhid 1"
    log_test $? 0 "Dump all nexthop buckets in a group"

    + sleep 0.1
    (( $($IP -j nexthop bucket list id 101 |
    jq '[.[] | select(.bucket.idle_time > 0 and
    .bucket.idle_time < 2)] | length') == 4 ))
    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-16 02:11    [W:4.033 / U:0.044 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site