>>But you also now only add tap devices to vmbrX instead of vmbrXvY, >>which is wrong, the vmbrXvY bridges are now virtually unused.
Yes, this is expected. we don't need vmbrXvY anymore, because we tag vlan on vmbrX ports directly. >>After: >>32: tap1000i0@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc >>pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 >>33: tap1000i1@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc >>pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 >>34: tap1000i2@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc >>pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 try, #brigge -c vlan you should like vlan tag list for each tap interface. ----- Mail original ----- De: "Wolfgang Bumiller" <w.bumil...@proxmox.com> À: "aderumier" <aderum...@odiso.com> Cc: "pve-devel" <pve-devel@pve.proxmox.com> Envoyé: Lundi 3 Août 2015 11:54:34 Objet: Re: [pve-devel] [PATCH] tap_plug : add support for vlan aware linux bridge > vlan tagging is always done on the main bridge, not firewall bridge. But you also now only add tap devices to vmbrX instead of vmbrXvY, which is wrong, the vmbrXvY bridges are now virtually unused. I have a VM with 3 interfaces, two of them tagged to 5 and 6. Before: 32: tap1000i0@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 33: tap1000i1@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0v5 state UNKNOWN mode DEFAULT group default qlen 500 34: tap1000i2@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0v6 state UNKNOWN mode DEFAULT group default qlen 500 After: 32: tap1000i0@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 33: tap1000i1@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 34: tap1000i2@NONE: <BROADCAST,MULTICAST,PROMISC,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast master vmbr0 state UNKNOWN mode DEFAULT group default qlen 500 On Mon, Jul 27, 2015 at 04:09:45PM +0200, Alexandre DERUMIER wrote: > does somebody have tested my vlan bridges patches ? (note that that need > iproute2 from debian sid, for vlan ranges) > > It's working really fine here, I'm looking to add a patch for Q-in-Q bridge > too. (I think Stefan Priebe use them) > > ----- Mail original ----- > De: "aderumier" <aderum...@odiso.com> > À: "Wolfgang Bumiller" <w.bumil...@proxmox.com> > Cc: "pve-devel" <pve-devel@pve.proxmox.com> > Envoyé: Vendredi 24 Juillet 2015 18:49:18 > Objet: Re: [pve-devel] [PATCH] tap_plug : add support for vlan aware linux > bridge > > >>Why is `bridge_add_interface` now restricted to the firewall-else > >>branch? > > I manage it like openvswitch, > > vlan tagging is always done on the main bridge, not firewall bridge. > > > > + if ($firewall) { > > + &$create_firewall_bridge_linux($iface, $bridge, $tag); > > create_firewall_bridge_linux($iface, $bridge, $tag) > have > > - &$bridge_add_interface($bridge, $vethfwpeer); > + &$bridge_add_interface($bridge, $vethfwpeer, $tag); #tag on the main bridge > - return $fwbr; > + &$bridge_add_interface($fwbr, $iface); # add vm tap interface on fwbridge > without vlan tag > > ----- Mail original ----- > De: "Wolfgang Bumiller" <w.bumil...@proxmox.com> > À: "aderumier" <aderum...@odiso.com> > Cc: "pve-devel" <pve-devel@pve.proxmox.com> > Envoyé: Vendredi 24 Juillet 2015 15:20:06 > Objet: Re: [pve-devel] [PATCH] tap_plug : add support for vlan aware linux > bridge > > On Fri, Jul 24, 2015 at 01:52:59PM +0200, Alexandre Derumier wrote: > > - $newbridge = &$create_firewall_bridge_linux($iface, $newbridge) if > > $firewall; > > + if (!$vlan_aware) { > > + my $newbridge = activate_bridge_vlan($bridge, $tag); > > + copy_bridge_config($bridge, $newbridge) if $bridge ne $newbridge; > > + $tag = undef; > > + } > > + > > + if ($firewall) { > > + &$create_firewall_bridge_linux($iface, $bridge, $tag); > > + } else { > > + &$bridge_add_interface($bridge, $iface, $tag); > > + } > > > > - &$bridge_add_interface($newbridge, $iface); > > > Why is `bridge_add_interface` now restricted to the firewall-else > branch? > _______________________________________________ > pve-devel mailing list > pve-devel@pve.proxmox.com > http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel > > _______________________________________________ > pve-devel mailing list > pve-devel@pve.proxmox.com > http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel