On Tue, Jul 29, 2025 at 06:07:21PM +0200, Casey Connolly wrote: > Implement support for printing stack backtraces on ARM64, make > framepointer support configurable by the build system with > CONFIG_FRAMEPOINTER and teach U-Boot to walk the framepointers and > unwind the stack when an exception occurs. Also show all 64 bits of the > exception syndrom register (ESR). > > Additionally, a new global unwind_stack() function is added, this can > be called from anywhere to print a backtrace which can be useful when > debugging certain problems. It is now used during panic() when available > to provide more useful information for debugging. > > A new printf format specifier is added: %pS will print a symbol name > from an address, this is particularly useful for function pointers
This fails on sandbox because the prototype for unwind_stack is #if
guarded but usage is now if (IS_ENABLED(...)) guarded. The next problem
however is that the vsprintf.c changes aren't safe for non-ARM64:
lib/vsprintf.c: In function 'pointer':
lib/vsprintf.c:527:43: warning: cast from pointer to integer of different size
[-Wpointer-to-int-cast]
527 | return ptr_symbol((phys_addr_t)ptr, buf, end,
|
Handling for %S will need to be #if guarded I believe. Please push the
next iteration through CI, thanks.
--
Tom
signature.asc
Description: PGP signature
