Bug 25676 - 'failsafe vga=normal' parameters used for non-failsafe menu items
Summary: 'failsafe vga=normal' parameters used for non-failsafe menu items
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: grub2 (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Vitaly Kuznetsov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-27 14:53 MSK by Sergey Vlasov
Modified: 2020-07-15 16:39 MSK (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Vlasov 2011-05-27 14:53:14 MSK 
Начиная с grub2-1.99-alt1, в /etc/grub.d/10_linux при генерации grub.cfg параметры GRUB_CMDLINE_LINUX='failsafe vga=normal' добавляются ко всем вариантам загрузки, а не только к failsafe, как было в предыдущих версиях.

Вообще, если делать как следует, там должно быть не два параметра конфигурации, а три - помимо существующих GRUB_CMDLINE_LINUX и GRUB_CMDLINE_LINUX_DEFAULT, ещё должен быть GRUB_CMDLINE_LINUX_RECOVERY (куда следует унести прибитую гвоздями в скрипте опцию single, которая, кстати, в предыдущей сборке пакета тоже убиралась патчем, а теперь не убирается).
Comment 1 Vitaly Kuznetsov 2011-05-27 15:30:42 MSK 
Логика в самом grub2 следующая:
GRUB_CMDLINE_LINUX - и для failsafe, и для normal
GRUB_CMDLINE_LINUX_DEFAULT - для normal

для 1.98 я её немного менял в следующем ключе:
GRUB_CMDLINE_LINUX - для failsafe
GRUB_CMDLINE_LINUX_DEFAULT - для normal и failsafe

какова должна быть логика, если параметров будет 3?
Comment 2 Vitaly Kuznetsov 2011-05-27 17:45:55 MSK 
(In reply to comment #1)
> для 1.98 я её немного менял в следующем ключе:
> GRUB_CMDLINE_LINUX - для failsafe
> GRUB_CMDLINE_LINUX_DEFAULT - для normal и failsafe

не совсем так, 
GRUB_CMDLINE_LINUX - для failsafe
GRUB_CMDLINE_LINUX_DEFAULT - для normal

я сейчас верну эту логику, а потом можно подумать о том, как сделать правильно
Comment 3 Repository Robot 2011-05-27 18:20:27 MSK 
grub2-1.99-alt3 -> sisyphus:

* Fri May 27 2011 Vitaly Kuznetsov <vitty@altlinux> 1.99-alt3
- fix adding failsafe options to non-failsave cmdline (ALT #25676)
- change 'splash=silent' to 'splash' in default sysconfig
Comment 4 Sergey Vlasov 2011-05-27 19:27:42 MSK 
(В ответ на комментарий №1)
> какова должна быть логика, если параметров будет 3?

GRUB_CMDLINE_LINUX - и для failsafe, и для normal
GRUB_CMDLINE_LINUX_DEFAULT - для normal
GRUB_CMDLINE_LINUX_RECOVERY - для failsafe
(первые два варианта совпадают с апстримным поведением, добавляется только одна дополнительная переменная)

Хотя это надо бы предлагать апстриму.