[PATCH v2 1/1] ethdev: support congestion management

[Jerin Jacob]

On Tue, Sep 27, 2022 at 8:39 PM Bruce Richardson
<bruce.richardson at intel.com> wrote:
>
> On Tue, Sep 27, 2022 at 04:36:18PM +0200, Thomas Monjalon wrote:

> > >  ++
> >
> > I feel EAL is not the right place for CMAN definitions.
> >
> > After a discussion with Jerin, I understand we could use the same
> > definitions in other API classes, not only ethdev.  However I think this
> > .h file should be better hosted in lib/ethdev/ with its own namespace
> > rte_cman.  Then other libs could include this rte_cman.h without having a
> > strong dependency on ethdev.
> >
> > Deal?
> >
> Would rte_net also be an option? Alternatively, is this related to the work

Looks like the net is a good option. Since it is just a header file
any place would be OK.

+ @Olivier Matz

Any objections to keeping rte_cman.h to lib/net?

> and structures defined in the meter library header, since it seems also
> related to congestion management on ingress?
>
> /Bruce