[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use So
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress |
Date: |
Mon, 19 Oct 2015 16:20:09 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 10/12/2015 05:14 AM, Daniel P. Berrange wrote:
> The VNC code is currently using QemuOpts to configure the
> sockets connections / listeners it needs. Convert it to
> use SocketAddress to bring it in line with modern QAPI
> based code elsewhere in QEMU.
>
> Signed-off-by: Daniel P. Berrange <address@hidden>
> ---
> ui/vnc.c | 161
> ++++++++++++++++++++++++++++++++++++---------------------------
> 1 file changed, 91 insertions(+), 70 deletions(-)
>
> @@ -3539,44 +3535,83 @@ void vnc_display_open(const char *id, Error **errp)
> return;
> }
>
> - sopts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
> - wsopts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
> -
> h = strrchr(vnc, ':');
> if (h) {
> - char *host;
> size_t hlen = h - vnc;
>
> - if (vnc[0] == '[' && vnc[hlen - 1] == ']') {
> - host = g_strndup(vnc + 1, hlen - 2);
> + const char *websocket = qemu_opt_get(opts, "websocket");
> + int to = qemu_opt_get_number(opts, "to", 0);
We really ought to improve the qapi notion of InetSocketAddress to allow
{ 'alternate':'StrOrInt', 'data': { 'i':'int', 's':'str' } } rather than
only 'str' for port. But that's a project for another day. I guess
things get a bit weird if "to" is present but doesn't parse as a number,
but I don't think your patch is making things any worse.
> +
> + if (strncmp(vnc, "unix:", 5) == 0) {
> + saddr->kind = SOCKET_ADDRESS_KIND_UNIX;
> + saddr->q_unix = g_new0(UnixSocketAddress, 1);
More clashes with my pending qapi patches; rebase fun for one of us :)
> + if (to) {
> + saddr->inet->has_to = true;
> + saddr->inet->to = to;
> + }
> + saddr->inet->ipv4 = saddr->inet->has_ipv4 = has_ipv4;
> + saddr->inet->ipv6 = saddr->inet->has_ipv6 = has_ipv6;
Do we want to specify has_ipvX as true even when setting inet->ipvX to
false?
> @@ -3770,37 +3795,32 @@ void vnc_display_open(const char *id, Error **errp)
> int csock;
> vs->lsock = -1;
> vs->lwebsock = -1;
> - if (strncmp(vnc, "unix:", 5) == 0) {
> - csock = unix_connect(vnc+5, errp);
> - } else {
> - csock = inet_connect(vnc, errp);
> + if (vs->ws_enabled) {
> + error_setg(errp, "Cannot use websockets in reverse mode");
> + goto fail;
> }
> + csock = socket_connect(saddr, errp,
> + NULL, NULL);
Looks like the line-wrapping isn't needed here.
Overall, looks like a sane conversion.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 00/16] Introduce I/O channels framework, Daniel P. Berrange, 2015/10/15
- [Qemu-devel] [PATCH v2 03/16] sockets: allow port to be NULL when listening on IP address, Daniel P. Berrange, 2015/10/12
- [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Daniel P. Berrange, 2015/10/12
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress,
Eric Blake <=
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Daniel P. Berrange, 2015/10/20
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Peter Maydell, 2015/10/20
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Daniel P. Berrange, 2015/10/20
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Peter Maydell, 2015/10/20
- Re: [Qemu-devel] [PATCH v2 04/16] ui: convert VNC startup code to use SocketAddress, Daniel P. Berrange, 2015/10/21
[Qemu-devel] [PATCH v2 01/16] sockets: add helpers for creating SocketAddress from a socket, Daniel P. Berrange, 2015/10/12
[Qemu-devel] [PATCH v2 06/16] coroutine: move into libqemuutil.a library, Daniel P. Berrange, 2015/10/12
[Qemu-devel] [PATCH v2 07/16] util: pull Buffer code out of VNC module, Daniel P. Berrange, 2015/10/12