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=-17.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT 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 C2C3AC4338F for ; Thu, 12 Aug 2021 19:05:18 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 822A16108C for ; Thu, 12 Aug 2021 19:05:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 822A16108C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Ko/F/eO+4LMWQo7OHA++SRf2M+XGcUm6GPzgrbuIusw=; b=TLy+wnMCc60+yP FrIUyHjPA3NXX/4gAwOBmXoPIxN5Wv4/y7uqF15hNlY+EYHvyMixSR/giI6c7H04u+cqUr82AmpRS Peoaoxs43LupcYnHibRWEGHbecI/1vyd9sSwPu8NbnHzGzCyFpNATHy37XmeffQUNI4kTvxWlrfeh 8/KHLMbHVyR2WsHOy96nVDOuTjozsExOiv8DDwLXZpikpekh+UwywopcOYCE1jnQzEAO+jLJM9xt6 pTpQLRlZ+DiM01wW7kReEJ611vmXtVxus0n2RMkTB1vLAJSRA4Bmd98oWpBIohwoz92rqzhxeT0/p BZDrBIptMcb+riYFzPWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mEFym-00AxfK-Dg; Thu, 12 Aug 2021 19:03:00 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mEFyN-00AxZE-3y for linux-arm-kernel@lists.infradead.org; Thu, 12 Aug 2021 19:02:37 +0000 Received: from disco-boy.misterjones.org (disco-boy.misterjones.org [51.254.78.96]) (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 C2E50610A4; Thu, 12 Aug 2021 19:02:34 +0000 (UTC) Received: from sofa.misterjones.org ([185.219.108.64] helo=why.lan) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1mEFyL-004c9r-68; Thu, 12 Aug 2021 20:02:33 +0100 From: Marc Zyngier To: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= , Will Deacon , Catalin Marinas , Mark Rutland , Ard Biesheuvel , Florian Fainelli , bcm-kernel-feedback-list@broadcom.com, kernel-team@android.com Subject: [PATCH 4/5] arm64: Warn on booting at EL2 with HVC disabled Date: Thu, 12 Aug 2021 20:02:12 +0100 Message-Id: <20210812190213.2601506-5-maz@kernel.org> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210812190213.2601506-1-maz@kernel.org> References: <20210812190213.2601506-1-maz@kernel.org> MIME-Version: 1.0 X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, zajec5@gmail.com, will@kernel.org, catalin.marinas@arm.com, mark.rutland@arm.com, ardb@kernel.org, f.fainelli@gmail.com, bcm-kernel-feedback-list@broadcom.com, kernel-team@android.com X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210812_120235_236436_32940A93 X-CRM114-Status: GOOD ( 16.16 ) 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 Now that we are able to paper over the gigantic stupidity that booting at EL2 with SCR_EL3.HCE==0 is, let's taint WARN_TAINT() when detecting this situation. Yes, this is *LOUD*. Signed-off-by: Marc Zyngier --- arch/arm64/include/asm/virt.h | 10 ++++++++++ arch/arm64/kernel/head.S | 10 ++++++++++ arch/arm64/kernel/smp.c | 3 +++ 3 files changed, 23 insertions(+) diff --git a/arch/arm64/include/asm/virt.h b/arch/arm64/include/asm/virt.h index 7379f35ae2c6..89bf5ae522da 100644 --- a/arch/arm64/include/asm/virt.h +++ b/arch/arm64/include/asm/virt.h @@ -49,6 +49,9 @@ #define BOOT_CPU_MODE_EL1 (0xe11) #define BOOT_CPU_MODE_EL2 (0xe12) +/* Flags associated to the boot mode */ +#define BOOT_CPU_MODE_DOWNGRADED (1 << 0) + #ifndef __ASSEMBLY__ #include @@ -67,6 +70,13 @@ */ extern u32 __boot_cpu_mode[2]; +/* + * __boot_cpu_mode_flags records events that are associated with CPUs + * coming online. A CPU having been downgraded from EL2 to EL1 because + * of HVC not being enabled will have BOOT_CPU_MODE_DOWNGRADED set. + */ +extern u32 __boot_cpu_mode_flags[1]; + void __hyp_set_vectors(phys_addr_t phys_vector_base); void __hyp_reset_vectors(void); diff --git a/arch/arm64/kernel/head.S b/arch/arm64/kernel/head.S index d6b2b05f5d3a..fdad6805868b 100644 --- a/arch/arm64/kernel/head.S +++ b/arch/arm64/kernel/head.S @@ -530,7 +530,13 @@ SYM_INNER_LABEL(init_el2, SYM_L_LOCAL) /* * HVC is unusable, so pretend we actually booted at EL1. * Once we have left EL2, there will be no going back. + * set_cpu_boot_mode_flag will do the necessary CMOs for us. */ + adr_l x1, __boot_cpu_mode_flags + ldr w0, [x1] + orr w0, w0, BOOT_CPU_MODE_DOWNGRADED + str w0, [x1] + mov_q x0, INIT_SCTLR_EL1_MMU_OFF msr sctlr_el1, x0 @@ -623,6 +629,10 @@ SYM_DATA_START(__early_cpu_boot_status) .quad 0 SYM_DATA_END(__early_cpu_boot_status) +SYM_DATA_START(__boot_cpu_mode_flags) + .long 0 +SYM_DATA_END(__boot_cpu_mode_flags) + .popsection /* diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c index 6f6ff072acbd..43fad7ca9110 100644 --- a/arch/arm64/kernel/smp.c +++ b/arch/arm64/kernel/smp.c @@ -425,6 +425,9 @@ static void __init hyp_mode_check(void) else if (is_hyp_mode_mismatched()) WARN_TAINT(1, TAINT_CPU_OUT_OF_SPEC, "CPU: CPUs started in inconsistent modes"); + else if (__boot_cpu_mode_flags[0] & BOOT_CPU_MODE_DOWNGRADED) + WARN_TAINT(1, TAINT_CPU_OUT_OF_SPEC, + "CPU: CPUs downgraded to EL1, HVC disabled"); else pr_info("CPU: All CPU(s) started at EL1\n"); if (IS_ENABLED(CONFIG_KVM) && !is_kernel_in_hyp_mode()) { -- 2.30.2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel