[RFC PATCH v3 1/4] arm64: Introduce stack trace reliability checks in the unwinder

Josh Poimboeuf jpoimboe at redhat.com
Tue May 4 14:52:48 PDT 2021


On Mon, May 03, 2021 at 12:36:12PM -0500, madvenka at linux.microsoft.com wrote:
> @@ -44,6 +44,8 @@ int notrace unwind_frame(struct task_struct *tsk, struct stackframe *frame)
>  	unsigned long fp = frame->fp;
>  	struct stack_info info;
>  
> +	frame->reliable = true;
> +

Why set 'reliable' to true on every invocation of unwind_frame()?
Shouldn't it be remembered across frames?

Also, it looks like there are several error scenarios where it returns
-EINVAL but doesn't set 'reliable' to false.

-- 
Josh




More information about the linux-arm-kernel mailing list