[EXTERNAL] Re: [PATCH v7 1/1] ethdev: add support to provide link type

Sunil Kumar Kori skori at marvell.com
Fri Aug 22 06:34:49 CEST 2025


> On Thu, 21 Aug 2025 04:51:47 +0000
> Sunil Kumar Kori <skori at marvell.com> wrote:
> 
> > >
> > > IMHO adding a trace point to a helper function like this is just
> > > extra clutter, remove it.
> > >
> > Ack, I will share a separate patch to add trace in this API.
> 
> No, please no tracepoint. Tracing is meant for datapath and control path
> operations not every helper library. For example, the string functions should not
> have tracing.
Ack.


More information about the dev mailing list