Re: [PATCH v5 00/46] x86: Use qemu-x86_64 to boot EFI installers

Hi Simon,

On Sun, Jul 16, 2023 at 11:39 AM Simon Glass sjg@chromium.org wrote:

This series adds various minor features so that qemu-x86_64 can boot the Ubuntu 2022.04 installer using a virtio device:

qemu-system-x86_64 -M pc -drive format=raw,file=root.img
    -bios /tmp/b/qemu-x86_64/u-boot.rom
    -drive if=virtio,file=ubuntu-22.04.2-desktop-amd64.iso
    -smp 4 -m 4G -serial mon:stdio

The main changes include:

• Enable video in SPL while running in 32-bit mode
• Drop the duplicate ACPI tables with EFI
• Support PCI autoconfig in SPL
• Support FAT on a CDROM filesystem
• Improved bootstd rules around device tree and efi_set_bootdev()

There are also quite a number of minor tweaks and fixes to make things easier to use.

This series is based on an older version of the SPL-video series from Nikhil M Jain. It is available at u-boot-dm/bryc-working

Changes in v5:

• Make mtrr_type_name[] static
• Fix 'abrievated' typo
• Drop extra space in penultimate example
• Drop unnecessary 'it' in commit message
• Adjust the prompt for clarity (although it is now >12 chars)
• Reword commit message
• Drop unnecessary 'allow' word
• Add tests for bdinfo additions
• Drop extra 'qfw list' line
• Update commit message to explain 'md' reference
• Drop period at end of commit subject
• Drop extra 'a' in commit message
• Add 'bdinfo' tag to commit subject
• Use the existing QEMU code instead
• Use u16 instead of ushort
• Split out banner code into a separate patch
• Drop code to ensure the SPL banner is only shown once
• Fix 'litlle' typo in subject
• Correct indentation
• Add a comment about QFW putting its tables up high
• Add a comment that the bloblist is always in high memory
• Use "gpio" for log return
• Use QEMU instead of qemu
• Drop patch 'Show the number of physical address bits'

Except the following patches:

[v5,02/46] x86: Return mtrr_add_request() to its old purpose [v5,23/46] pci: Allow the video BIOS to work in SPL with QEMU [v5,34/46] fs: fat: Support reading from a larger block size

series applied to u-boot-x86, thanks!

I will take a look at the MTRR one.

Regards, Bin