[libvirt] [PATCHv2] qemu: Redundant listen address entry in quest xml
Ján Tomko
jtomko at redhat.com
Thu Aug 14 13:17:16 UTC 2014
On 08/14/2014 02:44 PM, Erik Skultety wrote:
> When editing guest's XML (on QEMU), it was possible to add multiple
> listen elements into graphics parent element. However QEMU does not
> support listening on multiple addresses. Configuration is tested for
> multiple 'listen address' and if positive, an error is raised.
We have a public bug open for this issue. It's nice to add those in the commit
message, if someone wants to know why the commit was added in the future.
https://bugzilla.redhat.com/show_bug.cgi?id=1119212
> ---
> src/qemu/qemu_process.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index 9e6a9ae..1810e6c 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -3934,6 +3934,14 @@ int qemuProcessStart(virConnectPtr conn,
> }
> graphics->listens[0].fromConfig = true;
> }
> + /* multiple listen addresses are unsupported configuration in qemu
> + */
This comment is redundant - it basically repeats the error message.
> + else if (graphics->nListens > 1) {
We prefer putting 'else' on the same line as the closing brace of the 'if' block:
http://libvirt.org/hacking.html#curly_braces
> + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> + _("QEMU does not support multiple listen "
> + "addresses for a domain."));
This is not true. For example: you can use one VNC and one SPICE graphics,
both with different listen addresses. How about 'QEMU does not support
multiple listen addresses for one graphics device'?
Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140814/fb91b1bb/attachment-0001.sig>
More information about the libvir-list
mailing list