<div dir="ltr">Hi Alexandre,<div><br></div><div>We also use QinQ and have submitted patches for the previous network implementation that made use of a "bridge in bridge" design to achieve the QinQ functionality.</div><div><br></div><div>The new vlan aware bridge implementation will be a lot cleaner.</div><div><br></div><div>When your patches are ready we will test them and provide feedback.</div><div><br></div><div><br></div><div>Thanks,</div><div><br></div><div><br></div><div><br></div><div>Andrew</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 2:09 AM, Alexandre DERUMIER <span dir="ltr"><<a href="mailto:aderumier@odiso.com" target="_blank">aderumier@odiso.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">does somebody have tested my vlan bridges patches ?  (note that that need iproute2 from debian sid, for vlan ranges)<br>
<br>
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)<br>
<br>
<br>
<br>
<br>
<br>
----- Mail original -----<br>
De: "aderumier" <<a href="mailto:aderumier@odiso.com">aderumier@odiso.com</a>><br>
À: "Wolfgang Bumiller" <<a href="mailto:w.bumiller@proxmox.com">w.bumiller@proxmox.com</a>><br>
Cc: "pve-devel" <<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a>><br>
Envoyé: Vendredi 24 Juillet 2015 18:49:18<br>
<div class="HOEnZb"><div class="h5">Objet: Re: [pve-devel] [PATCH] tap_plug : add support for vlan aware linux bridge<br>
<br>
>>Why is `bridge_add_interface` now restricted to the firewall-else<br>
>>branch?<br>
<br>
I manage it like openvswitch,<br>
<br>
vlan tagging is always done on the main bridge, not firewall bridge.<br>
<br>
<br>
> + if ($firewall) {<br>
> + &$create_firewall_bridge_linux($iface, $bridge, $tag);<br>
<br>
create_firewall_bridge_linux($iface, $bridge, $tag)<br>
have<br>
<br>
- &$bridge_add_interface($bridge, $vethfwpeer);<br>
+ &$bridge_add_interface($bridge, $vethfwpeer, $tag); #tag on the main bridge<br>
- return $fwbr;<br>
+ &$bridge_add_interface($fwbr, $iface); # add vm tap interface on fwbridge without vlan tag<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
----- Mail original -----<br>
De: "Wolfgang Bumiller" <<a href="mailto:w.bumiller@proxmox.com">w.bumiller@proxmox.com</a>><br>
À: "aderumier" <<a href="mailto:aderumier@odiso.com">aderumier@odiso.com</a>><br>
Cc: "pve-devel" <<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a>><br>
Envoyé: Vendredi 24 Juillet 2015 15:20:06<br>
Objet: Re: [pve-devel] [PATCH] tap_plug : add support for vlan aware linux bridge<br>
<br>
On Fri, Jul 24, 2015 at 01:52:59PM +0200, Alexandre Derumier wrote:<br>
> - $newbridge = &$create_firewall_bridge_linux($iface, $newbridge) if $firewall;<br>
> + if (!$vlan_aware) {<br>
> + my $newbridge = activate_bridge_vlan($bridge, $tag);<br>
> + copy_bridge_config($bridge, $newbridge) if $bridge ne $newbridge;<br>
> + $tag = undef;<br>
> + }<br>
> +<br>
> + if ($firewall) {<br>
> + &$create_firewall_bridge_linux($iface, $bridge, $tag);<br>
> + } else {<br>
> + &$bridge_add_interface($bridge, $iface, $tag);<br>
> + }<br>
><br>
> - &$bridge_add_interface($newbridge, $iface);<br>
<br>
<br>
Why is `bridge_add_interface` now restricted to the firewall-else<br>
branch?<br>
_______________________________________________<br>
pve-devel mailing list<br>
<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a><br>
<a href="http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel" rel="noreferrer" target="_blank">http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel</a><br>
<br>
_______________________________________________<br>
pve-devel mailing list<br>
<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a><br>
<a href="http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel" rel="noreferrer" target="_blank">http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel</a><br>
</div></div></blockquote></div><br></div>