From patchwork Sun Feb 9 15:34:09 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dirk Gouders X-Patchwork-Id: 3613651 Return-Path: X-Original-To: patchwork-linux-kbuild@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 38D1F9F2F6 for ; Sun, 9 Feb 2014 15:49:32 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 1D6FF20172 for ; Sun, 9 Feb 2014 15:49:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 78E432016C for ; Sun, 9 Feb 2014 15:49:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751525AbaBIPtY (ORCPT ); Sun, 9 Feb 2014 10:49:24 -0500 Received: from services.gouders.net ([141.101.32.176]:35523 "EHLO services.gouders.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751424AbaBIPtX (ORCPT ); Sun, 9 Feb 2014 10:49:23 -0500 X-Greylist: delayed 891 seconds by postgrey-1.27 at vger.kernel.org; Sun, 09 Feb 2014 10:49:22 EST Received: from localhost (ip-80-226-24-4.vodafone-net.de [80.226.24.4]) (authenticated bits=0) by services.gouders.net (8.14.7/8.14.7) with ESMTP id s19FOr2N025410 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Sun, 9 Feb 2014 16:24:54 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gouders.net; s=gnet; t=1391959495; bh=9a2niCfJyEx+QfJahHS6/8SRFM1T31kZvrCUIAB6UV0=; h=From:To:Cc:Subject:In-Reply-To:References:Date; b=ZhjdZRmP8crFTyS4kVQ/MAf+tSTR+dNpR+drrJev7pgLuD1xmbG3RsPPamUHJkanr +zTtpc7PGEj5D89fAQoWgg+b1J/A9UOM6Ewpp8OY3BY+0KXdkU+fRsxscK55Xdskgx 13VJbue0rkgZ92q+mH1Veeca4FrOAUoMfCJ3H9Ak= From: Dirk Gouders To: Andi Kleen Cc: linux-kernel@vger.kernel.org, mmarek@suse.cz, linux-kbuild@vger.kernel.org, torvalds@linux-foundation.org Subject: Re: [PATCH] kbuild: Support split debug info In-Reply-To: <1391900703-18163-1-git-send-email-ak@linux.intel.com> (Andi Kleen's message of "Sun, 9 Feb 2014 00:05:03 +0100") References: <1391900703-18163-1-git-send-email-ak@linux.intel.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) Date: Sun, 09 Feb 2014 16:34:09 +0100 Message-ID: MIME-Version: 1.0 Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00, DKIM_ADSP_DISCARD, DKIM_SIGNED, RCVD_IN_DNSWL_HI, RCVD_IN_SORBS_WEB, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Andi Kleen writes: > This is an alternative approach to lower the overhead of debug info > (as we discussed a few days ago) > > gcc 4.7+ and newer binutils have a new "split debug info" debug info > model where the debug info is only written once into central ".dwo" files. > > This avoids having to copy it around multiple times, from the object > files to the final executable. It also lowers the disk space > requirement. In addition it defaults to compressed debug data. > > More details here: http://gcc.gnu.org/wiki/DebugFission > > This patch adds a new option to enable it. It has to be an option, > because it'll undoubtedly break everyone's debuginfo packaging scheme. > gdb/objdump/etc. all still work, if you have new enough versions. > > I don't see big compile wins (maybe a second or two faster or so), but the > object dirs with debuginfo get significantly smaller. My standard kernel > config (slightly bigger than defconfig) goes from 2.9G disk space > to 1.1G objdir (without debuginfo reduced). I presume if you are IO limited > it will also compile faster. > > Only problem I've seen so far is that it doesn't play well with ccache. Hi Andi, there are some rather superficial things I noticed with this patch. First, I was wondering if it would probably be a good idea to add the More-Details-Link to the Kconfig helpt text. I also noticed that the commit message references gcc 4.7+ whereas in the help text it is gcc 4.8+. Then, I tested the patch and got a trailing whitespace warning when applying the patch: "Applying: kbuild: Support split debug info /usr/src/torvalds/.git/rebase-apply/patch:45: trailing whitespace. in the object files and the vmlinux, as it only needs to warning: 1 line adds whitespace errors." I also noticed that `make clean' does not remove all .dwo files. Probably there are more elegant ways to do that, but this at least works here (I also tried to use globstar patterns in scripts/Makefile.clean but felt that wasn't a good idea): Dirk > Signed-off-by: Andi Kleen > --- > Makefile | 6 ++++++ > lib/Kconfig.debug | 13 +++++++++++++ > 2 files changed, 19 insertions(+) > > diff --git a/Makefile b/Makefile > index 606ef7c..2f3d9d1 100644 > --- a/Makefile > +++ b/Makefile > @@ -638,7 +638,11 @@ endif > endif > > ifdef CONFIG_DEBUG_INFO > +ifdef CONFIG_DEBUG_INFO_SPLIT > +KBUILD_CFLAGS += $(call cc-option, -gsplit-dwarf, -g) > +else > KBUILD_CFLAGS += -g > +endif > KBUILD_AFLAGS += -Wa,--gdwarf-2 > endif > > @@ -1072,6 +1076,8 @@ MRPROPER_FILES += .config .config.old .version .old_version $(version_h) \ > extra_certificates signing_key.x509.keyid \ > signing_key.x509.signer > > +CLEAN_FILES += .*.dwo > + > # clean - Delete most, but leave enough to build external modules > # > clean: rm-dirs := $(CLEAN_DIRS) > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug > index a48abea..367a6ba 100644 > --- a/lib/Kconfig.debug > +++ b/lib/Kconfig.debug > @@ -143,6 +143,19 @@ config DEBUG_INFO_REDUCED > DEBUG_INFO build and compile times are reduced too. > Only works with newer gcc versions. > > +config DEBUG_INFO_SPLIT > + bool "Produce split debuginfo in .dwo files" > + depends on DEBUG_INFO > + help > + Generate debug info into separate .dwo files. This can be > + faster for building than including the debug information directly > + in the object files and the vmlinux, as it only needs to > + be stored once to disk, not multiple times in object files. > + Requires recent gcc (4.8+) and recent gdb/binutils. > + Any tool that packages or reads debug information would need > + to know about the .dwo files and include them. > + Currently incompatible with ccache. > + > config ENABLE_WARN_DEPRECATED > bool "Enable __deprecated logic" > default y --- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/Makefile b/Makefile index 2f3d9d1..3a41147 100644 --- a/Makefile +++ b/Makefile @@ -1076,8 +1076,6 @@ MRPROPER_FILES += .config .config.old .version .old_version $(version_h) \ extra_certificates signing_key.x509.keyid \ signing_key.x509.signer -CLEAN_FILES += .*.dwo - # clean - Delete most, but leave enough to build external modules # clean: rm-dirs := $(CLEAN_DIRS) @@ -1324,6 +1322,7 @@ clean: $(clean-dirs) @find $(if $(KBUILD_EXTMOD), $(KBUILD_EXTMOD), .) $(RCS_FIND_IGNORE) \ \( -name '*.[oas]' -o -name '*.ko' -o -name '.*.cmd' \ -o -name '*.ko.*' \ + -o -name '*.dwo' \ -o -name '.*.d' -o -name '.*.tmp' -o -name '*.mod.c' \ -o -name '*.symtypes' -o -name 'modules.order' \ -o -name modules.builtin -o -name '.tmp_*.o.*' \ Finally, git doesn't know about the .dwo files, so I am wondering if .gitignore should be updated, too: diff --git a/.gitignore b/.gitignore index 7e9932e..cb43ed9 100644 --- a/.gitignore +++ b/.gitignore @@ -17,6 +17,7 @@ *.ko *.so *.so.dbg +*.dwo *.mod.c *.i *.lst