Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0E528C38142 for ; Fri, 27 Jan 2023 13:33:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234262AbjA0NdC (ORCPT ); Fri, 27 Jan 2023 08:33:02 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48062 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234452AbjA0Nc6 (ORCPT ); Fri, 27 Jan 2023 08:32:58 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3401D173B; Fri, 27 Jan 2023 05:32:46 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id DB56EB820FD; Fri, 27 Jan 2023 13:32:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 071B5C4339B; Fri, 27 Jan 2023 13:32:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1674826363; bh=ymT22baLsWOgI9yW22p0SoFzksqIRSb/XvXQWExE82Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=DQV7OLdi4Bvb+bmI9n7W/xsuRIuYHvbZ4fLvTwnfxR6N0P0naXidj6hAlsGXZsWND HdaK/zUKy6at2px0tqbzcubbQBiQOnB3O0qXhmgX7OOx8ZSRKdohAuRqLoJMPOU/AR p9Si+3Csy39xt31HiCWV9TvSqcFm0X4xwCNMIGRY= Date: Fri, 27 Jan 2023 14:32:40 +0100 From: Greg Kroah-Hartman To: Tom Saeger Cc: Naresh Kamboju , John Paul Adrian Glaubitz , Rich Felker , Masahiro Yamada , Ard Biesheuvel , Nathan Chancellor , Dennis Gilmore , Palmer Dabbelt , Arnd Bergmann , stable@vger.kernel.org, linux-kernel@vger.kernel.org, linux-sh@vger.kernel.org Subject: Re: [PATCH 5.4 fix build id for arm64 6/6] sh: define RUNTIME_DISCARD_EXIT Message-ID: References: <7301d95edaf5bd0926bc93a67cb0cc1256549c95.1674588616.git.tom.saeger@oracle.com> <20230127132540.agmyuzg64wlcwglo@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230127132540.agmyuzg64wlcwglo@oracle.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 27, 2023 at 07:25:40AM -0600, Tom Saeger wrote: > On Fri, Jan 27, 2023 at 08:29:23AM +0100, Greg Kroah-Hartman wrote: > > On Tue, Jan 24, 2023 at 02:14:23PM -0700, Tom Saeger wrote: > > > sh vmlinux fails to link with GNU ld < 2.40 (likely < 2.36) since > > > commit 99cb0d917ffa ("arch: fix broken BuildID for arm64 and riscv"). > > > > > > This is similar to fixes for powerpc and s390: > > > commit 4b9880dbf3bd ("powerpc/vmlinux.lds: Define RUNTIME_DISCARD_EXIT"). > > > commit a494398bde27 ("s390: define RUNTIME_DISCARD_EXIT to fix link error > > > with GNU ld < 2.36"). > > > > > > $ sh4-linux-gnu-ld --version | head -n1 > > > GNU ld (GNU Binutils for Debian) 2.35.2 > > > > > > $ make ARCH=sh CROSS_COMPILE=sh4-linux-gnu- microdev_defconfig > > > $ make ARCH=sh CROSS_COMPILE=sh4-linux-gnu- > > > > > > `.exit.text' referenced in section `__bug_table' of crypto/algboss.o: > > > defined in discarded section `.exit.text' of crypto/algboss.o > > > `.exit.text' referenced in section `__bug_table' of > > > drivers/char/hw_random/core.o: defined in discarded section > > > `.exit.text' of drivers/char/hw_random/core.o > > > make[2]: *** [scripts/Makefile.vmlinux:34: vmlinux] Error 1 > > > make[1]: *** [Makefile:1252: vmlinux] Error 2 > > > > > > arch/sh/kernel/vmlinux.lds.S keeps EXIT_TEXT: > > > > > > /* > > > * .exit.text is discarded at runtime, not link time, to deal with > > > * references from __bug_table > > > */ > > > .exit.text : AT(ADDR(.exit.text)) { EXIT_TEXT } > > > > > > However, EXIT_TEXT is thrown away by > > > DISCARD(include/asm-generic/vmlinux.lds.h) because > > > sh does not define RUNTIME_DISCARD_EXIT. > > > > > > GNU ld 2.40 does not have this issue and builds fine. > > > This corresponds with Masahiro's comments in a494398bde27: > > > "Nathan [Chancellor] also found that binutils > > > commit 21401fc7bf67 ("Duplicate output sections in scripts") cured this > > > issue, so we cannot reproduce it with binutils 2.36+, but it is better > > > to not rely on it." > > > > > > Fixes: 99cb0d917ffa ("arch: fix broken BuildID for arm64 and riscv") > > > Link: https://lore.kernel.org/all/Y7Jal56f6UBh1abE@dev-arch.thelio-3990X/ > > > Link: https://lore.kernel.org/all/20230123194218.47ssfzhrpnv3xfez@oracle.com/ > > > Signed-off-by: Tom Saeger > > > --- > > > arch/sh/kernel/vmlinux.lds.S | 1 + > > > 1 file changed, 1 insertion(+) > > > > No upstream git id? > > > > :( > > No, not yet. I'll try resending. This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read: https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html for how to do this properly.