Hello Sascha, On 5/21/26 10:06 AM, Sascha Hauer wrote: > The very first function in the barebox binary starts at __image_start > aka _stext, so resolving its name with %pS results in "__image_start" > instead of the function name. This can be confusing, so add a small > offset to the first function so that its name resolves properly. > > Signed-off-by: Sascha Hauer <[email protected]>
Reviewed-by: Ahmad Fatoum <[email protected]> This patch confused me a bit at first, so I would suggest adding following clarification to the commit message: """ This is safe to do for barebox proper, because it's always an ELF on ARM64 and thus there is no special significance to code at the start of the text segment. """ Thanks, Ahmad > --- > arch/arm/lib64/barebox.lds.S | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/lib64/barebox.lds.S b/arch/arm/lib64/barebox.lds.S > index bd76a0ca96..1dddd6d1a9 100644 > --- a/arch/arm/lib64/barebox.lds.S > +++ b/arch/arm/lib64/barebox.lds.S > @@ -26,6 +26,7 @@ SECTIONS > .text : > { > _stext = .; > + . += 16; > *(.text*) > } :text > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
