Re: [PATCH 0/5] OvmfPkg: complete the 4MB flash image support ("-bios" / emulated variables)

[Gary Lin <glin@suse.com>]

On Sat, May 06, 2017 at 09:30:18PM +0200, Laszlo Ersek wrote:
> (All hail Saturday!)
> 
> Gary, can you please fetch this from my repo (URL & branch name below)
> and test it with Xen? Please test both the 4MB and the 2MB build. (I
> also tested both, with qemu + "-bios".)
Hi Laszlo,

I have done some simples test with xen, and the 2MB build seems fine.
It booted into grub2 menu successfully. However, the 4MB build never boots.
The QEMU window showed less than 1 sec and then disappeared.

Here is the snippet from 'xl dmesg'

(d15)  - CPU0 ... 39-bit phys ... fixed MTRRs ... var MTRRs [1/8] ... done.
(d15) Testing HVM environment:
(d15)  - REP INSB across page boundaries ... passed
(d15)  - GS base MSRs and SWAPGS ... passed
(d15) Passed 2 of 2 tests
(d15) Writing SMBIOS tables ...
(d15) Loading OVMF ...
(d15) no BIOS ROM image found
(d15) *** HVMLoader bug at hvmloader.c:381
(d15) *** HVMLoader crashed.

I'm pretty sure that the ovmf path is right, so it seems Xen just rejected
the 4MB build :-\

I'll try to dig more information.

Cheeres,

Gary Lin

> 
> Note: this series depends on:
> 
>   [edk2] [PATCH 0/7] OvmfPkg: small cleanups and tweaks
>   https://lists.01.org/pipermail/edk2-devel/2017-May/010527.html
> 
> and it has been pushed to my github repo as such.
> 
> Repo:   https://github.com/lersek/edk2.git
> Branch: emu4k
> 
> Cc: Gary Ching-Pang Lin <glin@suse.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (5):
>   OvmfPkg/EmuVariableFvbRuntimeDxe: strip trailing whitespace
>   OvmfPkg/EmuVariableFvbRuntimeDxe: change block size to 4KB
>   OvmfPkg/PlatformPei: align EmuVariableNvStore at any page boundary
>   OvmfPkg/README: document 4MB flash layout
>   OvmfPkg: make the 4MB flash size the default (again)
> 
>  OvmfPkg/OvmfPkgIa32.dsc                |   2 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc             |   2 +-
>  OvmfPkg/OvmfPkgX64.dsc                 |   2 +-
>  OvmfPkg/EmuVariableFvbRuntimeDxe/Fvb.h |  26 ++-
>  OvmfPkg/EmuVariableFvbRuntimeDxe/Fvb.c | 191 +++++++++-----------
>  OvmfPkg/PlatformPei/Platform.c         |  20 +-
>  OvmfPkg/README                         |  39 +++-
>  7 files changed, 143 insertions(+), 139 deletions(-)
> 
> -- 
> 2.9.3
> 
>