On 11.12.2024 03:04, Volodymyr Babchuk wrote: > This patch is preparation for making stack protector > configurable. First step is to remove -fno-stack-protector flag from > EMBEDDED_EXTRA_CFLAGS so separate components (Hypervisor in this case) > can enable/disable this feature by themselves. > > Signed-off-by: Volodymyr Babchuk <volodymyr_babc...@epam.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> with ... > --- a/stubdom/Makefile > +++ b/stubdom/Makefile > @@ -14,6 +14,8 @@ export debug=y > # Moved from config/StdGNU.mk > CFLAGS += -O1 -fno-omit-frame-pointer > > +CFLAGS += -fno-stack-protector > + > ifeq (,$(findstring clean,$(MAKECMDGOALS))) > ifeq ($(wildcard $(MINI_OS)/Config.mk),) > $(error Please run 'make mini-os-dir' in top-level directory) > @@ -54,6 +56,7 @@ TARGET_CFLAGS += $(CFLAGS) > TARGET_CPPFLAGS += $(CPPFLAGS) > $(call cc-options-add,TARGET_CFLAGS,CC,$(EMBEDDED_EXTRA_CFLAGS)) > > + > # Do not use host headers and libs > GCC_INSTALL = $(shell LANG=C gcc -print-search-dirs | sed -n -e 's/install: > \(.*\)/\1/p') > TARGET_CPPFLAGS += -U __linux__ -U __FreeBSD__ -U __sun__ ... this stray (and wrong) hunk dropped. Can likely be done while committing. Jan
