Re: [Xen-devel] [PATCH]Load the cirrus vga rom for the OVMF UEFI support in hvmloader

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 22.03.12 at 09:57, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> On Thu, 2012-03-22 at 08:29 +0000, Bei Guan wrote:
>> Hi All,
>> 
>> 
>> Now, OVMF UEFI can boot on the Xen hvmloader. But, there is no VGA
>> bios for OVMF from the source code hvmloader/ovmf.c in the upstream
>> Xen. 
>> I have tried the vgabios provided by Xen, but I can see nothing
>> through the vnc client or sdl graphic window. Actually, if we check
>> the boot trace log of OVMF, we find that OVMF has booted into the OS
>> installer grub. 
>> When I try the cirrus_vga bios (named OvmfVideo.rom in edk2) provided
>> by OVMF, I can see the OVMF boot successfully through the vnc client
>> or sdl graphic window. 
>> 
>> 
>> So, this patch makes the hvmloader load the cirrus_vga bios for OVMF
>> on Xen. Any suggestions and comments are appreciated.
> 
> I'd prefer to not expose this behaviour any further than the existing
> ROMBIOS use case.
> 
> Can Tianocore not deploy option ROMs directly from the hardware (e.g. by
> using the ROM BAR on the PCI device). This would be a much preferable
> approach, it is supported by the QEMU emulations and ensures you get a
> ROM image which matches the hardware (emulated or passthrough).

UEFI should prefer not to make use of (legacy) option ROMs at all.
Instead, a proper EFI driver should be loaded (and hence provided
in the firmware image).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel