From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8FD36C433E7 for ; Fri, 9 Oct 2020 06:15:42 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D6B1521527 for ; Fri, 9 Oct 2020 06:15:41 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Ym+lpaNF"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="QTMvjHeV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D6B1521527 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=fTiRgUtB0muty0v+SFQwH0FXJmXozKbjSmSf/HqOWGM=; b=Ym+lpaNFHFVr4bL7GcLsyUWCa Qwx3g97u84HbPGhh8rvnpLHZ94rjPis4CQRcAqPP28t4ur+Ofz/n7LS0u1uWrvk4orbQGkCeU5rA4 2OLQGVEibps4qbZh+kvyhvaY1rV2yfgj6wT0RV3XDxNE3DdHtP4a5dfUl/ROd3opfpI0mUdnvrn6A QQzaqZZbv8d11Py8FTjrnmBfFmmgeJ56XGC8vdj4Z2TKNb/oFNVW4nJWKAqKg9OJ7xuPNgFcOmgjP v3yL2WdGh+78INacaIW3TNE8ff/oEPU9w7eIRpEQfdJxG3amBDZaXFEzF9HM5mthp0zIE6K5ZsLXl BpR/hRg7w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kQlfJ-0005Ag-Cc; Fri, 09 Oct 2020 06:14:05 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kQlfF-00059k-UN for linux-arm-kernel@lists.infradead.org; Fri, 09 Oct 2020 06:14:03 +0000 Received: from localhost (83-86-74-64.cable.dynamic.v4.ziggo.nl [83.86.74.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 7CE3121527; Fri, 9 Oct 2020 06:13:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1602224039; bh=BpBCyZtqZokE3MkM9aNGAmttL9r/254fEF1BvkawdrA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QTMvjHeVPdr5ffAAySVPXLlAS3AEd+rahcbf+nkgtnKqT93zCxSMUHRTXhQGEP4aG tZK+qjonRfA8mPMqlgxQnCPjU4LALz38mQhDNXsBDsTcMWjQ68yqFHYLaIV5OMI5C8 Jz3aexXqVTtDxaskaX14J94d3ghCMbiaX5iQQi+8= Date: Fri, 9 Oct 2020 08:13:56 +0200 From: Greg Kroah-Hartman To: Qais Yousef Subject: Re: [RFC PATCH 2/3] arm64: Add support for asymmetric AArch32 EL0 configurations Message-ID: <20201009061356.GA120580@kroah.com> References: <20201008181641.32767-1-qais.yousef@arm.com> <20201008181641.32767-3-qais.yousef@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201008181641.32767-3-qais.yousef@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201009_021402_118762_30371777 X-CRM114-Status: GOOD ( 29.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arch@vger.kernel.org, Will Deacon , "Peter Zijlstra \(Intel\)" , Catalin Marinas , Marc Zyngier , Linus Torvalds , Morten Rasmussen , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Oct 08, 2020 at 07:16:40PM +0100, Qais Yousef wrote: > When the CONFIG_ASYMMETRIC_AARCH32 option is enabled (EXPERT), the type > of the ARM64_HAS_32BIT_EL0 capability becomes WEAK_LOCAL_CPU_FEATURE. > The kernel will now return true for system_supports_32bit_el0() and > checks 32-bit tasks are affined to AArch32 capable CPUs only in > do_notify_resume(). If the affinity contains a non-capable AArch32 CPU, > the tasks will get SIGKILLed. If the last CPU supporting 32-bit is > offlined, the kernel will SIGKILL any scheduled 32-bit tasks (the > alternative is to prevent offlining through a new .cpu_disable feature > entry). > > In addition to the relaxation of the ARM64_HAS_32BIT_EL0 capability, > this patch factors out the 32-bit cpuinfo and features setting into > separate functions: __cpuinfo_store_cpu_32bit(), > init_cpu_32bit_features(). The cpuinfo of the booting CPU > (boot_cpu_data) is now updated on the first 32-bit capable CPU even if > it is a secondary one. The ID_AA64PFR0_EL0_64BIT_ONLY feature is relaxed > to FTR_NONSTRICT and FTR_HIGHER_SAFE when the asymmetric AArch32 support > is enabled. The compat_elf_hwcaps are only verified for the > AArch32-capable CPUs to still allow hotplugging AArch64-only CPUs. > > Make sure that KVM never sees the asymmetric 32bit system. Guest can > still ignore ID registers and force run 32bit at EL0. > > Co-developed-by: Qais Yousef > Signed-off-by: Catalin Marinas > Signed-off-by: Qais Yousef > --- > arch/arm64/Kconfig | 14 ++++++ > arch/arm64/include/asm/cpu.h | 2 + > arch/arm64/include/asm/cpucaps.h | 3 +- > arch/arm64/include/asm/cpufeature.h | 20 +++++++- > arch/arm64/include/asm/thread_info.h | 5 +- > arch/arm64/kernel/cpufeature.c | 66 +++++++++++++++----------- > arch/arm64/kernel/cpuinfo.c | 71 ++++++++++++++++++---------- > arch/arm64/kernel/process.c | 17 +++++++ > arch/arm64/kernel/signal.c | 18 +++++++ > arch/arm64/kvm/arm.c | 5 +- > arch/arm64/kvm/guest.c | 2 +- > arch/arm64/kvm/sys_regs.c | 14 +++++- > 12 files changed, 176 insertions(+), 61 deletions(-) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 6d232837cbee..591853504dc4 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -1868,6 +1868,20 @@ config DMI > > endmenu > > +config ASYMMETRIC_AARCH32 > + bool "Allow support for asymmetric AArch32 support" > + depends on COMPAT && EXPERT Why EXPERT? You don't want this able to be enabled by anyone? > + help > + Enable this option to allow support for asymmetric AArch32 EL0 > + CPU configurations. Once the AArch32 EL0 support is detected > + on a CPU, the feature is made available to user space to allow > + the execution of 32-bit (compat) applications. If the affinity > + of the 32-bit application contains a non-AArch32 capable CPU > + or the last AArch32 capable CPU is offlined, the application > + will be killed. > + > + If unsure say N. > + > config SYSVIPC_COMPAT > def_bool y > depends on COMPAT && SYSVIPC > diff --git a/arch/arm64/include/asm/cpu.h b/arch/arm64/include/asm/cpu.h > index 7faae6ff3ab4..c920fa45e502 100644 > --- a/arch/arm64/include/asm/cpu.h > +++ b/arch/arm64/include/asm/cpu.h > @@ -15,6 +15,7 @@ > struct cpuinfo_arm64 { > struct cpu cpu; > struct kobject kobj; > + bool aarch32_valid; Do you mean to cause holes in this structure? :) Isn't "valid" the common thing? Do you now have to explicitly enable this everywhere instead of just dealing with the uncommon case of this cpu variant? I don't see this information being exported to userspace anywhere. I know Intel has submitted a patch to export this "type" of thing to the cpu sysfs directories, can you do the same thing here? Otherwise, how is userspace supposed to know where to place programs that are 32bit? thanks, greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel