Re: [PATCH 0/3] OvmfPkg/QemuVideoDxe/VbeShim: handle PAM1 register on Q35 correctly

[Aleksei <alex3kov@zoho.com>]

Hi Laszlo,

I've built it (despite edk2/ovmf build process being as convoluted as 
ever) and am trying to run it with qemu 2.10.0 - it's not booting at 
all, stuck at "Guest has not initialized the display (yet)."

Maybe the problem is gcc version? I have 7.2.0 while edk2 instructions 
mention gcc5.

On 19/09/17 22:18, Laszlo Ersek wrote:
> Repo:   https://github.com/lersek/edk2.git
> Branch: vbe_shim_q35_pam
>
> Fix the long-standing OVMF/Q35 bug recently exposed by a QEMU change,
> and reported under <https://bugs.launchpad.net/qemu/+bug/1715700>.
>
> Aleksei, can you please fetch the branch, build it, and report back with
> your Tested-by if it works for you?
>
> I performed my own tests as well; I'll include those in a separate
> email.
>
> Cc: Aleksei Kovura <alex3kov@zoho.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Igor Mammedov <imammedo@redhat.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
>
> Thank you,
> Laszlo
>
> Laszlo Ersek (3):
>    OvmfPkg/CsmSupportLib: move PAM register addresses to IndustryStandard
>    OvmfPkg/QemuVideoDxe/VbeShim: rename Status to
>      Segment0AllocationStatus
>    OvmfPkg/QemuVideoDxe/VbeShim: handle PAM1 register on Q35 correctly
>
>   OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c       | 62 ++++++++++----------
>   OvmfPkg/Csm/CsmSupportLib/LegacyRegion.h       | 22 +------
>   OvmfPkg/Include/IndustryStandard/I440FxPiix4.h | 13 ++++
>   OvmfPkg/Include/IndustryStandard/Q35MchIch9.h  |  8 +++
>   OvmfPkg/QemuVideoDxe/QemuVideoDxe.inf          |  3 +-
>   OvmfPkg/QemuVideoDxe/VbeShim.c                 | 47 ++++++++++++---
>   6 files changed, 95 insertions(+), 60 deletions(-)
>

-- 
/--Regards, Aleksei/