On Jul 2, 2013, at 7:11 PM, Ben Pfaff <[email protected]> wrote: > On Tue, Jul 02, 2013 at 06:27:18PM -0700, Justin Pettit wrote: >> From: Jarno Rajahalme <[email protected]> >> >> Do not clear tunnel metadata on tunnel input. >> >> This is a backport of 4110a57 (Keep all of tunnel metadata in >> flow.) to branch-1.10. This fixes an issue with received tunnel >> traffic and megaflows. xlate_actions() calls tnl_port_should_receive() >> to determine whether it should un-wildcard the tunnel fields. Without >> this commit, the original flow's tunnel information is zeroed and is >> unavailable for tnl_port_should_receive()'s use to determine whether >> it's a tunnel flow or not. This resulted in some tunnel traffic >> being wildcarded inappropriately. >> >> Bug #18277. >> >> Signed-off-by: Justin Pettit <[email protected]> >> Signed-off-by: Jarno Rajahalme <[email protected]> >> Signed-off-by: Ben Pfaff <[email protected]> > > This looks good to me, but I believe that it is in fact a backport to > 1.11 (not 1.10) since only 1.11 uses megaflows.
Whoops. I fixed the commit message and pushed. Thanks for the review. --Justin _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
