On 05/02/2021 08:11, Jan Beulich wrote: > On 04.02.2021 14:38, Jan Beulich wrote: >> Our linker capability check fails with the recent binutils release's ld: >> >> .../check.o:(.debug_aranges+0x6): relocation truncated to fit: R_X86_64_32 >> against `.debug_info' >> .../check.o:(.debug_info+0x6): relocation truncated to fit: R_X86_64_32 >> against `.debug_abbrev' >> .../check.o:(.debug_info+0xc): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+76 >> .../check.o:(.debug_info+0x11): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+d >> .../check.o:(.debug_info+0x15): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+2b >> .../check.o:(.debug_info+0x29): relocation truncated to fit: R_X86_64_32 >> against `.debug_line' >> .../check.o:(.debug_info+0x30): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+19 >> .../check.o:(.debug_info+0x37): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+71 >> .../check.o:(.debug_info+0x3e): relocation truncated to fit: R_X86_64_32 >> against `.debug_str' >> .../check.o:(.debug_info+0x45): relocation truncated to fit: R_X86_64_32 >> against `.debug_str'+5e >> .../check.o:(.debug_info+0x4c): additional relocation overflows omitted from >> the output >> >> Tell the linker to strip debug info as a workaround. Oddly enough debug >> info has been getting stripped when linking the actual xen.efi, without >> me being able to tell why this would be. > I've changed this to > > "Tell the linker to strip debug info as a workaround. Debug info has been > getting stripped already anyway when linking the actual xen.efi." > > as I noticed I did look for -S only yesterday, while we have > > EFI_LDFLAGS += --image-base=$(1) --stack=0,0 --heap=0,0 --strip-debug
So, in terms of the bugfix, Acked-by: Andrew Cooper <andrew.coop...@citrix.com> However, we ought be keeping the debug symbols for xen-syms.efi (or equiv) seeing as there is logic included here which isn't in the regular xen-syms. ~Andrew
