linux-kernel - Re: [PATCH] arm64: Kconfig.platforms: fix warning unmet direct dependencies

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1548431829.28802.50.camel@pengutronix.de>
Date:   Fri, 25 Jan 2019 16:57:09 +0100
From:   Lucas Stach <l.stach@...gutronix.de>
To:     Catalin Marinas <catalin.marinas@....com>,
        Anders Roxell <anders.roxell@...aro.org>
Cc:     will.deacon@....com, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: Kconfig.platforms: fix warning unmet direct
 dependencies

Am Freitag, den 25.01.2019, 14:32 +0000 schrieb Catalin Marinas:
> On Tue, Jan 15, 2019 at 08:18:39PM +0100, Anders Roxell wrote:
> > When ARCH_MXC get enabled, ARM64_ERRATUM_845719 will be selected and
> > this warning will happen when COMPAT isn't set.
> > 
> > WARNING: unmet direct dependencies detected for ARM64_ERRATUM_845719
> >   Depends on [n]: COMPAT [=n]
> >   Selected by [y]:
> >   - ARCH_MXC [=y]
> > 
> > Rework to add 'if COMPAT' before ARM64_ERRATUM_845719 gets selected,
> > since ARM64_ERRATUM_845719 depends on COMPAT.
> > 
> > > > Signed-off-by: Anders Roxell <anders.roxell@...aro.org>
> > ---
> >  arch/arm64/Kconfig.platforms | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index 251ecf34cb02..d4faca775d9c 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -145,7 +145,7 @@ config ARCH_MVEBU
> >  config ARCH_MXC
> > > >  	bool "ARMv8 based NXP i.MX SoC family"
> > > >  	select ARM64_ERRATUM_843419
> > > > -	select ARM64_ERRATUM_845719
> > > > +	select ARM64_ERRATUM_845719 if COMPAT
> > > >  	help
> > > >  	  This enables support for the ARMv8 based SoCs in the
> >  	  NXP i.MX family.
> 
> Actually, do we need to select the errata workarounds explicitly? That
> seems to be the only case where we do it (commit 930507c18304, "arm64:
> add basic Kconfig symbols for i.MX8"). They are default y, so we
> shouldn't need to force them on.

This is based on past experience. We've had a lot of cases were people
did not enable the necessary CPU errata workaround, which then usually
lead to very hard to debug system failures. It is on our list of things
to look out for now, but I would feel much better if there is just no
chance for a user to misconfigure the kernel in this way.

Regards,
Lucas