Hi,

it could be a silly question, but better to ask...

> +     if (regs)
> +             start_backtrace(&frame, regs->regs[29], regs->pc);
> +     else
> +             start_backtrace(&frame, thread_saved_fp(task),
> +                             thread_saved_pc(task));

Would this also work for task == current? Given that the original code had

> -             start_backtrace(&frame,
> -                             (unsigned long)__builtin_frame_address(0),
> -                             (unsigned long)__save_stack_trace);

for the case, which seems correct (but I don't know much about arm64 arch 
in the kernel).

Otherwise, I did not spot anything suspicious or wrong.

Regards
Miroslav

Reply via email to