On Thu, 11 Jul 2019 09:26:19 +0200 Olivier Matz <olivier.m...@6wind.com> wrote:
> For generic fields, I think they should be declared in this file. For > instance, if we decide to replace the current m->timestamp field by a > dynamic field, we should add like this: > > #define RTE_MBUF_DYN_TIMESTAMP_ID "rte_timestamp" > #define RTE_MBUF_DYN_TIMESTAMP_SIZE sizeof(uint64_t) > #define RTE_MBUF_DYN_TIMESTAMP_ALIGN __alignof__(uint64_t) Let's use structures (like rte_flow) rather that macros for this?
