diff options
| author | Namhyung Kim <namhyung@kernel.org> | 2023-08-25 08:25:52 -0700 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2023-08-29 14:16:14 -0300 |
| commit | c091c78b73c82f1e2811ab19b8e287ddc7807651 (patch) | |
| tree | 78830e5420ad96b60545a381724268e2135e291b /tools/lib/perf | |
| parent | f174341d0da1cb2fb8888e1fa228c31523eaec90 (diff) | |
| download | linux-c091c78b73c82f1e2811ab19b8e287ddc7807651.tar.gz linux-c091c78b73c82f1e2811ab19b8e287ddc7807651.tar.bz2 linux-c091c78b73c82f1e2811ab19b8e287ddc7807651.zip | |
libperf: Get rid of attr.id field
Now there's no in-tree user of the field. To remove the possible bug
later, let's get rid of the 'id' field and add a comment for that.
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: https://lore.kernel.org/r/20230825152552.112913-4-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/lib/perf')
| -rw-r--r-- | tools/lib/perf/include/perf/event.h | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/tools/lib/perf/include/perf/event.h b/tools/lib/perf/include/perf/event.h index e563dd8c3628..ae64090184d3 100644 --- a/tools/lib/perf/include/perf/event.h +++ b/tools/lib/perf/include/perf/event.h @@ -148,7 +148,13 @@ struct perf_record_switch { struct perf_record_header_attr { struct perf_event_header header; struct perf_event_attr attr; - __u64 id[]; + /* + * Array of u64 id follows here but we cannot use a flexible array + * because size of attr in the data can be different then current + * version. Please use perf_record_header_attr_id() below. + * + * __u64 id[]; // do not use this + */ }; /* Returns the pointer to id array based on the actual attr size. */ |