ARM: unwind: track location of LR value in stack frame
authorArd Biesheuvel <ardb@kernel.org>
Mon, 24 Jan 2022 15:49:09 +0000 (16:49 +0100)
committerArd Biesheuvel <ardb@kernel.org>
Wed, 9 Feb 2022 08:13:43 +0000 (09:13 +0100)
commit538b9265c063f081ca6b1228d242575a1db60711
treebefdd399f7bf56691bf58e7fb09bbd3107cddb46
parent953f534a7ed6b725d4f101d2949393acc9262880
ARM: unwind: track location of LR value in stack frame

The ftrace graph tracer needs to override the return address of an
instrumented function, in order to install a hook that gets invoked when
the function returns again.

Currently, we only support this when building for ARM using GCC with
frame pointers, as in this case, it is guaranteed that the function will
reload LR from [FP, #-4] in all cases, and we can simply pass that
address to the ftrace code.

In order to support this for configurations that rely on the EABI
unwinder, such as Thumb2 builds, make the unwinder keep track of the
address from which LR was unwound, permitting ftrace to make use of this
in a subsequent patch.

Drop the call to is_kernel_text_address(), which is problematic in terms
of ftrace recursion, given that it may be instrumented itself. The call
is redundant anyway, as no unwind directives will be found unless the PC
points to memory that is known to contain executable code.

Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
arch/arm/include/asm/stacktrace.h
arch/arm/kernel/Makefile
arch/arm/kernel/unwind.c