Re: [libvirt] [PATCH 1/2] Add domain support for virtio channel

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

 



On Thu, Feb 18, 2010 at 03:29:13PM +0000, Matthew Booth wrote:
> On 18/02/10 14:09, Daniel P. Berrange wrote:
> > On Thu, Feb 18, 2010 at 01:16:02PM +0000, Matthew Booth wrote:
> >> On 18/02/10 12:34, Daniel P. Berrange wrote:
> >>> On Wed, Feb 17, 2010 at 05:11:00PM +0000, Matthew Booth wrote:
> >>>> Add support for virtio-serial by defining a new 'virtio' channel target type and
> >>>> a virtio-serial controller. Allows the following to be specified in a domain:
[...]
> Updated patch attached.
> 
[...]
> @@ -4905,11 +5055,18 @@ virDomainChrDefFormat(virBufferPtr buf,
>              break;
>          }
>  
> +    case VIR_DOMAIN_CHR_TARGET_TYPE_VIRTIO:
> +        virBufferAddLit(buf, "      <target type='virtio'");
> +        if (def->target.name) {
> +            virBufferVSprintf(buf, " name='%s'", def->target.name);

 I changed that to use virBufferEscapeString as the string comes from
the user XML input.

> +        }
> +        virBufferAddLit(buf, "/>\n");
> +        break;
> +

 ACK, and pushed, thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

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