>
> On Thu, Jan 14, 2021 at 12:07 PM Bruce Richardson
> <bruce.richard...@intel.com> wrote:
> >
> > Add in the missing includes into the header file for c11 memory model,
> > so it can be compiled up independently of the other headers.
> >
> > Fixes: 39368ebfc606 ("ring: introduce C11 memory model barrier option")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com>
> > ---
> > lib/librte_ring/rte_ring_c11_mem.h | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/lib/librte_ring/rte_ring_c11_mem.h
> > b/lib/librte_ring/rte_ring_c11_mem.h
> > index 0fb73a3371..12caf06a31 100644
> > --- a/lib/librte_ring/rte_ring_c11_mem.h
> > +++ b/lib/librte_ring/rte_ring_c11_mem.h
> > @@ -10,6 +10,10 @@
> > #ifndef _RTE_RING_C11_MEM_H_
> > #define _RTE_RING_C11_MEM_H_
> >
> > +#include <stdint.h>
> > +#include <rte_common.h>
> > +#include <rte_ring_core.h>
> > +
>
> This header is never directly included, like rte_ring_generic.h.
> I'd rather flag it as not being checked.
Yep, it is sort of 'internal' header that we need to keep public,
because of inlined ring enqueue/dequeue functions.
>
>
> --
> David Marchand
