kernel/arch/s390/lib/test_unwind.c, branch linux-6.2.y

s390: current_stack_pointer shouldn't be a function

2022-04-12T09:56:08Z

s390 defines current_stack_pointer as function while all other architectures use 'register unsigned long asm(""). This make codes like the following from check_stack_object() fail: if (IS_ENABLED(CONFIG_STACK_GROWSUP)) { if ((void *)current_stack_pointer < obj + len) return BAD_STACK; } else { if (obj < (void *)current_stack_pointer) return BAD_STACK; } because this would compare the address of current_stack_pointer() and not the stackpointer value. Reported-by: Karsten Graul Fixes: 2792d84e6da5 ("usercopy: Check valid lifetime via stack depth") Cc: Kees Cook Cc: Vasily Gorbik Cc: Alexander Gordeev Signed-off-by: Sven Schnelle Reviewed-by: Heiko Carstens Signed-off-by: Heiko Carstens

s390/test_unwind: verify __kretprobe_trampoline is replaced

2022-03-27T20:18:39Z

Reviewed-by: Tobias Huschle Signed-off-by: Vasily Gorbik

s390/test_unwind: extend kretprobe test

2022-03-27T20:18:39Z

Verify unwinding from kretprobed function. Reviewed-by: Tobias Huschle Signed-off-by: Vasily Gorbik

s390/test_unwind: add kretprobe tests

2022-03-10T14:58:17Z

Add tests to verify that s390 kretprobes maintain a correct stack chain and ensure their proper function. Reviewed-by: Vasily Gorbik Signed-off-by: Tobias Huschle Signed-off-by: Vasily Gorbik

s390/test_unwind: avoid build warning with W=1

2022-03-07T23:33:01Z

Fix the following build warning with W=1 arch/s390/lib/test_unwind.c:172:21: warning: variable 'fops' set but not used [-Wunused-but-set-variable] struct ftrace_ops *fops; Reported-by: kernel test robot Signed-off-by: Vasily Gorbik

s390/test_unwind: fix and extend kprobes test

2022-03-01T20:05:09Z

Running kprobe test on a kernel built with clang 14 didn't actually trigger pgm_pre_handler() and no unwinder code was called. Even though do_report_trap() is a global symbol, clang inlined it in several local functions including illegal_op() handler, so that kprobbing a global symbol didn't have a desired effect. To achieve the same test result (unwinding from a program check handler) introduce a local function and probe an instruction in the middle, so that kprobe doesn't take KPROBE_ON_FTRACE path. While at it, add another test for KPROBE_ON_FTRACE. Signed-off-by: Vasily Gorbik

s390/test_unwind: add ftrace test

2022-03-01T20:05:09Z

Signed-off-by: Vasily Gorbik

s390/test_unwind: add "backtrace" module parameter

2022-03-01T20:05:09Z

By default no backtraces are printed when a test succeeds, but sometimes it is useful to spot issues automated test doesn't cover. Add "backtrace" module parameter to force it. Signed-off-by: Vasily Gorbik

s390/test_unwind: minor cleanup

2022-03-01T20:05:09Z

- make current_test static - use current_test consistently - add TEST_WITH_FLAGS macro to contract parametrized tests definition Signed-off-by: Vasily Gorbik

s390/test_unwind: show tests as skipped if unsupported

2022-03-01T20:05:09Z

Signed-off-by: Vasily Gorbik