Re: [PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Richard Henderson
On 3/12/24 04:08, Ilias Apalodimas wrote: - .bss_start (NOLOAD) : { - . = ALIGN(8); This alignment got lost. - KEEP(*(.__bss_start)); - } >.sdram - - .bss (NOLOAD) : { + .bss : { If it is required, the best replacement would be here on the

Re: [PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Ilias Apalodimas
Hi Richard, Thanks for taking a look On Tue, 12 Mar 2024 at 17:51, Richard Henderson wrote: > > On 3/12/24 04:08, Ilias Apalodimas wrote: > > - .bss_start (NOLOAD) : { > > - . = ALIGN(8); > > This alignment got lost. > > > - KEEP(*(.__bss_start)); > > - } >.sdram

[PATCH 2/7 v2] arm: clean up v7 and v8 linker scripts for bss_start/end

2024-03-12 Thread Ilias Apalodimas
commit 3ebd1cbc49f0 ("arm: make __bss_start and __bss_end__ compiler-generated") and commit f84a7b8f54db ("ARM: Fix __bss_start and __bss_end in linker scripts") were moving the bss_start/end on c generated variables that were injected in their own sections. The reason was that we needed relative r