On Sat, 3 Dec 2016 22:06:28 +1100
Michael Ellerman <m...@ellerman.id.au> wrote:
> From: Nicholas Piggin <npig...@gmail.com>
>
> The boot wrapper performs its own relocations and does not require
> PT_INTERP segment. However currently we don't tell the linker that.
>
> Prior to binutils 2.28 that works OK. But since binutils commit
> 1a9ccd70f9a7 ("Fix the linker so that it will not silently generate ELF
> binaries with invalid program headers. Fix readelf to report such
> invalid binaries.") binutils tries to create a program header segment
> due to PT_INTERP, and the link fails because there is no space for it:
>
> ld: arch/powerpc/boot/zImage.pseries: Not enough room for program headers,
> try linking with -N
> ld: final link failed: Bad value
>
> So tell the linker not to do that, by passing --no-dynamic-linker.
>
> Reported-by: Anton Blanchard <an...@samba.org>
> Signed-off-by: Nicholas Piggin <npig...@gmail.com>
> [mpe: Drop dependency on ld-version.sh and massage change log]
Thanks for fixing that up, it looks good.
Thanks,
Nick
