Eitan,
Looks good but for the following comments.

> +UINT32
> +static __inline NlAttrSize(UINT32 payload)

Can we call this: NlAttrSizeForPayload() or NlAttrSizeForData? The reason is, 
there are existing APIs such as NlAttrLen() which operate on a nlattr. These 
functions that you are adding don't operate on an nlattr.

> +static __inline NlAttrTotalSize(UINT32 payload)

Can we call this: NlAttrTotalSizeForPayload() or NlAttrTotalSizeForData?

Acked-by: Nithin Raju <nit...@vmware.com>

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

Reply via email to