On Tue, Dec 04, 2018 at 11:24:28AM +1030, Joel Stanley wrote: > Alan Modra <amo...@gmail.com> explains: > > > Likely you could discard .interp > and .dynstr too, and .dynsym when > > !CONFIG_PPC32. > > Discarding of interp and dynstr happened in a previous patch. The dynsym > cleanup was a bit less straightforward, so it gets it's own patch.
> diff --git a/arch/powerpc/kernel/vmlinux.lds.S > b/arch/powerpc/kernel/vmlinux.lds.S > index 6570209b0671..2c93a420f456 100644 > --- a/arch/powerpc/kernel/vmlinux.lds.S > +++ b/arch/powerpc/kernel/vmlinux.lds.S > @@ -266,14 +266,13 @@ SECTIONS > } > #ifdef CONFIG_RELOCATABLE > . = ALIGN(8); > +#ifdef CONFIG_PPC32 > .dynsym : AT(ADDR(.dynsym) - LOAD_OFFSET) > { > -#ifdef CONFIG_PPC32 > __dynamic_symtab = .; > -#endif > *(.dynsym) > } > - .dynstr : AT(ADDR(.dynstr) - LOAD_OFFSET) { *(.dynstr) } So this last line belongs in the previous patch then, right? Segher