From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933077AbcHKIy1 (ORCPT ); Thu, 11 Aug 2016 04:54:27 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:58744 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932787AbcHKIyX (ORCPT ); Thu, 11 Aug 2016 04:54:23 -0400 From: Arnd Bergmann To: "Zhangjian (Bamvor)" Cc: Yury Norov , catalin.marinas@arm.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, linux-arch@vger.kernel.org, libc-alpha@sourceware.org, schwidefsky@de.ibm.com, heiko.carstens@de.ibm.com, pinskia@gmail.com, broonie@kernel.org, joseph@codesourcery.com, christoph.muellner@theobroma-systems.com, szabolcs.nagy@arm.com, klimov.linux@gmail.com, Nathan_Lynch@mentor.com, agraf@suse.de, Prasun.Kapoor@caviumnetworks.com, kilobyte@angband.pl, geert@linux-m68k.org, philipp.tomsich@theobroma-systems.com, manuel.montezelo@gmail.com, linyongting@huawei.com, maxim.kuvyrkov@linaro.org, davem@davemloft.net, Andrew Pinski , Andrew Pinski , Bamvor Jian Zhang , Hanjun Guo Subject: Re: [PATCH 05/19] arm64: rename COMPAT to AARCH32_EL0 in Kconfig Date: Thu, 11 Aug 2016 10:53:01 +0200 Message-ID: <3043310.yXUR8Er0VK@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-31-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <57AC2AA5.1030808@huawei.com> References: <1466207668-10549-1-git-send-email-ynorov@caviumnetworks.com> <1466207668-10549-6-git-send-email-ynorov@caviumnetworks.com> <57AC2AA5.1030808@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:p4SzknCY4GtaaH1xLbbiXZLyFR+ztr2DbYQih1hyzwHA2O5oHAq t2ORzbE8Ja3kEB0vaBOy6IDOQRU2j67MVZ5NxZvL8/TOweTRAFAS01DrZs5fUJr71liRHuY sqJEA26upoGFGecqqAFKZjQq8Gg1oS8HcPcFJ/tfV6PGmfdevYL5zaUy1FxqU/sL9XKTFCS d+mx9LlhUGgdfISfUMuvQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:IDcDU4YJTuw=:IQzSUTwcK03u+5AGjKpHwq Xhf/TL/9HUqAVIYIP7wi5GMa4OhK4RYwZORjqSZ6Hg7DXES2VxYVJlghxjgBd+Rv1mYAGR/gi O1MxLbliImEC4Msy+mifp0ttehuRtr9JqlYL5gjWkeNuvUXVcl1DbGOW9Rw2yV+PJQsJ2Bo5c 8+2artDqAfrb1OWX4JBNRKxgJw4J4Q5Hay+x3ZVa1+JTp4hWY61akqyg8DXrOgdQ+9AFZOEgs STjaHDN6hb4FVKJtSNRBrtDOHlcfCSTwgME5OUky7sSmBQq75BI/cWaMdVozoI7F0fM7rcaPL pAOOOA4EQNFL2Jzs5BQg8XYmJ4OZtGQNkdUoSA0QLh0pEgCD/x7yHhJM6aky79GIoP0WO/oJd G6M56fPu9hFNV4xLQzHYtALFXCg4TzVXI6VhDW0G917ScCCRyPd3MMsShvoevi3qBokg/FyvB Unzx5L78Vi9JAzy4WAmhpH+fTiQKqMuYRZia5RI61irnW57zYscdq3unUZam13jixXfwMoPML CiUQ8lCNeySuPiNumVd+YSBGNeuuHKxT73PKdKy5eVhVFpUO9Co99PwKNyfztAaD2SlbpsIDx 0uZuaVmTrGiEyTTarjwRzSU/RnclaQ1xk4MigkFZABMdWUaWmJJ81NGUV5jXI4Xi6LRk86cHI mUVw/qXrlxmTq+DkibPTf0Qp2DJgVD6sleFdRMUqgeycqXqiHEmYfkqf+2Y6Rg3O0n3OUs3wf XkFvho33HBwxrcV2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday, August 11, 2016 3:35:01 PM CEST Zhangjian (Bamvor) wrote: > On 2016/6/18 7:54, Yury Norov wrote: > > From: Andrew Pinski > > > > In this patchset ILP32 ABI support is added. Additionally to AARCH32, > > which is binary-compatible with ARM, ILP32 is (mostly) ABI-compatible. > > > > From now, AARCH32_EL0 (former COMPAT) config option means the support of > > AARCH32 userspace, ARM64_ILP32 - support of ILP32 ABI (see next patches), > > and COMPAT indicates that one of them, or both, is enabled. > > > > Where needed, CONFIG_COMPAT is changed over to use CONFIG_AARCH32_EL0 instead > > > > Reviewed-by: David Daney > > Signed-off-by: Andrew Pinski > > Signed-off-by: Philipp Tomsich > > Signed-off-by: Christoph Muellner > > Signed-off-by: Bamvor Jian Zhang > > Signed-off-by: Yury Norov > ... > > diff --git a/arch/arm64/kernel/cpuinfo.c b/arch/arm64/kernel/cpuinfo.c > > index c173d32..af200a8 100644 > > --- a/arch/arm64/kernel/cpuinfo.c > > +++ b/arch/arm64/kernel/cpuinfo.c > > @@ -134,15 +134,17 @@ static int c_show(struct seq_file *m, void *v) > > */ > > seq_puts(m, "Features\t:"); > > if (compat) { > > -#ifdef CONFIG_COMPAT > > - for (j = 0; compat_hwcap_str[j]; j++) > > - if (compat_elf_hwcap & (1 << j)) > > - seq_printf(m, " %s", compat_hwcap_str[j]); > > - > > - for (j = 0; compat_hwcap2_str[j]; j++) > > - if (compat_elf_hwcap2 & (1 << j)) > > - seq_printf(m, " %s", compat_hwcap2_str[j]); > > -#endif /* CONFIG_COMPAT */ > > +#ifdef CONFIG_AARCH32_EL0 > I saw that compat_hwcap_str and compat_hwcap2_str is defined when > "CONFIG_COMPAT" is true. Why we only change it to CONFIG_AARCH32_EL0 > in c show()? > > + if (personality(current->personality) == PER_LINUX32) { > And "compat" is "personality(current->personality) == PER_LINUX32;", > it seems that there is no need to add this twice. I think it would be best to remove the #ifdef here completely, the PER_LINUX32 concept is not strictly tied to the emulation of ARM binaries, it literally just changes the output of /proc/cpuinfo and 'uname', and you can have ARM binaries with PER_LINUX (using the arm64 uname) just like you can have arm64 binaries running with PER_LINUX32. Arnd