> On Aug 26, 2015, at 6:56 PM, Jesse Gross <je...@nicira.com> wrote: > > On Wed, Aug 26, 2015 at 5:42 PM, Jarno Rajahalme <jrajaha...@nicira.com> > wrote: >> Initialize the whole 'exact_match_mask' in >> mf_mask_field_and_prereqs(), as geneve options can be bigger than 16 >> bytes. >> >> Requested-by: Jesse Gross <je...@nicira.com> >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > Acked-by: Jesse Gross <je...@nicira.com> > > I think we can actually do the same memset optimization in > mf_mask_field() as well.
I moved the exact_match_mask to global scope and used it in both. I exposed the global variable in meta-flow.h and moved the initializers to meta-flow.c, and pushed to master with these changes. Thanks for the review and the improvement suggestion! Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev