Re: [PATCH v7 2/9] perf c2c: add function view browser skeleton
From: Jiebin Sun
Date: Mon Aug 10 2026 - 01:28:54 EST
Hi Namhyung,
> I don't think it's a proper fix. The c2c-function.o should belong to
> libperf-ui.a. Perhabs we can split libperf-tui.a and not link it to the
> python module. But still, it's better to move the common code into the
> util directory.
Thanks. I reworked this in v8 to keep c2c-function.o in libperf-ui.a and
removed the v7 tools/perf/Build change.
builtin-c2c.c now passes the cacheline histograms, the --coalesce field
list, symbol_full, and the cacheline detail callback through
struct c2c_function_view_args. The browser no longer references the
command-private c2c instance or perf_c2c__browse_cacheline().
The shared c2c types and formatter callbacks are now in util/c2c.h and
util/c2c.c. struct perf_c2c and the command state remain private to
builtin-c2c.c.
Thanks,
Jiebin