diff options
author | Laszlo Ersek <lersek@redhat.com> | 2014-08-29 17:27:20 +0000 |
---|---|---|
committer | jljusten <jljusten@6f19259b-4bc3-4df7-8a09-765794883524> | 2014-08-29 17:27:20 +0000 |
commit | b37bcfd6bd0254cd7a67ca6af3743d28eaffceec (patch) | |
tree | 3ee8ae449c216e727f08fb68a1ecd7b11af8ca6f /OvmfPkg/QemuVideoDxe/Qemu.h | |
parent | fd9abd045e41387ef8e5cfbb4588220cf7a1261d (diff) | |
download | edk2-b37bcfd6bd0254cd7a67ca6af3743d28eaffceec.tar.gz edk2-b37bcfd6bd0254cd7a67ca6af3743d28eaffceec.tar.bz2 edk2-b37bcfd6bd0254cd7a67ca6af3743d28eaffceec.zip |
OvmfPkg: QemuVideoDxe: work around misreported QXL framebuffer size
When setting up the list of GOP modes offered on QEMU's stdvga ("VGA") and
QXL ("qxl-vga") video devices, QemuVideoBochsModeSetup() filters those
modes against the available framebuffer size. (Refer to SVN r15288 / git
commit ec88061e.)
The VBE_DISPI_INDEX_VIDEO_MEMORY_64K register of both stdvga and QXL is
supposed to report the size of the drawable, VGA-compatibility
framebuffer. Instead, up to and including qemu-2.1, this register actually
reports the full video RAM (PCI BAR 0) size.
In case of stdvga, this happens to be correct, because on that card the
full PCI BAR 0 is usable for drawing; there is no difference between
"drawable framebuffer size" and "video RAM (PCI BAR 0) size".
However, on the QXL card, only an initial portion of the video RAM is
suitable for drawing, as compatibility framebuffer; and the value
currently reported by VBE_DISPI_INDEX_VIDEO_MEMORY_64K overshoots the
valid size. Beyond the drawable range, the video RAM contains buffers and
structures for the QXL guest-host protocol.
Luckily, the size of the drawable QXL framebuffer can also be read from a
register in the QXL ROM BAR (PCI BAR 2), so let's retrieve it from there.
Without this fix, OVMF offers too large resolutions on the QXL card (up to
the full size of the video RAM). If a GOP client selects such a resolution
and draws into the video RAM past the compatibility segment, then the
guest corrupts its communication structures (which is invalid guest
behavior).
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15978 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'OvmfPkg/QemuVideoDxe/Qemu.h')
-rw-r--r-- | OvmfPkg/QemuVideoDxe/Qemu.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/OvmfPkg/QemuVideoDxe/Qemu.h b/OvmfPkg/QemuVideoDxe/Qemu.h index 4bf51c7150..52ee20d8ba 100644 --- a/OvmfPkg/QemuVideoDxe/Qemu.h +++ b/OvmfPkg/QemuVideoDxe/Qemu.h @@ -499,7 +499,8 @@ QemuVideoCirrusModeSetup ( EFI_STATUS
QemuVideoBochsModeSetup (
- QEMU_VIDEO_PRIVATE_DATA *Private
+ QEMU_VIDEO_PRIVATE_DATA *Private,
+ BOOLEAN IsQxl
);
VOID
|