[Xen-devel] [PATCH v3] xen: add DEBUG_INFO Kconfig symbol

[Olaf Hering <olaf@xxxxxxxxx>]

Creating debug info during build is not strictly required at runtime.
Make it optional by introducing a new Kconfig knob "DEBUG_INFO".
This slightly reduces build time and diskusage, if disabled.

Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
---
 xen/Kconfig.debug | 7 +++++++
 xen/Rules.mk      | 3 ++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/xen/Kconfig.debug b/xen/Kconfig.debug
index 380c4e8d75..4d5d7f87cb 100644
--- a/xen/Kconfig.debug
+++ b/xen/Kconfig.debug
@@ -20,6 +20,13 @@ config CRASH_DEBUG
          If you want to attach gdb to Xen to debug Xen if it crashes
          then say Y.
 
+config DEBUG_INFO
+       bool "Compile Xen with debug info"
+       default y
+       ---help---
+         If you say Y here the resulting Xen will include debugging info
+         resulting in a larger binary image.
+
 config FRAME_POINTER
        bool "Compile Xen with frame pointers"
        default DEBUG
diff --git a/xen/Rules.mk b/xen/Rules.mk
index 47c954425d..41eb7fa0d4 100644
--- a/xen/Rules.mk
+++ b/xen/Rules.mk
@@ -55,7 +55,7 @@ endif
 
 CFLAGS += -nostdinc -fno-builtin -fno-common
 CFLAGS += -Werror -Wredundant-decls -Wno-pointer-arith
-CFLAGS += -pipe -g -D__XEN__ -include $(BASEDIR)/include/xen/config.h
+CFLAGS += -pipe -D__XEN__ -include $(BASEDIR)/include/xen/config.h
 CFLAGS += '-D__OBJECT_FILE__="$@"'
 
 ifneq ($(clang),y)
@@ -65,6 +65,7 @@ ifneq ($(clang),y)
 CFLAGS += -Wa,--strip-local-absolute
 endif
 
+CFLAGS-$(CONFIG_DEBUG_INFO) += -g
 ifneq ($(max_phys_irqs),)
 CFLAGS-y                += -DMAX_PHYS_IRQS=$(max_phys_irqs)
 endif

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel