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=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 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 BEBEEC2D0DB for ; Thu, 30 Jan 2020 20:19:36 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [203.11.71.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 779F120CC7 for ; Thu, 30 Jan 2020 20:19:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 779F120CC7 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ghiti.fr Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 487sCB5vHFzDqZq for ; Fri, 31 Jan 2020 07:19:34 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; spf=none (no SPF record) smtp.mailfrom=ghiti.fr (client-ip=217.70.178.242; helo=mslow2.mail.gandi.net; envelope-from=alex@ghiti.fr; receiver=) Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=ghiti.fr Received: from mslow2.mail.gandi.net (mslow2.mail.gandi.net [217.70.178.242]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 487s8d0lyLzDqCt for ; Fri, 31 Jan 2020 07:17:18 +1100 (AEDT) Received: from relay4-d.mail.gandi.net (unknown [217.70.183.196]) by mslow2.mail.gandi.net (Postfix) with ESMTP id 9D3C63A84FA for ; Thu, 30 Jan 2020 20:08:42 +0000 (UTC) X-Originating-IP: 79.86.19.127 Received: from [192.168.0.12] (127.19.86.79.rev.sfr.net [79.86.19.127]) (Authenticated sender: alex@ghiti.fr) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id EC913E0008; Thu, 30 Jan 2020 20:08:10 +0000 (UTC) Subject: Re: [PATCH v2] powerpc: Do not consider weak unresolved symbol relocations as bad To: Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, Stephen Rothwell , Alexei Starovoitov , linux-next@vger.kernel.org, Zong Li , Palmer Dabbelt References: <20200118170335.21440-1-alex@ghiti.fr> From: Alex Ghiti Message-ID: <8a8d45c6-4ad2-c682-abfb-3d97188d0d45@ghiti.fr> Date: Thu, 30 Jan 2020 15:07:10 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.3.0 MIME-Version: 1.0 In-Reply-To: <20200118170335.21440-1-alex@ghiti.fr> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On 1/18/20 12:03 PM, Alexandre Ghiti wrote: > Commit 8580ac9404f6 ("bpf: Process in-kernel BTF") introduced two weak > symbols that may be unresolved at link time which result in an absolute > relocation to 0. relocs_check.sh emits the following warning: > > "WARNING: 2 bad relocations > c000000001a41478 R_PPC64_ADDR64 _binary__btf_vmlinux_bin_start > c000000001a41480 R_PPC64_ADDR64 _binary__btf_vmlinux_bin_end" > > whereas those relocations are legitimate even for a relocatable kernel > compiled with -pie option. > > relocs_check.sh already excluded some weak unresolved symbols explicitly: > remove those hardcoded symbols and add some logic that parses the symbols > using nm, retrieves all the weak unresolved symbols and excludes those from > the list of the potential bad relocations. > > Reported-by: Stephen Rothwell > Signed-off-by: Alexandre Ghiti > --- > > Changes in v2: > - Follow Stephen advice of using grep -F instead of looping over weak symbols > using read, patch is way smaller and cleaner. > - Add missing nm in comment > > arch/powerpc/Makefile.postlink | 4 ++-- > arch/powerpc/tools/relocs_check.sh | 20 ++++++++++++-------- > 2 files changed, 14 insertions(+), 10 deletions(-) > > diff --git a/arch/powerpc/Makefile.postlink b/arch/powerpc/Makefile.postlink > index 134f12f89b92..2268396ff4bb 100644 > --- a/arch/powerpc/Makefile.postlink > +++ b/arch/powerpc/Makefile.postlink > @@ -17,11 +17,11 @@ quiet_cmd_head_check = CHKHEAD $@ > quiet_cmd_relocs_check = CHKREL $@ > ifdef CONFIG_PPC_BOOK3S_64 > cmd_relocs_check = \ > - $(CONFIG_SHELL) $(srctree)/arch/powerpc/tools/relocs_check.sh "$(OBJDUMP)" "$@" ; \ > + $(CONFIG_SHELL) $(srctree)/arch/powerpc/tools/relocs_check.sh "$(OBJDUMP)" "$(NM)" "$@" ; \ > $(BASH) $(srctree)/arch/powerpc/tools/unrel_branch_check.sh "$(OBJDUMP)" "$@" > else > cmd_relocs_check = \ > - $(CONFIG_SHELL) $(srctree)/arch/powerpc/tools/relocs_check.sh "$(OBJDUMP)" "$@" > + $(CONFIG_SHELL) $(srctree)/arch/powerpc/tools/relocs_check.sh "$(OBJDUMP)" "$(NM)" "$@" > endif > > # `@true` prevents complaint when there is nothing to be done > diff --git a/arch/powerpc/tools/relocs_check.sh b/arch/powerpc/tools/relocs_check.sh > index 7b9fe0a567cf..014e00e74d2b 100755 > --- a/arch/powerpc/tools/relocs_check.sh > +++ b/arch/powerpc/tools/relocs_check.sh > @@ -10,14 +10,21 @@ > # based on relocs_check.pl > # Copyright © 2009 IBM Corporation > > -if [ $# -lt 2 ]; then > - echo "$0 [path to objdump] [path to vmlinux]" 1>&2 > +if [ $# -lt 3 ]; then > + echo "$0 [path to objdump] [path to nm] [path to vmlinux]" 1>&2 > exit 1 > fi > > -# Have Kbuild supply the path to objdump so we handle cross compilation. > +# Have Kbuild supply the path to objdump and nm so we handle cross compilation. > objdump="$1" > -vmlinux="$2" > +nm="$2" > +vmlinux="$3" > + > +# Remove from the bad relocations those that match an undefined weak symbol > +# which will result in an absolute relocation to 0. > +# Weak unresolved symbols are of that form in nm output: > +# " w _binary__btf_vmlinux_bin_end" > +undef_weak_symbols=$($nm "$vmlinux" | awk '$1 ~ /w/ { print $2 }') > > bad_relocs=$( > $objdump -R "$vmlinux" | > @@ -26,8 +33,6 @@ $objdump -R "$vmlinux" | > # These relocations are okay > # On PPC64: > # R_PPC64_RELATIVE, R_PPC64_NONE > - # R_PPC64_ADDR64 mach_ > - # R_PPC64_ADDR64 __crc_ > # On PPC: > # R_PPC_RELATIVE, R_PPC_ADDR16_HI, > # R_PPC_ADDR16_HA,R_PPC_ADDR16_LO, > @@ -39,8 +44,7 @@ R_PPC_ADDR16_HI > R_PPC_ADDR16_HA > R_PPC_RELATIVE > R_PPC_NONE' | > - grep -E -v '\ - grep -E -v '\ + ([ "$undef_weak_symbols" ] && grep -F -w -v "$undef_weak_symbols" || cat) > ) > > if [ -z "$bad_relocs" ]; then Hi guys, Any thought about that ? I do think this patch makes the whole check about absolute relocations clearer. And in the future, it will avoid anyone to spend some time on those "bad" relocations which actually aren't. Thanks, Alex