On Mon, Nov 4, 2024 at 4:31 PM Fabio Valentini <decathorpe@xxxxxxxxx> wrote: > > On Mon, Nov 4, 2024 at 4:20 PM Jaroslav Škarvada <jskarvad@xxxxxxxxxx> wrote: > > > > On Mon, Nov 4, 2024 at 3:02 PM Ian Pilcher <arequipeno@xxxxxxxxx> wrote: > > > > > > On 11/4/24 7:42 AM, Fabio Valentini wrote: > > > > Oof ... the fact that tuned mangles bootloader parameters wasn't > > > > mentioned at all in the Change Proposal to enable it by default. To me > > > > that would have been a no-go. (I already voted -1 for this change for > > > > other reasons, but these bootloader shenanigans definitely don't make > > > > me regret that decision ...) > > > > > > Should there be a packaging guideline about this? > > > > > > > Hi, > > > > kernel command line parameters can be changed by the bootloader plugin > > which is not used in the profiles that were used in the tuned-ppd > > replacement. So that's probably why it wasn't explicitly mentioned. > > Patching of bootloader entries is done to clearly mark what can be > > changed by TuneD, i.e. by the $tuned_* variables. If the bootloader > > plugin is not used, these variables are empty. It's interesting that > > this problem wasn't reported during the F41 test days > > I wonder if it would be possible to just ... not set bootloader > parameters at all if the bootloader plugin isn't used? > The current implementation seems like a very brittle solution to me > (as witnessed by the systemd-boot breakage). > > Fabio IMHO this is the first problem with it in a very long time (not counting the s390x zipl problem). To be honest this feature was in TuneD years before the Boot Loader Specification (BLS) appeared and when Fedora switched to BLS we used grub as the reference BLS implementation (because AFAIK there was nothing else BLS ready at that time). The bootloader entries are conservatively (excluding rescue entries) patched in the kernel installation script, which IMHO should run as the root during kernel update, so probably in a transaction. IMHO there is a lower chance that something could go wrong than patching it in the daemon. The problem appeared because systemd-boot doesn't support variable expansion as grub does, because it is not written in the BLS. I am going to patch the 92-tuned.install script to disable the bootloader feature in such case as we already did with the s390x zipl loader thanks & regards Jaroslav -- _______________________________________________ devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
