From: Stoiko Ivanov <s.ivanov@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH pve-kernel-meta/pve-installer v2] improve boot-mode switching
Date: Thu, 3 Mar 2022 20:07:49 +0100 [thread overview]
Message-ID: <20220303190759.3527703-1-s.ivanov@proxmox.com> (raw)
these patches are still based on v3 of the pin/unpin addtion [1].
Not sending the pin/unpin again, since the feedback was mostly positive and
I think Thomas had already some improvements on top of it prepared locally.
changes v1->v2:
* added a fix for #3729 to pve-kernel-meta - not directly related to the
series - but since the series moves the affected lines around quite a bit
I thought we can merge them together (can gladly send it on top of master)
* else mostly incorporated Fabian's feedback - huge thanks!
* comments where appropriate are added to the individual patches
original cover-letter for the v1:
This patchset is the result from the discussion on the rfc for using grub as
only boot loader [0].
The patches for pve-kernel-meta are based on top of the v3 of the
`pin/unpin` addition [1].
The patches for pve-installer are independent from the pve-kernel-meta ones
(my tests used the current PVE 7.1 ISO w/o updating the pve-kernel-helper
package) - but I still consider them linked.
changes from the rfc:
* instead of using grub everywhere - we now use systemd-boot always for efi
systems and leave grub only for legacy booted systems
* instead of unconditionally installing the boot-loaders for both modes on
all ESPs users now can opt-in to installing the boot-loader for the other
bootmode (both are updated if their configs are present, by default we
only install the boot-loader for the current boot mode)
Tested the pve-kernel-meta with 3 VMs (ZFS+uefi, ZFS+legacy, ext4+uefi) and
switching boot-modes (after the necessary preparations).
Tested the pve-installer by scping proxinstall into a VM and installing
on btrfs+uefi, ext4+uefi, btrfs+legacy
[0] https://lists.proxmox.com/pipermail/pve-devel/2022-February/051562.html
[1] https://lists.proxmox.com/pipermail/pve-devel/2022-February/051732.html
pve-kernel-meta:
Stoiko Ivanov (8):
rename pve-efiboot-manual-kernels to proxmox-boot-manual-kernels
proxmox-boot: add reinit subcommand
proxmox-boot: add helpers to check for bootloader configs
proxmox-boot: refresh based on bootloader config instead of bootmode
proxmox-boot: remove now obsolete EFI/proxmoxdir if it exists
proxmox-boot: init bootloaders based on esp contents
proxmox-boot: allow to manually specify bootloader
proxmox-boot: fix #3729 add --graceful to bootctl invocation
bin/grub-install-wrapper | 19 +-----
bin/proxmox-boot-tool | 105 ++++++++++++++++++++++++------
debian/pve-kernel-helper.postinst | 4 ++
proxmox-boot/functions | 18 ++++-
proxmox-boot/zz-proxmox-boot | 84 +++++++++---------------
5 files changed, 141 insertions(+), 89 deletions(-)
pve-installer:
Stoiko Ivanov (2):
remove /mnt/hostrun after install
use proxmox-boot-tool for all uefi installs
proxinstall | 81 ++++++++++-------------------------------------------
1 file changed, 15 insertions(+), 66 deletions(-)
--
2.30.2
next reply other threads:[~2022-03-03 19:08 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-03 19:07 Stoiko Ivanov [this message]
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 1/8] rename pve-efiboot-manual-kernels to proxmox-boot-manual-kernels Stoiko Ivanov
2022-03-04 10:51 ` Thomas Lamprecht
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 2/8] proxmox-boot: add reinit subcommand Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 3/8] proxmox-boot: add helpers to check for bootloader configs Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 4/8] proxmox-boot: refresh based on bootloader config instead of bootmode Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 5/8] proxmox-boot: remove now obsolete EFI/proxmoxdir if it exists Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 6/8] proxmox-boot: init bootloaders based on esp contents Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 7/8] proxmox-boot: allow to manually specify bootloader Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH pve-kernel-meta v2 8/8] proxmox-boot: fix #3729 add --graceful to bootctl invocation Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH installer v2 1/2] remove /mnt/hostrun after install Stoiko Ivanov
2022-03-03 19:07 ` [pve-devel] [PATCH installer v2 2/2] use proxmox-boot-tool for all uefi installs Stoiko Ivanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220303190759.3527703-1-s.ivanov@proxmox.com \
--to=s.ivanov@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.