diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2025-04-08 10:51:01 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2025-04-10 10:44:12 -0300 |
| commit | 0afeacbc8d0828140410ce144a60f6159a8e8437 (patch) | |
| tree | c75f03e1fc6260c8e130b7a92b22e73ee28dbbfa | |
| parent | 6994c6374a4e4a4fbb67fbe231420ca6c3bee4b6 (diff) | |
perf check: Move the FEATURE_STATUS() macro to its only user source file
It is just 'perf check' that uses that macro, to initialize the list of
features built into perf, so move it there.
This also avoids depending on things that are not included from
builtin.h, like is_builtin(), the CONFIG_ macros, etc, that are all
included in 'builtin-check.c' and before where this macro was moved to.
Tested-by: Ingo Molnar <mingo@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Dmitriy Vyukov <dvyukov@google.com>
Cc: Howard Chu <howardchu95@gmail.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: https://lore.kernel.org/r/Z_dkNDj9EPFwPqq1@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
| -rw-r--r-- | tools/perf/builtin-check.c | 5 | ||||
| -rw-r--r-- | tools/perf/builtin.h | 5 |
2 files changed, 5 insertions, 5 deletions
diff --git a/tools/perf/builtin-check.c b/tools/perf/builtin-check.c index bed9df0d7912..a451fa25f50d 100644 --- a/tools/perf/builtin-check.c +++ b/tools/perf/builtin-check.c @@ -22,6 +22,11 @@ static const char *check_feature_usage[] = { NULL }; +#define FEATURE_STATUS(name_, macro_) { \ + .name = name_, \ + .macro = #macro_, \ + .is_builtin = IS_BUILTIN(macro_) } + struct feature_status supported_features[] = { FEATURE_STATUS("aio", HAVE_AIO_SUPPORT), FEATURE_STATUS("bpf", HAVE_LIBBPF_SUPPORT), diff --git a/tools/perf/builtin.h b/tools/perf/builtin.h index b7e60d19fad9..7db7bc054f6e 100644 --- a/tools/perf/builtin.h +++ b/tools/perf/builtin.h @@ -8,11 +8,6 @@ struct feature_status { int is_builtin; }; -#define FEATURE_STATUS(name_, macro_) { \ - .name = name_, \ - .macro = #macro_, \ - .is_builtin = IS_BUILTIN(macro_) } - extern struct feature_status supported_features[]; void feature_status__printf(const struct feature_status *feature); |