kselftest/arm64: Fix test numbering when skipping tests
authorMark Brown <broonie@kernel.org>
Tue, 10 Jan 2023 20:49:58 +0000 (20:49 +0000)
committerCatalin Marinas <catalin.marinas@arm.com>
Thu, 12 Jan 2023 17:10:01 +0000 (17:10 +0000)
Currently when skipping tests in the BTI testsuite we assign the same
number to every test since we forget to increment the current test number
as we skip, causing warnings about not running the expected test count and
potentially otherwise confusing result parsers. Fix this by adding an
appropriate increment.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20230110-arm64-bti-selftest-skip-v1-1-143ecdc84567@kernel.org
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
tools/testing/selftests/arm64/bti/test.c

index 67b77ab..4b6dda9 100644 (file)
@@ -112,7 +112,7 @@ static void __do_test(void (*trampoline)(void (*)(void)),
        if (skip_all) {
                test_skipped++;
                putstr("ok ");
-               putnum(test_num);
+               putnum(test_num++);
                putstr(" ");
                puttestname(name, trampoline_name);
                putstr(" # SKIP\n");