Hi Heinrich, On Sat, 29 Aug 2020 at 02:31, Heinrich Schuchardt <[email protected]> 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
