[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 0/2] qapi/ui: add change-vnc-listen
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[PATCH 0/2] qapi/ui: add change-vnc-listen |
Date: |
Mon, 20 Dec 2021 16:44:16 +0100 |
Hi all!
Recently our customer requested a possibility to change VNC listen port
dynamically.
Happily in Rhel7-based Qemu we already have this possibility: through
deprecated "change" qmp command.
But since 6.0 "change" qmp command was removed, with recommendation to
use change-vnc-password or blockdev-change-medium instead. Of course,
neither of them allow change VNC listen port.
So, let's reimplement the possibility.
Note: now, reconnecting may trigger existing deadlock, as I described
in my message "Re: [PULL 09/11] ui/vnc: clipboard support":
<973ddebe-14a9-4ba7-c389-7a97d6017237@virtuozzo.com>
Simple hack helps, but I'm not sure it's safe itself:
diff --git a/ui/vnc.c b/ui/vnc.c
index 69bbf3b6f6..8c6b378e2e 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -1354,12 +1354,12 @@ void vnc_disconnect_finish(VncState *vs)
/* last client gone */
vnc_update_server_surface(vs->vd);
}
+ vnc_unlock_output(vs);
+
if (vs->cbpeer.update.notify) {
qemu_clipboard_peer_unregister(&vs->cbpeer);
}
- vnc_unlock_output(vs);
-
qemu_mutex_destroy(&vs->output_mutex);
if (vs->bh != NULL) {
qemu_bh_delete(vs->bh);
Vladimir Sementsov-Ogievskiy (2):
ui/vnc: refactor arrays of addresses to SocketAddressList
qapi/ui: introduce change-vnc-listen
docs/about/removed-features.rst | 3 +-
qapi/ui.json | 12 +++
ui/vnc.c | 155 ++++++++++++++++----------------
3 files changed, 91 insertions(+), 79 deletions(-)
--
2.31.1
- [PATCH 0/2] qapi/ui: add change-vnc-listen,
Vladimir Sementsov-Ogievskiy <=