[build] Fix the REQUIRE_SYMBOL mechanism
authorMichael Brown <mcb30@ipxe.org>
Wed, 4 Mar 2015 18:48:19 +0000 (18:48 +0000)
committerMichael Brown <mcb30@ipxe.org>
Thu, 5 Mar 2015 00:59:38 +0000 (00:59 +0000)
commitfbc4ba4b4ed13cc86cb8fdea0bac6c3be0164ed5
treea0c504e297806aa88e89a04e694333a7c5172757
parent86ae6e6c1836e43993a14db278398fc54e5419bd
[build] Fix the REQUIRE_SYMBOL mechanism

At some point in the past few years, binutils became more aggressive
at removing unused symbols.  To function as a symbol requirement, a
relocation record must now be in a section marked with @progbits and
must not be in a section which gets discarded during the link (either
via --gc-sections or via /DISCARD/).

Update REQUIRE_SYMBOL() to generate relocation records meeting these
criteria.  To minimise the impact upon the final binary size, we use
existing symbols (specified via the REQUIRING_SYMBOL() macro) as the
relocation targets where possible.  We use R_386_NONE or R_X86_64_NONE
relocation types to prevent any actual unwanted relocation taking
place.  Where no suitable symbol exists for REQUIRING_SYMBOL() (such
as in config.c), the macro PROVIDE_REQUIRING_SYMBOL() can be used to
generate a one-byte-long symbol to act as the relocation target.

If there are versions of binutils for which this approach fails, then
the fallback will probably involve killing off REQUEST_SYMBOL(),
redefining REQUIRE_SYMBOL() to use the current definition of
REQUEST_SYMBOL(), and postprocessing the linked ELF file with
something along the lines of "nm -u | wc -l" to check that there are
no undefined symbols remaining.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
37 files changed:
src/arch/i386/include/bits/compiler.h
src/arch/i386/interface/pxe/pxe_call.c
src/arch/i386/prefix/kkkpxeprefix.S
src/arch/i386/prefix/romprefix.S
src/arch/i386/transitions/librm_test.c
src/arch/x86/drivers/hyperv/hyperv.c
src/arch/x86/drivers/xen/hvm.c
src/arch/x86_64/include/bits/compiler.h
src/config/config.c
src/config/config_ethernet.c
src/config/config_fc.c
src/config/config_infiniband.c
src/config/config_net80211.c
src/config/config_romprefix.c
src/config/config_route.c
src/config/config_usb.c
src/crypto/x509.c
src/drivers/bus/usb.c
src/drivers/net/rtl818x/rtl8180.c
src/drivers/net/rtl818x/rtl8185.c
src/drivers/net/vxge/vxge.c
src/hci/commands/image_trust_cmd.c
src/include/compiler.h
src/include/nic.h
src/net/80211/net80211.c
src/net/80211/wpa.c
src/net/ethernet.c
src/net/fc.c
src/net/infiniband.c
src/net/ipv4.c
src/net/ipv6.c
src/tests/cms_test.c
src/tests/ocsp_test.c
src/tests/tests.c
src/tests/x509_test.c
src/usr/route.c
src/util/elf2efi.c