24/02/2024 09:21, Tyler Retzlaff:
> Provide inline functions to access rearm data and rx descriptor fields
> in rte_mbuf struct.
[...]
> +static inline
> +uint64_t *
> +rte_mbuf_rearm_data(struct rte_mbuf *m)
> +{
> +     RTE_BUILD_BUG_ON(offsetof(struct rte_mbuf, ol_flags) !=
> +                      offsetof(struct rte_mbuf, data_off) + 8);
> +     RTE_BUILD_BUG_ON(offsetof(struct rte_mbuf, rearm_data) !=
> +                      RTE_ALIGN(offsetof(struct rte_mbuf,
> +                                         data_off),
> +                                         16));
> +
> +     return (uint64_t *)&m->data_off;
> +}
> +
> +static inline
> +void *
> +rte_mbuf_rx_descriptor_fields1(struct rte_mbuf *m)
> +{
> +     return &m->packet_type;
> +}

We need to enforce usage of these accessors.
Please add these 2 things:
1/ comments in the mbuf structure to encourage using these functions
2/ checkpatch warning if using the markers directly in new code


Reply via email to