Re: [PATCH v1 3/5] perf trace-event: Free the global trace_event when a command ends

[sashiko-bot@kernel.org]

> The tep handle behind trace_event__tp_format() is a process wide cache
> of the running kernel's tracepoint formats and nothing ever released
> it, as the comment above it had noted since the code was added.
> 
> Add trace_event__exit() and call it from run_builtin() next to the
> existing perf_config__exit(). By then the builtin has returned, so the
> tep_event pointers it handed out, such as the ones cached in
> evsel->tp_format, are no longer reachable.
> 
> Note that this does not silence every libtraceevent report under
> leak sanitizer. Parsing a print fmt drops a token inside the library
> itself, and as those allocations are not linked into the handle,
> freeing the handle cannot reclaim them.
> 
> Assisted-by: Antigravity:gemini-3.1-pro
> Signed-off-by: Ian Rogers <irogers@google.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260918063249.2172589-1-irogers@google.com?part=3