From patchwork Fri May 16 22:06:03 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andi Kleen X-Patchwork-Id: 4195161 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 B53F29F32A for ; Fri, 16 May 2014 22:07:45 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id DECDF20386 for ; Fri, 16 May 2014 22:07:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 02304203AA for ; Fri, 16 May 2014 22:07:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753421AbaEPWHH (ORCPT ); Fri, 16 May 2014 18:07:07 -0400 Received: from mga03.intel.com ([143.182.124.21]:4343 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752758AbaEPWHG (ORCPT ); Fri, 16 May 2014 18:07:06 -0400 Received: from azsmga001.ch.intel.com ([10.2.17.19]) by azsmga101.ch.intel.com with ESMTP; 16 May 2014 15:07:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,1070,1389772800"; d="scan'208";a="433238618" Received: from laut.jf.intel.com (HELO localhost) ([10.23.232.120]) by azsmga001.ch.intel.com with ESMTP; 16 May 2014 15:06:54 -0700 Received: by localhost (Postfix, from userid 1000) id 3F5BF124B07; Sat, 17 May 2014 00:06:10 +0200 (CEST) From: Andi Kleen To: mmarek@suse.cz Cc: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, Andi Kleen , Dirk Gouders Subject: [PATCH 1/2] kbuild: Support split debug info v3 Date: Sat, 17 May 2014 00:06:03 +0200 Message-Id: <1400277964-15762-1-git-send-email-andi@firstfloor.org> X-Mailer: git-send-email 1.8.5.2 Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable 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 From: Andi Kleen 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 requirements. 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) shrinks from 2.9G disk space to 1.1G objdir (with non reduced debuginfo). I presume if you are IO limited the compile time difference will be larger. Only problem I've seen so far is that it doesn't play well with older versions of ccache (apparently fixed, see https://bugzilla.samba.org/show_bug.cgi?id=10005) v2: various fixes from Dirk Gouders. Improve commit message slightly. v3: Fix clean rules and improve Kconfig slightly Cc: Dirk Gouders Cc: mmarek@suse.cz Signed-off-by: Andi Kleen --- .gitignore | 1 + Makefile | 5 +++++ lib/Kconfig.debug | 13 +++++++++++++ 3 files changed, 19 insertions(+) diff --git a/.gitignore b/.gitignore index 42fa0d5..d1ed0b5 100644 --- a/.gitignore +++ b/.gitignore @@ -33,6 +33,7 @@ modules.builtin *.lzo *.patch *.gcno +*.dwo # # Top-level generic files diff --git a/Makefile b/Makefile index 8a8440a..29665fe 100644 --- a/Makefile +++ b/Makefile @@ -670,7 +670,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 @@ -1356,6 +1360,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.*' \ diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 819ac51..d500487 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.7+) and recent gdb/binutils. + Any tool that packages or reads debug information would need + to know about the .dwo files and include them. + Incompatible with older versions of ccache. + config ENABLE_WARN_DEPRECATED bool "Enable __deprecated logic" default y