Re: [PATCH v3 05/12] LoongArch: Fix EFI linking with -fdata-sections

On Wed, Jul 08, 2026 at 09:54:16AM -0400, Joe Lawrence wrote:
> On Tue, Jul 07, 2026 at 03:20:24PM +0800, George Guo wrote:
> > From: George Guo <[email protected]>
> > 
> > When building with func-fdata-sections, the .init.bss section gets split
> 
> nit: s/func-fdata-sections/-fdata-sections
> 
> > up into a bunch of .init.bss.<var> sections.  Make sure they get linked
> > into .init.data.
> > 
> > Co-developed-by: Kexin Liu <[email protected]>
> > Signed-off-by: Kexin Liu <[email protected]>
> > Signed-off-by: George Guo <[email protected]>
> > ---
> >  arch/loongarch/kernel/vmlinux.lds.S | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/loongarch/kernel/vmlinux.lds.S b/arch/loongarch/kernel/vmlinux.lds.S
> > index 840d944c2f73..dc5fa017db94 100644
> > --- a/arch/loongarch/kernel/vmlinux.lds.S
> > +++ b/arch/loongarch/kernel/vmlinux.lds.S
> > @@ -96,7 +96,7 @@ SECTIONS
> >  #endif
> >  
> >  	.init.bss : {
> > -		*(.init.bss)
> > +		*(.init.bss .init.bss.*)
> 
> Both arch/arm64/kernel/vmlinux.lds.S and
> arch/loongarch/kernel/vmlinux.lds.S contain a "/* from the EFI stub */"

^^ typo: I meant the riscv version as well

Also the commit message says, "Make sure they get linked into
.init.data," but should that read, "... into .init.bss"?

--
Joe