Unverified Commit c7a6065e authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!8238 arm: unwinder: Fix pc off-by-one in arm unwinder 

Merge Pull Request from: @ci-robot 
 
PR sync from: Chen Zhongjin <chenzhongjin@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/GDMX2FGMIYSVOI3MGEWIJUCJON7FR3RV/ 
 
https://gitee.com/openeuler/kernel/issues/I9T4EM 
 
Link:https://gitee.com/openeuler/kernel/pulls/8238

 

Reviewed-by: default avatarXu Kuohai <xukuohai@huawei.com>
Reviewed-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
Signed-off-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
parents dc3c1ebc 578fc9a6

arch/arm/include/asm/stacktrace.h

+1 −0
Original line number Diff line number Diff line
@@ -13,6 +13,7 @@ struct stackframe { 
	unsigned long sp;

	unsigned long lr;

	unsigned long pc;

	bool ex_frame;

};



static __always_inline

arch/arm/kernel/process.c

+1 −0
Original line number Diff line number Diff line
@@ -294,6 +294,7 @@ unsigned long get_wchan(struct task_struct *p) 
	frame.lr = 0;			/* recovered from the stack */

	frame.pc = thread_saved_pc(p);

	stack_page = (unsigned long)task_stack_page(p);

	frame.ex_frame = true;

	do {

		if (frame.sp < stack_page ||

		    frame.sp >= stack_page + THREAD_SIZE ||

arch/arm/kernel/stacktrace.c

+1 −0
Original line number Diff line number Diff line
@@ -73,6 +73,7 @@ int notrace unwind_frame(struct stackframe *frame) 
void notrace walk_stackframe(struct stackframe *frame,

		     int (*fn)(struct stackframe *, void *), void *data)

{

	frame->ex_frame = true;

	while (1) {

		int ret;

arch/arm/kernel/unwind.c

+4 −1
Original line number Diff line number Diff line
@@ -35,6 +35,7 @@ 
#include <asm/stacktrace.h>

#include <asm/traps.h>

#include <asm/unwind.h>

#include <asm/sections.h>



/* Dummy functions to avoid linker complaints */

void __aeabi_unwind_cpp_pr0(void)
@@ -415,7 +416,7 @@ int unwind_frame(struct stackframe *frame) 
	if (!kernel_text_address(frame->pc))

		return -URC_FAILURE;



	idx = unwind_find_idx(frame->pc);

	idx = unwind_find_idx(frame->ex_frame ? frame->pc : frame->pc - 1);

	if (!idx) {

		pr_warn("unwind: Index not found %08lx\n", frame->pc);

		return -URC_FAILURE;
@@ -478,6 +479,7 @@ int unwind_frame(struct stackframe *frame) 
	frame->sp = ctrl.vrs[SP];

	frame->lr = ctrl.vrs[LR];

	frame->pc = ctrl.vrs[PC];

	frame->ex_frame = in_entry_text(frame->pc);



	return URC_OK;

}
@@ -513,6 +515,7 @@ void unwind_backtrace(struct pt_regs *regs, struct task_struct *tsk, 
		frame.lr = 0;

		frame.pc = thread_saved_pc(tsk);

	}

	frame.ex_frame = true;



	while (1) {

		int urc;