]> xenbits.xensource.com Git - people/tklengyel/xen.git/commitdiff
Config.mk: correct PIE-related option(s) in EMBEDDED_EXTRA_CFLAGS
authorJan Beulich <jbeulich@suse.com>
Thu, 8 Sep 2022 07:25:26 +0000 (09:25 +0200)
committerJan Beulich <jbeulich@suse.com>
Thu, 8 Sep 2022 07:25:26 +0000 (09:25 +0200)
I haven't been able to find evidence of "-nopie" ever having been a
supported compiler option. The correct spelling is "-no-pie".
Furthermore like "-pie" this is an option which is solely passed to the
linker. The compiler only recognizes "-fpie" / "-fPIE" / "-fno-pie", and
it doesn't infer these options from "-pie" / "-no-pie".

Add the compiler recognized form, but for the possible case of the
variable also being used somewhere for linking keep the linker option as
well (with corrected spelling).

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Julien Grall <jgrall@amazon.com>
Config.mk

index e56844d964b354f1afb327ecdac568695b06914f..000e57e57b446c7cfaba941144582df5dbf36fb6 100644 (file)
--- a/Config.mk
+++ b/Config.mk
@@ -188,7 +188,7 @@ endif
 APPEND_LDFLAGS += $(foreach i, $(APPEND_LIB), -L$(i))
 APPEND_CFLAGS += $(foreach i, $(APPEND_INCLUDES), -I$(i))
 
-EMBEDDED_EXTRA_CFLAGS := -nopie -fno-stack-protector -fno-stack-protector-all
+EMBEDDED_EXTRA_CFLAGS := -fno-pie -no-pie -fno-stack-protector -fno-stack-protector-all
 EMBEDDED_EXTRA_CFLAGS += -fno-exceptions -fno-asynchronous-unwind-tables
 
 XEN_EXTFILES_URL ?= http://xenbits.xen.org/xen-extfiles