mirror of
https://github.com/torvalds/linux.git
synced 2024-11-21 19:46:16 +00:00
kmsan: make the tests compatible with kmsan.panic=1
It's useful to have both tests and kmsan.panic=1 during development, but right now the warnings, that the tests cause, lead to kernel panics. Temporarily set kmsan.panic=0 for the duration of the KMSAN testing. Link: https://lkml.kernel.org/r/20240621113706.315500-3-iii@linux.ibm.com Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com> Reviewed-by: Alexander Potapenko <glider@google.com> Cc: Alexander Gordeev <agordeev@linux.ibm.com> Cc: Christian Borntraeger <borntraeger@linux.ibm.com> Cc: Christoph Lameter <cl@linux.com> Cc: David Rientjes <rientjes@google.com> Cc: Dmitry Vyukov <dvyukov@google.com> Cc: Heiko Carstens <hca@linux.ibm.com> Cc: Hyeonggon Yoo <42.hyeyoo@gmail.com> Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com> Cc: <kasan-dev@googlegroups.com> Cc: Marco Elver <elver@google.com> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Masami Hiramatsu (Google) <mhiramat@kernel.org> Cc: Pekka Enberg <penberg@kernel.org> Cc: Roman Gushchin <roman.gushchin@linux.dev> Cc: Steven Rostedt (Google) <rostedt@goodmis.org> Cc: Sven Schnelle <svens@linux.ibm.com> Cc: Vasily Gorbik <gor@linux.ibm.com> Cc: Vlastimil Babka <vbabka@suse.cz> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
This commit is contained in:
parent
c02525a339
commit
7d1c8e99b2
@ -686,9 +686,13 @@ static void test_exit(struct kunit *test)
|
||||
{
|
||||
}
|
||||
|
||||
static int orig_panic_on_kmsan;
|
||||
|
||||
static int kmsan_suite_init(struct kunit_suite *suite)
|
||||
{
|
||||
register_trace_console(probe_console, NULL);
|
||||
orig_panic_on_kmsan = panic_on_kmsan;
|
||||
panic_on_kmsan = 0;
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -696,6 +700,7 @@ static void kmsan_suite_exit(struct kunit_suite *suite)
|
||||
{
|
||||
unregister_trace_console(probe_console, NULL);
|
||||
tracepoint_synchronize_unregister();
|
||||
panic_on_kmsan = orig_panic_on_kmsan;
|
||||
}
|
||||
|
||||
static struct kunit_suite kmsan_test_suite = {
|
||||
|
Loading…
Reference in New Issue
Block a user