On Sep 8, 2014, at 11:31 AM, Ben Pfaff <[email protected]> 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 <[email protected]>
>> ---
>> 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 <[email protected]>

Thanks for the review, pushed to master,

  Jarno

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to