Re: [PATCH v2] vepa: parsing for 802.1Qb{g|h} XML

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

 



* Stefan Berger (stefanb@xxxxxxxxxx) wrote:
> Gerhard Stenzel <gstenzel@xxxxxxxxxxxxxxxxxx> wrote on 05/12/2010 12:34:09 
> PM:
> 
> > 
> > On Wed, 2010-05-12 at 12:13 -0400, Stefan Berger wrote:
> > > I'd suggest to use this patch as a base for triggering the setup
> > > protocol with the 802.1Qb{g|h} switch.
> > 
> > Here is a RFC patch, which demonstrates how libvirt could communicate
> > with lldpad via the lldptool for the 802.1Qbg case. Please note, that
> > there is currently no public available version of lldptool which accepts
> > this command line. This is also work in progress. 
> > 
> 
> [...]
> 
> > +    virFormatMacAddr(mac, macaddr);
> > +
> > +     rc = ifaceGetRootIface(-1, linkdev, rootifname);
> > +     if (rc != 0) {
> > +        macvtapError(VIR_ERR_INTERNAL_ERROR,
> > +                     _("cannot get root interface for %s"),
> > +                     linkdev);
> > +         return rc;
> > +     }
> > +    VIR_DEBUG("root iface of %s is %s\n", linkdev, rootifname);
> > +
> > +    ifaceGetVlanID(linkdev, &vlanid);
> > +    VIR_DEBUG("vlan id of %s is %d\n", linkdev, vlanid);
> 
> I am wondering whether it would not be 'more general' to have lldpad 
> determine
> the root interface and handling bonding interfaces on the way rather 
> than libvirt trying to determine that here.

I think so, esp. since we will need to have bonding details in lldpad to
handle some active/backup failover cases.

> So we would just pass the
> linkdev or the even the macvtap to lldptool. The same would be true
> for finding the VLAN ID. For both the code I posted here on libvirt
> mailing list could be used in lldpad. For now I won't check it in.

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