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

Sam Edwards cfsworks at gmail.com
Mon May 22 21:25:50 CEST 2023


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. 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?

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

Cheers,
Sam


More information about the U-Boot mailing list