From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [IPv6:2a0f:8001:1:32::40]) by lore.proxmox.com (Postfix) with ESMTPS id 0899B1FF0E1 for ; Mon, 27 Jul 2026 17:27:54 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id 81EE12156A; Mon, 27 Jul 2026 17:27:39 +0200 (CEST) From: Fiona Ebner To: pve-devel@lists.proxmox.com Subject: [PATCH edk2-firmware v2 2/4] d/{rules,install}: build secureboot-enabled image for aarch64 Date: Mon, 27 Jul 2026 17:27:16 +0200 Message-ID: <20260727152730.947259-3-f.ebner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260727152730.947259-1-f.ebner@proxmox.com> References: <20260727152730.947259-1-f.ebner@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1785166019292 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.192 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) RCVD_IN_DNSWL_LOW -0.7 Sender listed at https://www.dnswl.org/, low trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: C2DGD65YEKLDLSIKX4YOM46T3THV2DVH X-Message-ID-Hash: C2DGD65YEKLDLSIKX4YOM46T3THV2DVH X-MailFrom: f.ebner@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Previously, a second copy of the CODE image was shipped as /usr/share/pve-edk2-firmware/aarch64/QEMU_EFI.fd, but this is not used by qemu-server and it was not yet expanded to 64MiB so it is wouldn't even have worked with QEMU. Just drop it. The BUILD_SHELL=FALSE flag is added for completeness. Not including the shell is already guarded by edk2 commit f881b4d129 ("OvmfPkg: only add shell to FV in case secure boot is disabled"), but it doesn't hurt to be explicit and Debian does it too. Signed-off-by: Fiona Ebner Tested-by: Lukas Sichert --- Changes in v2: * move hunk for .install to previous patch debian/rules | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/debian/rules b/debian/rules index 90a4f6a5cc..8abc6a7d70 100755 --- a/debian/rules +++ b/debian/rules @@ -44,6 +44,7 @@ AAVMF_FLAGS = $(COMMON_FLAGS) AAVMF_FLAGS += -DTPM2_ENABLE=TRUE AAVMF_FLAGS += -DTPM2_CONFIG_ENABLE=TRUE AAVMF_FLAGS += -DCAVIUM_ERRATUM_27456=TRUE +AAVMF_SECBOOT_FLAGS = $(AAVMF_FLAGS) -DBUILD_SHELL=FALSE -DSECURE_BOOT_ENABLE=TRUE RISCV64_FLAGS = $(COMMON_FLAGS) @@ -108,7 +109,7 @@ AAVMF_BUILD_DIR = $(AAVMF_BUILD_ROOT)/$(BUILD_TYPE)_$(EDK2_TOOLCHAIN) AAVMF_ENROLL = $(AAVMF_BUILD_DIR)/AARCH64/EnrollDefaultKeys.efi AAVMF_SHELL = $(AAVMF_BUILD_DIR)/AARCH64/Shell.efi AAVMF_BINARIES = $(AAVMF_ENROLL) $(AAVMF_SHELL) -AAVMF_IMAGES = $(addprefix $(AAVMF_INSTALL_DIR)/,AAVMF_CODE.fd AAVMF_VARS.fd) +AAVMF_IMAGES = $(addprefix $(AAVMF_INSTALL_DIR)/,AAVMF_CODE.fd AAVMF_CODE.secboot.fd AAVMF_VARS.fd) AAVMF_PREENROLLED_VARS = $(addprefix $(AAVMF_INSTALL_DIR)/,AAVMF_VARS.ms.fd AAVMF_VARS.snakeoil.fd) RISCV64_BUILD_ROOT = Build/RiscVVirtQemu @@ -201,7 +202,7 @@ enroll_snakeoil = virt-fw-vars --input $(1) --output $(2) \ --add-db OvmfEnrollDefaultKeys \ debian/PkKek-1-snakeoil.pem -%/AAVMF_VARS.ms.fd: %/AAVMF_CODE.fd %/AAVMF_VARS.fd debian/PkKek-1-vendor.pem $(AAVMF_ENROLL) $(AAVMF_SHELL) +%/AAVMF_VARS.ms.fd: %/AAVMF_CODE.secboot.fd %/AAVMF_VARS.fd debian/PkKek-1-vendor.pem $(AAVMF_ENROLL) $(AAVMF_SHELL) $(call enroll_vendor,$(AAVMF_INSTALL_DIR)/AAVMF_VARS.fd,$@,arm64) %/AAVMF_VARS.snakeoil.fd: %/AAVMF_CODE.fd %/AAVMF_VARS.fd debian/PkKek-1-snakeoil.pem $(AAVMF_ENROLL) $(AAVMF_SHELL) @@ -241,6 +242,15 @@ build-qemu-efi: debian/setup-build-stamp $(QEMU_EFI_INSTALL_DIR)/$(FW_NAME)_CODE.fd cp $(QEMU_EFI_BUILD_DIR)/FV/QEMU_VARS.fd \ $(QEMU_EFI_INSTALL_DIR)/$(FW_NAME)_VARS.fd + rm -rf $(QEMU_EFI_BUILD_ROOT) + set -e; . ./edksetup.sh; \ + build -a $(EDK2_HOST_ARCH) \ + -t $(EDK2_TOOLCHAIN) \ + -p ArmVirtPkg/ArmVirtQemu.dsc \ + $(AAVMF_SECBOOT_FLAGS) -b $(BUILD_TYPE) + truncate -s 64M $(QEMU_EFI_BUILD_DIR)/FV/QEMU_EFI.fd + cp $(QEMU_EFI_BUILD_DIR)/FV/QEMU_EFI.fd \ + $(QEMU_EFI_INSTALL_DIR)/$(FW_NAME)_CODE.secboot.fd build-qemu-efi-aarch64: $(AAVMF_BINARIES) $(AAVMF_IMAGES) $(AAVMF_PREENROLLED_VARS) $(AAVMF_BINARIES) $(AAVMF_IMAGES): BaseTools/Bin/GccLto/liblto-aarch64.a -- 2.47.3