lkml.org 
[lkml]   [2021]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.14 316/432] selftests/bpf: Correctly display subtest skip status
    Date
    From: Yucong Sun <fallentree@fb.com>

    [ Upstream commit f667d1d66760fcb27aee6c9964eefde39a464afe ]

    In skip_account(), test->skip_cnt is set to 0 at the end, this makes next print
    statement never display SKIP status for the subtest. This patch moves the
    accounting logic after the print statement, fixing the issue.

    This patch also added SKIP status display for normal tests.

    Signed-off-by: Yucong Sun <fallentree@fb.com>
    Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
    Link: https://lore.kernel.org/bpf/20210817044732.3263066-3-fallentree@fb.com
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/testing/selftests/bpf/test_progs.c | 25 ++++++++++++------------
    1 file changed, 13 insertions(+), 12 deletions(-)

    diff --git a/tools/testing/selftests/bpf/test_progs.c b/tools/testing/selftests/bpf/test_progs.c
    index 6f103106a39b..bfbf2277b61a 100644
    --- a/tools/testing/selftests/bpf/test_progs.c
    +++ b/tools/testing/selftests/bpf/test_progs.c
    @@ -148,18 +148,18 @@ void test__end_subtest()
    struct prog_test_def *test = env.test;
    int sub_error_cnt = test->error_cnt - test->old_error_cnt;

    - if (sub_error_cnt)
    - env.fail_cnt++;
    - else if (test->skip_cnt == 0)
    - env.sub_succ_cnt++;
    - skip_account();
    -
    dump_test_log(test, sub_error_cnt);

    fprintf(env.stdout, "#%d/%d %s:%s\n",
    test->test_num, test->subtest_num, test->subtest_name,
    sub_error_cnt ? "FAIL" : (test->skip_cnt ? "SKIP" : "OK"));

    + if (sub_error_cnt)
    + env.fail_cnt++;
    + else if (test->skip_cnt == 0)
    + env.sub_succ_cnt++;
    + skip_account();
    +
    free(test->subtest_name);
    test->subtest_name = NULL;
    }
    @@ -786,17 +786,18 @@ int main(int argc, char **argv)
    test__end_subtest();

    test->tested = true;
    - if (test->error_cnt)
    - env.fail_cnt++;
    - else
    - env.succ_cnt++;
    - skip_account();

    dump_test_log(test, test->error_cnt);

    fprintf(env.stdout, "#%d %s:%s\n",
    test->test_num, test->test_name,
    - test->error_cnt ? "FAIL" : "OK");
    + test->error_cnt ? "FAIL" : (test->skip_cnt ? "SKIP" : "OK"));
    +
    + if (test->error_cnt)
    + env.fail_cnt++;
    + else
    + env.succ_cnt++;
    + skip_account();

    reset_affinity();
    restore_netns();
    --
    2.30.2


    \
     
     \ /
      Last update: 2021-09-16 20:07    [W:4.110 / U:0.140 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site