[PATCH v7 2/9] perf c2c: add function view browser skeleton
Jiebin Sun <[email protected]> Thu, 6 Aug 2026 15:42:10 +0800
| Newsgroups | org.kernel.vger.linux-perf-users,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Add the skeleton of the c2c function view: a new TUI browser in tools/perf/ui/browsers/c2c-function.c reached by pressing TAB in the cacheline view. This commit wires up the entry point (a stub that will be filled in by later patches), declares perf_c2c__browse_function_view() in c2c.h, and adds the TAB key handler and help text to the cacheline browser. Link c2c-function.o directly into perf rather than libperf-ui.a. The browser is part of the c2c command and later patches make it depend on state and callbacks provided by builtin-c2c.o. libperf-ui.a is also linked into python/perf.so under --whole-archive, without the builtin command objects, which would leave those command-private symbols unresolved. Build the browser only with CONFIG_SLANG, matching its TUI-only entry point. Signed-off-by: Jiebin Sun <[email protected]> Cc: Adrian Hunter <[email protected]> Cc: Alexander Shishkin <[email protected]> Cc: Arnaldo Carvalho de Melo <[email protected]> Cc: Dapeng Mi <[email protected]> Cc: Ian Rogers <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: James Clark <[email protected]> Cc: Jiri Olsa <[email protected]> Cc: Mark Rutland <[email protected]> Cc: Namhyung Kim <[email protected]> Cc: Peter Zijlstra <[email protected]> Cc: Thomas Falcon <[email protected]> Reviewed-by: Tianyou Li <[email protected]> Reviewed-by: Wangyang Guo <[email protected]> --- tools/perf/Build | 1 + tools/perf/builtin-c2c.c | 4 ++ tools/perf/c2c.h | 12 +++-- tools/perf/ui/browsers/c2c-function.c | 77 +++++++++++++++++++++++++++ 4 files changed, 91 insertions(+), 3 deletions(-) create mode 100644 tools/perf/ui/browsers/c2c-function.c diff --git a/tools/perf/Build b/tools/perf/Build index e18c80a5c1bc..21b509f3a23d 100644 --- a/tools/perf/Build +++ b/tools/perf/Build @@ -21,6 +21,7 @@ perf-y += builtin-mem.o perf-y += builtin-data.o perf-y += builtin-version.o perf-y += builtin-c2c.o +perf-$(CONFIG_SLANG) += ui/browsers/c2c-function.o perf-y += builtin-daemon.o perf-$(CONFIG_LIBTRACEEVENT) += builtin-kmem.o diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c index 4a3cf8c906f9..12b18208d3dd 100644 --- a/tools/perf/builtin-c2c.c +++ b/tools/perf/builtin-c2c.c @@ -2737,6 +2737,7 @@ static int perf_c2c__hists_browse(struct hists *hists) static const char help[] = " d Display cacheline details \n" " ENTER Toggle callchains (if present) \n" + " TAB Switch to function view\n" " q Quit \n"; browser = perf_c2c_browser__new(hists); @@ -2758,6 +2759,9 @@ static int perf_c2c__hists_browse(struct hists *hists) case 'd': perf_c2c__browse_cacheline(browser->he_selection); break; + case '\t': + perf_c2c__browse_function_view(); + break; case '?': ui_browser__help_window(&browser->b, help); break; diff --git a/tools/perf/c2c.h b/tools/perf/c2c.h index aac01d4f6760..704b452aa390 100644 --- a/tools/perf/c2c.h +++ b/tools/perf/c2c.h @@ -132,18 +132,24 @@ void c2c_fmt_free(struct perf_hpp_fmt *fmt); bool c2c_fmt_equal(struct perf_hpp_fmt *a, struct perf_hpp_fmt *b); /* - * The TUI browser is only built with SLANG support. The stub below keeps the - * header self-contained for NO_SLANG builds, as util/hist.h does for its own - * TUI entry points. + * The TUI browsers are only built with SLANG support. The stubs below keep + * the header self-contained for NO_SLANG builds, as util/hist.h does for its + * own TUI entry points. */ #ifdef HAVE_SLANG_SUPPORT int perf_c2c__browse_cacheline(struct hist_entry *he); +int perf_c2c__browse_function_view(void); #else static inline int perf_c2c__browse_cacheline(struct hist_entry *he __maybe_unused) { return 0; } + +static inline int perf_c2c__browse_function_view(void) +{ + return 0; +} #endif #endif /* _PERF_C2C_H_ */ diff --git a/tools/perf/ui/browsers/c2c-function.c b/tools/perf/ui/browsers/c2c-function.c new file mode 100644 index 000000000000..c0fd7799e4f8 --- /dev/null +++ b/tools/perf/ui/browsers/c2c-function.c @@ -0,0 +1,77 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * C2C Function Browser - function-level cacheline sharing analysis + * + * Displays a 3-level hierarchy showing which functions share cachelines: + * Level 1: Read-side functions sorted by Cycles % (estimated load cycles) + * Level 2: Functions sampled writing the shared lines read by level 1 + * Level 3: The specific cachelines where the two functions contend + * + * Builds the hierarchy from the existing cacheline histograms + * (c2c_hist_entry->hists), reusing the shared c2c data structures. + */ + +#include <errno.h> +#include <inttypes.h> +#include <stdlib.h> +#include <string.h> +#include <tools/libc_compat.h> /* reallocarray */ +#include <asm/bug.h> +#include <linux/list.h> +#include <linux/rbtree.h> +#include <linux/zalloc.h> + +#include "../browser.h" +#include "../keysyms.h" +#include "../libslang.h" +#include "../ui.h" +#include "../../util/addr_location.h" +#include "../../util/cacheline.h" +#include "../../util/debug.h" +#include "../../util/hist.h" +#include "../../util/map.h" +#include "../../util/mem-events.h" +#include "../../util/mem-info.h" +#include "../../util/sort.h" +#include "../../util/symbol.h" +#include "../../util/thread.h" +#include "../../c2c.h" +#include "hists.h" + +struct perf_c2c_ext { + struct c2c_hists function_hists; + /* Total estimated cycles across all level-1 entries. */ + u64 total_cycles; +}; + +static struct perf_c2c_ext c2c_ext __maybe_unused; + +struct c2c_function_browser { + struct hist_browser hb; +}; + +static inline __maybe_unused u64 c2c_hitm_count(const struct c2c_stats *stats) +{ + return stats->tot_hitm; +} + +static inline __maybe_unused bool symbol_name_equal(struct symbol *a, struct symbol *b) +{ + /* Two unknown symbols compare equal, matching cmp_null() in util/sort.c. */ + if (!a || !b) + return a == b; + return arch__compare_symbol_names(a->name, b->name) == 0; +} + +static inline __maybe_unused u64 hist_entry__iaddr(struct hist_entry *he) +{ + if (he->mem_info) + return mem_info__iaddr(he->mem_info)->addr; + return he->ip; +} + +int perf_c2c__browse_function_view(void) +{ + ui__warning("C2C function view is not implemented yet.\n"); + return 0; +} -- 2.52.0