On Tue, Jan 07, 2014 at 10:56:53AM -0800, Andy Zhou wrote: > On Tue, Jan 7, 2014 at 9:58 AM, Ben Pfaff <[email protected]> wrote: > > > On Tue, Jan 07, 2014 at 01:11:18AM -0800, Andy Zhou wrote: > > > When vlan-splinter is enabled, ovs receives non-vlan flows from the > > > kernel vlan ports, vlan tag is then added to the incoming flow before > > > xlating, so that they look like those received from a trunk port. > > > > > > In case megaflow is enabled, xlating may set vlan masks during rule > > > processing as usual. If those vlan masks were serialized and downloaded > > > to the kernel (this bug), those mega flows will be rejected due to > > > unexpected vlan mask encapsulation, since the original kernel flows do > > > not have vlan tags. This bug does not break connectivity, but impacts > > > performance since all traffic received on vlan splinter ports will now > > > be handled by vswitchd, as no datapath flows can be successfully > > > installed. > > > > > > This fix is to make sure no vlan mask encapsulation is generated for > > > the datapath flow if its in_port was re-written by vlan-splinter > > > receiving logic. > > > > > > Bug fixed: 22567 > > > > > > Signed-off-by: Andy Zhou <[email protected]> > > > > Good work tracking this down, thanks. > > > > Thanks for the review. > > > > > Does this go all the way back to 1.11? > > > > No, Unfortunately. Should I also send the fix for 1.11 to dev-mailing list?
I mean, does the bug go back that far? I guess you confirmed that, though. > > Why is compose_output_action__() changing? I suspect that it is still > > necessary, because when VLAN splinters are in use the choice of the > > datapath output port depends on the VLAN in use (that is, output to > > OpenFlow port eth0 might go to network device eth0 for VLAN 0 and > > e.g. eth0.9 for VLAN 9). > > > > I see. I will drop this change since it is not relevant to the fix > anyways. > > Will send out a V2. Please let me know what to do with 1.11+. If you think the backport needs additional review, please send it and we'll take a look. Thanks, Ben. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
