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 cEx1LagBw17nKQAA0tVLHw (envelope-from ) for ; Mon, 18 May 2020 21:44:08 +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 sHlhKagBw16EfAAA1q6Kng (envelope-from ) for ; Mon, 18 May 2020 21:44:08 +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 F3F86940B62 for ; Mon, 18 May 2020 21:44:07 +0000 (UTC) Received: from localhost ([::1]:57344 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1janYM-0001lr-Cv for larch@yhetil.org; Mon, 18 May 2020 17:44:06 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:39848) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1janYJ-0001kq-1o for guix-patches@gnu.org; Mon, 18 May 2020 17:44:03 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:36931) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1janYI-0002Z1-Ix for guix-patches@gnu.org; Mon, 18 May 2020 17:44:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1janYI-0001oa-Eh for guix-patches@gnu.org; Mon, 18 May 2020 17:44:02 -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: Mon, 18 May 2020 21:44:02 +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.15898382396959 (code B ref 41011); Mon, 18 May 2020 21:44:02 +0000 Received: (at 41011) by debbugs.gnu.org; 18 May 2020 21:43:59 +0000 Received: from localhost ([127.0.0.1]:48477 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1janY0-0001nt-DN for submit@debbugs.gnu.org; Mon, 18 May 2020 17:43:59 -0400 Received: from vsmx012.vodafonemail.xion.oxcs.net ([153.92.174.90]:23615) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1janXy-0001ne-9n for 41011@debbugs.gnu.org; Mon, 18 May 2020 17:43:43 -0400 Received: from vsmx004.vodafonemail.xion.oxcs.net (unknown [192.168.75.198]) by mta-8-out.mta.xion.oxcs.net (Postfix) with ESMTP id DF72DF34D61; Mon, 18 May 2020 21:43:35 +0000 (UTC) Received: from macbook-pro.kuh-wiese.my-router.de (unknown [2.206.141.223]) by mta-8-out.mta.xion.oxcs.net (Postfix) with ESMTPA id 8943C19AEC9; Mon, 18 May 2020 21:43:31 +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: <1179D890-7D6C-43D8-A286-DA7A0F61D585@vodafonemail.de> Date: Mon, 18 May 2020 23:43:30 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: <0CE089DD-54B6-43AD-8CE7-8EC7004768EB@vodafonemail.de> References: <9AAFEFF4-8ACE-4C95-975F-67C3F4FDAF81@vodafonemail.de> <87a72gi4kz.fsf@gmail.com> <1179D890-7D6C-43D8-A286-DA7A0F61D585@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: WIrQ6ogAi/pd * 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 | 128 +++++++++++++++++++++++++++++----------- gnu/system.scm | 3 +- 2 files changed, 95 insertions(+), 36 deletions(-) diff --git a/gnu/bootloader/grub.scm b/gnu/bootloader/grub.scm index 3f61b4a963..6ea1b38eb7 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) @@ -53,6 +53,8 @@ =20 grub-bootloader grub-efi-bootloader + make-grub-efi-net-bootloader + grub-efi-net-bootloader grub-mkrescue-bootloader grub-minimal-bootloader =20 @@ -93,7 +95,7 @@ denoting a file name." (default '((fg . cyan) (bg . blue)))) (color-highlight grub-theme-color-highlight (default '((fg . white) (bg . blue)))) - (gfxmode grub-gfxmode + (gfxmode grub-theme-gfxmode (default '("auto")))) ;list of string =20 (define %background-image @@ -143,40 +145,24 @@ WIDTH/HEIGHT, or #f if none was found." #:width width #:height height)))) =20 (define* (eye-candy config store-device store-mount-point - #:key system port) + #:key port) "Return a gexp that writes to PORT (a port-valued gexp) the 'grub.cfg' part 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\"." - (define setup-gfxterm-body - (let ((gfxmode - (or (and-let* ((theme (bootloader-configuration-theme = config)) - (gfxmode (grub-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) - ""))) - +background image and fonts must be searched for." (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) @@ -194,8 +180,6 @@ fi~%" #$font-file) =20 (and image #~(format #$port " -function setup_gfxterm {~a} - # Set 'root' to the partition that contains /gnu/store. ~a =20 @@ -210,7 +194,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) @@ -317,6 +300,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 @@ -356,7 +342,6 @@ entries corresponding to old generations of the = system." (eye-candy config (menu-entry-device (first all-entries)) (menu-entry-device-mount-point (first all-entries)) - #:system system #:port #~port)) =20 (define keyboard-layout-config @@ -444,6 +429,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/guix\" or \"efi/boot\"." + (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/guix\" or \"efi/boot\" 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 ;;; @@ -473,7 +520,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 cd75e4d4ba..c92bf57f1e 100644 --- a/gnu/system.scm +++ b/gnu/system.scm @@ -300,7 +300,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