[RFC PATCH 07/10] arm: efi_loader: discard hash, unwind information

Ilias Apalodimas ilias.apalodimas at linaro.org
Mon May 22 22:28:01 CEST 2023

Hi Sam,

On Mon, 22 May 2023 at 22:25, Sam Edwards <cfsworks at gmail.com> wrote:
> Hi Ilias,
> On 5/22/23 01:00, Ilias Apalodimas wrote:
> > The reason we end up with both hash and gnu.hash is because the hash
> > style is set to 'both'.  Should we perhaps use (and strip) only one of
> > them?
> If we do keep one, it should probably be .hash -- see commit b02bfc4dfc.
> I admit I'm completely mystified as to why we need the hash tables at
> all. The ELF spec says those are just for the dynamic linker, but
> neither the EFI code nor the self-relocating thunk require it, and I
> don't know of any target where the u-boot ELF itself is the shipped
> binary.

Me neither

> For all I know, there never was a need to include .hash and
> Albert's commit fixed whatever problem he was facing only accidentally.
> Do you have any insights?

Unfortunately not.  I just started looking up the linker scripts myself.

> LLD's --hash-style option doesn't appear to have a "none" option or I'd
> just be making use of that here.

Indeed.  I am fine with the patch regardless, switching the makefile
to only produce one of them is a nit anyway, since we'll eventually
get rid of them

> Cheers,
> Sam

More information about the U-Boot mailing list