[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZwYVCTsf8p3v22HQ@google.com>
Date: Tue, 8 Oct 2024 22:30:49 -0700
From: Namhyung Kim <namhyung@...nel.org>
To: linux@...blig.org, kan.liang@...ux.intel.com
Cc: peterz@...radead.org, mingo@...hat.com, acme@...nel.org,
mark.rutland@....com, alexander.shishkin@...ux.intel.com,
jolsa@...nel.org, irogers@...gle.com, adrian.hunter@...el.com,
linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf callchain: Remove unused callchain_branch_counts
On Sun, Oct 06, 2024 at 02:22:04AM +0100, linux@...blig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@...blig.org>
>
> callchain_branch_counts() was added in 2016 by commit
> 3dd029ef9401 ("perf report: Calculate and return the branch flag counting")
> but unused.
>
> Remove it and it's helpers.
Kan, are you ok with this?
Thanks,
Namhyung
>
> Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>
> ---
> tools/perf/util/callchain.c | 71 -------------------------------------
> tools/perf/util/callchain.h | 4 ---
> 2 files changed, 75 deletions(-)
>
> diff --git a/tools/perf/util/callchain.c b/tools/perf/util/callchain.c
> index 0c7564747a14..11435b72afbe 100644
> --- a/tools/perf/util/callchain.c
> +++ b/tools/perf/util/callchain.c
> @@ -1266,77 +1266,6 @@ int callchain_node__fprintf_value(struct callchain_node *node,
> return 0;
> }
>
> -static void callchain_counts_value(struct callchain_node *node,
> - u64 *branch_count, u64 *predicted_count,
> - u64 *abort_count, u64 *cycles_count)
> -{
> - struct callchain_list *clist;
> -
> - list_for_each_entry(clist, &node->val, list) {
> - if (branch_count)
> - *branch_count += clist->branch_count;
> -
> - if (predicted_count)
> - *predicted_count += clist->predicted_count;
> -
> - if (abort_count)
> - *abort_count += clist->abort_count;
> -
> - if (cycles_count)
> - *cycles_count += clist->cycles_count;
> - }
> -}
> -
> -static int callchain_node_branch_counts_cumul(struct callchain_node *node,
> - u64 *branch_count,
> - u64 *predicted_count,
> - u64 *abort_count,
> - u64 *cycles_count)
> -{
> - struct callchain_node *child;
> - struct rb_node *n;
> -
> - n = rb_first(&node->rb_root_in);
> - while (n) {
> - child = rb_entry(n, struct callchain_node, rb_node_in);
> - n = rb_next(n);
> -
> - callchain_node_branch_counts_cumul(child, branch_count,
> - predicted_count,
> - abort_count,
> - cycles_count);
> -
> - callchain_counts_value(child, branch_count,
> - predicted_count, abort_count,
> - cycles_count);
> - }
> -
> - return 0;
> -}
> -
> -int callchain_branch_counts(struct callchain_root *root,
> - u64 *branch_count, u64 *predicted_count,
> - u64 *abort_count, u64 *cycles_count)
> -{
> - if (branch_count)
> - *branch_count = 0;
> -
> - if (predicted_count)
> - *predicted_count = 0;
> -
> - if (abort_count)
> - *abort_count = 0;
> -
> - if (cycles_count)
> - *cycles_count = 0;
> -
> - return callchain_node_branch_counts_cumul(&root->node,
> - branch_count,
> - predicted_count,
> - abort_count,
> - cycles_count);
> -}
> -
> static int count_pri64_printf(int idx, const char *str, u64 value, char *bf, int bfsize)
> {
> return scnprintf(bf, bfsize, "%s%s:%" PRId64 "", (idx) ? " " : " (", str, value);
> diff --git a/tools/perf/util/callchain.h b/tools/perf/util/callchain.h
> index 86ed9e4d04f9..d7741fa9e9de 100644
> --- a/tools/perf/util/callchain.h
> +++ b/tools/perf/util/callchain.h
> @@ -298,10 +298,6 @@ void free_callchain(struct callchain_root *root);
> void decay_callchain(struct callchain_root *root);
> int callchain_node__make_parent_list(struct callchain_node *node);
>
> -int callchain_branch_counts(struct callchain_root *root,
> - u64 *branch_count, u64 *predicted_count,
> - u64 *abort_count, u64 *cycles_count);
> -
> void callchain_param_setup(u64 sample_type, const char *arch);
>
> bool callchain_cnode_matched(struct callchain_node *base_cnode,
> --
> 2.46.2
>
Powered by blists - more mailing lists