Re: [PATCH 13/52] perf annotate-data: Add dso->data_types tree
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH 13/52] perf annotate-data: Add dso->data_types tree
- From: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
- Date: Thu, 21 Dec 2023 17:10:53 -0300
- Cc: Jiri Olsa <jolsa@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Ian Rogers <irogers@xxxxxxxxxx>, Adrian Hunter <adrian.hunter@xxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx>, linux-perf-users@xxxxxxxxxxxxxxx, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Stephane Eranian <eranian@xxxxxxxxxx>, Masami Hiramatsu <mhiramat@xxxxxxxxxx>, Andi Kleen <ak@xxxxxxxxxxxxxxx>, linux-trace-devel@xxxxxxxxxxxxxxx, linux-toolchains@xxxxxxxxxxxxxxx
- In-reply-to: <20231110000012.3538610-14-namhyung@kernel.org>
- References: <20231110000012.3538610-1-namhyung@kernel.org> <20231110000012.3538610-14-namhyung@kernel.org>
Em Thu, Nov 09, 2023 at 03:59:32PM -0800, Namhyung Kim escreveu:
> +++ b/tools/perf/util/dso.h
> @@ -154,6 +154,8 @@ struct dso {
> size_t symbol_names_len;
> struct rb_root_cached inlined_nodes;
> struct rb_root_cached srclines;
> + struct rb_root data_types;
> +
> struct {
> u64 addr;
> struct symbol *symbol;
At some point we need to make these feature specific members to be
associated on demand, maybe thru some hash table, etc.
I.e. the most basic workflow, what everybody needs should be in 'struct
dso', something one _may_ want, like data profiling, should be in
associated with that DSO thru some other way.
I'm applying this now as this is a super cool feature, but think about
it.
- Arnaldo
[Index of Archives]
[Linux USB Development]
[Linux USB Development]
[Linux Audio Users]
[Yosemite Hiking]
[Linux Kernel]
[Linux SCSI]