Re: [PATCHv5 6/9] spicevmc: support new qemu chardev

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

 



On Thu, Feb 03, 2011 at 21:16:08 -0700, Eric Blake wrote:
> From: Daniel P. Berrange <berrange@xxxxxxxxxx>
> 
> Inspired by https://bugzilla.redhat.com/show_bug.cgi?id=615757
> 
> Add a new character device backend for virtio serial channels that
> activates the QEMU spice agent on the main channel using the vdagent
> spicevmc connection.  The <target> must be type='virtio', and supports
> an optional name that specifies how the guest will see the channel
> (for now, name must be com.redhat.spice.0).
> 
> <channel type='spicevmc'>
>   <target type='virtio'/>
>   <address type='virtio-serial' controller='1' bus='0' port='3'/>
> </channel>
> 
> * docs/schemas/domain.rng: Support new XML.
> * docs/formatdomain.html.in: Document it.
> * src/conf/domain_conf.h (virDomainChrType): New enum value.
> * src/conf/domain_conf.c (virDomainChr): Add spicevmc.
> (virDomainChrDefParseXML, virDomainChrSourceDefParseXML)
> (virDomainChrDefParseTargetXML): Parse and enforce proper use.
> (virDomainChrSourceDefFormat, virDomainChrDefFormat): Format.
> * src/qemu/qemu_command.c (qemuBuildChrChardevStr)
> (qemuBuildCommandLine): Add qemu support.
> * tests/qemuxml2argvtest.c (domain): New test.
> * tests/qemuxml2argvdata/qemuxml2argv-channel-spicevmc.xml: New
> file.
> * tests/qemuxml2argvdata/qemuxml2argv-channel-spicevmc.args:
> Likewise.
> 
> Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>

I didn't check if the qemu command line generated by this patch follows what
qemu requires for spicemvc channel but the patch looks good and the XML
modeling for this is the same as what Dan used in a RHEL-specific patch
before. Thus,

ACK

Jirka

--
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]