From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id 9B2AB6947B for ; Fri, 11 Mar 2022 20:06:07 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 4EFDC2803D for ; Fri, 11 Mar 2022 20:06:07 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 215BA27F00 for ; Fri, 11 Mar 2022 20:06:02 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id EDB7346389 for ; Fri, 11 Mar 2022 20:06:01 +0100 (CET) From: Stoiko Ivanov To: pve-devel@lists.proxmox.com Date: Fri, 11 Mar 2022 20:05:42 +0100 Message-Id: <20220311190548.2937377-5-s.ivanov@proxmox.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20220311190548.2937377-1-s.ivanov@proxmox.com> References: <20220311190548.2937377-1-s.ivanov@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.060 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_SHORT 0.001 Use of a URL Shortener for very short URL POISEN_SPAM_PILL 0.1 Meta: its spam POISEN_SPAM_PILL_1 0.1 random spam to be learned in bayes POISEN_SPAM_PILL_3 0.1 random spam to be learned in bayes SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record T_SCC_BODY_TEXT_LINE -0.01 - Subject: [pve-devel] [PATCH pve-kernel-meta v3 4/8] proxmox-boot: refresh based on bootloader config instead of bootmode X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Mar 2022 19:06:07 -0000 ignore the current bootmode (uefi/legacy) when deciding which configs to generate - make this decision based on the found boot loader configs on the ESP. Change systemd-boot to load the kernel+initrd from the ESPs root where grub needs them. This prevents the double use of space for systems having both boot-loaders present (since FAT does not support symlinks). While this is against the recommendations of the boot-loader-specification [0]) it works fine (and we deviate from [0] by not having the machine-id in the kernel subdirs anyways). The UEFI spec on the directory structure [1], also should be fine with it. Additionally adapt the output of `p-b-t status` to the new paths [0] https://systemd.io/BOOT_LOADER_SPECIFICATION/ [1] section 13.3.1.3 of https://uefi.org/sites/default/files/resources/UEFI_Spec_2_9_2021_03_18.pdf Signed-off-by: Stoiko Ivanov --- bin/proxmox-boot-tool | 14 +++---- proxmox-boot/zz-proxmox-boot | 78 ++++++++++++------------------------ 2 files changed, 32 insertions(+), 60 deletions(-) diff --git a/bin/proxmox-boot-tool b/bin/proxmox-boot-tool index d91786e..d2425d2 100755 --- a/bin/proxmox-boot-tool +++ b/bin/proxmox-boot-tool @@ -415,20 +415,18 @@ _status_detail() { result="" if mountpoint_has_sd_boot "${mountpoint}"; then - if [ ! -d "${mountpoint}/$PMX_ESP_DIR" ]; then - warn "${path}/$PMX_ESP_DIR does not exist" - fi - versions_uefi=$(ls -1 ${mountpoint}/$PMX_ESP_DIR | awk '{printf (NR>1?", ":"") $0}') - result="uefi (versions: ${versions_uefi})" + result="uefi" fi if mountpoint_has_grub "${mountpoint}"; then - versions_grub=$(ls -1 ${mountpoint}/vmlinuz-* | awk '{ gsub(/.*\/vmlinuz-/, ""); printf (NR>1?", ":"") $0 }') if [ -n "$result" ]; then - result="${result}, grub (versions: ${versions_grub})" + result="${result}, grub" else - result="grub (versions: ${versions_grub})" + result="grub" fi fi + versions=$(find "${mountpoint}" -maxdepth 1 -name 'vmlinuz-*' | awk '{ gsub(/.*\/vmlinuz-/, ""); printf (NR>1?", ":"") $0 }') + result="${result} (versions: ${versions})" + echo "$curr_uuid is configured with: $result" umount "${mountpoint}" || \ { warn "umount of ${path} failed - failure"; exit 0; } diff --git a/proxmox-boot/zz-proxmox-boot b/proxmox-boot/zz-proxmox-boot index a6629aa..fa0ea4a 100755 --- a/proxmox-boot/zz-proxmox-boot +++ b/proxmox-boot/zz-proxmox-boot @@ -75,33 +75,33 @@ update_esp_func() { { warn "creation of mountpoint ${mountpoint} failed - skipping"; return; } mount "${path}" "${mountpoint}" || \ { warn "mount of ${path} failed - skipping"; return; } - if [ -d /sys/firmware/efi ]; then - if ! mountpoint_has_sd_boot "${mountpoint}"; then - warn "${path} contains no loader.conf - skipping" - return - fi - if [ ! -d "${mountpoint}/$PMX_ESP_DIR" ]; then - warn "${path}/$PMX_ESP_DIR does not exist- skipping" - return - fi - elif ! mountpoint_has_grub "${mountpoint}"; then - warn "${path} contains no grub directory - skipping" + sd_boot="" + grub="" + if mountpoint_has_sd_boot "${mountpoint}"; then + sd_boot=1 + fi + if mountpoint_has_grub "${mountpoint}"; then + grub=1 + fi + if [ -z "$sd_boot" ] && [ -z "$grub" ]; then + warn "${path} contains no bootloader config - skipping!" return fi warn "Copying and configuring kernels on ${path}" - copy_and_config_kernels "${mountpoint}" + copy_and_config_kernels "${mountpoint}" "${sd_boot}" pinned_kernel=$(get_first_line "${PINNED_KERNEL_CONF}") if [ -e "${NEXT_BOOT_PIN}" ]; then pinned_kernel=$(get_first_line "${NEXT_BOOT_PIN}") fi - if [ -d /sys/firmware/efi ]; then + if [ -n "$sd_boot" ]; then set_systemd_boot_default "${mountpoint}" "${pinned_kernel}" - remove_old_kernels_efi "${mountpoint}" - else + remove_old_kernels "${mountpoint}" + fi + if [ -n "$grub" ]; then set_grub_default "${pinned_kernel}" - remove_old_kernels_legacy "${mountpoint}" + remove_old_kernels "${mountpoint}" mount --bind "${mountpoint}" "/boot" update-grub umount /boot @@ -116,7 +116,7 @@ update_esp_func() { copy_and_config_kernels() { esp="$1" - + sd_boot="$2" for kver in ${BOOT_KVERS}; do @@ -132,52 +132,25 @@ copy_and_config_kernels() { continue fi - if [ -d /sys/firmware/efi ]; then - - warn " Copying kernel and creating boot-entry for ${kver}" - KERNEL_ESP_DIR="${PMX_ESP_DIR}/${kver}" - KERNEL_LIVE_DIR="${esp}/${KERNEL_ESP_DIR}" - mkdir -p "${KERNEL_LIVE_DIR}" - cp --preserve=timestamps "${linux_image}" "${KERNEL_LIVE_DIR}/" - cp --preserve=timestamps "${initrd}" "${KERNEL_LIVE_DIR}/" + warn " Preparing kernel and initrd for ${kver}" + cp --preserve=timestamps "${linux_image}" "${esp}/" + cp --preserve=timestamps "${initrd}" "${esp}/" - # create loader entry + if [ -n "$sd_boot" ]; then + # create loader entry cat > "${esp}/loader/entries/proxmox-${kver}.conf" <<- EOF title ${LOADER_TITLE} version ${kver} options ${CMDLINE} - linux /${KERNEL_ESP_DIR}/vmlinuz-${kver} - initrd /${KERNEL_ESP_DIR}/initrd.img-${kver} + linux /vmlinuz-${kver} + initrd /initrd.img-${kver} EOF - else - warn " Copying kernel ${kver}" - cp --preserve=timestamps "${linux_image}" "${esp}/" - cp --preserve=timestamps "${initrd}" "${esp}/" - fi - done - -} - -remove_old_kernels_efi() { - esp="$1" - - for kerneldir in "${esp}/${PMX_ESP_DIR}"/*; do - if [ ! -d "${kerneldir}" ]; then - warn " ${kerneldir} is not a directory - skipping" - continue fi - - kver="$(echo "${kerneldir}" | sed -r "s#^${esp}/${PMX_ESP_DIR}/(.+)\$#\\1#")" - - echo "${BOOT_KVERS}" | grep -q "${kver}" && continue; - warn " Removing old version ${kver}" - rm -rf "${kerneldir}" - rm -f "${esp}/loader/entries/proxmox-${kver}.conf" done } -remove_old_kernels_legacy() { +remove_old_kernels() { esp="$1" for kernel in "${esp}/"vmlinuz-*; do @@ -187,6 +160,7 @@ remove_old_kernels_legacy() { warn " Removing old version ${kver}" rm -rf "${esp}/vmlinuz-${kver}" rm -rf "${esp}/initrd.img-${kver}" + rm -rf "${esp}/loader/entries/proxmox-${kver}.conf" done } -- 2.30.2