Bug 12837

Summary:	adding vga=792 to all kernel cmdline in grub.cfg of system and live image
Product:	[ROSA-based products] ROSA Fresh	Reporter:	a.betkher <a.betkher>
Component:	Installer and live mode	Assignee:	ROSA Linux Bugs <bugs>
Status:	UNCONFIRMED ---	QA Contact:	ROSA Linux Bugs <bugs>
Severity:	normal
Priority:	Normal	CC:	leshikne, m.novosyolov, pastordidi
Version:	All	Flags:	pastordidi: qa_verified-
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Platform:	---	ROSA Vulnerability identifier:
RPM Package:		ISO-related:
Bad POT generating:		Upstream:

Description a.betkher 2022-11-08 14:25:43 MSK

Added vga=792 to all grub2 menu items, in need to set 1024x768 resolution on virtualbox boot

livecd-tools:
https://abf.io/build_lists/4174232 - x86_64
https://abf.io/build_lists/4174233
https://abf.io/build_lists/4174234
https://abf.io/build_lists/4174235
https://abf.io/build_lists/4174236

grub2:
https://abf.io/build_lists/4174247 - x86_64
https://abf.io/build_lists/4174248 
https://abf.io/build_lists/4174249
 
ROSA fresh lxqt image build with this packages
https://abf.io/platforms/rosa2021.1/products/316/product_build_lists/44912

Advisory:
- find 'vga=792' in file /EFI/BOOT/grub.cfg  in unpacked or mounted iso 
- find 'vga=792' in file /boot/grub2/grub.cfg in installed system
- find 'vga=792' in file /boot/grub2/grub.cfg in installed system after update-grub2

Comment 1 Mikhail Novosyolov 2022-11-08 14:34:31 MSK

Предлагаю не забивать конфиг граба сомнительными параметрами, а внести vga в список параметров ядра, которые Анаконда сохраняет после установки.
https://abf.io/import/anaconda/blob/rosa2021.1/0007-Do-not-preserve-SELinux-related-kernel-cmdline.patch
Если его понадобится поправить для запуска лайва, сохранится измененное значение.

Comment 2 Mikhail Novosyolov 2022-11-08 14:36:01 MSK

И еще в патче livecd-tools не дано полного описания проблематики. Например, что gfx* в grub не помогает, что после загрузки драйвера видеокарты этот параметр перестает влиять (предположительно) и т.д.

Comment 3 Mikhail Novosyolov 2022-11-08 14:44:15 MSK

По коммиту в грабе вообще непонятно, зачем это добавлено. через пару лет будет не разобрать.

Comment 4 Mikhail Novosyolov 2022-11-08 14:44:57 MSK

При установке системы из, например, rootfs, этот параметр не нужен.

Comment 5 Mikhail Novosyolov 2022-11-08 15:06:12 MSK

Перенес гиты в https://abf.io/groups/vga_cmdline_tmp , сделайте, пожалуйста, нормальную документацию по багу, в текущем виде не понять со стороны и нам же через несколько лет, что это и зачем сделано. Как будет документация, в костылях можно будет добавить отсылку в ней (в патче и коммите).
контейнеры "отклоняю", чтоб их случайно не опубликовали.

Comment 6 a.betkher 2022-11-08 15:11:58 MSK

Считаю, что этот параметр вообще не нужен. То что это как-то влияет на разрешение иксов в виртуалбоксе скорее баг ибо vga= совсем не об этом. Впрочем и как либо мешать параметр не должен. 1024x768 - стандарт, мало вероятно что хоть какая-то карта его не поддерживает. Добавлял по просьбе Володи.