* Sections .dynamic and .dynsym in UEFI binaries
@ 2020-08-29 8:31 Heinrich Schuchardt
2020-08-29 17:19 ` Simon Glass
0 siblings, 1 reply; 2+ messages in thread
From: Heinrich Schuchardt @ 2020-08-29 8:31 UTC (permalink / raw)
To: u-boot
Hello Simon,
with commit c7ae3dfdccc1 ("efi: Add support for a hello world test
program") you added section .dynamic and .dynsym to generated UEFI binaries.
It is unclear to me why those sections should be needed. According to
https://docs.oracle.com/cd/E23824_01/html/819-0690/chapter6-42444.html
section .dynamic is only used for dynamic linking.
On RISC-V inclusion of .dynsym leeds to a file that is longer then the
value of SizeOfImage in the optional header.
On x86 and x86_64 we have a general problem that SizeOfImage does not
match the actual file size.
Best regards
Heinrich
^ permalink raw reply [flat|nested] 2+ messages in thread
* Sections .dynamic and .dynsym in UEFI binaries
2020-08-29 8:31 Sections .dynamic and .dynsym in UEFI binaries Heinrich Schuchardt
@ 2020-08-29 17:19 ` Simon Glass
0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2020-08-29 17:19 UTC (permalink / raw)
To: u-boot
Hi Heinrich,
On Sat, 29 Aug 2020 at 02:31, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> Hello Simon,
>
> with commit c7ae3dfdccc1 ("efi: Add support for a hello world test
> program") you added section .dynamic and .dynsym to generated UEFI binaries.
>
> It is unclear to me why those sections should be needed. According to
> https://docs.oracle.com/cd/E23824_01/html/819-0690/chapter6-42444.html
> section .dynamic is only used for dynamic linking.
>
> On RISC-V inclusion of .dynsym leeds to a file that is longer then the
> value of SizeOfImage in the optional header.
>
> On x86 and x86_64 we have a general problem that SizeOfImage does not
> match the actual file size.
I think this was brought over from arch/x86/config.mk
If it still works without it then it is OK to drop those two sections.
Regards,
Simon
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-08-29 17:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-29 8:31 Sections .dynamic and .dynsym in UEFI binaries Heinrich Schuchardt
2020-08-29 17:19 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.