[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190808094346.31383383@canb.auug.org.au>
Date: Thu, 8 Aug 2019 09:43:46 +1000
From: Stephen Rothwell <sfr@...b.auug.org.au>
To: Peter Collingbourne <pcc@...gle.com>
Cc: Will Deacon <will@...nel.org>,
Masahiro Yamada <yamada.masahiro@...ionext.com>,
Catalin Marinas <catalin.marinas@....com>,
Linux Next Mailing List <linux-next@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Paul Mackerras <paulus@...ba.org>,
Michael Ellerman <mpe@...erman.id.au>
Subject: Re: linux-next: build failure after merge of the arm64 tree
Hi all,
On Wed, 7 Aug 2019 09:33:07 -0700 Peter Collingbourne <pcc@...gle.com> wrote:
>
> On Wed, Aug 7, 2019 at 8:25 AM Will Deacon <will@...nel.org> wrote:
> >
> > From 71c67a31f09fa8fdd1495dffd96a5f0d4cef2ede Mon Sep 17 00:00:00 2001
> > From: Will Deacon <will@...nel.org>
> > Date: Wed, 7 Aug 2019 12:48:33 +0100
> > Subject: [PATCH] init/Kconfig: Fix infinite Kconfig recursion on PPC
> >
> > Commit 5cf896fb6be3 ("arm64: Add support for relocating the kernel with
> > RELR relocations") introduced CONFIG_TOOLS_SUPPORT_RELR, which checks
> > for RELR support in the toolchain as part of the kernel configuration.
> > During this procedure, "$(NM)" is invoked to see if it supports the new
> > relocation format, however PowerPC conditionally overrides this variable
> > in the architecture Makefile in order to pass '--synthetic' when
> > targetting PPC64.
> >
> > This conditional override causes Kconfig to recurse forever, since
> > CONFIG_TOOLS_SUPPORT_RELR cannot be determined without $(NM) being
> > defined, but that in turn depends on CONFIG_PPC64:
> >
> > $ make ARCH=powerpc CROSS_COMPILE=powerpc-linux-gnu-
> > scripts/kconfig/conf --syncconfig Kconfig
> > scripts/kconfig/conf --syncconfig Kconfig
> > scripts/kconfig/conf --syncconfig Kconfig
> > [...]
> >
> > In this particular case, it looks like PowerPC may be able to pass
> > '--synthetic' unconditionally to nm or even drop it altogether. While
> > that is being resolved, let's just bodge the RELR check by picking up
> > $(NM) directly from the environment in whatever state it happens to be
> > in.
> >
> > Cc: Peter Collingbourne <pcc@...gle.com>
> > Reported-by: Stephen Rothwell <sfr@...b.auug.org.au>
> > Suggested-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
> > Signed-off-by: Will Deacon <will@...nel.org>
>
> Tested-by: Peter Collingbourne <pcc@...gle.com>
> Reviewed-by: Peter Collingbourne <pcc@...gle.com>
Thanks for sorting this out (even temporarily).
--
Cheers,
Stephen Rothwell
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists