[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 3/5] vnc: fix update stalls
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 3/5] vnc: fix update stalls |
Date: |
Mon, 11 Mar 2019 09:26:17 +0100 |
vnc aborts display update jobs on video mode switches and page flips.
That can cause vnc update stalls in case an unfinished vnc job gets
aborted. The vnc client will never receive the requested update then.
Fix that by copying the state from job_update back to update in that
case.
Reports complain about stalls with two or more clients being connected
at the same time, on some but not all connections. I suspect it can
also happen with a single connection, multiple connections only make
this more much likely to happen.
Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1662260
Reported-by: Ying Fang <address@hidden>
Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Ying Fang <address@hidden>
Message-id: address@hidden
---
ui/vnc.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/ui/vnc.c b/ui/vnc.c
index da4a21d4ce94..2f2ab62fcf71 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -700,6 +700,12 @@ static void vnc_abort_display_jobs(VncDisplay *vd)
}
QTAILQ_FOREACH(vs, &vd->clients, next) {
vnc_lock_output(vs);
+ if (vs->update == VNC_STATE_UPDATE_NONE &&
+ vs->job_update != VNC_STATE_UPDATE_NONE) {
+ /* job aborted before completion */
+ vs->update = vs->job_update;
+ vs->job_update = VNC_STATE_UPDATE_NONE;
+ }
vs->abort = false;
vnc_unlock_output(vs);
}
--
2.18.1
- [Qemu-devel] [PULL 0/5] Ui 20190311 v2 patches, Gerd Hoffmann, 2019/03/11
- [Qemu-devel] [PULL 5/5] monitor: deprecate acl_show, acl_reset, acl_policy, acl_add, acl_remove, Gerd Hoffmann, 2019/03/11
- [Qemu-devel] [PULL 3/5] vnc: fix update stalls,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 2/5] curses: support wide input, Gerd Hoffmann, 2019/03/11
- [Qemu-devel] [PULL 1/5] Reduce curses escdelay from 1s to 25ms, Gerd Hoffmann, 2019/03/11
- [Qemu-devel] [PULL 4/5] vnc: allow specifying a custom authorization object name, Gerd Hoffmann, 2019/03/11
- Re: [Qemu-devel] [PULL 0/5] Ui 20190311 v2 patches, Peter Maydell, 2019/03/11