From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F02F0606A33B for ; Tue, 10 Nov 2026 12:18:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=DjI+sBvv; dkim=fail reason="signature verification failed" (2048-bit key) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20260707 header.b=C+tY6Ma1 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:Cc:To:From:Subject:Message-ID: References:Mime-Version:In-Reply-To:Date:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Unsubscribe-Post:List-Owner; bh=tOvPPStHK523KlXXwsOwSQebYUXavKMOrWsFxvvN/tk=; b=DjI+sBvvJBXoKBVnEF6dv/nvcM 19pkLZW5TRsD1Z1tnleJ/rVxYxk4qcfcP0B5xTaHm5bM4T+zgT+jwL4NPWJ6JZs84+CeJY10oh6IA Zj9Uj33NiPBHtr5In6rFIoGmALniltMt77VKyl7wwVyZqClwl2Dr9b12ByJqvz197OlpFn7eLHgCt y2vVVZwC+ZQZGin31IN/fLZ+QHkEwhrEcX4uKTaFBsqcsNW1cItHdJ0q6h4eR93rkgw04xyPNCP8R Q4sUwwq+cJB+9FpOWBKtHyIPvDlWj9gATeveE7QezYIDuq7wCQjdsf9bts1v8zU4mwDHL8sW05A++ MldSSS0g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xQknf-00000003nN5-2KLr; Tue, 10 Nov 2026 12:18:07 +0000 Received: from mail-wm1-x348.google.com ([2a00:1450:4864:20::348]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xQknQ-00000003n3P-1dTb for linux-riscv@lists.infradead.org; Tue, 10 Nov 2026 12:17:53 +0000 Received: by mail-wm1-x348.google.com with SMTP id 5b1f17b1804b1-4a0284121c6so14462675e9.3 for ; Tue, 10 Nov 2026 04:17:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20260707; t=1794313070; x=1794917870; darn=lists.infradead.org; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:from:to:cc:subject:date:message-id:reply-to :content-type; bh=/NOGXHiMglGNlkAHWoIUlx58rvJ+/z9H2Q0oQTuWMl4=; b=C+tY6Ma1QD4C2MfoB9BZ7aHw2Qnjsyqe/EodVXWGEHypGhmSZg/+mX/4jiQ2U27kKG Sen0Ew2KFgRtbPjpgNT4wL3lIXGXee6T51MEOJfrEquZRefq9pV1Vl7du+6aOYEWIoIY 5nxzOHdA+d2jiXNKaXQP7dLFroeK6DNUkgRUJ6PvOBZE5I7Bqh1FbG3IzViukj3S1qH2 GrfxTNbyG97R6Fc9r4vWDHwQmmIFCmStHLy9aB9ROuXalxhtKfiITLCvhNUcDYTCfYev LxnfsrMSX0jIu5ftMEx6azM6t14m8AltbT8zK/pQP7xz4GA+C5QnK3ePG2CrGYNCgSM8 wmDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1794313070; x=1794917870; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=/NOGXHiMglGNlkAHWoIUlx58rvJ+/z9H2Q0oQTuWMl4=; b=EaAoSF1vB2H9rAUsLVYHJ/ZEUyeKBc92o7UhnY6xwPpUQzU5ThjqHL9lb5Uh++xA+G wlQ9317GhX8f+Q1cR9Arw0711iEfcv5ZmAcytxAdCWaY0aZTjrZw8ahwl3Wv6N8VJE7T TfMW6kvde7EwOFEX9PSyg5QgsOmcg40qVNhWBlwJ0QIV4ZvPxvFDyGugyfyqsBcQHUZM IycRFORHCbKXBrWq74cfQNTAvQUKiXFo7d+NweZ8CZ+3U4KyXeMz1EVsdY5jlEiM5STW XLSoWz7VzbTWVCw0ejTsKwau5qIZQTpNutmbbZPF/nfDtOtBiraMwOQg/qTiz8jHfNUX J72Q== X-Forwarded-Encrypted: i=1; AL+hErEiwy1CyDl7iChTKscdKr3nKYZUL97n7PF0xAvj/XBI+LyVap5Qcc23tetZEZKn0/KG/j6guYAoaKLYlA==@lists.infradead.org X-Gm-Message-State: APuU4Z9ukc5igVM0nNxmeboXSzftM21Iw26dMxt6lz1t/33WExv7se86 i0dKMHQNy8h5oZIOiBW1xdf+46p0HDCmQWvA0G17WtctB1+7CzVLvn4BYVCTce+alcUujOEX9w= = X-Received: from wmbgx15.prod.google.com ([2002:a05:600c:858f:b0:4a2:8ef:a5af]) (user=ardb job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:1392:b0:4a2:1e1:d55 with SMTP id 5b1f17b1804b1-4ba02355ab0mr37537145e9.28.1794313070010; Tue, 10 Nov 2026 04:17:50 -0800 (PST) Date: Tue, 10 Nov 2026 13:17:41 +0100 In-Reply-To: <20261110121731.2728824-12-ardb+git@google.com> X-Archived-By: Mime-Version: 1.0 References: <20261110121731.2728824-12-ardb+git@google.com> X-Mailer: git-send-email 2.56.0.542.g3ef89b7020-goog Message-ID: <20261110121731.2728824-21-ardb+git@google.com> Subject: [PATCH v2 09/10] efi/zboot: Disentangle efi_remap_image() from efi_cache_sync_image() From: Ard Biesheuvel To: linux-efi@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, Ard Biesheuvel , Jason Gunthorpe , Paul Walmsley , Palmer Dabbelt , Albert Ou , Huacai Chen , loongarch@lists.linux.dev, linux-riscv@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20261110_041752_455933_180C4924 X-CRM114-Status: GOOD ( 10.32 ) X-Spam-Score: -9.5 (---------) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" X-Spam-Status: No, score=-16.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_ADSP_CUSTOM_MED,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 From: Ard Biesheuvel Refactor efi_cache_sync_image() so that it is not in charge of calling efi_remap_image(), which is really the wrong way around. Instead, call efi_remap_image() from the primary zboot procedure, where it is more straight-forward to obtain the code size from the loaded image. Signed-off-by: Ard Biesheuvel --- drivers/firmware/efi/libstub/arm64.c | 2 -- drivers/firmware/efi/libstub/zboot-decompress-gzip.c | 2 -- drivers/firmware/efi/libstub/zboot-decompress-zstd.c | 2 -- drivers/firmware/efi/libstub/zboot.c | 14 +++++++++++++- 4 files changed, 13 insertions(+), 7 deletions(-) diff --git a/drivers/firmware/efi/libstub/arm64.c b/drivers/firmware/efi/libstub/arm64.c index 62fc8e53c8a0..fdd48ba0f9f1 100644 --- a/drivers/firmware/efi/libstub/arm64.c +++ b/drivers/firmware/efi/libstub/arm64.c @@ -114,8 +114,6 @@ void efi_cache_sync_image(unsigned long image_base, asm("ic ialluis"); dsb(ish); isb(); - - efi_remap_image(image_base, alloc_size, code_size); } unsigned long __weak primary_entry_offset(void) diff --git a/drivers/firmware/efi/libstub/zboot-decompress-gzip.c b/drivers/firmware/efi/libstub/zboot-decompress-gzip.c index e97a7e9d3c98..79cf8c48b033 100644 --- a/drivers/firmware/efi/libstub/zboot-decompress-gzip.c +++ b/drivers/firmware/efi/libstub/zboot-decompress-gzip.c @@ -62,7 +62,5 @@ efi_status_t efi_zboot_decompress(u8 *out, unsigned long outlen) return EFI_LOAD_ERROR; } - efi_cache_sync_image((unsigned long)out, outlen); - return EFI_SUCCESS; } diff --git a/drivers/firmware/efi/libstub/zboot-decompress-zstd.c b/drivers/firmware/efi/libstub/zboot-decompress-zstd.c index bde9d94dd2e3..dd7f11238154 100644 --- a/drivers/firmware/efi/libstub/zboot-decompress-zstd.c +++ b/drivers/firmware/efi/libstub/zboot-decompress-zstd.c @@ -43,7 +43,5 @@ efi_status_t efi_zboot_decompress(u8 *out, unsigned long outlen) return EFI_LOAD_ERROR; } - efi_cache_sync_image((unsigned long)out, outlen); - return EFI_SUCCESS; } diff --git a/drivers/firmware/efi/libstub/zboot.c b/drivers/firmware/efi/libstub/zboot.c index c6071b0e8f1a..25ff5960771c 100644 --- a/drivers/firmware/efi/libstub/zboot.c +++ b/drivers/firmware/efi/libstub/zboot.c @@ -38,6 +38,17 @@ const struct efi_image_info *zboot_get_image_info(unsigned long image_base) return (void *)image_base + efistub_image_info_offset; } +static efi_status_t efi_zboot_remap_image(const struct efi_image_info *image_info, + unsigned long image_base, + unsigned long image_size) +{ + unsigned long code_size = efistub_kernel_codesize(image_info); + + efi_cache_sync_image(image_base, code_size); + + return efi_remap_image(image_base, image_size, code_size); +} + asmlinkage efi_status_t __efiapi efi_zboot_entry(efi_handle_t handle, efi_system_table_t *systab) { @@ -105,7 +116,8 @@ efi_zboot_entry(efi_handle_t handle, efi_system_table_t *systab) goto error; auto image_info = zboot_get_image_info(image_base); - status = efi_stub_common(handle, image, image_base, cmdline_ptr, + status = efi_zboot_remap_image(image_info, image_base, alloc_size) ?: + efi_stub_common(handle, image, image_base, cmdline_ptr, image_info); error: -- 2.56.0.542.g3ef89b7020-goog _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv