From patchwork Tue Oct 18 11:04:36 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ard Biesheuvel X-Patchwork-Id: 13010265 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 DA083C4332F for ; Tue, 18 Oct 2022 11:06:03 +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: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=ZHnd1YmavKbum+9w8sY7Yayl05m/Y66jQegf9O/YJUs=; b=gwFj4MIB4aaI8V cSBH2RVHrOn9UAoRQhnzh1zrxDLjrX/ECgA4JoU9NA4AOEVPgWr3YwI/0JymnzO5p4omhhUnpXiSl HJU3LN0aZFe/dULYwtKEDmAsm2nP6va6JCwFeDogafSjuvHAC64H3h/HjMxcXZnn1soMc72cdXZ7I wVYstkuAGyKThyi/jq+PBsTd19n5o/+aCSl62VJw1wHfGe4os7qCmmNRuHO+mOHhRUfuswaNUQJg1 oJEMPdEdGguUD3T4qGAn078wglZalO3zVyeBNLeuevV4loPfVXoMVnmyAFDuUibg6VJ4Th55uw4eh +zalJFz6unV8psjZ/A6A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1okkPA-005yTH-DE; Tue, 18 Oct 2022 11:05:04 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1okkP0-005yD5-7E for linux-arm-kernel@lists.infradead.org; Tue, 18 Oct 2022 11:04:55 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 969B361520; Tue, 18 Oct 2022 11:04:53 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 47ED4C43470; Tue, 18 Oct 2022 11:04:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1666091093; bh=fOnIdv0Xyy55Ww2CKD9LOztyluMCPv+3nJyI3YpsVF0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=HdFogkM/YvR0ICZqmNuKMnJRUxIg05fqAyNB8X53twaIVCGy0rpvrArb+RjqxP0qQ yWosl6raCPKzhdJYhFxPxn8upliX5U7PdmU4Hew+GXC9HkMziehh8/joi4MRjKyYyp 0rNAW3WUH9pGE/x8KwfI4B3yO9pUOW6msC921tl99n54CVnjD3SIVZrS7rytWeFfSf PBcwCFosvZ6XatU1M3EBVQcPHlyqKXDOxaraurQde+BnZLUw5CPaXNTWktS5qXEh7Q saPYDWvCN3NUpZ1AFSmBDxgE89e8ITDr9kbtcbcNdC8rL2PYWcb8VO7KJv7jA/3kW7 IC/XATeC4e+5w== From: Ard Biesheuvel To: linux-arm-kernel@lists.infradead.org Cc: linux-efi@vger.kernel.org, keescook@chromium.org, Ard Biesheuvel , Will Deacon , Catalin Marinas , Marc Zyngier , Mark Rutland Subject: [PATCH v4 1/6] arm64: lds: reduce effective minimum image alignment to 64k Date: Tue, 18 Oct 2022 13:04:36 +0200 Message-Id: <20221018110441.3855148-2-ardb@kernel.org> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20221018110441.3855148-1-ardb@kernel.org> References: <20221018110441.3855148-1-ardb@kernel.org> MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=5081; i=ardb@kernel.org; h=from:subject; bh=fOnIdv0Xyy55Ww2CKD9LOztyluMCPv+3nJyI3YpsVF0=; b=owEB7QES/pANAwAKAcNPIjmS2Y8kAcsmYgBjTohAHq6+KfeoPUmam6DRiCZqZah9LWBjKGkmSMCj lX/kwg2JAbMEAAEKAB0WIQT72WJ8QGnJQhU3VynDTyI5ktmPJAUCY06IQAAKCRDDTyI5ktmPJDZ9DA C/xJIV+LWKu7IwFMZnGt8Sb7I5n0+z0K/H30/w9+y9UNKq/u1NK41hFtPAUwW9LX8Ol3vTpKNVsZyn s/Rz/zbNWjA3awb81VPt9fr3J0FSI9lG1yQd8RzEUoU514h1Wh9jwPyM45rDbDo0ZhU2YbPDNFvvGK 68xPQoUT6ez1Q4L7Rf8tyYPUc9PlMJy7Eh2JwPOP+j8Garmmv78wx7EORedFadIGfaSySh6zIRDLpV uGrECP75CJojvdDU5sVl04CVF/h5kkLP9ZCd1fD8WwrhxdORWLW6RH54H9qzJ6Lds64FDWZo5PRn1I E/fpZB89cyAuPZe8Z6EGN5QdiM1g5X5zqqbpKdJMCdc362MuiJ/lq8knJLMzpNScQeR49HO6/K4rTH BG9gwuNAzwXUAqsy7m5DAfBv6Xg0x2jRSR6GUrjAI2tvV93aOXnMN0tnXVO8Tt93EhQlc73VYqN7Il vzsxEJh6oKy+XoeLoiVnMUG6hot29+cOyNcyv6BzQNFFA= X-Developer-Key: i=ardb@kernel.org; a=openpgp; fpr=F43D03328115A198C90016883D200E9CA6329909 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221018_040454_368937_F98D809B X-CRM114-Status: GOOD ( 23.58 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Our segment alignment is 64k for all configurations, and coincidentally, this is the largest alignment supported by the PE/COFF executable format used by EFI. This means that generally, there is no need to move the image around in memory after it has been loaded by the firmware, which can be advantageous as it also permits us to rely on the memory attributes set by the firmware (R-X for [_text, __inittext_end] and RW- for [__initdata_begin, _end]. However, the minimum alignment of the image is actually 128k on 64k pages configurations with CONFIG_VMAP_STACK=y, due to the existence of a single 128k aligned object in the image, which is the stack of the init task. Let's work around this by adding some padding before the init stack allocation, so we can round down the stack pointer to a suitably aligned value if the image is not aligned to 128k in memory. Note that this does not affect the boot protocol, which still requires 2 MiB alignment for bare metal boot, but is only part of the internal contract between the EFI stub and the kernel proper. Signed-off-by: Ard Biesheuvel --- arch/arm64/include/asm/efi.h | 7 ------- arch/arm64/kernel/head.S | 3 +++ arch/arm64/kernel/vmlinux.lds.S | 11 ++++++++++- drivers/firmware/efi/libstub/arm64-stub.c | 2 +- include/linux/efi.h | 6 +----- 5 files changed, 15 insertions(+), 14 deletions(-) diff --git a/arch/arm64/include/asm/efi.h b/arch/arm64/include/asm/efi.h index 439e2bc5d5d8..3177e76de708 100644 --- a/arch/arm64/include/asm/efi.h +++ b/arch/arm64/include/asm/efi.h @@ -54,13 +54,6 @@ efi_status_t __efi_rt_asm_wrapper(void *, const char *, ...); /* arch specific definitions used by the stub code */ -/* - * In some configurations (e.g. VMAP_STACK && 64K pages), stacks built into the - * kernel need greater alignment than we require the segments to be padded to. - */ -#define EFI_KIMG_ALIGN \ - (SEGMENT_ALIGN > THREAD_ALIGN ? SEGMENT_ALIGN : THREAD_ALIGN) - /* * On arm64, we have to ensure that the initrd ends up in the linear region, * which is a 1 GB aligned region of size '1UL << (VA_BITS_MIN - 1)' that is diff --git a/arch/arm64/kernel/head.S b/arch/arm64/kernel/head.S index 2196aad7b55b..f168e3309704 100644 --- a/arch/arm64/kernel/head.S +++ b/arch/arm64/kernel/head.S @@ -398,6 +398,9 @@ SYM_FUNC_END(create_kernel_mapping) msr sp_el0, \tsk ldr \tmp1, [\tsk, #TSK_STACK] +#if THREAD_ALIGN > SEGMENT_ALIGN + bic \tmp1, \tmp1, #THREAD_ALIGN - 1 +#endif add sp, \tmp1, #THREAD_SIZE sub sp, sp, #PT_REGS_SIZE diff --git a/arch/arm64/kernel/vmlinux.lds.S b/arch/arm64/kernel/vmlinux.lds.S index 45131e354e27..0efccdf52be2 100644 --- a/arch/arm64/kernel/vmlinux.lds.S +++ b/arch/arm64/kernel/vmlinux.lds.S @@ -274,7 +274,16 @@ SECTIONS _data = .; _sdata = .; - RW_DATA(L1_CACHE_BYTES, PAGE_SIZE, THREAD_ALIGN) +#if THREAD_ALIGN > SEGMENT_ALIGN + /* + * Add some padding for the init stack so we can fix up any potential + * misalignment at runtime. In practice, this can only occur on 64k + * pages configurations with CONFIG_VMAP_STACK=y. + */ + . += THREAD_ALIGN - SEGMENT_ALIGN; + ASSERT(. == init_stack, "init_stack not at start of RW_DATA as expected") +#endif + RW_DATA(L1_CACHE_BYTES, PAGE_SIZE, SEGMENT_ALIGN) /* * Data written with the MMU off but read with the MMU on requires diff --git a/drivers/firmware/efi/libstub/arm64-stub.c b/drivers/firmware/efi/libstub/arm64-stub.c index e767a5ac8c3d..6229f42c797f 100644 --- a/drivers/firmware/efi/libstub/arm64-stub.c +++ b/drivers/firmware/efi/libstub/arm64-stub.c @@ -97,7 +97,7 @@ efi_status_t handle_kernel_image(unsigned long *image_addr, * 2M alignment if KASLR was explicitly disabled, even if it was not * going to be activated to begin with. */ - u64 min_kimg_align = efi_nokaslr ? MIN_KIMG_ALIGN : EFI_KIMG_ALIGN; + u64 min_kimg_align = efi_nokaslr ? MIN_KIMG_ALIGN : SEGMENT_ALIGN; if (IS_ENABLED(CONFIG_RANDOMIZE_BASE)) { efi_guid_t li_fixed_proto = LINUX_EFI_LOADED_IMAGE_FIXED_GUID; diff --git a/include/linux/efi.h b/include/linux/efi.h index 256e70e42114..1a395c24fdc0 100644 --- a/include/linux/efi.h +++ b/include/linux/efi.h @@ -422,11 +422,7 @@ void efi_native_runtime_setup(void); /* * This GUID may be installed onto the kernel image's handle as a NULL protocol * to signal to the stub that the placement of the image should be respected, - * and moving the image in physical memory is undesirable. To ensure - * compatibility with 64k pages kernels with virtually mapped stacks, and to - * avoid defeating physical randomization, this protocol should only be - * installed if the image was placed at a randomized 128k aligned address in - * memory. + * and moving the image in physical memory is undesirable. */ #define LINUX_EFI_LOADED_IMAGE_FIXED_GUID EFI_GUID(0xf5a37b6d, 0x3344, 0x42a5, 0xb6, 0xbb, 0x97, 0x86, 0x48, 0xc1, 0x89, 0x0a)