Messages in this thread | | | Date | Thu, 9 May 2024 08:03:49 +0900 | From | Dominique Martinet <> | Subject | Re: [PATCH v2 1/3] perf parse-events: pass parse_state to add_tracepoint |
| |
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
| |