[libvirt] [PATCH 2/5] macvtap support for libvirt -- parse new interface XML

[Stefan Berger]

Daniel Veillard <veillard at redhat.com> wrote on 02/11/2010 05:04:46 AM:


> Please respond to veillard
> 
> On Mon, Feb 08, 2010 at 02:36:51PM -0500, Stefan Berger wrote:
> > This part adds support to domain_conf.{c|h} for parsing the new
> > interface XML of type 'direct'. The parsed mode is now stored as 
> > an int.
> > 
> > Signed-off-by: Stefan Berger <stefanb at us.ibm.com>
> [...]
> > +    case VIR_DOMAIN_NET_TYPE_DIRECT:
> > +        virBufferEscapeString(buf, "      <source dev='%s'",
> > +                              def->data.direct.linkdev);
> > +        virBufferEscapeString(buf, " mode='%s'",
> > +                   virDomainNetdevMacvtapTypeToString
> (def->data.direct.mode));
> 
>   Nitpicking, virDomainNetdevMacvtapTypeToString return doesn't need to
>   be escaped because it's coming from an ASCII string list embedded in
>   the code.

Fixed in next patch.

> 
> [...]
> > @@ -594,6 +609,8 @@ struct _virSecurityLabelDef {
> >      int type;
> >  };
> > 
> > +
> > +
> >  #define VIR_DOMAIN_CPUMASK_LEN 1024
> > 
> >  /* Guest VM main configuration */
> 
>   random indentation change

Oops. Fixed in next patch.

> 
> > @@ -915,4 +932,6 @@ VIR_ENUM_DECL(virDomainGraphics)
> >  VIR_ENUM_DECL(virDomainState)
> >  VIR_ENUM_DECL(virDomainSeclabel)
> > 
> > +VIR_ENUM_DECL(virDomainNetdevMacvtap)
> > +
> >  #endif /* __DOMAIN_CONF_H */
> 
>   But that's fine. ACK,

Will repost.

  Stefan

> 
> Daniel
> 
> -- 
> Daniel Veillard      | libxml Gnome XML XSLT toolkit  
http://xmlsoft.org/
> daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
> http://veillard.com/ | virtualization library  http://libvirt.org/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100211/9ca0dcda/attachment-0001.htm>