Dear Kevin,
On 10/25/16 18:14, Kevin O'Connor wrote:
On Tue, Oct 25, 2016 at 06:08:08PM +0200, Paul Menzel wrote:
On 10/25/16 17:27, Paul Menzel wrote:
On 10/25/16 17:21, Kevin O'Connor wrote:
Does adding '-fno-pie' help?
Isn’t that already used due to `COMMONCFLAGS += $(call
cc-option,$(CC),-nopie,)` in the `Makefile`?
I’ll try it anyway.
That also worked. Should I sent an updated patch?
Does the patch below work?
-Kevin
commit 589fe2803b110014a84dee2056f1e2fec744d2e6
Author: Kevin O'Connor <ke...@koconnor.net>
Date: Tue Oct 25 12:09:41 2016 -0400
build: Add -fno-pie to the gcc flags when available
Signed-off-by: Kevin O'Connor <ke...@koconnor.net>
diff --git a/Makefile b/Makefile
index 1916ecf..3b94ee0 100644
--- a/Makefile
+++ b/Makefile
@@ -63,6 +63,7 @@ COMMONCFLAGS := -I$(OUT) -Isrc -Os -MD -g \
-freg-struct-return -ffreestanding -fno-delete-null-pointer-checks \
-ffunction-sections -fdata-sections -fno-common -fno-merge-constants
COMMONCFLAGS += $(call cc-option,$(CC),-nopie,)
+COMMONCFLAGS += $(call cc-option,$(CC),-fno-pie,)
COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector,)
COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector-all,)
COMMONCFLAGS += $(call cc-option,$(CC),-fstack-check=no,)
Yes, I was just about to send that.
Do you know, why SeaBIOS won’t build without it?
Kind regards,
Paul
_______________________________________________
SeaBIOS mailing list
SeaBIOS@seabios.org
https://www.coreboot.org/mailman/listinfo/seabios
