Re: [PATCH 05/19] arm64: rename COMPAT to AARCH32_EL0 in Kconfig

From: Yury Norov <ynorov@caviumnetworks.com>
To: Catalin Marinas <catalin.marinas@arm.com>
Cc: Arnd Bergmann <arnd@arndb.de>, <linux-doc@vger.kernel.org>,
	<szabolcs.nagy@arm.com>, <heiko.carstens@de.ibm.com>,
	Hanjun Guo <guohanjun@huawei.com>,
	<philipp.tomsich@theobroma-systems.com>,
	<joseph@codesourcery.com>, <linux-arch@vger.kernel.org>,
	<Prasun.Kapoor@caviumnetworks.com>, <agraf@suse.de>,
	Andrew Pinski <Andrew.Pinski@caviumnetworks.com>,
	<geert@linux-m68k.org>, <kilobyte@angband.pl>,
	<manuel.montezelo@gmail.com>, <pinskia@gmail.com>,
	<linyongting@huawei.com>, <klimov.linux@gmail.com>,
	<broonie@kernel.org>,
	"Zhangjian (Bamvor)" <bamvor.zhangjian@huawei.com>,
	Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<maxim.kuvyrkov@linaro.org>, <libc-alpha@sourceware.org>,
	<Nathan_Lynch@mentor.com>, <linux-kernel@vger.kernel.org>,
	Andrew Pinski <apinski@cavium.com>, <schwidefsky@de.ibm.com>,
	<davem@davemloft.net>, <christoph.muellner@theobroma-systems.com>
Subject: Re: [PATCH 05/19] arm64: rename COMPAT to AARCH32_EL0 in Kconfig
Date: Sat, 13 Aug 2016 18:17:03 +0300	[thread overview]
Message-ID: <20160813151703.GC24335@yury-N73SV> (raw)
In-Reply-To: <20160812143612.GH12939@e104818-lin.cambridge.arm.com>

On Fri, Aug 12, 2016 at 03:36:12PM +0100, Catalin Marinas wrote:
> On Thu, Aug 11, 2016 at 10:29:03PM +0200, Arnd Bergmann wrote:
> > On Thursday, August 11, 2016 5:30:03 PM CEST Catalin Marinas wrote:
> > > > > > and you can have ARM binaries with
> > > > > > PER_LINUX (using the arm64 uname) just like you can have
> > > > > > arm64 binaries running with PER_LINUX32.
> > > > > 
> > > > > I was actually looking to enforce the 32-bit binaries to only see
> > > > > PER_LINUX32, though with a risk of breaking the ABI. OTOH, people are
> > > > > abusing this and write 32-bit apps relying on the 64-bit /proc/cpuinfo:
> > > > > 
> > > > > http://lkml.kernel.org/g/1464706504-25224-3-git-send-email-catalin.marinas@arm.com
> > > > > 
> > > > > (you were summoned on that discussion couple of times ;))
> > > > 
> > > > Hmm, I thought I saw the thread and didn't have any good idea for
> > > > the uname information, but didn't notice it was for /proc/cpuinfo.
> > > > 
> > > > What's wrong with always showing both the 32-bit and the 64-bit
> > > > hwcap strings here (minus the duplicates, which hopefully have
> > > > the same meaning here)?
> > > 
> > > As I said above, some of them have the same name (which may be a good
> > > thing at a first look) but we don't have an architecture guarantee that
> > > the feature is present in both AArch32 and AArch64 modes (e.g. AES may
> > > only be available in AArch64).
> > 
> > Is this the case on actual implementations that exist today? If they
> > are actually always both present, we might be able to get away with it.
> 
> It may be fine on current implementations but what would we do when/if
> we actually find such discrepancy? It's not just ARM Ltd designing the
> chips, so as long as the architecture doesn't mandate it you may find
> strange implementations.
> 
> Imposing such restriction in the architecture doesn't make sense if the
> only reason is the /proc/cpuinfo file (and I can't think of any other
> reason why this should be enforced).
> 
> What I'm worried about is 32-bit apps running on an arm64 kernel and
> making use of the 64-bit /proc/cpuinfo without any guarantee that the
> AArch32 state has such features. In my patch proposal linked above I
> wanted to always force the compat /proc/cpuinfo for 32-bit tasks.

The link doesn't work for me. Is there other link, or what's the
maillist there?

So, what we decided finally? Is my understanding correct that we leave
everything as is in ilp32 series, and it will be resolved separately?

Yury.

> 
> -- 
> Catalin