[Qemu-devel] [PATCH v3 3/3] vfio/display: delay link up event
Alex Williamson
alex.williamson at redhat.com
Fri Mar 8 21:16:23 UTC 2019
On Fri, 22 Feb 2019 11:19:11 +0000
Liam Merwick <liam.merwick at oracle.com> wrote:
> On 22/02/2019 05:49, Gerd Hoffmann wrote:
> > Kick the display link up event with a 0.1 sec delay,
> > so the guest has a chance to notice the link down first.
> >
> > Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>
>
> Depending on your thoughts on the suggestion in patch 1 regarding a
> comment at the 'err' label - another candidate in
> vfio_display_edid_link_up().
This would also get the following fixup rolled in. Thanks,
Alex
hw/vfio/display.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/vfio/display.c b/hw/vfio/display.c
index e8f312dc3308..a3d9c8f5beac 100644
--- a/hw/vfio/display.c
+++ b/hw/vfio/display.c
@@ -44,7 +44,9 @@ static void vfio_display_edid_link_up(void *opaque)
int fd = vdev->vbasedev.fd;
dpy->edid_regs->link_state = VFIO_DEVICE_GFX_LINK_STATE_UP;
- pwrite_field(fd, dpy->edid_info, dpy->edid_regs, link_state);
+ if (pwrite_field(fd, dpy->edid_info, dpy->edid_regs, link_state)) {
+ goto err;
+ }
trace_vfio_display_edid_link_up();
return;
More information about the intel-gvt-dev
mailing list