Re: [libvirt] [PATCH 2/4] Add a <graphics> type for SPICE protocol

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks for the patches (and sorry for the late response).

The patches are fine, though I am still missing a means to control which
of spice channels are to be encrypted. Also missing is a way to set
key/cert files and cipher suite per domain. (I aksed spice folks to
avoid this problem by using reasonable defaults). More of a problem is
the need to set/reset spice "ticket" (one time password), or at least
disable it completely on the command line (with ,disable-ticketing).

On Tue, Sep 29, 2009 at 04:43:51PM +0100, Daniel P. Berrange wrote:
> This supports the -qxl argument in RHEL-5's fork of KVM
> which has SPICE support. QXL is a graphics card, but
> inexplicably doesn't use the standard -vga syntax for
> generic configuration. Also -qxl is rather useless unless
> you also supply -spice (coming in next patch)

> +
> +                if (virAsprintf(&optstr, "%u,ram=%u",
> +                                def->videos[0]->heads,
> +                                (def->videos[0]->vram /1024)) < 0)
> +                    goto no_memory;

this hides spice's own default, and sends ",ram=0" if xml lacks vram
attribute. I think it would be better to drop ",ram" completely if
vram==0.

Regards,

Dan.

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]