DORMANTno replies

[PATCH] selftests/powerpc: Don't run spectre_v2 test by default

From: Russell Currey <hidden>
Date: 2022-08-05 01:56:33
Subsystem: kernel selftest framework, the rest · Maintainers: Shuah Khan, Shuah Khan, Linus Torvalds

The spectre_v2 selftest has issues that I'm unsure of how to resolve.
It uses context to determine intended behaviour, but that context is
unreliable - as an example, when running as a KVM guest, qemu can
deliberately misreport mitigation status for compatibility purposes.

As a result, the selftest is unreliable as a pass/fail test without
the test runner knowing what they expect its behaviour to be.  I don't
think the selftest is useless so we should keep it around, but we
shouldn't have run_tests run it by default.

Suggested-by: Eirik Fuller <redacted>
Signed-off-by: Russell Currey <redacted>
---
 tools/testing/selftests/powerpc/security/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/powerpc/security/Makefile b/tools/testing/selftests/powerpc/security/Makefile
index 7488315fd847..c954d79aeb80 100644
--- a/tools/testing/selftests/powerpc/security/Makefile
+++ b/tools/testing/selftests/powerpc/security/Makefile
@@ -1,6 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0+
 
-TEST_GEN_PROGS := rfi_flush entry_flush uaccess_flush spectre_v2
+TEST_GEN_PROGS := rfi_flush entry_flush uaccess_flush
+TEST_GEN_PROGS_EXTENDED := spectre_v2
 TEST_PROGS := mitigation-patching.sh
 
 top_srcdir = ../../../../..
@@ -10,6 +11,7 @@ CFLAGS += -I../../../../../usr/include
 include ../../lib.mk
 
 $(TEST_GEN_PROGS): ../harness.c ../utils.c
+$(TEST_GEN_PROGS_EXTENDED): ../harness.c ../utils.c
 
 $(OUTPUT)/spectre_v2: CFLAGS += -m64
 $(OUTPUT)/spectre_v2: ../pmu/event.c branch_loops.S
-- 
2.37.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help