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