Re: [PATCH v2 1/3] perf parse-events: pass parse_state to add_tracepoint
From: Dominique Martinet
Date: Wed May 08 2024 - 19:04:22 EST
Thanks for the review!
Ian Rogers wrote on Wed, May 08, 2024 at 02:37:16PM -0700:
> Nit: evsel__newtp_idx typo
Fixed in v3
> Fwiw, I think the idx value is possibly something to be done away
> with. We renumber the evsels here:
> https://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git/tree/tools/perf/util/parse-events.c?h=perf-tools-next#n2077
(I'm not sure what's immediately actionable here, so I'll consider this
a higher level comment for later cleanup)
Ian Rogers wrote on Wed, May 08, 2024 at 03:47:29PM -0700:
> Note, this isn't applying cleanly to the perf-tools-next branch on:
> https://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git/
Thanks for this note, I've rebased on this branch for v3
--
Dominique Martinet | Asmadeus