mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-11-01 00:48:50 +00:00
6f9885a36c
There have been some reports of "bad bp value" warnings printed by the
frame pointer unwinder:
WARNING: kernel stack regs at 000000005bac7112 in sh:1014 has bad 'bp' value 0000000000000000
This warning happens when unwinding from an interrupt in
ret_from_fork(). If entry code gets interrupted, the state of the
frame pointer (rbp) may be undefined, which can confuse the unwinder,
resulting in warnings like the above.
There's an in_entry_code() check which normally silences such
warnings for entry code. But in this case, ret_from_fork() is getting
interrupted. It recently got moved out of .entry.text, so the
in_entry_code() check no longer works.
It could be moved back into .entry.text, but that would break the
noinstr validation because of the call to schedule_tail().
Instead, initialize each new task's RBP to point to the task's entry
regs via an encoded frame pointer. That will allow the unwinder to
reach the end of the stack gracefully.
Fixes: b9f6976bfb ("x86/entry/64: Move non entry code into .text section")
Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Reported-by: Logan Gunthorpe <logang@deltatee.com>
Signed-off-by: Josh Poimboeuf <jpoimboe@redhat.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/f366bbf5a8d02e2318ee312f738112d0af74d16f.1600103007.git.jpoimboe@redhat.com
113 lines
2.5 KiB
C
113 lines
2.5 KiB
C
/* SPDX-License-Identifier: GPL-2.0 */
|
|
#ifndef _ASM_X86_FRAME_H
|
|
#define _ASM_X86_FRAME_H
|
|
|
|
#include <asm/asm.h>
|
|
|
|
/*
|
|
* These are stack frame creation macros. They should be used by every
|
|
* callable non-leaf asm function to make kernel stack traces more reliable.
|
|
*/
|
|
|
|
#ifdef CONFIG_FRAME_POINTER
|
|
|
|
#ifdef __ASSEMBLY__
|
|
|
|
.macro FRAME_BEGIN
|
|
push %_ASM_BP
|
|
_ASM_MOV %_ASM_SP, %_ASM_BP
|
|
.endm
|
|
|
|
.macro FRAME_END
|
|
pop %_ASM_BP
|
|
.endm
|
|
|
|
#ifdef CONFIG_X86_64
|
|
/*
|
|
* This is a sneaky trick to help the unwinder find pt_regs on the stack. The
|
|
* frame pointer is replaced with an encoded pointer to pt_regs. The encoding
|
|
* is just setting the LSB, which makes it an invalid stack address and is also
|
|
* a signal to the unwinder that it's a pt_regs pointer in disguise.
|
|
*
|
|
* NOTE: This macro must be used *after* PUSH_AND_CLEAR_REGS because it corrupts
|
|
* the original rbp.
|
|
*/
|
|
.macro ENCODE_FRAME_POINTER ptregs_offset=0
|
|
leaq 1+\ptregs_offset(%rsp), %rbp
|
|
.endm
|
|
#else /* !CONFIG_X86_64 */
|
|
/*
|
|
* This is a sneaky trick to help the unwinder find pt_regs on the stack. The
|
|
* frame pointer is replaced with an encoded pointer to pt_regs. The encoding
|
|
* is just clearing the MSB, which makes it an invalid stack address and is also
|
|
* a signal to the unwinder that it's a pt_regs pointer in disguise.
|
|
*
|
|
* NOTE: This macro must be used *after* SAVE_ALL because it corrupts the
|
|
* original ebp.
|
|
*/
|
|
.macro ENCODE_FRAME_POINTER
|
|
mov %esp, %ebp
|
|
andl $0x7fffffff, %ebp
|
|
.endm
|
|
#endif /* CONFIG_X86_64 */
|
|
|
|
#else /* !__ASSEMBLY__ */
|
|
|
|
#define FRAME_BEGIN \
|
|
"push %" _ASM_BP "\n" \
|
|
_ASM_MOV "%" _ASM_SP ", %" _ASM_BP "\n"
|
|
|
|
#define FRAME_END "pop %" _ASM_BP "\n"
|
|
|
|
#ifdef CONFIG_X86_64
|
|
|
|
#define ENCODE_FRAME_POINTER \
|
|
"lea 1(%rsp), %rbp\n\t"
|
|
|
|
static inline unsigned long encode_frame_pointer(struct pt_regs *regs)
|
|
{
|
|
return (unsigned long)regs + 1;
|
|
}
|
|
|
|
#else /* !CONFIG_X86_64 */
|
|
|
|
#define ENCODE_FRAME_POINTER \
|
|
"movl %esp, %ebp\n\t" \
|
|
"andl $0x7fffffff, %ebp\n\t"
|
|
|
|
static inline unsigned long encode_frame_pointer(struct pt_regs *regs)
|
|
{
|
|
return (unsigned long)regs & 0x7fffffff;
|
|
}
|
|
|
|
#endif /* CONFIG_X86_64 */
|
|
|
|
#endif /* __ASSEMBLY__ */
|
|
|
|
#define FRAME_OFFSET __ASM_SEL(4, 8)
|
|
|
|
#else /* !CONFIG_FRAME_POINTER */
|
|
|
|
#ifdef __ASSEMBLY__
|
|
|
|
.macro ENCODE_FRAME_POINTER ptregs_offset=0
|
|
.endm
|
|
|
|
#else /* !__ASSEMBLY */
|
|
|
|
#define ENCODE_FRAME_POINTER
|
|
|
|
static inline unsigned long encode_frame_pointer(struct pt_regs *regs)
|
|
{
|
|
return 0;
|
|
}
|
|
|
|
#endif
|
|
|
|
#define FRAME_BEGIN
|
|
#define FRAME_END
|
|
#define FRAME_OFFSET 0
|
|
|
|
#endif /* CONFIG_FRAME_POINTER */
|
|
|
|
#endif /* _ASM_X86_FRAME_H */
|