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

Stephen Hemminger stephen at networkplumber.org
Thu Aug 21 17:20:12 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.


More information about the dev mailing list