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 67b77ab83c20ed2098578c0ab14877b63259035a..4b6dda987c58646fea69f284b8e5fc4e5188a229 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");