[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 6/6] ui: introduce vfio_display_reset
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 6/6] ui: introduce vfio_display_reset |
Date: |
Fri, 27 Apr 2018 11:54:34 +0200 |
From: Tina Zhang <address@hidden>
During guest OS reboot, guest framebuffer is invalid. It will cause
bugs, if the invalid guest framebuffer is still used by host.
This patch is to introduce vfio_display_reset which is invoked
during vfio display reset. This vfio_display_reset function is used
to release the invalid display resource, disable scanout mode and
replace the invalid surface with QemuConsole's DisplaySurafce.
This patch can fix the GPU hang issue caused by gd_egl_draw during
guest OS reboot.
Changes v3->v4:
- Move dma-buf based display check into the vfio_display_reset().
(Gerd)
Changes v2->v3:
- Limit vfio_display_reset to dma-buf based vfio display. (Gerd)
Changes v1->v2:
- Use dpy_gfx_update_full() update screen after reset. (Gerd)
- Remove dpy_gfx_switch_surface(). (Gerd)
Signed-off-by: Tina Zhang <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
---
hw/vfio/pci.h | 1 +
hw/vfio/display.c | 11 +++++++++++
hw/vfio/pci.c | 4 ++++
3 files changed, 16 insertions(+)
diff --git a/hw/vfio/pci.h b/hw/vfio/pci.h
index 629c875701..59ab7757a3 100644
--- a/hw/vfio/pci.h
+++ b/hw/vfio/pci.h
@@ -176,6 +176,7 @@ int vfio_pci_igd_opregion_init(VFIOPCIDevice *vdev,
struct vfio_region_info *info,
Error **errp);
+void vfio_display_reset(VFIOPCIDevice *vdev);
int vfio_display_probe(VFIOPCIDevice *vdev, Error **errp);
void vfio_display_finalize(VFIOPCIDevice *vdev);
diff --git a/hw/vfio/display.c b/hw/vfio/display.c
index 7d727ce910..59c0e5d1d7 100644
--- a/hw/vfio/display.c
+++ b/hw/vfio/display.c
@@ -198,6 +198,17 @@ static void vfio_display_dmabuf_exit(VFIODisplay *dpy)
}
/* ---------------------------------------------------------------------- */
+void vfio_display_reset(VFIOPCIDevice *vdev)
+{
+ if (!vdev || !vdev->dpy || !vdev->dpy->con ||
+ !vdev->dpy->dmabuf.primary) {
+ return;
+ }
+
+ dpy_gl_scanout_disable(vdev->dpy->con);
+ vfio_display_dmabuf_exit(vdev->dpy);
+ dpy_gfx_update_full(vdev->dpy->con);
+}
static void vfio_display_region_update(void *opaque)
{
diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
index b9bc6cd310..4947fe39a2 100644
--- a/hw/vfio/pci.c
+++ b/hw/vfio/pci.c
@@ -3103,6 +3103,10 @@ static void vfio_pci_reset(DeviceState *dev)
vfio_pci_pre_reset(vdev);
+ if (vdev->display != ON_OFF_AUTO_OFF) {
+ vfio_display_reset(vdev);
+ }
+
if (vdev->resetfn && !vdev->resetfn(vdev)) {
goto post_reset;
}
--
2.9.3
- [Qemu-devel] [PULL 0/6] Ui 20180427 patches, Gerd Hoffmann, 2018/04/27
- [Qemu-devel] [PULL 4/6] sdl: Allow OpenGL ES context creation, Gerd Hoffmann, 2018/04/27
- [Qemu-devel] [PULL 6/6] ui: introduce vfio_display_reset,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 5/6] console: introduce dpy_gfx_update_full, Gerd Hoffmann, 2018/04/27
- [Qemu-devel] [PULL 2/6] qapi: Parameter gl of DisplayType now accept an enum, Gerd Hoffmann, 2018/04/27
- [Qemu-devel] [PULL 3/6] sdl: Move DisplayOptions global to sdl2_console, Gerd Hoffmann, 2018/04/27
- [Qemu-devel] [PULL 1/6] vnc: fix use-after-free, Gerd Hoffmann, 2018/04/27
- Re: [Qemu-devel] [PULL 0/6] Ui 20180427 patches, Peter Maydell, 2018/04/27