From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp0 ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms11 with LMTPS id 2HB7GEjLxl6cfgAA0tVLHw (envelope-from ) for ; Thu, 21 May 2020 18:41:12 +0000 Received: from aspmx1.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp0 with LMTPS id VChcFEjLxl7TTwAA1q6Kng (envelope-from ) for ; Thu, 21 May 2020 18:41:12 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id A1E8D94038D for ; Thu, 21 May 2020 18:41:11 +0000 (UTC) Received: from localhost ([::1]:58344 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jbq7w-0000F2-Qz for larch@yhetil.org; Thu, 21 May 2020 14:41:08 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45502) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jbq7q-0000CC-5b for guix-patches@gnu.org; Thu, 21 May 2020 14:41:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:45711) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jbq7p-0001jp-RK for guix-patches@gnu.org; Thu, 21 May 2020 14:41:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jbq7p-0003kO-OG for guix-patches@gnu.org; Thu, 21 May 2020 14:41:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#41011] [PATCH] gnu: grub: Support for network boot via tftp/nfs. Resent-From: Stefan Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Thu, 21 May 2020 18:41:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 41011 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Mathieu Othacehe Cc: 41011@debbugs.gnu.org Received: via spool by 41011-submit@debbugs.gnu.org id=B41011.159008645814387 (code B ref 41011); Thu, 21 May 2020 18:41:01 +0000 Received: (at 41011) by debbugs.gnu.org; 21 May 2020 18:40:58 +0000 Received: from localhost ([127.0.0.1]:57257 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jbq7l-0003jy-Mv for submit@debbugs.gnu.org; Thu, 21 May 2020 14:40:58 -0400 Received: from vsmx011.vodafonemail.xion.oxcs.net ([153.92.174.89]:41073) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jbq7j-0003jk-MQ for 41011@debbugs.gnu.org; Thu, 21 May 2020 14:40:57 -0400 Received: from vsmx003.vodafonemail.xion.oxcs.net (unknown [192.168.75.197]) by mta-5-out.mta.xion.oxcs.net (Postfix) with ESMTP id 63C5A59D49B; Thu, 21 May 2020 18:40:49 +0000 (UTC) Received: from macbook-pro.kuh-wiese.my-router.de (unknown [2.206.141.243]) by mta-7-out.mta.xion.oxcs.net (Postfix) with ESMTPA id 0E355539BB2; Thu, 21 May 2020 18:40:44 +0000 (UTC) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\)) From: Stefan In-Reply-To: Date: Thu, 21 May 2020 20:40:44 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: References: <9AAFEFF4-8ACE-4C95-975F-67C3F4FDAF81@vodafonemail.de> <87a72gi4kz.fsf@gmail.com> <1179D890-7D6C-43D8-A286-DA7A0F61D585@vodafonemail.de> <0CE089DD-54B6-43AD-8CE7-8EC7004768EB@vodafonemail.de> X-Mailer: Apple Mail (2.3124) X-VADE-STATUS: LEGIT X-Spam-Score: -0.7 (/) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-Spam-Score: -1.7 (-) X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Scanner: scn0 Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Spam-Score: -0.51 X-TUID: gx6cxnF7Kcbb * gnu/bootloader/grub.scm (grub-efi-net-bootloader): New efi bootloader = for network booting via tftp/nfs, prepared for chain loading. (make-grub-efi-net-bootloader): New macro to define a customized = bootloader based on 'grub-efi-net-bootloader'. (install-grub-efi-net): New bootloader installer for tftp. (grub-root-search): Adding support for tftp root. (eye-candy): Use 'gfxterm' for all systems if selected via = 'terminal-outputs'. * gnu/system.scm (read-boot-parameters): Prevent devices with ":/" from = being treated as a file system label. --- gnu/bootloader/grub.scm | 125 +++++++++++++++++++++++++++++----------- gnu/system.scm | 3 +- 2 files changed, 94 insertions(+), 34 deletions(-) diff --git a/gnu/bootloader/grub.scm b/gnu/bootloader/grub.scm index bb40c551a7..2b0ecec279 100644 --- a/gnu/bootloader/grub.scm +++ b/gnu/bootloader/grub.scm @@ -23,7 +23,7 @@ =20 (define-module (gnu bootloader grub) #:use-module (guix records) - #:use-module ((guix utils) #:select (%current-system)) + #:use-module ((guix utils) #:select (%current-system = %current-target-system)) #:use-module (guix gexp) #:use-module (gnu artwork) #:use-module (gnu bootloader) @@ -47,6 +47,8 @@ =20 grub-bootloader grub-efi-bootloader + make-grub-efi-net-bootloader + grub-efi-net-bootloader grub-mkrescue-bootloader grub-minimal-bootloader =20 @@ -140,36 +142,20 @@ file with the resolution provided in CONFIG." concerned with graphics mode, background images, colors, and all that. STORE-DEVICE designates the device holding the store, and = STORE-MOUNT-POINT is its mount point; these are used to determine where the background image = and -fonts must be searched for. SYSTEM must be the target system = string---e.g., -\"x86_64-linux\". BTRFS-STORE-SUBVOLUME-FILE-NAME is the file name of = the -Btrfs subvolume, to be prepended to any store path, if any." - (define setup-gfxterm-body - (let ((gfxmode - (or (and-let* ((theme (bootloader-configuration-theme = config)) - (gfxmode (grub-theme-gfxmode theme))) - (string-join gfxmode ";")) - "auto"))) - - ;; Intel and EFI systems need to be switched into graphics mode, = whereas - ;; most other modern architectures have no other mode and = therefore - ;; don't need to be switched. - - ;; XXX: Do we really need to restrict to x86 systems? We could = imitate - ;; what the GRUB default configuration does and decide based on = whether - ;; a user provided 'gfxterm' in the terminal-outputs field of = their - ;; bootloader-configuration record. - (if (string-match "^(x86_64|i[3-6]86)-" system) - (format #f " - set gfxmode=3D~a - insmod all_video - insmod gfxterm~%" gfxmode) - ""))) - +fonts must be searched for. BTRFS-STORE-SUBVOLUME-FILE-NAME is the = file name +of the Btrfs subvolume, to be prepended to any store path, if any." (define (setup-gfxterm config font-file) (if (memq 'gfxterm (bootloader-configuration-terminal-outputs = config)) - #~(format #f "if loadfont ~a; then - setup_gfxterm -fi~%" #+font-file) + #~(format #f " +if loadfont ~a; then + set gfxmode=3D~a + insmod all_video + insmod gfxterm +fi~%" + #$font-file + #$(string-join + (grub-theme-gfxmode (bootloader-theme config)) + ";")) "")) =20 (define (theme-colors type) @@ -190,8 +176,6 @@ fi~%" #+font-file) =20 (and image #~(format #$port " -function setup_gfxterm {~a} - # Set 'root' to the partition that contains /gnu/store. ~a =20 @@ -206,7 +190,6 @@ else set menu_color_normal=3Dcyan/blue set menu_color_highlight=3Dwhite/blue fi~%" - #$setup-gfxterm-body #$(grub-root-search store-device font-file) #$(setup-gfxterm config font-file) #$(grub-setup-io config) @@ -313,6 +296,9 @@ code." ((? file-system-label? label) (format #f "search --label --set ~a" (file-system-label->string label))) + ((? (lambda (device) + (and (string? device) (string-contains device ":/"))) = nfs-uri) + "set root=3D(tftp)") ((or #f (? string?)) #~(format #f "search --file --set ~a" #$file))))) =20 @@ -454,6 +440,68 @@ fi~%")))) "--bootloader-id=3DGuix" "--efi-directory" target-esp)))) =20 +(define (install-grub-efi-net efi-subdir) + "Define a grub-efi bootloader installer for installation in = EFI-SUBDIR, +which is usually \"efi/boot\" or \"efi/Guix\"." + (let* ((arch (car (string-split (or (%current-target-system) + (%current-system)) + #\-))) + (efi-bootloader-link (string-append "boot" + (match arch + ("i686" "ia32") + ("x86_64" "x64") + ("arm" "arm") + ("armhf" "arm") + ("aarch64" "aa64") + ("riscv" "riscv32") + ("riscv64" "riscv64")) + ".efi")) + (efi-bootloader (string-append (match arch + ("i686" "i386") + ("x86_64" "x86_64") + ("arm" "arm") + ("armhf" "arm") + ("aarch64" "arm64") + ("riscv" "riscv32") + ("riscv64" "riscv64")) + "-efi/core.efi"))) + #~(lambda (bootloader target mount-point) + "Install GRUB as e.g. \"bootx64.efi\" or \"bootarm64.efi\" = \"into +EFI-SUBDIR, which is usually \"efi/boot\" or \"efi/Guix\" below the = directory +TARGET for the system whose root is mounted at MOUNT-POINT." + (let* ((mount-point-list (delete "" (string-split mount-point = #\/))) + (target-list (delete "" (string-split target #\/))) + (net-dir + (string-append "/" (string-join (append + mount-point-list + target-list) + "/"))) + (subdir #$efi-subdir) + (efi-bootloader-link + (string-append net-dir "/" subdir "/" = #$efi-bootloader-link)) + (store-name (car (delete "" (string-split bootloader = #\/)))) + (store + ;; Use target-list to construct a "../gnu" link with a = correct + ;; number of "../" to the store. + (string-join (append (make-list (length target-list) = "..") + (list store-name)) + "/")) + (store-link (string-append net-dir "/" store-name))) + ;; Tell 'grub-install' that there might be a LUKS-encrypted = /boot or + ;; root partition. + (setenv "GRUB_ENABLE_CRYPTODISK" "y") + (invoke/quiet (string-append bootloader "/bin/grub-mknetdir") + (string-append "--net-directory=3D" net-dir) + (string-append "--subdir=3D" subdir)) + (false-if-exception + (delete-file efi-bootloader-link)) + (symlink #$efi-bootloader + efi-bootloader-link) + (false-if-exception + (delete-file store-link)) + (symlink store + store-link))))) + ^L =20 ;;; @@ -483,7 +531,18 @@ fi~%")))) (name 'grub-efi) (package grub-efi))) =20 -(define* grub-mkrescue-bootloader +(define-syntax-rule (make-grub-efi-net-bootloader bootloader-name = target efi-subdir) + (define bootloader-name + (bootloader + (inherit grub-bootloader) + (name (quote bootloader-name)) + (package grub-efi) + (installer (install-grub-efi-net efi-subdir)) + (configuration-file (string-append target "/" efi-subdir = "/grub.cfg"))))) + +(make-grub-efi-net-bootloader grub-efi-net-bootloader "boot" = "efi/boot") + +(define grub-mkrescue-bootloader (bootloader (inherit grub-efi-bootloader) (package grub-hybrid))) diff --git a/gnu/system.scm b/gnu/system.scm index d929187695..2035235549 100644 --- a/gnu/system.scm +++ b/gnu/system.scm @@ -301,7 +301,8 @@ file system labels." ((? string? device) ;; It used to be that we would not distinguish between labels = and ;; device names. Try to infer the right thing here. - (if (string-prefix? "/dev/" device) + (if (or (string-prefix? "/dev/" device) + (string-contains device ":/")) ; nfs device (file-system-label device))))) =20 --=20 2.26.0