On 11/12/18 15:16, Philippe Mathieu-Daudé wrote: > On 2/11/18 4:23, [email protected] wrote: >> From: yuchenlin <[email protected]> >> >> In this series, replace the original vmsvga driver to Bochs >> interface. >> >> Simply revert vmsvga driver implementation. After it, use Bochs >> interface for initializing vmsvga. >> >> Because of the PCI BARs difference between std vga and vmsvga. >> We can not simply recognize the "QEMU VMWare SVGA" as the >> QEMU_VIDEO_BOCHS_MMIO variant. >> >> BAR | std vga | vmsvga >> --------------------------------- >> 0 | Framebuffer | I/O space >> 1 | Reserved | Framebuffer >> 2 | MMIO | FIFO >> >> To overcome this problem, we remain variant QEMU_VIDEO_VMWARE_SVGA, >> and use it for: >> >> (1) Get framebuffer from correct PCI BAR >> (2) Prevent using BAR2 for MMIO >> >> We have tested on qemu before and after commit 104bd1dc70 and all >> worked. >> >> Contributed-under: TianoCore Contribution Agreement 1.1 >> Signed-off-by: yuchenlin <[email protected]> >> >> Changelog: >> v1 -> v2 >> * use 'else' clause (Thanks Philippe). >> * add more comment in revert patches (Thanks Philippe). >> * reorder the revert patches, we should revert the last commit first. >> * use correct framebuffer to ClearScreen. >> * revert VMWare svga definitions. >> >> yuchenlin (5): >> Revert "OvmfPkg/QemuVideoDxe: list "UnalignedIoInternal.h" in the INF >> file" >> Revert "OvmfPkg/QemuVideoDxe: VMWare SVGA device support" >> Revert "OvmfPkg/QemuVideoDxe: Helper functions for unaligned port >> I/O." >> Revert "OvmfPkg: VMWare SVGA display device register definitions" >> OvmfPkg: simply use the Bochs interface for vmsvga > > FWIW Patches 1-4: > Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Thanks! > Patch 5 (not yet merged) is still on my TODO because I want to run > detailed testing. Thanks for that too. :) Please remember to use the v3 posting for that: [edk2] [PATCH v3] OvmfPkg: simply use the Bochs interface for vmsvga (msgid: <[email protected]>) Cheers! Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
