On Wed, Dec 23, 2015 at 4:23 PM, Ben Pfaff <[email protected]> wrote: > On Wed, Dec 23, 2015 at 03:41:04PM -0500, Russell Bryant wrote: > > On 12/22/2015 04:17 PM, Ben Pfaff wrote: > > > Until now, the flow table treated localnet logical ports that have a > VLAN > > > quite differently from those that don't. The ones without a VLAN were > > > essentially trunk ports: any packets that came in, that weren't picked > off > > > by a localnet port with a VLAN, were passed to the ones without a VLAN. > > > This wasn't the intended behavior. > > > > > > This commit changes behavior to the intended behavior. Now, localnet > ports > > > without a specific VLAN only receive packets without a VLAN header or > those > > > with VLAN ID 0 (with that header stripped off). > > > > > > Found by inspection. > > > > > > Signed-off-by: Ben Pfaff <[email protected]> > > > > Acked-by: Russell Bryant <[email protected]> > > > > I was able to test this using a new test case that I just posted to the > > list. It passes with v2 applied. > > Thanks. > > Is this for all three patches? >
Sorry about that. Sure, consider the ACK for all three patches. The second one is trivial, so that's fine. I didn't review the first one quite as closely as the 3rd, but it seems sane and passes tests for me locally. -- Russell Bryant _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
