On Sun, Dec 17, 2023 at 01:33:39AM +0100, Marek Vasut wrote: > Avoid allocating and loading the BSS section. > > $ aarch64-linux-gnu-objdump -Sh u-boot > > Before: > 10 .bss_start 00000000 00000000000f21d8 00000000000f21d8 001021d8 > 2**0 > CONTENTS, ALLOC, LOAD, DATA > 11 .bss 000068f8 00000000000f2200 00000000000f2200 001021d8 > 2**6 > ALLOC > 12 .bss_end 00000000 00000000000f8af8 00000000000f8af8 00108af8 > 2**0 > CONTENTS, ALLOC, LOAD, DATA > > After: > 10 .bss_start 00000000 00000000000bf990 00000000000bf990 001021e0 > 2**0 > CONTENTS > 11 .bss 000068e8 00000000000bf990 00000000000bf990 001021e0 > 2**4 > CONTENTS > 12 .bss_end 00000000 00000000000c6278 00000000000c6278 00108ac8 > 2**0 > CONTENTS > > Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org> > --- > Cc: Simon Glass <s...@chromium.org> > Cc: Tom Rini <tr...@konsulko.com> > --- > V2: Replicate arch/arm/cpu/u-boot.lds BSS part verbatim > --- > arch/arm/cpu/armv8/u-boot.lds | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-)
The point I was trying to make before was that I wasn't 100% confident that we need the linker scripts to match in that most of the relevant code is in foo_64.S and not foo.S. But from a very quick grep it does look similar enough that perhaps it was ported over directly enough that we should be doing this with the linker scripts here too. Hopefully throwing this in to -next soon enough will get any problems to fall out quickly if we're wrong. -- Tom
signature.asc
Description: PGP signature