lkml.org 
[lkml]   [2022]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.18 26/93] selftests: mlxsw: resource_scale: Allow skipping a test
    Date
    From: Petr Machata <petrm@nvidia.com>

    [ Upstream commit 8cad339db339a39cb82b1188e4be4070a433abac ]

    The scale tests are currently testing two things: that some number of
    instances of a given resource can actually be created; and that when an
    attempt is made to create more than the supported amount, the failures are
    noted and handled gracefully.

    Sometimes the scale test depends on more than one resource. In particular,
    a following patch will add a RIF counter scale test, which depends on the
    number of RIF counters that can be bound, and also on the number of RIFs
    that can be created.

    When the test is limited by the auxiliary resource and not by the primary
    one, there's no point trying to run the overflow test, because it would be
    testing exhaustion of the wrong resource.

    To support this use case, when the $test_get_target yields 0, skip the test
    instead.

    Signed-off-by: Petr Machata <petrm@nvidia.com>
    Reviewed-by: Amit Cohen <amcohen@nvidia.com>
    Signed-off-by: Ido Schimmel <idosch@nvidia.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    .../selftests/drivers/net/mlxsw/spectrum-2/resource_scale.sh | 5 +++++
    .../selftests/drivers/net/mlxsw/spectrum/resource_scale.sh | 4 ++++
    2 files changed, 9 insertions(+)

    diff --git a/tools/testing/selftests/drivers/net/mlxsw/spectrum-2/resource_scale.sh b/tools/testing/selftests/drivers/net/mlxsw/spectrum-2/resource_scale.sh
    index e9f65bd2e299..df920b6ed7c4 100755
    --- a/tools/testing/selftests/drivers/net/mlxsw/spectrum-2/resource_scale.sh
    +++ b/tools/testing/selftests/drivers/net/mlxsw/spectrum-2/resource_scale.sh
    @@ -36,6 +36,11 @@ for current_test in ${TESTS:-$ALL_TESTS}; do
    for should_fail in 0 1; do
    RET=0
    target=$(${current_test}_get_target "$should_fail")
    + if ((target == 0)); then
    + log_test_skip "'$current_test' should_fail=$should_fail test"
    + continue
    + fi
    +
    ${current_test}_setup_prepare
    setup_wait $num_netifs
    ${current_test}_test "$target" "$should_fail"
    diff --git a/tools/testing/selftests/drivers/net/mlxsw/spectrum/resource_scale.sh b/tools/testing/selftests/drivers/net/mlxsw/spectrum/resource_scale.sh
    index dea33dc93790..b75d1fcd2db2 100755
    --- a/tools/testing/selftests/drivers/net/mlxsw/spectrum/resource_scale.sh
    +++ b/tools/testing/selftests/drivers/net/mlxsw/spectrum/resource_scale.sh
    @@ -41,6 +41,10 @@ for current_test in ${TESTS:-$ALL_TESTS}; do
    for should_fail in 0 1; do
    RET=0
    target=$(${current_test}_get_target "$should_fail")
    + if ((target == 0)); then
    + log_test_skip "'$current_test' [$profile] should_fail=$should_fail test"
    + continue
    + fi
    ${current_test}_setup_prepare
    setup_wait $num_netifs
    ${current_test}_test "$target" "$should_fail"
    --
    2.35.1
    \
     
     \ /
      Last update: 2022-08-11 17:59    [W:3.264 / U:0.888 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site