[<prev] [next>] [day] [month] [year] [list]
Message-ID: <49E3E944.2040505@oracle.com>
Date: Tue, 14 Apr 2009 09:39:16 +0800
From: Wenji Huang <wenji.huang@...cle.com>
To: linux-kbuild@...r.kernel.org
CC: tytso@....EDU, Elena Zannoni <elena.zannoni@...cle.com>,
Roland McGrath <roland@...hat.com>,
LKML <linux-kernel@...r.kernel.org>
Subject: [Fwd: [PATCH 1/1] Separate the debuginfo out from kbuild]
Add Roland to cc
-------- Original Message --------
Subject: [PATCH 1/1] Separate the debuginfo out from kbuild
Date: Wed, 8 Apr 2009 22:06:21 -0400
From: Wenji Huang <wenji.huang@...cle.com>
To: linux-kbuild@...r.kernel.org
CC: linux-kernel@...r.kernel.org, tytso@....EDU,
elena.zannoni@...cle.com, Wenji Huang <wenji.huang@...cle.com>
This patch will add "make debug_info" to split debuginfo files. The
separate debug files are under debug directory of building tree.
It is based on two considerations:
* The debuginfo extracting machinery could be integrated into the
kbuild system. This way it's not just done via RPM macros. So
generating debuginfo RPMS or something alike can be simplified
and standardized, also distribution independent.
* Reduce the size of current file which mixes executable and
debug sections. For example, 7% off for vmlinux.
Signed-off-by: Wenji Huang <wenji.huang@...cle.com>
---
Makefile | 14 ++++++++++++++
scripts/Makefile.modpost | 12 ++++++++++++
2 files changed, 26 insertions(+), 0 deletions(-)
diff --git a/Makefile b/Makefile
index e5ad5fd..0df7581 100644
--- a/Makefile
+++ b/Makefile
@@ -832,6 +832,19 @@ define rule_vmlinux-modpost
$(Q)echo 'cmd_$@ := $(cmd_vmlinux-modpost)' > $(dot-target).cmd
endef
+quiet_cmd_vmlinux_debug = GEN $<.debug
+ cmd_vmlinux_debug = mkdir -p debug; \
+ $(OBJCOPY) --only-keep-debug \
+ $< debug/$<.debug
+targets += vmlinux.debug
+
+PHONY += debug_info
+debug_info: vmlinux FORCE
+ifdef CONFIG_DEBUG_INFO
+ $(call if_changed,vmlinux_debug)
+ $(Q)$(MAKE) -f $(srctree)/scripts/Makefile.modpost $@
+endif
+
# vmlinux image - including updated kernel symbols
vmlinux: $(vmlinux-lds) $(vmlinux-init) $(vmlinux-main) vmlinux.o
$(kallsyms.o) FORCE
ifdef CONFIG_HEADERS_CHECK
@@ -1278,6 +1291,7 @@ help:
@echo ' all - Build all targets marked with [*]'
@echo '* vmlinux - Build the bare kernel'
@echo '* modules - Build all modules'
+ @echo ' debug_info - Extract debug info to debug directory'
@echo ' modules_install - Install all modules to INSTALL_MOD_PATH
(default: /)'
@echo ' firmware_install- Install all firmware to INSTALL_FW_PATH'
@echo ' (default: $$(INSTALL_MOD_PATH)/lib/firmware)'
diff --git a/scripts/Makefile.modpost b/scripts/Makefile.modpost
index f4053dc..c120261 100644
--- a/scripts/Makefile.modpost
+++ b/scripts/Makefile.modpost
@@ -137,6 +137,18 @@ $(modules): %.ko :%.o %.mod.o FORCE
targets += $(modules)
+modules-debug := $(modules:.ko=.ko.debug)
+quiet_cmd_debug_ko = GEN $@
+ cmd_debug_ko = mkdir -p debug/$(dir $@); \
+ $(OBJCOPY) --only-keep-debug $< debug/$@
+targets += $(modules-debug)
+
+debug_info: $(modules-debug) FORCE
+
+ifdef CONFIG_DEBUG_INFO
+$(modules-debug): $(modules) FORCE
+ $(call if_changed,debug_ko)
+endif
# Add FORCE to the prequisites of a target to force it to be always
rebuilt.
#
---------------------------------------------------------------------------
--
1.5.6
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists