From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fangrui Song Subject: Re: [PATCH v3 2/9] vmlinux.lds.h: Add .symtab, .strtab, and .shstrtab to STABS_DEBUG Date: Wed, 24 Jun 2020 09:16:43 -0700 Message-ID: <20200624161643.73x6navnwryckuit@google.com> References: <20200624014940.1204448-1-keescook@chromium.org> <20200624014940.1204448-3-keescook@chromium.org> <20200624153930.GA1337895@rani.riverdale.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Return-path: Content-Disposition: inline In-Reply-To: <20200624153930.GA1337895@rani.riverdale.lan> Sender: linux-kernel-owner@vger.kernel.org To: Arvind Sankar Cc: Kees Cook , Will Deacon , Catalin Marinas , Mark Rutland , Ard Biesheuvel , Peter Collingbourne , James Morse , Borislav Petkov , Thomas Gleixner , Ingo Molnar , Russell King , Masahiro Yamada , Nick Desaulniers , Nathan Chancellor , Arnd Bergmann , x86@kernel.org, clang-built-linux@googlegroups.com, linux-arch@vger.kernel.org, linux-efi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org List-Id: linux-arch.vger.kernel.org On 2020-06-24, Arvind Sankar wrote: >On Tue, Jun 23, 2020 at 06:49:33PM -0700, Kees Cook wrote: >> When linking vmlinux with LLD, the synthetic sections .symtab, .strtab, >> and .shstrtab are listed as orphaned. Add them to the STABS_DEBUG section >> so there will be no warnings when --orphan-handling=warn is used more >> widely. (They are added above comment as it is the more common > >Nit 1: is "after .comment" better than "above comment"? It's above in the >sense of higher file offset, but it's below in readelf output. I mean this order:) .comment .symtab .shstrtab .strtab This is the case in the absence of a linker script if at least one object file has .comment (mostly for GCC/clang version information) or the linker is LLD which adds a .comment >Nit 2: These aren't actually debugging sections, no? Is it better to add >a new macro for it, and is there any plan to stop LLD from warning about >them? https://reviews.llvm.org/D75149 "[ELF] --orphan-handling=: don't warn/error for unused synthesized sections" described that .symtab .shstrtab .strtab are different in GNU ld. Since many other GNU ld synthesized sections (.rela.dyn .plt ...) can be renamed or dropped via output section descriptions, I don't understand why the 3 sections can't be customized. I created a feature request: https://sourceware.org/bugzilla/show_bug.cgi?id=26168 (If this is supported, it is a consistent behavior to warn for orphan .symtab/.strtab/.shstrtab There may be 50% chance that the maintainer decides that "LLD diverges" I would disagree: there is no fundamental problems with .symtab/.strtab/.shstrtab which make them special in output section descriptions or orphan handling.) >> order[1].) >> >> ld.lld: warning: :(.symtab) is being placed in '.symtab' >> ld.lld: warning: :(.shstrtab) is being placed in '.shstrtab' >> ld.lld: warning: :(.strtab) is being placed in '.strtab' >> >> [1] https://lore.kernel.org/lkml/20200622224928.o2a7jkq33guxfci4@google.com/ >> >> Reported-by: Fangrui Song >> Reviewed-by: Fangrui Song >> Signed-off-by: Kees Cook >> --- >> include/asm-generic/vmlinux.lds.h | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h >> index 1248a206be8d..8e71757f485b 100644 >> --- a/include/asm-generic/vmlinux.lds.h >> +++ b/include/asm-generic/vmlinux.lds.h >> @@ -792,7 +792,10 @@ >> .stab.exclstr 0 : { *(.stab.exclstr) } \ >> .stab.index 0 : { *(.stab.index) } \ >> .stab.indexstr 0 : { *(.stab.indexstr) } \ >> - .comment 0 : { *(.comment) } >> + .comment 0 : { *(.comment) } \ >> + .symtab 0 : { *(.symtab) } \ >> + .strtab 0 : { *(.strtab) } \ >> + .shstrtab 0 : { *(.shstrtab) } >> >> #ifdef CONFIG_GENERIC_BUG >> #define BUG_TABLE \ >> -- >> 2.25.1 >> From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39116 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404829AbgFXQQr (ORCPT ); Wed, 24 Jun 2020 12:16:47 -0400 Received: from mail-pl1-x643.google.com (mail-pl1-x643.google.com [IPv6:2607:f8b0:4864:20::643]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A0B27C0613ED for ; Wed, 24 Jun 2020 09:16:47 -0700 (PDT) Received: by mail-pl1-x643.google.com with SMTP id g17so1224272plq.12 for ; Wed, 24 Jun 2020 09:16:47 -0700 (PDT) Date: Wed, 24 Jun 2020 09:16:43 -0700 From: Fangrui Song Subject: Re: [PATCH v3 2/9] vmlinux.lds.h: Add .symtab, .strtab, and .shstrtab to STABS_DEBUG Message-ID: <20200624161643.73x6navnwryckuit@google.com> References: <20200624014940.1204448-1-keescook@chromium.org> <20200624014940.1204448-3-keescook@chromium.org> <20200624153930.GA1337895@rani.riverdale.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <20200624153930.GA1337895@rani.riverdale.lan> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arvind Sankar Cc: Kees Cook , Will Deacon , Catalin Marinas , Mark Rutland , Ard Biesheuvel , Peter Collingbourne , James Morse , Borislav Petkov , Thomas Gleixner , Ingo Molnar , Russell King , Masahiro Yamada , Nick Desaulniers , Nathan Chancellor , Arnd Bergmann , x86@kernel.org, clang-built-linux@googlegroups.com, linux-arch@vger.kernel.org, linux-efi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Message-ID: <20200624161643.Yi1nnXL0cQeHl2aC4klA_z7XWr_xKV4PAv1d4xl_XEI@z> On 2020-06-24, Arvind Sankar wrote: >On Tue, Jun 23, 2020 at 06:49:33PM -0700, Kees Cook wrote: >> When linking vmlinux with LLD, the synthetic sections .symtab, .strtab, >> and .shstrtab are listed as orphaned. Add them to the STABS_DEBUG section >> so there will be no warnings when --orphan-handling=warn is used more >> widely. (They are added above comment as it is the more common > >Nit 1: is "after .comment" better than "above comment"? It's above in the >sense of higher file offset, but it's below in readelf output. I mean this order:) .comment .symtab .shstrtab .strtab This is the case in the absence of a linker script if at least one object file has .comment (mostly for GCC/clang version information) or the linker is LLD which adds a .comment >Nit 2: These aren't actually debugging sections, no? Is it better to add >a new macro for it, and is there any plan to stop LLD from warning about >them? https://reviews.llvm.org/D75149 "[ELF] --orphan-handling=: don't warn/error for unused synthesized sections" described that .symtab .shstrtab .strtab are different in GNU ld. Since many other GNU ld synthesized sections (.rela.dyn .plt ...) can be renamed or dropped via output section descriptions, I don't understand why the 3 sections can't be customized. I created a feature request: https://sourceware.org/bugzilla/show_bug.cgi?id=26168 (If this is supported, it is a consistent behavior to warn for orphan .symtab/.strtab/.shstrtab There may be 50% chance that the maintainer decides that "LLD diverges" I would disagree: there is no fundamental problems with .symtab/.strtab/.shstrtab which make them special in output section descriptions or orphan handling.) >> order[1].) >> >> ld.lld: warning: :(.symtab) is being placed in '.symtab' >> ld.lld: warning: :(.shstrtab) is being placed in '.shstrtab' >> ld.lld: warning: :(.strtab) is being placed in '.strtab' >> >> [1] https://lore.kernel.org/lkml/20200622224928.o2a7jkq33guxfci4@google.com/ >> >> Reported-by: Fangrui Song >> Reviewed-by: Fangrui Song >> Signed-off-by: Kees Cook >> --- >> include/asm-generic/vmlinux.lds.h | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h >> index 1248a206be8d..8e71757f485b 100644 >> --- a/include/asm-generic/vmlinux.lds.h >> +++ b/include/asm-generic/vmlinux.lds.h >> @@ -792,7 +792,10 @@ >> .stab.exclstr 0 : { *(.stab.exclstr) } \ >> .stab.index 0 : { *(.stab.index) } \ >> .stab.indexstr 0 : { *(.stab.indexstr) } \ >> - .comment 0 : { *(.comment) } >> + .comment 0 : { *(.comment) } \ >> + .symtab 0 : { *(.symtab) } \ >> + .strtab 0 : { *(.strtab) } \ >> + .shstrtab 0 : { *(.shstrtab) } >> >> #ifdef CONFIG_GENERIC_BUG >> #define BUG_TABLE \ >> -- >> 2.25.1 >>