Re: [Qemu-devel] [PATCH 2/2] ui: introduce vfio_display_reset

2018-04-25 Thread Zhang, Tina
> -Original Message- > From: intel-gvt-dev [mailto:intel-gvt-dev-boun...@lists.freedesktop.org] On > Behalf Of Gerd Hoffmann > Sent: Tuesday, April 24, 2018 4:16 PM > To: Zhang, Tina > Cc: alex.william...@redhat.com; intel-gvt-...@lists.freedesktop.org; qemu- > de...@nongnu.org; zhen...@

Re: [Qemu-devel] [PATCH 2/2] ui: introduce vfio_display_reset

2018-04-24 Thread Gerd Hoffmann
Hi, > After reviewing this patch-set again, I think we might not need the > proposed dpy_gfx_switch_surface() any more. The reason I proposed it > was because I thought gfx.ds is changed to the surface related to > guest dma-buf framebuffer and we need to switch it with QemuConsole's > DisplayS

Re: [Qemu-devel] [PATCH 2/2] ui: introduce vfio_display_reset

2018-04-23 Thread Zhang, Tina
> -Original Message- > From: intel-gvt-dev [mailto:intel-gvt-dev-boun...@lists.freedesktop.org] On > Behalf Of Gerd Hoffmann > Sent: Monday, April 23, 2018 9:12 PM > To: Zhang, Tina > Cc: alex.william...@redhat.com; intel-gvt-...@lists.freedesktop.org; qemu- > de...@nongnu.org; zhen...@l

Re: [Qemu-devel] [PATCH 2/2] ui: introduce vfio_display_reset

2018-04-23 Thread Gerd Hoffmann
> +surface = qemu_console_surface(vdev->dpy->con); > +dpy_gl_scanout_disable(vdev->dpy->con); > +dpy_gfx_switch_surface(vdev->dpy->con, surface); Hmm, so you ask which surface is active, then switch to it. Why? Maybe you just need a full display update after disabling the gl scanout?