> On Thu, 21 Aug 2025 04:51:47 +0000 > Sunil Kumar Kori <[email protected]> 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.
- RE: [EXTERNAL] Re: [PATCH v4 1/1] ethdev: add suppo... Sunil Kumar Kori
- [PATCH v5 1/1] ethdev: add support to provide link type skori
- Re: [PATCH v5 1/1] ethdev: add support to provide l... Stephen Hemminger
- RE: [EXTERNAL] Re: [PATCH v5 1/1] ethdev: add s... Sunil Kumar Kori
- [PATCH v6 1/1] ethdev: add support to provide link ... skori
- RE: [PATCH v6 1/1] ethdev: add support to provi... Morten Brørup
- [PATCH v7 1/1] ethdev: add support to provide l... skori
- Re: [PATCH v7 1/1] ethdev: add support to p... Stephen Hemminger
- RE: [EXTERNAL] Re: [PATCH v7 1/1] ethde... Sunil Kumar Kori
- Re: [EXTERNAL] Re: [PATCH v7 1/1] ... Stephen Hemminger
- RE: [EXTERNAL] Re: [PATCH v7 1/1] ... Sunil Kumar Kori
- [PATCH v8 1/1] ethdev: add support to provi... skori
- Re: [PATCH v8 1/1] ethdev: add support ... Stephen Hemminger
- RE: [EXTERNAL] Re: [PATCH v8 1/1] ... Sunil Kumar Kori
- [PATCH v9 1/1] ethdev: add support to p... skori
- Re: [PATCH v9 1/1] ethdev: add sup... Stephen Hemminger
- RE: [EXTERNAL] Re: [PATCH v9 1/1] ... Sunil Kumar Kori
- RE: [EXTERNAL] Re: [PATCH v9 1/1] ... Sunil Kumar Kori
- [PATCH v10 1/1] ethdev: add support to ... skori
- Re: [PATCH v10 1/1] ethdev: add su... Stephen Hemminger
- Re: [PATCH v10 1/1] ethdev: add su... Thomas Monjalon

