> 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

Reply via email to