On Sep 8, 2014, at 11:31 AM, Ben Pfaff <b...@nicira.com> wrote:

> On Fri, Sep 05, 2014 at 04:05:08PM -0700, Jarno Rajahalme wrote:
>> Add a new action type OVS_ACTION_ATTR_SET_MASKED, and support for
>> parsing, printing, and committing them.
>> 
>> Masked set actions add a mask, immediately following the netlink
>> attribute data, within the netlink attribute itself.  Thus the key
>> attribute size for a masked set action is exactly double of the
>> non-masked set action.
>> 
>> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>
>> ---
>> v5: Addressed Ben's comments, moved OVS_KEY_ATTR_SET_MASKED comment to
>>    this patch.
> 
> I think that you can use nl_attr_get() in place of the new get_value()
> function-like macro.
> 

Done!

> Acked-by: Ben Pfaff <b...@nicira.com>

Thanks for the review, pushed to master,

  Jarno

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to