From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from atuin.qyliss.net (localhost [IPv6:::1]) by atuin.qyliss.net (Postfix) with ESMTP id C5C5415561; Wed, 12 Nov 2025 01:00:46 +0000 (UTC) Received: by atuin.qyliss.net (Postfix, from userid 993) id EF33D154DC; Wed, 12 Nov 2025 01:00:34 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on atuin.qyliss.net X-Spam-Level: X-Spam-Status: No, score=-0.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,DMARC_PASS,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE autolearn=unavailable autolearn_force=no version=4.0.1 Received: from mail-yx1-xb130.google.com (mail-yx1-xb130.google.com [IPv6:2607:f8b0:4864:20::b130]) by atuin.qyliss.net (Postfix) with ESMTPS id 7C1D3154CF for ; Wed, 12 Nov 2025 01:00:25 +0000 (UTC) Received: by mail-yx1-xb130.google.com with SMTP id 956f58d0204a3-640f88b873bso305700d50.3 for ; Tue, 11 Nov 2025 17:00:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1762909223; x=1763514023; darn=spectrum-os.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=/2Dt7mb/MZQanRi8DqS+9ULxt1N1Cth0v0MNfufBil0=; b=ILNT8VVPnPdXYAhyeE0yTFjFdTA00k9XUqXW9SPa5NRUYYPZ4GDKV/WqkEF9dgM33c jww6AiUDx3C1b4HE5eRyEcYVdUx8FlcxkEjKS4brNHRIm4PIZ/Hwa3ur5u+mgzPBr6bb L7Tbx2DbL/icHTXa7OhcqdOsekrAjJnFNtxXEEJq2mhI9Q4zePA5utYZ+mn3gobP8l8T mFoGHdi82s/FMYGCrqRX2k11eyz9vpORlcs0ax7KVMP27FHtGaF+IwWY3LgLompzjbaT urLkFLivegrKcDM+4RNi/0V+zED0K2D2pJN+C0bstXy3DTDCL29Yp4gy5n3OJY3AZeMm JjKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762909223; x=1763514023; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=/2Dt7mb/MZQanRi8DqS+9ULxt1N1Cth0v0MNfufBil0=; b=JsiMajVbQlxwuAXWNmOIHGSb9qYwDFK2gl+X4oyHeRYLV9Tg2sB2V13+u7GAm8IqpM 3TB6LbD5hE6/BLTAYh7mflzfOJb70qCFEgN6gCjjhvH3tonPFPQcI5gM0A64n6M0gc/T mGizwLm8yUKOSarqG/AT3mJtu/dnNqawvuRFWp2gC9Zu7gJNR39t3F2Ug7X8nu4g8F3H 26S1sxk6Dj4ElsN76AUM8yw6rj+SKuOkrIH31vsWcy9mGft5Yp2kpXr2+aq89cerGGAo 7NfNGf/l1W+HSJ6p6FE4L2xE40A8upM9tvLWi6ycn/znywy+rNvKKps6m4KJXBECZhYm tOdg== X-Gm-Message-State: AOJu0YxnPwU5aV8h4b9MwdD02cdeBA67zHiJnIpU7O97weBEizighHIv NpeYv0prKf3XESdDgP054au1czlP7XyfLuBhFsShBF7zu4nz2MLcoIe7eT4Vhu5J X-Gm-Gg: ASbGncvirRfgS077DO9xHVlPEEO43yrQ03eGya0rTn37XpQkMfHp4T9l7xsEELDInlY 19Y3s8fHRdvQi7Fsx/mquXYRgUGq3tlumlJb7BwLYdKuWYT1VM8Uq6GCFmyjo8KA68QVsnvSKcJ EvcKgWRJYfY+wNQvq1zfUiF79D6tbvrVJK1fwz67vc16L/6QacYNXdr4eoK+VZmp8NJBCC2NE9v imkRKJv4NkVJ8S18WqJ4/tuAlh38J1qXfjliZ/73ou0BBYjGa+ghu88XAAlQqlq8RdNlYjEtV77 Nk2D9FU5xaTode1rrPGlBRPoQp9NezsOWEIdrvMMhbIFZBk8thXL/UJJYho10nBwAWwjBiBiqKj y8Y9OcQPV18CzS05F5qJD+A9N+D+HFLeqyfyaijPOczquP28HSc0j5h+G6jR54uid7/KHbE48Pn Z1Tx6hBkp1A4hL6rvU1kgrGqe8pVhptlNc0ytyKR1pNJrOzmQOu+zi6RMTmmpUE8o6KuQGE7P9c 8biL38Y/zgGs6kYBR95bjMJ X-Google-Smtp-Source: AGHT+IGSeic19fhG/gp4LKIq8RtnDAVEAfagvLHLHV1A6l6XIPIqK3ITO6QhvM/poRKN+BMGWcj0ww== X-Received: by 2002:a05:690e:d8b:b0:63f:b624:4c14 with SMTP id 956f58d0204a3-64101a2b8efmr1175200d50.21.1762909223039; Tue, 11 Nov 2025 17:00:23 -0800 (PST) Received: from localhost.localdomain (h96-60-249-169.cncrtn.broadband.dynamic.tds.net. [96.60.249.169]) by smtp.gmail.com with UTF8SMTPSA id 00721157ae682-78811130fb1sm4701487b3.39.2025.11.11.17.00.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 Nov 2025 17:00:22 -0800 (PST) From: Demi Marie Obenour Date: Tue, 11 Nov 2025 19:59:09 -0500 Subject: [PATCH v3 2/2] Move UKI creation to a separate derivation MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20251111-refactor-verity-v3-2-575726639f9e@gmail.com> References: <20251111-refactor-verity-v3-0-575726639f9e@gmail.com> In-Reply-To: <20251111-refactor-verity-v3-0-575726639f9e@gmail.com> To: Spectrum OS Development X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1762909147; l=5927; i=demiobenour@gmail.com; s=20250729; h=from:subject:message-id; bh=oYb4pKOFmIiLHYwvZJJQBzVY14osr+g10EDXznM438I=; b=rdjroMgBBhhm8ek0lXFiwz/ij0RD6B4PleMxgViq9hFZ/O4lSKg3chO3wPN4L55rRs9p84AaS QGiiw6PCgrKD2tiaVDRENqHVV8YZYyJB49pvSSDcrp5p6NOXNlbbmNG X-Developer-Key: i=demiobenour@gmail.com; a=ed25519; pk=X57Q4/YQDj9t4SBeKaDwvXYKB6quZJVx/DE2Ly2out0= Message-ID-Hash: CPNWCLDJ555IW3GLKNS2OJD53WAUJA2V X-Message-ID-Hash: CPNWCLDJ555IW3GLKNS2OJD53WAUJA2V X-MailFrom: demiobenour@gmail.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-devel.spectrum-os.org-0; header-match-devel.spectrum-os.org-1; header-match-devel.spectrum-os.org-2; header-match-devel.spectrum-os.org-3; header-match-devel.spectrum-os.org-4; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Demi Marie Obenour , Alyssa Ross X-Mailman-Version: 3.3.9 Precedence: list List-Id: Patches and low-level development discussion Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: It will be used by the update code later. No functional change intended, other than a trivial shell script refactoring. Signed-off-by: Demi Marie Obenour --- host/efi.nix | 46 ++++++++++++++++++++++++++++++++++++++++++++++ pkgs/default.nix | 1 + release/live/Makefile | 15 ++------------- release/live/default.nix | 19 +++++-------------- 4 files changed, 54 insertions(+), 27 deletions(-) diff --git a/host/efi.nix b/host/efi.nix new file mode 100644 index 0000000000000000000000000000000000000000..a2b47fd050fbf00050473a0d5a1373eb96c341b5 --- /dev/null +++ b/host/efi.nix @@ -0,0 +1,46 @@ +# SPDX-License-Identifier: EUPL-1.2+ +# SPDX-FileCopyrightText: 2021-2024 Alyssa Ross +# SPDX-FileCopyrightText: 2025 Demi Marie Obenour + +import ../lib/call-package.nix ( +{ bash, callSpectrumPackage, cryptsetup, runCommand +, stdenv, systemdUkify, rootfs +}: +let + initramfs = callSpectrumPackage ./initramfs {}; + kernel = "${rootfs.kernel}/${stdenv.hostPlatform.linux-kernel.target}"; + systemd = systemdUkify.overrideAttrs ({ mesonFlags ? [], ... }: { + # The default limit is too low to build a generic aarch64 distro image: + # https://github.com/systemd/systemd/pull/37417 + mesonFlags = mesonFlags ++ [ "-Defi-stub-extra-sections=3000" ]; + }); +in + +runCommand "spectrum-efi" { + nativeBuildInputs = [ cryptsetup systemd bash ]; + __structuredAttrs = true; + unsafeDiscardReferences = { out = true; }; + dontFixup = true; + passthru = { inherit systemd; }; + env = { + DTBS = "${rootfs.kernel}/dtbs"; + KERNEL = kernel; + INITRAMFS = initramfs; + ROOTFS = rootfs; + }; +} '' + read -r roothash < "$ROOTFS/rootfs.verity.roothash" + { \ + printf "[UKI]\nDeviceTreeAuto=" + if [ -d "$DTBS" ]; then + find "$DTBS" -name '*.dtb' -print0 | tr '\0' ' ' + fi + } | ukify build \ + --output "$out" \ + --config /dev/stdin \ + --linux "$KERNEL" \ + --initrd "$INITRAMFS" \ + --os-release $'NAME="Spectrum"\n' \ + --cmdline "ro intel_iommu=on roothash=$roothash" + '' +) (_: {}) diff --git a/pkgs/default.nix b/pkgs/default.nix index cc60228a10cddcb70e5ab9faa1bab7d74f3ebb35..c9f6dcfad9369567468b30d1c5697e3551a7b236 100644 --- a/pkgs/default.nix +++ b/pkgs/default.nix @@ -36,6 +36,7 @@ let path: (import path { inherit (self) callPackage; }).override; rootfs = self.callSpectrumPackage ../host/rootfs {}; + efi = self.callSpectrumPackage ../host/efi.nix {}; spectrum-build-tools = self.callSpectrumPackage ../tools { appSupport = false; buildSupport = true; diff --git a/release/live/Makefile b/release/live/Makefile index 191b44944af0adf965e1d5f2785719b236bfd99c..4de8743f42dec65aa863c3020cd70124316a6118 100644 --- a/release/live/Makefile +++ b/release/live/Makefile @@ -19,19 +19,8 @@ $(dest): ../../scripts/format-uuid.sh ../../scripts/make-gpt.sh ../../scripts/sf build/empty: mkdir -p $@ -build/spectrum.efi: $(DTBS) $(KERNEL) $(INITRAMFS) $(ROOT_FS_VERITY_ROOTHASH) - { \ - printf "[UKI]\nDeviceTreeAuto=" && \ - find $(DTBS) -name '*.dtb' -print0 | tr '\0' ' ' ;\ - } | $(UKIFY) build \ - --output $@ \ - --config /dev/stdin \ - --linux $(KERNEL) \ - --initrd $(INITRAMFS) \ - --os-release $$'NAME="Spectrum"\n' \ - --cmdline "ro intel_iommu=on roothash=$$(cat "$$ROOT_FS_VERITY_ROOTHASH")" - -build/boot.fat: $(SYSTEMD_BOOT_EFI) build/spectrum.efi +build/boot.fat: $(SYSTEMD_BOOT_EFI) $(EFI_IMAGE) build/empty + ln -sf -- "$$EFI_IMAGE" build/spectrum.efi $(TRUNCATE) -s 440401920 $@ $(MKFS_FAT) $@ $(MMD) -i $@ ::/EFI ::/EFI/BOOT ::/EFI/Linux diff --git a/release/live/default.nix b/release/live/default.nix index 9a62d4da9cfea11d94d2a1d5764d41587efd5ad5..c234d87e62cc9ae65ba60f94bab6e58b43beddbc 100644 --- a/release/live/default.nix +++ b/release/live/default.nix @@ -6,7 +6,7 @@ import ../../lib/call-package.nix ( { callSpectrumPackage, spectrum-build-tools, rootfs, src , lib, pkgsStatic, stdenvNoCC , cryptsetup, dosfstools, jq, mtools, util-linux -, systemdUkify +, systemdUkify, efi }: let @@ -14,13 +14,6 @@ let stdenv = stdenvNoCC; - systemd = systemdUkify.overrideAttrs ({ mesonFlags ? [], ... }: { - # The default limit is too low to build a generic aarch64 distro image: - # https://github.com/systemd/systemd/pull/37417 - mesonFlags = mesonFlags ++ [ "-Defi-stub-extra-sections=3000" ]; - }); - - initramfs = callSpectrumPackage ../../host/initramfs {}; efiArch = stdenv.hostPlatform.efiArch; in @@ -40,19 +33,17 @@ stdenv.mkDerivation { sourceRoot = "source/release/live"; nativeBuildInputs = [ - cryptsetup dosfstools jq spectrum-build-tools mtools systemd util-linux + cryptsetup dosfstools jq spectrum-build-tools mtools util-linux ]; env = { - INITRAMFS = initramfs; KERNEL = "${rootfs.kernel}/${stdenv.hostPlatform.linux-kernel.target}"; ROOT_FS = "${rootfs}/rootfs"; ROOT_FS_VERITY = "${rootfs}/rootfs.verity.superblock"; ROOT_FS_VERITY_ROOTHASH = "${rootfs}/rootfs.verity.roothash"; - SYSTEMD_BOOT_EFI = "${systemd}/lib/systemd/boot/efi/systemd-boot${efiArch}.efi"; + SYSTEMD_BOOT_EFI = "${efi.systemd}/lib/systemd/boot/efi/systemd-boot${efiArch}.efi"; + EFI_IMAGE = efi; EFINAME = "BOOT${toUpper efiArch}.EFI"; - } // lib.optionalAttrs stdenv.hostPlatform.linux-kernel.DTB or false { - DTBS = "${rootfs.kernel}/dtbs"; }; buildFlags = [ "dest=$(out)" ]; @@ -65,6 +56,6 @@ stdenv.mkDerivation { unsafeDiscardReferences = { out = true; }; dontFixup = true; - passthru = { inherit initramfs rootfs; }; + passthru = { inherit rootfs; }; } ) (_: {}) -- 2.51.2