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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 141EACDB47E for ; Wed, 18 Oct 2023 13:03:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=9l1ug58IP1xPHXAUt/wTdxq+lH1xlMT1WlyQyWQgU7I=; b=SnnnKWI4gJOWDO GB2aqjARStG8kWB5OlE86MiMLQLVrUmBiPGHx2rQBKrrz88+gyquac25r5+jGUdSE9+g/vvyFC+pJ lw2Rim78WPiWMGPGPkJAt26bq9SJvg/d/VAQITsdsfQbPw/uuh9sZZwE+Ulp370AY/CW4QPmrU1Kl 5lzCvTlxNfn6KjoGNdS1prVprSZF2d0Ckdt8BIvSX5MfUVVHQ7meykCRK8W5EuieUAP3O1apHYKVc x8zqS43H9JYkaDRKNK5GatVxJK/Mp47MzngxErXR6iHxEoHzXts9oqAH94KBbm/6/2pz0s7yZ5Pxp HdRAxSI7RAC1vDj96b3w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qt6C5-00EjGd-0O; Wed, 18 Oct 2023 13:02:37 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qt6C2-00EjG1-0g for linux-arm-kernel@lists.infradead.org; Wed, 18 Oct 2023 13:02:35 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3DCA12F4; Wed, 18 Oct 2023 06:03:12 -0700 (PDT) Received: from FVFF77S0Q05N.cambridge.arm.com (FVFF77S0Q05N.cambridge.arm.com [10.1.31.158]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 44B493F762; Wed, 18 Oct 2023 06:02:30 -0700 (PDT) Date: Wed, 18 Oct 2023 14:02:27 +0100 From: Mark Rutland To: Andrea della Porta Cc: Catalin Marinas , Will Deacon , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, nik.borisov@suse.com Subject: Re: [PATCH 4/4] arm64: Make Aarch32 emulation boot time configurable Message-ID: References: <1029761eb218702d4aafa58d83c4bf9d3a760264.1697614386.git.andrea.porta@suse.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1029761eb218702d4aafa58d83c4bf9d3a760264.1697614386.git.andrea.porta@suse.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231018_060234_366906_B6D15922 X-CRM114-Status: GOOD ( 26.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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 Wed, Oct 18, 2023 at 01:13:22PM +0200, Andrea della Porta wrote: > Distributions would like to reduce their attack surface as much as > possible but at the same time they'd want to retain flexibility to > cater to a variety of legacy software. This stems from the conjecture > that compat layer is likely rarely tested and could have latent > security bugs. Ideally distributions will set their default policy > and also give users the ability to override it as appropriate. > > To enable this use case, introduce CONFIG_AARCH32_EMULATION_DEFAULT_DISABLED > compile time option, which controls whether 32bit processes/syscalls > should be allowed or not. This option is aimed mainly at distributions > to set their preferred default behavior in their kernels. > > To allow users to override the distro's policy, introduce the > 'aarch32_emulation' parameter which allows overriding > CONFIG_AARCH32_EMULATION_DEFAULT_DISABLED state at boot time. > > Signed-off-by: Andrea della Porta > --- > Documentation/admin-guide/kernel-parameters.txt | 7 +++++++ > arch/arm64/Kconfig | 9 +++++++++ > arch/arm64/kernel/entry-common.c | 8 +++++++- > 3 files changed, 23 insertions(+), 1 deletion(-) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index 0a1731a0f0ef..a41c5e6f5d2e 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -1,3 +1,10 @@ > + aarch32_emulation= [ARM64] > + Format: > + When true, allows loading 32-bit programs and executing > + 32-bit syscalls, essentially overriding > + AARCH32_EMULATION_DEFAULT_DISABLED at boot time. when false, > + unconditionally disables AARCH32 emulation. Can we please drop the 'emulation' part of the name? We don't use that terminology on arm64 for regular execution of compat tasks, and only use that to refer to true emulation of deprecated instructions. We already have the 'allow_mismatched_32bit_el0' option; can we please us a name that aligns with that? e.g. 'allow_32bit_el0=false' to disable support. Mark. > + > acpi= [HW,ACPI,X86,ARM64,RISCV64] > Advanced Configuration and Power Interface > Format: { force | on | off | strict | noirq | rsdt | > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index b10515c0200b..66c4cb273550 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -1725,6 +1725,15 @@ config SETEND_EMULATION > If unsure, say Y > endif # ARMV8_DEPRECATED > > +config AARCH32_EMULATION_DEFAULT_DISABLED > + bool "Aarch32 emulation disabled by default" > + default n > + depends on COMPAT > + help > + Make Aarch32 emulation disabled by default. This prevents loading 32-bit > + processes and access to 32-bit syscalls. > + > + If unsure, leave it to its default value. > endif # COMPAT > > menu "ARMv8.1 architectural features" > diff --git a/arch/arm64/kernel/entry-common.c b/arch/arm64/kernel/entry-common.c > index 32761760d9dd..07f2b4e632b8 100644 > --- a/arch/arm64/kernel/entry-common.c > +++ b/arch/arm64/kernel/entry-common.c > @@ -897,7 +897,13 @@ asmlinkage void noinstr el0t_32_error_handler(struct pt_regs *regs) > __el0_error_handler_common(regs); > } > > -bool __aarch32_enabled __ro_after_init = true; > +bool __aarch32_enabled __ro_after_init = !IS_ENABLED(CONFIG_AARCH32_EMULATION_DEFAULT_DISABLED); > + > +static int aarch32_emulation_override_cmdline(char *arg) > +{ > + return kstrtobool(arg, &__aarch32_enabled); > +} > +early_param("aarch32_emulation", aarch32_emulation_override_cmdline); > #else /* CONFIG_COMPAT */ > UNHANDLED(el0t, 32, sync) > UNHANDLED(el0t, 32, irq) > -- > 2.35.3 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel