From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5B94CC43381 for ; Sun, 31 Jan 2021 00:41:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3BDC564E18 for ; Sun, 31 Jan 2021 00:41:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232577AbhAaAk5 (ORCPT ); Sat, 30 Jan 2021 19:40:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55282 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232290AbhAaAkp (ORCPT ); Sat, 30 Jan 2021 19:40:45 -0500 Received: from mail-il1-x129.google.com (mail-il1-x129.google.com [IPv6:2607:f8b0:4864:20::129]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E36B4C061574; Sat, 30 Jan 2021 16:40:04 -0800 (PST) Received: by mail-il1-x129.google.com with SMTP id z18so11934610ile.9; Sat, 30 Jan 2021 16:40:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:reply-to:from:date:message-id :subject:to:cc:content-transfer-encoding; bh=+TFRVbIqY5B7UoIMdCTWTdFFbAf760FFWhjVOC/xl1w=; b=hdRG+niLabsNzspeS+5Hsulzup6i3H4nW/NOjSYN7Vr3fCcxYHY8zysMH6wIcbxB68 JXom2iYYTzvIija/npWSfhkq0cMTuKorZyzRMtvrjP1A4X3wtCuafuERBksJ+4lkpZhA DVJbv1yrTaGBYPhkS76pm3j0TcsUxv6E1s+RvZI6L2Ixuxjbw/5g5nYjREQb9X2vFh7Z MIWkRw/1SNbJ+ZOw5D9ZHybGN7Ht/7DZIMJ+N2102gMJ+4mxIH2MhSYeFYha5T7WEV8R zkQzoBfiLB78Gv1aNC6XUEmLBcQqqjHXWPmWBiqTgbpYxwO7gK2mMvuLegMu0twG36Rg 2m/A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc:content-transfer-encoding; bh=+TFRVbIqY5B7UoIMdCTWTdFFbAf760FFWhjVOC/xl1w=; b=DtYQMK8uVdYEUPXJlWJQIDKj+1t2WKIjjVsNLzHagnJUQqobHCPIi1MyfMG7PcUdBj aN5PxcL5wmMPOZ1Y0lsIY6KiTOKlJ1hz97tkRVfBXVT7SWyL0WHcD68C65bUivv97Izl IoqKbvubyvfvNMyAO3XPIs4XYjWau0N6PA7DB583vWluHUjw60vLg8Jg4Vg0k/g+R8M6 HCmNz+uwj6835wFIZ6WsKoX6VW8g6JN4CBHWU3okRGbQHxPPRlsUvgQ4L08REPqHcF6A UxzO4W817nUn2a1PJo/27yFUrlGZ1x09zVKu1umB8eB4tlIrD2Q5kP+VZP4B+mks9Vn2 1BPQ== X-Gm-Message-State: AOAM531Igv1GyB8zJONztRNXm3nxfVtDtsBIRXBHvv5ScL3m5S1z9/1s +wyv6UZLrdBg+Yl0dJLDe/3qIZqbIR3mBkz21kk= X-Google-Smtp-Source: ABdhPJynxSxw+WY3q/3YMrp64U11oWWtmm99rfs1gXMySu6NGLftmCvUtyXdjPjgtveEVX4j0YfyXpTzPBk9BVUDUlw= X-Received: by 2002:a92:c80b:: with SMTP id v11mr7776249iln.215.1612053604328; Sat, 30 Jan 2021 16:40:04 -0800 (PST) MIME-Version: 1.0 References: <20210130004401.2528717-1-ndesaulniers@google.com> <20210130004401.2528717-3-ndesaulniers@google.com> In-Reply-To: Reply-To: sedat.dilek@gmail.com From: Sedat Dilek Date: Sun, 31 Jan 2021 01:39:52 +0100 Message-ID: Subject: Re: [PATCH v7 2/2] Kbuild: implement support for DWARF v5 To: =?UTF-8?B?RsSBbmctcnXDrCBTw7JuZw==?= Cc: Nick Desaulniers , Masahiro Yamada , Nathan Chancellor , Andrew Morton , LKML , Clang-Built-Linux ML , Linux Kbuild mailing list , linux-arch , Jakub Jelinek , Caroline Tice , Nick Clifton , Yonghong Song , Jiri Olsa , Andrii Nakryiko , Arnaldo Carvalho de Melo , Arvind Sankar Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jan 31, 2021 at 1:37 AM F=C4=81ng-ru=C3=AC S=C3=B2ng wrote: > > On Sat, Jan 30, 2021 at 3:10 PM Sedat Dilek wrote= : > > > > On Sat, Jan 30, 2021 at 1:44 AM Nick Desaulniers > > wrote: > > > > > > DWARF v5 is the latest standard of the DWARF debug info format. > > > > > > Feature detection of DWARF5 is onerous, especially given that we've > > > removed $(AS), so we must query $(CC) for DWARF5 assembler directive > > > support. > > > > > > The DWARF version of a binary can be validated with: > > > $ llvm-dwarfdump vmlinux | head -n 4 | grep version > > > or > > > $ readelf --debug-dump=3Dinfo vmlinux 2>/dev/null | grep Version > > > > > > DWARF5 wins significantly in terms of size when mixed with compressio= n > > > (CONFIG_DEBUG_INFO_COMPRESSED). > > > > > > 363M vmlinux.clang12.dwarf5.compressed > > > 434M vmlinux.clang12.dwarf4.compressed > > > 439M vmlinux.clang12.dwarf2.compressed > > > 457M vmlinux.clang12.dwarf5 > > > 536M vmlinux.clang12.dwarf4 > > > 548M vmlinux.clang12.dwarf2 > > > > > > 515M vmlinux.gcc10.2.dwarf5.compressed > > > 599M vmlinux.gcc10.2.dwarf4.compressed > > > 624M vmlinux.gcc10.2.dwarf2.compressed > > > 630M vmlinux.gcc10.2.dwarf5 > > > 765M vmlinux.gcc10.2.dwarf4 > > > 809M vmlinux.gcc10.2.dwarf2 > > > > > > Though the quality of debug info is harder to quantify; size is not a > > > proxy for quality. > > > > > > Jakub notes: > > > All [GCC] 5.1 - 6.x did was start accepting -gdwarf-5 as experiment= al > > > option that enabled some small DWARF subset (initially only a few > > > DW_LANG_* codes newly added to DWARF5 drafts). Only GCC 7 (release= d > > > after DWARF 5 has been finalized) started emitting DWARF5 section > > > headers and got most of the DWARF5 changes in... > > > > > > Version check GCC so that we don't need to worry about the difference= in > > > command line args between GNU readelf and llvm-readelf/llvm-dwarfdump= to > > > validate the DWARF Version in the assembler feature detection script. > > > > > > GNU `as` only recently gained support for specifying -gdwarf-5, so wh= en > > > compiling with Clang but without Clang's integrated assembler > > > (LLVM_IAS=3D1 is not set), explicitly add -Wa,-gdwarf-5 to DEBUG_CFLA= GS. > > > > > > Disabled for now if CONFIG_DEBUG_INFO_BTF is set; pahole doesn't yet > > > recognize the new additions to the DWARF debug info. Thanks to Sedat = for > > > the report. > > > > > > Link: http://www.dwarfstd.org/doc/DWARF5.pdf > > > Reported-by: Sedat Dilek > > > Suggested-by: Arvind Sankar > > > Suggested-by: Caroline Tice > > > Suggested-by: Fangrui Song > > > Suggested-by: Jakub Jelinek > > > Suggested-by: Masahiro Yamada > > > Suggested-by: Nathan Chancellor > > > Signed-off-by: Nick Desaulniers > > > --- > > > Makefile | 1 + > > > include/asm-generic/vmlinux.lds.h | 7 ++++++- > > > lib/Kconfig.debug | 18 ++++++++++++++++++ > > > scripts/test_dwarf5_support.sh | 8 ++++++++ > > > 4 files changed, 33 insertions(+), 1 deletion(-) > > > create mode 100755 scripts/test_dwarf5_support.sh > > > > > > diff --git a/Makefile b/Makefile > > > index d2b4980807e0..5387a6f2f62d 100644 > > > --- a/Makefile > > > +++ b/Makefile > > > @@ -831,6 +831,7 @@ KBUILD_AFLAGS +=3D -Wa,-gdwarf-2 > > > endif > > > > > > dwarf-version-$(CONFIG_DEBUG_INFO_DWARF4) :=3D 4 > > > +dwarf-version-$(CONFIG_DEBUG_INFO_DWARF5) :=3D 5 > > > DEBUG_CFLAGS +=3D -gdwarf-$(dwarf-version-y) > > > > > > ifdef CONFIG_DEBUG_INFO_REDUCED > > > diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/= vmlinux.lds.h > > > index 34b7e0d2346c..1e7cde4bd3f9 100644 > > > --- a/include/asm-generic/vmlinux.lds.h > > > +++ b/include/asm-generic/vmlinux.lds.h > > > @@ -842,8 +842,13 @@ > > > /* DWARF 4 */ = \ > > > .debug_types 0 : { *(.debug_types) } = \ > > > /* DWARF 5 */ = \ > > > + .debug_addr 0 : { *(.debug_addr) } = \ > > > + .debug_line_str 0 : { *(.debug_line_str) } = \ > > > + .debug_loclists 0 : { *(.debug_loclists) } = \ > > > .debug_macro 0 : { *(.debug_macro) } = \ > > > - .debug_addr 0 : { *(.debug_addr) } > > > + .debug_names 0 : { *(.debug_names) } = \ > > > + .debug_rnglists 0 : { *(.debug_rnglists) } = \ > > > + .debug_str_offsets 0 : { *(.debug_str_offsets) } > > > > > > > I just looked at binutils 2.36 in the Debian/experimental repositories. > > > > [1] says: > > > > + PR ld/27230 > > =EF=BF=BC+ * scripttempl/DWARF.sc: Add DWARF-5 .debug_* sections. > > > > ... > > > > - /* DWARF Extension. */ > > =EF=BF=BC- .debug_macro 0 : { *(.debug_macro) } > > + /* DWARF 5. */ > > .debug_addr 0 : { *(.debug_addr) } > > + .debug_line_str 0 : { *(.debug_line_str) } > > + .debug_loclists 0 : { *(.debug_loclists) } > > + .debug_macro 0 : { *(.debug_macro) } > > + .debug_names 0 : { *(.debug_names) } > > + .debug_rnglists 0 : { *(.debug_rnglists) } > > + .debug_str_offsets 0 : { *(.debug_str_offsets) } > > =EF=BF=BC+ .debug_sup 0 : { *(.debug_sup) } > > > > The list of DWARF-5 .debug_* sections is alphabetically sorted. > > AFAICS .debug_sup section is missing? > > > > - Sedat - > > No compiler produces .debug_sup section. It could be from some > post-processing tool which is unrelated to the linker. > Omitting it is fine. > Thanks for the clarification, Fangrui. - Sedat - > > [1] https://salsa.debian.org/toolchain-team/binutils/-/commit/f58f33081= 035672b01a04326a9c8daadbd09a430 > > > > > /* Stabs debugging sections. */ > > > #define STABS_DEBUG = \ > > > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug > > > index 94c1a7ed6306..ad6f78989d4f 100644 > > > --- a/lib/Kconfig.debug > > > +++ b/lib/Kconfig.debug > > > @@ -268,6 +268,24 @@ config DEBUG_INFO_DWARF4 > > > It makes the debug information larger, but it significantly > > > improves the success of resolving variables in gdb on optim= ized code. > > > > > > +config DEBUG_INFO_DWARF5 > > > + bool "Generate DWARF Version 5 debuginfo" > > > + depends on GCC_VERSION >=3D 50000 || CC_IS_CLANG > > > + depends on CC_IS_GCC || $(success,$(srctree)/scripts/test_dwa= rf5_support.sh $(CC) $(CLANG_FLAGS)) > > > + depends on !DEBUG_INFO_BTF > > > + help > > > + Generate DWARF v5 debug info. Requires binutils 2.35.2, gcc= 5.0+ (gcc > > > + 5.0+ accepts the -gdwarf-5 flag but only had partial suppor= t for some > > > + draft features until 7.0), and gdb 8.0+. > > > + > > > + Changes to the structure of debug info in Version 5 allow f= or around > > > + 15-18% savings in resulting image and debug info section si= zes as > > > + compared to DWARF Version 4. DWARF Version 5 standardizes p= revious > > > + extensions such as accelerators for symbol indexing and the= format > > > + for fission (.dwo/.dwp) files. Users may not want to select= this > > > + config if they rely on tooling that has not yet been update= d to > > > + support DWARF Version 5. > > > + > > > endchoice # "DWARF version" > > > > > > config DEBUG_INFO_BTF > > > diff --git a/scripts/test_dwarf5_support.sh b/scripts/test_dwarf5_sup= port.sh > > > new file mode 100755 > > > index 000000000000..c46e2456b47a > > > --- /dev/null > > > +++ b/scripts/test_dwarf5_support.sh > > > @@ -0,0 +1,8 @@ > > > +#!/bin/sh > > > +# SPDX-License-Identifier: GPL-2.0 > > > + > > > +# Test that the assembler doesn't need -Wa,-gdwarf-5 when presented = with DWARF > > > +# v5 input, such as `.file 0` and `md5 0x00`. Should be fixed in GNU= binutils > > > +# 2.35.2. https://sourceware.org/bugzilla/show_bug.cgi?id=3D25611 > > > +echo '.file 0 "filename" md5 0x7a0b65214090b6693bd1dc24dd248245' | \ > > > + $* -gdwarf-5 -Wno-unused-command-line-argument -c -x assembler -o = /dev/null - > > > -- > > > 2.30.0.365.g02bc693789-goog > > >