> @@ -607,8 +603,7 @@ struct rte_mbuf {
>       struct rte_mempool *pool; /**< Pool from which mbuf was allocated. */
> 
>       /* second cache line - fields only used in slow path or on TX */
> -     RTE_MARKER cacheline1 __rte_cache_min_aligned;
> -
> +     alignas(RTE_CACHE_LINE_MIN_SIZE)

This comment is a matter of taste...

The alignas() is too far away from the field it applies to, and might get 
overlooked.
I would prefer it immediately before "struct rte_mbuf *next;" and "uint64_t 
dynfield2;".

>  #if RTE_IOVA_IN_MBUF
>       /**
>        * Next segment of scattered packet. Must be NULL in the last


Reply via email to