diff options
author | Jessica Clarke <jrtc27@jrtc27.com> | 2021-08-05 20:25:45 +0100 |
---|---|---|
committer | Michael Roth <michael.roth@amd.com> | 2021-10-27 12:01:30 -0500 |
commit | db882c5c1871874d0b79dad380129479cb7b3c09 (patch) | |
tree | 88a0f66230afd6b348d5a4582af34e9940403023 /stubs/pci-host-piix.c | |
parent | 62a012b0f40658fe6a5e75a7d6373080dcbcc5dd (diff) |
Partially revert "build: -no-pie is no functional linker flag"
This partially reverts commit bbd2d5a8120771ec59b86a80a1f51884e0a26e53.
This commit was misguided and broke using --disable-pie on any distro
that enables PIE by default in their compiler driver, including Debian
and its derivatives. Whilst -no-pie is not a linker flag, it is a
compiler driver flag that ensures -pie is not automatically passed by it
to the linker. Without it, all compile_prog checks will fail as any code
built with the explicit -fno-pie will fail to link with the implicit
default -pie due to trying to use position-dependent relocations. The
only bug that needed fixing was LDFLAGS_NOPIE being used as a flag for
the linker itself in pc-bios/optionrom/Makefile.
Note this does not reinstate exporting LDFLAGS_NOPIE, as it is unused,
since the only previous use was the one that should not have existed. I
have also updated the comment for the -fno-pie and -no-pie checks to
reflect what they're actually needed for.
Fixes: bbd2d5a8120771ec59b86a80a1f51884e0a26e53
Cc: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-stable@nongnu.org
Signed-off-by: Jessica Clarke <jrtc27@jrtc27.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Michael Roth <michael.roth@amd.com>
Diffstat (limited to 'stubs/pci-host-piix.c')
0 files changed, 0 insertions, 0 deletions