lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210203205024.GM854763@kernel.org>
Date:   Wed, 3 Feb 2021 17:50:24 -0300
From:   Arnaldo Carvalho de Melo <acme@...nel.org>
To:     Ian Rogers <irogers@...gle.com>
Cc:     Peter Zijlstra <peterz@...radead.org>,
        Ingo Molnar <mingo@...hat.com>,
        Mark Rutland <mark.rutland@....com>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Jiri Olsa <jolsa@...hat.com>,
        Namhyung Kim <namhyung@...nel.org>,
        LKML <linux-kernel@...r.kernel.org>,
        Stephane Eranian <eranian@...gle.com>
Subject: Re: [PATCH 2/2] perf parse-events: Break out tracepoint and printing.

Em Wed, Feb 03, 2021 at 10:38:41AM -0800, Ian Rogers escreveu:
> On Wed, Feb 3, 2021 at 8:21 AM Arnaldo Carvalho de Melo <acme@...nel.org>
> wrote:
> 
> > Em Tue, Feb 02, 2021 at 09:26:59PM -0800, Ian Rogers escreveu:
> > > Move print_*_events functions out of parse-events.c into a new
> > > print-events.c. Move tracepoint code into tracepoint.c or
> > > trace-event-info.c (sole user). This reduces the dependencies of
> > > parse-events.c and makes it more amenable to being a library in the
> > > future.
> > > Remove some unnecessary definitions from parse-events.h. Fix a
> > > checkpatch.pl warning on using unsigned rather than unsigned int.
> >
> > Thanks, applied.
> >
> 
> Wow, cool :-) I wasn't sure about the macros in the headers. Are you
> staging these patches before pushing to your perf/core or using
> tmp.perf/core ? I know you mentioned a different approach. When I look at:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/
> 
> I don't see anything newer than 9 days.

Right, I'm collecting some more patches, will do build tests and push,
till then, lemme push tmp.perf/core with what I have.
 
> Thanks,
> Ian
> 
> - Arnaldo
> >
> >
> > > Signed-off-by: Ian Rogers <irogers@...gle.com>
> > > ---
> > >  tools/perf/builtin-list.c          |   2 +-
> > >  tools/perf/builtin-lock.c          |   1 +
> > >  tools/perf/builtin-timechart.c     |   1 +
> > >  tools/perf/builtin-trace.c         |   1 +
> > >  tools/perf/util/Build              |   2 +
> > >  tools/perf/util/parse-events.c     | 620 +----------------------------
> > >  tools/perf/util/parse-events.h     |  29 --
> > >  tools/perf/util/print-events.c     | 472 ++++++++++++++++++++++
> > >  tools/perf/util/print-events.h     |  21 +
> > >  tools/perf/util/trace-event-info.c |  94 +++++
> > >  tools/perf/util/tracepoint.c       |  63 +++
> > >  tools/perf/util/tracepoint.h       |  25 ++
> > >  12 files changed, 687 insertions(+), 644 deletions(-)
> > >  create mode 100644 tools/perf/util/print-events.c
> > >  create mode 100644 tools/perf/util/print-events.h
> > >  create mode 100644 tools/perf/util/tracepoint.c
> > >  create mode 100644 tools/perf/util/tracepoint.h
> > >
> > > diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
> > > index 10ab5e40a34f..91327a321c36 100644
> > > --- a/tools/perf/builtin-list.c
> > > +++ b/tools/perf/builtin-list.c
> > > @@ -10,7 +10,7 @@
> > >   */
> > >  #include "builtin.h"
> > >
> > > -#include "util/parse-events.h"
> > > +#include "util/print-events.h"
> > >  #include "util/pmu.h"
> > >  #include "util/debug.h"
> > >  #include "util/metricgroup.h"
> > > diff --git a/tools/perf/builtin-lock.c b/tools/perf/builtin-lock.c
> > > index a2f1e53f37a7..486123cb106e 100644
> > > --- a/tools/perf/builtin-lock.c
> > > +++ b/tools/perf/builtin-lock.c
> > > @@ -13,6 +13,7 @@
> > >  #include <subcmd/pager.h>
> > >  #include <subcmd/parse-options.h>
> > >  #include "util/trace-event.h"
> > > +#include "util/tracepoint.h"
> > >
> > >  #include "util/debug.h"
> > >  #include "util/session.h"
> > > diff --git a/tools/perf/builtin-timechart.c
> > b/tools/perf/builtin-timechart.c
> > > index 4e380e7b5230..cdebcf26f408 100644
> > > --- a/tools/perf/builtin-timechart.c
> > > +++ b/tools/perf/builtin-timechart.c
> > > @@ -35,6 +35,7 @@
> > >  #include "util/tool.h"
> > >  #include "util/data.h"
> > >  #include "util/debug.h"
> > > +#include "util/tracepoint.h"
> > >  #include <linux/err.h>
> > >
> > >  #ifdef LACKS_OPEN_MEMSTREAM_PROTOTYPE
> > > diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
> > > index 85b6a46e85b6..b3b9fa1c7731 100644
> > > --- a/tools/perf/builtin-trace.c
> > > +++ b/tools/perf/builtin-trace.c
> > > @@ -53,6 +53,7 @@
> > >  #include "trace-event.h"
> > >  #include "util/parse-events.h"
> > >  #include "util/bpf-loader.h"
> > > +#include "util/tracepoint.h"
> > >  #include "callchain.h"
> > >  #include "print_binary.h"
> > >  #include "string2.h"
> > > diff --git a/tools/perf/util/Build b/tools/perf/util/Build
> > > index 188521f34347..c2c9f3f490e8 100644
> > > --- a/tools/perf/util/Build
> > > +++ b/tools/perf/util/Build
> > > @@ -23,6 +23,8 @@ perf-y += llvm-utils.o
> > >  perf-y += mmap.o
> > >  perf-y += memswap.o
> > >  perf-y += parse-events.o
> > > +perf-y += print-events.o
> > > +perf-y += tracepoint.o
> > >  perf-y += perf_regs.o
> > >  perf-y += path.o
> > >  perf-y += print_binary.o
> > > diff --git a/tools/perf/util/parse-events.c
> > b/tools/perf/util/parse-events.c
> > > index 42c84adeb2fb..5d5ebb700ef2 100644
> > > --- a/tools/perf/util/parse-events.c
> > > +++ b/tools/perf/util/parse-events.c
> > > @@ -5,42 +5,34 @@
> > >  #include <dirent.h>
> > >  #include <errno.h>
> > >  #include <sys/ioctl.h>
> > > -#include <sys/types.h>
> > > -#include <sys/stat.h>
> > > -#include <fcntl.h>
> > >  #include <sys/param.h>
> > >  #include "term.h"
> > > -#include "build-id.h"
> > >  #include "evlist.h"
> > >  #include "evsel.h"
> > > -#include <subcmd/pager.h>
> > >  #include <subcmd/parse-options.h>
> > >  #include "parse-events.h"
> > > -#include <subcmd/exec-cmd.h>
> > >  #include "string2.h"
> > > -#include "strlist.h"
> > > -#include "symbol.h"
> > > -#include "header.h"
> > >  #include "bpf-loader.h"
> > >  #include "debug.h"
> > >  #include <api/fs/tracing_path.h>
> > >  #include <perf/cpumap.h>
> > >  #include "parse-events-bison.h"
> > > -#define YY_EXTRA_TYPE void*
> > >  #include "parse-events-flex.h"
> > >  #include "pmu.h"
> > > -#include "thread_map.h"
> > > -#include "probe-file.h"
> > >  #include "asm/bug.h"
> > >  #include "util/parse-branch-options.h"
> > > -#include "metricgroup.h"
> > >  #include "util/evsel_config.h"
> > >  #include "util/event.h"
> > > -#include "util/pfm.h"
> > >  #include "perf.h"
> > > +#include "tracepoint.h"
> > >
> > >  #define MAX_NAME_LEN 100
> > >
> > > +struct perf_pmu_event_symbol {
> > > +     char    *symbol;
> > > +     enum perf_pmu_event_symbol_type type;
> > > +};
> > > +
> > >  #ifdef PARSER_DEBUG
> > >  extern int parse_events_debug;
> > >  #endif
> > > @@ -155,35 +147,6 @@ struct event_symbol
> > event_symbols_sw[PERF_COUNT_SW_MAX] = {
> > >  #define PERF_EVENT_TYPE(config)              __PERF_EVENT_FIELD(config,
> > TYPE)
> > >  #define PERF_EVENT_ID(config)                __PERF_EVENT_FIELD(config,
> > EVENT)
> > >
> > > -#define for_each_subsystem(sys_dir, sys_dirent)                      \
> > > -     while ((sys_dirent = readdir(sys_dir)) != NULL)         \
> > > -             if (sys_dirent->d_type == DT_DIR &&             \
> > > -                 (strcmp(sys_dirent->d_name, ".")) &&        \
> > > -                 (strcmp(sys_dirent->d_name, "..")))
> > > -
> > > -static int tp_event_has_id(const char *dir_path, struct dirent *evt_dir)
> > > -{
> > > -     char evt_path[MAXPATHLEN];
> > > -     int fd;
> > > -
> > > -     snprintf(evt_path, MAXPATHLEN, "%s/%s/id", dir_path,
> > evt_dir->d_name);
> > > -     fd = open(evt_path, O_RDONLY);
> > > -     if (fd < 0)
> > > -             return -EINVAL;
> > > -     close(fd);
> > > -
> > > -     return 0;
> > > -}
> > > -
> > > -#define for_each_event(dir_path, evt_dir, evt_dirent)                \
> > > -     while ((evt_dirent = readdir(evt_dir)) != NULL)         \
> > > -             if (evt_dirent->d_type == DT_DIR &&             \
> > > -                 (strcmp(evt_dirent->d_name, ".")) &&        \
> > > -                 (strcmp(evt_dirent->d_name, "..")) &&       \
> > > -                 (!tp_event_has_id(dir_path, evt_dirent)))
> > > -
> > > -#define MAX_EVENT_LENGTH 512
> > > -
> > >  void parse_events__handle_error(struct parse_events_error *err, int idx,
> > >                               char *str, char *help)
> > >  {
> > > @@ -217,92 +180,6 @@ void parse_events__handle_error(struct
> > parse_events_error *err, int idx,
> > >       err->num_errors++;
> > >  }
> > >
> > > -struct tracepoint_path *tracepoint_id_to_path(u64 config)
> > > -{
> > > -     struct tracepoint_path *path = NULL;
> > > -     DIR *sys_dir, *evt_dir;
> > > -     struct dirent *sys_dirent, *evt_dirent;
> > > -     char id_buf[24];
> > > -     int fd;
> > > -     u64 id;
> > > -     char evt_path[MAXPATHLEN];
> > > -     char *dir_path;
> > > -
> > > -     sys_dir = tracing_events__opendir();
> > > -     if (!sys_dir)
> > > -             return NULL;
> > > -
> > > -     for_each_subsystem(sys_dir, sys_dirent) {
> > > -             dir_path = get_events_file(sys_dirent->d_name);
> > > -             if (!dir_path)
> > > -                     continue;
> > > -             evt_dir = opendir(dir_path);
> > > -             if (!evt_dir)
> > > -                     goto next;
> > > -
> > > -             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > -
> > > -                     scnprintf(evt_path, MAXPATHLEN, "%s/%s/id",
> > dir_path,
> > > -                               evt_dirent->d_name);
> > > -                     fd = open(evt_path, O_RDONLY);
> > > -                     if (fd < 0)
> > > -                             continue;
> > > -                     if (read(fd, id_buf, sizeof(id_buf)) < 0) {
> > > -                             close(fd);
> > > -                             continue;
> > > -                     }
> > > -                     close(fd);
> > > -                     id = atoll(id_buf);
> > > -                     if (id == config) {
> > > -                             put_events_file(dir_path);
> > > -                             closedir(evt_dir);
> > > -                             closedir(sys_dir);
> > > -                             path = zalloc(sizeof(*path));
> > > -                             if (!path)
> > > -                                     return NULL;
> > > -                             if (asprintf(&path->system, "%.*s",
> > MAX_EVENT_LENGTH, sys_dirent->d_name) < 0) {
> > > -                                     free(path);
> > > -                                     return NULL;
> > > -                             }
> > > -                             if (asprintf(&path->name, "%.*s",
> > MAX_EVENT_LENGTH, evt_dirent->d_name) < 0) {
> > > -                                     zfree(&path->system);
> > > -                                     free(path);
> > > -                                     return NULL;
> > > -                             }
> > > -                             return path;
> > > -                     }
> > > -             }
> > > -             closedir(evt_dir);
> > > -next:
> > > -             put_events_file(dir_path);
> > > -     }
> > > -
> > > -     closedir(sys_dir);
> > > -     return NULL;
> > > -}
> > > -
> > > -struct tracepoint_path *tracepoint_name_to_path(const char *name)
> > > -{
> > > -     struct tracepoint_path *path = zalloc(sizeof(*path));
> > > -     char *str = strchr(name, ':');
> > > -
> > > -     if (path == NULL || str == NULL) {
> > > -             free(path);
> > > -             return NULL;
> > > -     }
> > > -
> > > -     path->system = strndup(name, str - name);
> > > -     path->name = strdup(str+1);
> > > -
> > > -     if (path->system == NULL || path->name == NULL) {
> > > -             zfree(&path->system);
> > > -             zfree(&path->name);
> > > -             zfree(&path);
> > > -     }
> > > -
> > > -     return path;
> > > -}
> > > -
> > >  const char *event_type(int type)
> > >  {
> > >       switch (type) {
> > > @@ -2451,491 +2328,6 @@ int exclude_perf(const struct option *opt,
> > >                                         NULL);
> > >  }
> > >
> > > -static const char * const event_type_descriptors[] = {
> > > -     "Hardware event",
> > > -     "Software event",
> > > -     "Tracepoint event",
> > > -     "Hardware cache event",
> > > -     "Raw hardware event descriptor",
> > > -     "Hardware breakpoint",
> > > -};
> > > -
> > > -static int cmp_string(const void *a, const void *b)
> > > -{
> > > -     const char * const *as = a;
> > > -     const char * const *bs = b;
> > > -
> > > -     return strcmp(*as, *bs);
> > > -}
> > > -
> > > -/*
> > > - * Print the events from <debugfs_mount_point>/tracing/events
> > > - */
> > > -
> > > -void print_tracepoint_events(const char *subsys_glob, const char
> > *event_glob,
> > > -                          bool name_only)
> > > -{
> > > -     DIR *sys_dir, *evt_dir;
> > > -     struct dirent *sys_dirent, *evt_dirent;
> > > -     char evt_path[MAXPATHLEN];
> > > -     char *dir_path;
> > > -     char **evt_list = NULL;
> > > -     unsigned int evt_i = 0, evt_num = 0;
> > > -     bool evt_num_known = false;
> > > -
> > > -restart:
> > > -     sys_dir = tracing_events__opendir();
> > > -     if (!sys_dir)
> > > -             return;
> > > -
> > > -     if (evt_num_known) {
> > > -             evt_list = zalloc(sizeof(char *) * evt_num);
> > > -             if (!evt_list)
> > > -                     goto out_close_sys_dir;
> > > -     }
> > > -
> > > -     for_each_subsystem(sys_dir, sys_dirent) {
> > > -             if (subsys_glob != NULL &&
> > > -                 !strglobmatch(sys_dirent->d_name, subsys_glob))
> > > -                     continue;
> > > -
> > > -             dir_path = get_events_file(sys_dirent->d_name);
> > > -             if (!dir_path)
> > > -                     continue;
> > > -             evt_dir = opendir(dir_path);
> > > -             if (!evt_dir)
> > > -                     goto next;
> > > -
> > > -             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > -                     if (event_glob != NULL &&
> > > -                         !strglobmatch(evt_dirent->d_name, event_glob))
> > > -                             continue;
> > > -
> > > -                     if (!evt_num_known) {
> > > -                             evt_num++;
> > > -                             continue;
> > > -                     }
> > > -
> > > -                     snprintf(evt_path, MAXPATHLEN, "%s:%s",
> > > -                              sys_dirent->d_name, evt_dirent->d_name);
> > > -
> > > -                     evt_list[evt_i] = strdup(evt_path);
> > > -                     if (evt_list[evt_i] == NULL) {
> > > -                             put_events_file(dir_path);
> > > -                             goto out_close_evt_dir;
> > > -                     }
> > > -                     evt_i++;
> > > -             }
> > > -             closedir(evt_dir);
> > > -next:
> > > -             put_events_file(dir_path);
> > > -     }
> > > -     closedir(sys_dir);
> > > -
> > > -     if (!evt_num_known) {
> > > -             evt_num_known = true;
> > > -             goto restart;
> > > -     }
> > > -     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > -     evt_i = 0;
> > > -     while (evt_i < evt_num) {
> > > -             if (name_only) {
> > > -                     printf("%s ", evt_list[evt_i++]);
> > > -                     continue;
> > > -             }
> > > -             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > > -
> >  event_type_descriptors[PERF_TYPE_TRACEPOINT]);
> > > -     }
> > > -     if (evt_num && pager_in_use())
> > > -             printf("\n");
> > > -
> > > -out_free:
> > > -     evt_num = evt_i;
> > > -     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > -             zfree(&evt_list[evt_i]);
> > > -     zfree(&evt_list);
> > > -     return;
> > > -
> > > -out_close_evt_dir:
> > > -     closedir(evt_dir);
> > > -out_close_sys_dir:
> > > -     closedir(sys_dir);
> > > -
> > > -     printf("FATAL: not enough memory to print %s\n",
> > > -                     event_type_descriptors[PERF_TYPE_TRACEPOINT]);
> > > -     if (evt_list)
> > > -             goto out_free;
> > > -}
> > > -
> > > -/*
> > > - * Check whether event is in <debugfs_mount_point>/tracing/events
> > > - */
> > > -
> > > -int is_valid_tracepoint(const char *event_string)
> > > -{
> > > -     DIR *sys_dir, *evt_dir;
> > > -     struct dirent *sys_dirent, *evt_dirent;
> > > -     char evt_path[MAXPATHLEN];
> > > -     char *dir_path;
> > > -
> > > -     sys_dir = tracing_events__opendir();
> > > -     if (!sys_dir)
> > > -             return 0;
> > > -
> > > -     for_each_subsystem(sys_dir, sys_dirent) {
> > > -             dir_path = get_events_file(sys_dirent->d_name);
> > > -             if (!dir_path)
> > > -                     continue;
> > > -             evt_dir = opendir(dir_path);
> > > -             if (!evt_dir)
> > > -                     goto next;
> > > -
> > > -             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > -                     snprintf(evt_path, MAXPATHLEN, "%s:%s",
> > > -                              sys_dirent->d_name, evt_dirent->d_name);
> > > -                     if (!strcmp(evt_path, event_string)) {
> > > -                             closedir(evt_dir);
> > > -                             closedir(sys_dir);
> > > -                             return 1;
> > > -                     }
> > > -             }
> > > -             closedir(evt_dir);
> > > -next:
> > > -             put_events_file(dir_path);
> > > -     }
> > > -     closedir(sys_dir);
> > > -     return 0;
> > > -}
> > > -
> > > -static bool is_event_supported(u8 type, unsigned config)
> > > -{
> > > -     bool ret = true;
> > > -     int open_return;
> > > -     struct evsel *evsel;
> > > -     struct perf_event_attr attr = {
> > > -             .type = type,
> > > -             .config = config,
> > > -             .disabled = 1,
> > > -     };
> > > -     struct perf_thread_map *tmap = thread_map__new_by_tid(0);
> > > -
> > > -     if (tmap == NULL)
> > > -             return false;
> > > -
> > > -     evsel = evsel__new(&attr);
> > > -     if (evsel) {
> > > -             open_return = evsel__open(evsel, NULL, tmap);
> > > -             ret = open_return >= 0;
> > > -
> > > -             if (open_return == -EACCES) {
> > > -                     /*
> > > -                      * This happens if the paranoid value
> > > -                      * /proc/sys/kernel/perf_event_paranoid is set to 2
> > > -                      * Re-run with exclude_kernel set; we don't do that
> > > -                      * by default as some ARM machines do not support
> > it.
> > > -                      *
> > > -                      */
> > > -                     evsel->core.attr.exclude_kernel = 1;
> > > -                     ret = evsel__open(evsel, NULL, tmap) >= 0;
> > > -             }
> > > -             evsel__delete(evsel);
> > > -     }
> > > -
> > > -     perf_thread_map__put(tmap);
> > > -     return ret;
> > > -}
> > > -
> > > -void print_sdt_events(const char *subsys_glob, const char *event_glob,
> > > -                   bool name_only)
> > > -{
> > > -     struct probe_cache *pcache;
> > > -     struct probe_cache_entry *ent;
> > > -     struct strlist *bidlist, *sdtlist;
> > > -     struct strlist_config cfg = {.dont_dupstr = true};
> > > -     struct str_node *nd, *nd2;
> > > -     char *buf, *path, *ptr = NULL;
> > > -     bool show_detail = false;
> > > -     int ret;
> > > -
> > > -     sdtlist = strlist__new(NULL, &cfg);
> > > -     if (!sdtlist) {
> > > -             pr_debug("Failed to allocate new strlist for SDT\n");
> > > -             return;
> > > -     }
> > > -     bidlist = build_id_cache__list_all(true);
> > > -     if (!bidlist) {
> > > -             pr_debug("Failed to get buildids: %d\n", errno);
> > > -             return;
> > > -     }
> > > -     strlist__for_each_entry(nd, bidlist) {
> > > -             pcache = probe_cache__new(nd->s, NULL);
> > > -             if (!pcache)
> > > -                     continue;
> > > -             list_for_each_entry(ent, &pcache->entries, node) {
> > > -                     if (!ent->sdt)
> > > -                             continue;
> > > -                     if (subsys_glob &&
> > > -                         !strglobmatch(ent->pev.group, subsys_glob))
> > > -                             continue;
> > > -                     if (event_glob &&
> > > -                         !strglobmatch(ent->pev.event, event_glob))
> > > -                             continue;
> > > -                     ret = asprintf(&buf, "%s:%s@%s", ent->pev.group,
> > > -                                     ent->pev.event, nd->s);
> > > -                     if (ret > 0)
> > > -                             strlist__add(sdtlist, buf);
> > > -             }
> > > -             probe_cache__delete(pcache);
> > > -     }
> > > -     strlist__delete(bidlist);
> > > -
> > > -     strlist__for_each_entry(nd, sdtlist) {
> > > -             buf = strchr(nd->s, '@');
> > > -             if (buf)
> > > -                     *(buf++) = '\0';
> > > -             if (name_only) {
> > > -                     printf("%s ", nd->s);
> > > -                     continue;
> > > -             }
> > > -             nd2 = strlist__next(nd);
> > > -             if (nd2) {
> > > -                     ptr = strchr(nd2->s, '@');
> > > -                     if (ptr)
> > > -                             *ptr = '\0';
> > > -                     if (strcmp(nd->s, nd2->s) == 0)
> > > -                             show_detail = true;
> > > -             }
> > > -             if (show_detail) {
> > > -                     path = build_id_cache__origname(buf);
> > > -                     ret = asprintf(&buf, "%s@%s(%.12s)", nd->s, path,
> > buf);
> > > -                     if (ret > 0) {
> > > -                             printf("  %-50s [%s]\n", buf, "SDT event");
> > > -                             free(buf);
> > > -                     }
> > > -                     free(path);
> > > -             } else
> > > -                     printf("  %-50s [%s]\n", nd->s, "SDT event");
> > > -             if (nd2) {
> > > -                     if (strcmp(nd->s, nd2->s) != 0)
> > > -                             show_detail = false;
> > > -                     if (ptr)
> > > -                             *ptr = '@';
> > > -             }
> > > -     }
> > > -     strlist__delete(sdtlist);
> > > -}
> > > -
> > > -int print_hwcache_events(const char *event_glob, bool name_only)
> > > -{
> > > -     unsigned int type, op, i, evt_i = 0, evt_num = 0;
> > > -     char name[64];
> > > -     char **evt_list = NULL;
> > > -     bool evt_num_known = false;
> > > -
> > > -restart:
> > > -     if (evt_num_known) {
> > > -             evt_list = zalloc(sizeof(char *) * evt_num);
> > > -             if (!evt_list)
> > > -                     goto out_enomem;
> > > -     }
> > > -
> > > -     for (type = 0; type < PERF_COUNT_HW_CACHE_MAX; type++) {
> > > -             for (op = 0; op < PERF_COUNT_HW_CACHE_OP_MAX; op++) {
> > > -                     /* skip invalid cache type */
> > > -                     if (!evsel__is_cache_op_valid(type, op))
> > > -                             continue;
> > > -
> > > -                     for (i = 0; i < PERF_COUNT_HW_CACHE_RESULT_MAX;
> > i++) {
> > > -                             __evsel__hw_cache_type_op_res_name(type,
> > op, i, name, sizeof(name));
> > > -                             if (event_glob != NULL &&
> > !strglobmatch(name, event_glob))
> > > -                                     continue;
> > > -
> > > -                             if (!is_event_supported(PERF_TYPE_HW_CACHE,
> > > -                                                     type | (op << 8) |
> > (i << 16)))
> > > -                                     continue;
> > > -
> > > -                             if (!evt_num_known) {
> > > -                                     evt_num++;
> > > -                                     continue;
> > > -                             }
> > > -
> > > -                             evt_list[evt_i] = strdup(name);
> > > -                             if (evt_list[evt_i] == NULL)
> > > -                                     goto out_enomem;
> > > -                             evt_i++;
> > > -                     }
> > > -             }
> > > -     }
> > > -
> > > -     if (!evt_num_known) {
> > > -             evt_num_known = true;
> > > -             goto restart;
> > > -     }
> > > -     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > -     evt_i = 0;
> > > -     while (evt_i < evt_num) {
> > > -             if (name_only) {
> > > -                     printf("%s ", evt_list[evt_i++]);
> > > -                     continue;
> > > -             }
> > > -             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > > -
> >  event_type_descriptors[PERF_TYPE_HW_CACHE]);
> > > -     }
> > > -     if (evt_num && pager_in_use())
> > > -             printf("\n");
> > > -
> > > -out_free:
> > > -     evt_num = evt_i;
> > > -     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > -             zfree(&evt_list[evt_i]);
> > > -     zfree(&evt_list);
> > > -     return evt_num;
> > > -
> > > -out_enomem:
> > > -     printf("FATAL: not enough memory to print %s\n",
> > event_type_descriptors[PERF_TYPE_HW_CACHE]);
> > > -     if (evt_list)
> > > -             goto out_free;
> > > -     return evt_num;
> > > -}
> > > -
> > > -static void print_tool_event(const char *name, const char *event_glob,
> > > -                          bool name_only)
> > > -{
> > > -     if (event_glob && !strglobmatch(name, event_glob))
> > > -             return;
> > > -     if (name_only)
> > > -             printf("%s ", name);
> > > -     else
> > > -             printf("  %-50s [%s]\n", name, "Tool event");
> > > -
> > > -}
> > > -
> > > -void print_tool_events(const char *event_glob, bool name_only)
> > > -{
> > > -     print_tool_event("duration_time", event_glob, name_only);
> > > -     if (pager_in_use())
> > > -             printf("\n");
> > > -}
> > > -
> > > -void print_symbol_events(const char *event_glob, unsigned type,
> > > -                             struct event_symbol *syms, unsigned max,
> > > -                             bool name_only)
> > > -{
> > > -     unsigned int i, evt_i = 0, evt_num = 0;
> > > -     char name[MAX_NAME_LEN];
> > > -     char **evt_list = NULL;
> > > -     bool evt_num_known = false;
> > > -
> > > -restart:
> > > -     if (evt_num_known) {
> > > -             evt_list = zalloc(sizeof(char *) * evt_num);
> > > -             if (!evt_list)
> > > -                     goto out_enomem;
> > > -             syms -= max;
> > > -     }
> > > -
> > > -     for (i = 0; i < max; i++, syms++) {
> > > -
> > > -             if (event_glob != NULL && syms->symbol != NULL &&
> > > -                 !(strglobmatch(syms->symbol, event_glob) ||
> > > -                   (syms->alias && strglobmatch(syms->alias,
> > event_glob))))
> > > -                     continue;
> > > -
> > > -             if (!is_event_supported(type, i))
> > > -                     continue;
> > > -
> > > -             if (!evt_num_known) {
> > > -                     evt_num++;
> > > -                     continue;
> > > -             }
> > > -
> > > -             if (!name_only && strlen(syms->alias))
> > > -                     snprintf(name, MAX_NAME_LEN, "%s OR %s",
> > syms->symbol, syms->alias);
> > > -             else
> > > -                     strlcpy(name, syms->symbol, MAX_NAME_LEN);
> > > -
> > > -             evt_list[evt_i] = strdup(name);
> > > -             if (evt_list[evt_i] == NULL)
> > > -                     goto out_enomem;
> > > -             evt_i++;
> > > -     }
> > > -
> > > -     if (!evt_num_known) {
> > > -             evt_num_known = true;
> > > -             goto restart;
> > > -     }
> > > -     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > -     evt_i = 0;
> > > -     while (evt_i < evt_num) {
> > > -             if (name_only) {
> > > -                     printf("%s ", evt_list[evt_i++]);
> > > -                     continue;
> > > -             }
> > > -             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > event_type_descriptors[type]);
> > > -     }
> > > -     if (evt_num && pager_in_use())
> > > -             printf("\n");
> > > -
> > > -out_free:
> > > -     evt_num = evt_i;
> > > -     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > -             zfree(&evt_list[evt_i]);
> > > -     zfree(&evt_list);
> > > -     return;
> > > -
> > > -out_enomem:
> > > -     printf("FATAL: not enough memory to print %s\n",
> > event_type_descriptors[type]);
> > > -     if (evt_list)
> > > -             goto out_free;
> > > -}
> > > -
> > > -/*
> > > - * Print the help text for the event symbols:
> > > - */
> > > -void print_events(const char *event_glob, bool name_only, bool
> > quiet_flag,
> > > -                     bool long_desc, bool details_flag, bool deprecated)
> > > -{
> > > -     print_symbol_events(event_glob, PERF_TYPE_HARDWARE,
> > > -                         event_symbols_hw, PERF_COUNT_HW_MAX,
> > name_only);
> > > -
> > > -     print_symbol_events(event_glob, PERF_TYPE_SOFTWARE,
> > > -                         event_symbols_sw, PERF_COUNT_SW_MAX,
> > name_only);
> > > -     print_tool_events(event_glob, name_only);
> > > -
> > > -     print_hwcache_events(event_glob, name_only);
> > > -
> > > -     print_pmu_events(event_glob, name_only, quiet_flag, long_desc,
> > > -                     details_flag, deprecated);
> > > -
> > > -     if (event_glob != NULL)
> > > -             return;
> > > -
> > > -     if (!name_only) {
> > > -             printf("  %-50s [%s]\n",
> > > -                    "rNNN",
> > > -                    event_type_descriptors[PERF_TYPE_RAW]);
> > > -             printf("  %-50s [%s]\n",
> > > -                    "cpu/t1=v1[,t2=v2,t3 ...]/modifier",
> > > -                    event_type_descriptors[PERF_TYPE_RAW]);
> > > -             if (pager_in_use())
> > > -                     printf("   (see 'man perf-list' on how to encode
> > it)\n\n");
> > > -
> > > -             printf("  %-50s [%s]\n",
> > > -                    "mem:<addr>[/len][:access]",
> > > -                     event_type_descriptors[PERF_TYPE_BREAKPOINT]);
> > > -             if (pager_in_use())
> > > -                     printf("\n");
> > > -     }
> > > -
> > > -     print_tracepoint_events(NULL, NULL, name_only);
> > > -
> > > -     print_sdt_events(NULL, NULL, name_only);
> > > -
> > > -     metricgroup__print(true, true, NULL, name_only, details_flag);
> > > -
> > > -     print_libpfm_events(name_only, long_desc);
> > > -}
> > > -
> > >  int parse_events__is_hardcoded_term(struct parse_events_term *term)
> > >  {
> > >       return term->type_term != PARSE_EVENTS__TERM_TYPE_USER;
> > > diff --git a/tools/perf/util/parse-events.h
> > b/tools/perf/util/parse-events.h
> > > index e80c9b74f2f2..7cea425e80f9 100644
> > > --- a/tools/perf/util/parse-events.h
> > > +++ b/tools/perf/util/parse-events.h
> > > @@ -11,7 +11,6 @@
> > >  #include <linux/perf_event.h>
> > >  #include <string.h>
> > >
> > > -struct list_head;
> > >  struct evsel;
> > >  struct evlist;
> > >  struct parse_events_error;
> > > @@ -19,14 +18,6 @@ struct parse_events_error;
> > >  struct option;
> > >  struct perf_pmu;
> > >
> > > -struct tracepoint_path {
> > > -     char *system;
> > > -     char *name;
> > > -     struct tracepoint_path *next;
> > > -};
> > > -
> > > -struct tracepoint_path *tracepoint_id_to_path(u64 config);
> > > -struct tracepoint_path *tracepoint_name_to_path(const char *name);
> > >  bool have_tracepoints(struct list_head *evlist);
> > >
> > >  const char *event_type(int type);
> > > @@ -46,8 +37,6 @@ int parse_events_terms(struct list_head *terms, const
> > char *str);
> > >  int parse_filter(const struct option *opt, const char *str, int unset);
> > >  int exclude_perf(const struct option *opt, const char *arg, int unset);
> > >
> > > -#define EVENTS_HELP_MAX (128*1024)
> > > -
> > >  enum perf_pmu_event_symbol_type {
> > >       PMU_EVENT_SYMBOL_ERR,           /* not a PMU EVENT */
> > >       PMU_EVENT_SYMBOL,               /* normal style PMU event */
> > > @@ -55,11 +44,6 @@ enum perf_pmu_event_symbol_type {
> > >       PMU_EVENT_SYMBOL_SUFFIX,        /* suffix of pre-suf style event */
> > >  };
> > >
> > > -struct perf_pmu_event_symbol {
> > > -     char    *symbol;
> > > -     enum perf_pmu_event_symbol_type type;
> > > -};
> > > -
> > >  enum {
> > >       PARSE_EVENTS__TERM_TYPE_NUM,
> > >       PARSE_EVENTS__TERM_TYPE_STR,
> > > @@ -216,8 +200,6 @@ void parse_events_update_lists(struct list_head
> > *list_event,
> > >  void parse_events_evlist_error(struct parse_events_state *parse_state,
> > >                              int idx, const char *str);
> > >
> > > -void print_events(const char *event_glob, bool name_only, bool quiet,
> > > -               bool long_desc, bool details_flag, bool deprecated);
> > >
> > >  struct event_symbol {
> > >       const char      *symbol;
> > > @@ -225,18 +207,7 @@ struct event_symbol {
> > >  };
> > >  extern struct event_symbol event_symbols_hw[];
> > >  extern struct event_symbol event_symbols_sw[];
> > > -void print_symbol_events(const char *event_glob, unsigned type,
> > > -                             struct event_symbol *syms, unsigned max,
> > > -                             bool name_only);
> > > -void print_tool_events(const char *event_glob, bool name_only);
> > > -void print_tracepoint_events(const char *subsys_glob, const char
> > *event_glob,
> > > -                          bool name_only);
> > > -int print_hwcache_events(const char *event_glob, bool name_only);
> > > -void print_sdt_events(const char *subsys_glob, const char *event_glob,
> > > -                   bool name_only);
> > > -int is_valid_tracepoint(const char *event_string);
> > >
> > > -int valid_event_mount(const char *eventfs);
> > >  char *parse_events_formats_error_string(char *additional_terms);
> > >
> > >  void parse_events_print_error(struct parse_events_error *err,
> > > diff --git a/tools/perf/util/print-events.c
> > b/tools/perf/util/print-events.c
> > > new file mode 100644
> > > index 000000000000..584888353d13
> > > --- /dev/null
> > > +++ b/tools/perf/util/print-events.c
> > > @@ -0,0 +1,472 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> > > +#include <dirent.h>
> > > +#include <errno.h>
> > > +#include <stdio.h>
> > > +#include <stdlib.h>
> > > +#include <string.h>
> > > +#include <sys/param.h>
> > > +
> > > +#include <api/fs/tracing_path.h>
> > > +#include <linux/perf_event.h>
> > > +#include <linux/zalloc.h>
> > > +#include <subcmd/pager.h>
> > > +
> > > +#include "build-id.h"
> > > +#include "debug.h"
> > > +#include "evsel.h"
> > > +#include "metricgroup.h"
> > > +#include "parse-events.h"
> > > +#include "pmu.h"
> > > +#include "print-events.h"
> > > +#include "probe-file.h"
> > > +#include "string2.h"
> > > +#include "strlist.h"
> > > +#include "thread_map.h"
> > > +#include "tracepoint.h"
> > > +#include "pfm.h"
> > > +
> > > +#define MAX_NAME_LEN 100
> > > +
> > > +static int cmp_string(const void *a, const void *b)
> > > +{
> > > +     const char * const *as = a;
> > > +     const char * const *bs = b;
> > > +
> > > +     return strcmp(*as, *bs);
> > > +}
> > > +
> > > +static const char * const event_type_descriptors[] = {
> > > +     "Hardware event",
> > > +     "Software event",
> > > +     "Tracepoint event",
> > > +     "Hardware cache event",
> > > +     "Raw hardware event descriptor",
> > > +     "Hardware breakpoint",
> > > +};
> > > +
> > > +/*
> > > + * Print the events from <debugfs_mount_point>/tracing/events
> > > + */
> > > +void print_tracepoint_events(const char *subsys_glob,
> > > +                          const char *event_glob, bool name_only)
> > > +{
> > > +     DIR *sys_dir, *evt_dir;
> > > +     struct dirent *sys_dirent, *evt_dirent;
> > > +     char evt_path[MAXPATHLEN];
> > > +     char *dir_path;
> > > +     char **evt_list = NULL;
> > > +     unsigned int evt_i = 0, evt_num = 0;
> > > +     bool evt_num_known = false;
> > > +
> > > +restart:
> > > +     sys_dir = tracing_events__opendir();
> > > +     if (!sys_dir)
> > > +             return;
> > > +
> > > +     if (evt_num_known) {
> > > +             evt_list = zalloc(sizeof(char *) * evt_num);
> > > +             if (!evt_list)
> > > +                     goto out_close_sys_dir;
> > > +     }
> > > +
> > > +     for_each_subsystem(sys_dir, sys_dirent) {
> > > +             if (subsys_glob != NULL &&
> > > +                 !strglobmatch(sys_dirent->d_name, subsys_glob))
> > > +                     continue;
> > > +
> > > +             dir_path = get_events_file(sys_dirent->d_name);
> > > +             if (!dir_path)
> > > +                     continue;
> > > +             evt_dir = opendir(dir_path);
> > > +             if (!evt_dir)
> > > +                     goto next;
> > > +
> > > +             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > +                     if (event_glob != NULL &&
> > > +                         !strglobmatch(evt_dirent->d_name, event_glob))
> > > +                             continue;
> > > +
> > > +                     if (!evt_num_known) {
> > > +                             evt_num++;
> > > +                             continue;
> > > +                     }
> > > +
> > > +                     snprintf(evt_path, MAXPATHLEN, "%s:%s",
> > > +                              sys_dirent->d_name, evt_dirent->d_name);
> > > +
> > > +                     evt_list[evt_i] = strdup(evt_path);
> > > +                     if (evt_list[evt_i] == NULL) {
> > > +                             put_events_file(dir_path);
> > > +                             goto out_close_evt_dir;
> > > +                     }
> > > +                     evt_i++;
> > > +             }
> > > +             closedir(evt_dir);
> > > +next:
> > > +             put_events_file(dir_path);
> > > +     }
> > > +     closedir(sys_dir);
> > > +
> > > +     if (!evt_num_known) {
> > > +             evt_num_known = true;
> > > +             goto restart;
> > > +     }
> > > +     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > +     evt_i = 0;
> > > +     while (evt_i < evt_num) {
> > > +             if (name_only) {
> > > +                     printf("%s ", evt_list[evt_i++]);
> > > +                     continue;
> > > +             }
> > > +             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > > +
> >  event_type_descriptors[PERF_TYPE_TRACEPOINT]);
> > > +     }
> > > +     if (evt_num && pager_in_use())
> > > +             printf("\n");
> > > +
> > > +out_free:
> > > +     evt_num = evt_i;
> > > +     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > +             zfree(&evt_list[evt_i]);
> > > +     zfree(&evt_list);
> > > +     return;
> > > +
> > > +out_close_evt_dir:
> > > +     closedir(evt_dir);
> > > +out_close_sys_dir:
> > > +     closedir(sys_dir);
> > > +
> > > +     printf("FATAL: not enough memory to print %s\n",
> > > +                     event_type_descriptors[PERF_TYPE_TRACEPOINT]);
> > > +     if (evt_list)
> > > +             goto out_free;
> > > +}
> > > +
> > > +void print_sdt_events(const char *subsys_glob, const char *event_glob,
> > > +                   bool name_only)
> > > +{
> > > +     struct probe_cache *pcache;
> > > +     struct probe_cache_entry *ent;
> > > +     struct strlist *bidlist, *sdtlist;
> > > +     struct strlist_config cfg = {.dont_dupstr = true};
> > > +     struct str_node *nd, *nd2;
> > > +     char *buf, *path, *ptr = NULL;
> > > +     bool show_detail = false;
> > > +     int ret;
> > > +
> > > +     sdtlist = strlist__new(NULL, &cfg);
> > > +     if (!sdtlist) {
> > > +             pr_debug("Failed to allocate new strlist for SDT\n");
> > > +             return;
> > > +     }
> > > +     bidlist = build_id_cache__list_all(true);
> > > +     if (!bidlist) {
> > > +             pr_debug("Failed to get buildids: %d\n", errno);
> > > +             return;
> > > +     }
> > > +     strlist__for_each_entry(nd, bidlist) {
> > > +             pcache = probe_cache__new(nd->s, NULL);
> > > +             if (!pcache)
> > > +                     continue;
> > > +             list_for_each_entry(ent, &pcache->entries, node) {
> > > +                     if (!ent->sdt)
> > > +                             continue;
> > > +                     if (subsys_glob &&
> > > +                         !strglobmatch(ent->pev.group, subsys_glob))
> > > +                             continue;
> > > +                     if (event_glob &&
> > > +                         !strglobmatch(ent->pev.event, event_glob))
> > > +                             continue;
> > > +                     ret = asprintf(&buf, "%s:%s@%s", ent->pev.group,
> > > +                                     ent->pev.event, nd->s);
> > > +                     if (ret > 0)
> > > +                             strlist__add(sdtlist, buf);
> > > +             }
> > > +             probe_cache__delete(pcache);
> > > +     }
> > > +     strlist__delete(bidlist);
> > > +
> > > +     strlist__for_each_entry(nd, sdtlist) {
> > > +             buf = strchr(nd->s, '@');
> > > +             if (buf)
> > > +                     *(buf++) = '\0';
> > > +             if (name_only) {
> > > +                     printf("%s ", nd->s);
> > > +                     continue;
> > > +             }
> > > +             nd2 = strlist__next(nd);
> > > +             if (nd2) {
> > > +                     ptr = strchr(nd2->s, '@');
> > > +                     if (ptr)
> > > +                             *ptr = '\0';
> > > +                     if (strcmp(nd->s, nd2->s) == 0)
> > > +                             show_detail = true;
> > > +             }
> > > +             if (show_detail) {
> > > +                     path = build_id_cache__origname(buf);
> > > +                     ret = asprintf(&buf, "%s@%s(%.12s)", nd->s, path,
> > buf);
> > > +                     if (ret > 0) {
> > > +                             printf("  %-50s [%s]\n", buf, "SDT event");
> > > +                             free(buf);
> > > +                     }
> > > +                     free(path);
> > > +             } else
> > > +                     printf("  %-50s [%s]\n", nd->s, "SDT event");
> > > +             if (nd2) {
> > > +                     if (strcmp(nd->s, nd2->s) != 0)
> > > +                             show_detail = false;
> > > +                     if (ptr)
> > > +                             *ptr = '@';
> > > +             }
> > > +     }
> > > +     strlist__delete(sdtlist);
> > > +}
> > > +
> > > +static bool is_event_supported(u8 type, unsigned int config)
> > > +{
> > > +     bool ret = true;
> > > +     int open_return;
> > > +     struct evsel *evsel;
> > > +     struct perf_event_attr attr = {
> > > +             .type = type,
> > > +             .config = config,
> > > +             .disabled = 1,
> > > +     };
> > > +     struct perf_thread_map *tmap = thread_map__new_by_tid(0);
> > > +
> > > +     if (tmap == NULL)
> > > +             return false;
> > > +
> > > +     evsel = evsel__new(&attr);
> > > +     if (evsel) {
> > > +             open_return = evsel__open(evsel, NULL, tmap);
> > > +             ret = open_return >= 0;
> > > +
> > > +             if (open_return == -EACCES) {
> > > +                     /*
> > > +                      * This happens if the paranoid value
> > > +                      * /proc/sys/kernel/perf_event_paranoid is set to 2
> > > +                      * Re-run with exclude_kernel set; we don't do that
> > > +                      * by default as some ARM machines do not support
> > it.
> > > +                      *
> > > +                      */
> > > +                     evsel->core.attr.exclude_kernel = 1;
> > > +                     ret = evsel__open(evsel, NULL, tmap) >= 0;
> > > +             }
> > > +             evsel__delete(evsel);
> > > +     }
> > > +
> > > +     perf_thread_map__put(tmap);
> > > +     return ret;
> > > +}
> > > +
> > > +int print_hwcache_events(const char *event_glob, bool name_only)
> > > +{
> > > +     unsigned int type, op, i, evt_i = 0, evt_num = 0;
> > > +     char name[64];
> > > +     char **evt_list = NULL;
> > > +     bool evt_num_known = false;
> > > +
> > > +restart:
> > > +     if (evt_num_known) {
> > > +             evt_list = zalloc(sizeof(char *) * evt_num);
> > > +             if (!evt_list)
> > > +                     goto out_enomem;
> > > +     }
> > > +
> > > +     for (type = 0; type < PERF_COUNT_HW_CACHE_MAX; type++) {
> > > +             for (op = 0; op < PERF_COUNT_HW_CACHE_OP_MAX; op++) {
> > > +                     /* skip invalid cache type */
> > > +                     if (!evsel__is_cache_op_valid(type, op))
> > > +                             continue;
> > > +
> > > +                     for (i = 0; i < PERF_COUNT_HW_CACHE_RESULT_MAX;
> > i++) {
> > > +                             __evsel__hw_cache_type_op_res_name(type,
> > op, i, name, sizeof(name));
> > > +                             if (event_glob != NULL &&
> > !strglobmatch(name, event_glob))
> > > +                                     continue;
> > > +
> > > +                             if (!is_event_supported(PERF_TYPE_HW_CACHE,
> > > +                                                     type | (op << 8) |
> > (i << 16)))
> > > +                                     continue;
> > > +
> > > +                             if (!evt_num_known) {
> > > +                                     evt_num++;
> > > +                                     continue;
> > > +                             }
> > > +
> > > +                             evt_list[evt_i] = strdup(name);
> > > +                             if (evt_list[evt_i] == NULL)
> > > +                                     goto out_enomem;
> > > +                             evt_i++;
> > > +                     }
> > > +             }
> > > +     }
> > > +
> > > +     if (!evt_num_known) {
> > > +             evt_num_known = true;
> > > +             goto restart;
> > > +     }
> > > +     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > +     evt_i = 0;
> > > +     while (evt_i < evt_num) {
> > > +             if (name_only) {
> > > +                     printf("%s ", evt_list[evt_i++]);
> > > +                     continue;
> > > +             }
> > > +             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > > +
> >  event_type_descriptors[PERF_TYPE_HW_CACHE]);
> > > +     }
> > > +     if (evt_num && pager_in_use())
> > > +             printf("\n");
> > > +
> > > +out_free:
> > > +     evt_num = evt_i;
> > > +     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > +             zfree(&evt_list[evt_i]);
> > > +     zfree(&evt_list);
> > > +     return evt_num;
> > > +
> > > +out_enomem:
> > > +     printf("FATAL: not enough memory to print %s\n",
> > event_type_descriptors[PERF_TYPE_HW_CACHE]);
> > > +     if (evt_list)
> > > +             goto out_free;
> > > +     return evt_num;
> > > +}
> > > +
> > > +static void print_tool_event(const char *name, const char *event_glob,
> > > +                          bool name_only)
> > > +{
> > > +     if (event_glob && !strglobmatch(name, event_glob))
> > > +             return;
> > > +     if (name_only)
> > > +             printf("%s ", name);
> > > +     else
> > > +             printf("  %-50s [%s]\n", name, "Tool event");
> > > +
> > > +}
> > > +
> > > +void print_tool_events(const char *event_glob, bool name_only)
> > > +{
> > > +     print_tool_event("duration_time", event_glob, name_only);
> > > +     if (pager_in_use())
> > > +             printf("\n");
> > > +}
> > > +
> > > +void print_symbol_events(const char *event_glob, unsigned int type,
> > > +                      struct event_symbol *syms, unsigned int max,
> > > +                      bool name_only)
> > > +{
> > > +     unsigned int i, evt_i = 0, evt_num = 0;
> > > +     char name[MAX_NAME_LEN];
> > > +     char **evt_list = NULL;
> > > +     bool evt_num_known = false;
> > > +
> > > +restart:
> > > +     if (evt_num_known) {
> > > +             evt_list = zalloc(sizeof(char *) * evt_num);
> > > +             if (!evt_list)
> > > +                     goto out_enomem;
> > > +             syms -= max;
> > > +     }
> > > +
> > > +     for (i = 0; i < max; i++, syms++) {
> > > +
> > > +             if (event_glob != NULL && syms->symbol != NULL &&
> > > +                 !(strglobmatch(syms->symbol, event_glob) ||
> > > +                   (syms->alias && strglobmatch(syms->alias,
> > event_glob))))
> > > +                     continue;
> > > +
> > > +             if (!is_event_supported(type, i))
> > > +                     continue;
> > > +
> > > +             if (!evt_num_known) {
> > > +                     evt_num++;
> > > +                     continue;
> > > +             }
> > > +
> > > +             if (!name_only && strlen(syms->alias))
> > > +                     snprintf(name, MAX_NAME_LEN, "%s OR %s",
> > syms->symbol, syms->alias);
> > > +             else
> > > +                     strlcpy(name, syms->symbol, MAX_NAME_LEN);
> > > +
> > > +             evt_list[evt_i] = strdup(name);
> > > +             if (evt_list[evt_i] == NULL)
> > > +                     goto out_enomem;
> > > +             evt_i++;
> > > +     }
> > > +
> > > +     if (!evt_num_known) {
> > > +             evt_num_known = true;
> > > +             goto restart;
> > > +     }
> > > +     qsort(evt_list, evt_num, sizeof(char *), cmp_string);
> > > +     evt_i = 0;
> > > +     while (evt_i < evt_num) {
> > > +             if (name_only) {
> > > +                     printf("%s ", evt_list[evt_i++]);
> > > +                     continue;
> > > +             }
> > > +             printf("  %-50s [%s]\n", evt_list[evt_i++],
> > event_type_descriptors[type]);
> > > +     }
> > > +     if (evt_num && pager_in_use())
> > > +             printf("\n");
> > > +
> > > +out_free:
> > > +     evt_num = evt_i;
> > > +     for (evt_i = 0; evt_i < evt_num; evt_i++)
> > > +             zfree(&evt_list[evt_i]);
> > > +     zfree(&evt_list);
> > > +     return;
> > > +
> > > +out_enomem:
> > > +     printf("FATAL: not enough memory to print %s\n",
> > event_type_descriptors[type]);
> > > +     if (evt_list)
> > > +             goto out_free;
> > > +}
> > > +
> > > +/*
> > > + * Print the help text for the event symbols:
> > > + */
> > > +void print_events(const char *event_glob, bool name_only, bool
> > quiet_flag,
> > > +                     bool long_desc, bool details_flag, bool deprecated)
> > > +{
> > > +     print_symbol_events(event_glob, PERF_TYPE_HARDWARE,
> > > +                         event_symbols_hw, PERF_COUNT_HW_MAX,
> > name_only);
> > > +
> > > +     print_symbol_events(event_glob, PERF_TYPE_SOFTWARE,
> > > +                         event_symbols_sw, PERF_COUNT_SW_MAX,
> > name_only);
> > > +     print_tool_events(event_glob, name_only);
> > > +
> > > +     print_hwcache_events(event_glob, name_only);
> > > +
> > > +     print_pmu_events(event_glob, name_only, quiet_flag, long_desc,
> > > +                     details_flag, deprecated);
> > > +
> > > +     if (event_glob != NULL)
> > > +             return;
> > > +
> > > +     if (!name_only) {
> > > +             printf("  %-50s [%s]\n",
> > > +                    "rNNN",
> > > +                    event_type_descriptors[PERF_TYPE_RAW]);
> > > +             printf("  %-50s [%s]\n",
> > > +                    "cpu/t1=v1[,t2=v2,t3 ...]/modifier",
> > > +                    event_type_descriptors[PERF_TYPE_RAW]);
> > > +             if (pager_in_use())
> > > +                     printf("   (see 'man perf-list' on how to encode
> > it)\n\n");
> > > +
> > > +             printf("  %-50s [%s]\n",
> > > +                    "mem:<addr>[/len][:access]",
> > > +                     event_type_descriptors[PERF_TYPE_BREAKPOINT]);
> > > +             if (pager_in_use())
> > > +                     printf("\n");
> > > +     }
> > > +
> > > +     print_tracepoint_events(NULL, NULL, name_only);
> > > +
> > > +     print_sdt_events(NULL, NULL, name_only);
> > > +
> > > +     metricgroup__print(true, true, NULL, name_only, details_flag);
> > > +
> > > +     print_libpfm_events(name_only, long_desc);
> > > +}
> > > diff --git a/tools/perf/util/print-events.h
> > b/tools/perf/util/print-events.h
> > > new file mode 100644
> > > index 000000000000..cf64e8129c1b
> > > --- /dev/null
> > > +++ b/tools/perf/util/print-events.h
> > > @@ -0,0 +1,21 @@
> > > +/* SPDX-License-Identifier: GPL-2.0 */
> > > +#ifndef __PERF_PRINT_EVENTS_H
> > > +#define __PERF_PRINT_EVENTS_H
> > > +
> > > +#include <stdbool.h>
> > > +
> > > +struct event_symbol;
> > > +
> > > +void print_events(const char *event_glob, bool name_only, bool quiet,
> > > +               bool long_desc, bool details_flag, bool deprecated);
> > > +int print_hwcache_events(const char *event_glob, bool name_only);
> > > +void print_sdt_events(const char *subsys_glob, const char *event_glob,
> > > +                   bool name_only);
> > > +void print_symbol_events(const char *event_glob, unsigned int type,
> > > +                      struct event_symbol *syms, unsigned int max,
> > > +                      bool name_only);
> > > +void print_tool_events(const char *event_glob, bool name_only);
> > > +void print_tracepoint_events(const char *subsys_glob, const char
> > *event_glob,
> > > +                          bool name_only);
> > > +
> > > +#endif /* __PERF_PRINT_EVENTS_H */
> > > diff --git a/tools/perf/util/trace-event-info.c
> > b/tools/perf/util/trace-event-info.c
> > > index a65f65d0857e..fbc6d14aabbb 100644
> > > --- a/tools/perf/util/trace-event-info.c
> > > +++ b/tools/perf/util/trace-event-info.c
> > > @@ -19,16 +19,24 @@
> > >  #include <linux/kernel.h>
> > >  #include <linux/zalloc.h>
> > >  #include <internal/lib.h> // page_size
> > > +#include <sys/param.h>
> > >
> > >  #include "trace-event.h"
> > > +#include "tracepoint.h"
> > >  #include <api/fs/tracing_path.h>
> > >  #include "evsel.h"
> > >  #include "debug.h"
> > >
> > >  #define VERSION "0.6"
> > > +#define MAX_EVENT_LENGTH 512
> > >
> > >  static int output_fd;
> > >
> > > +struct tracepoint_path {
> > > +     char *system;
> > > +     char *name;
> > > +     struct tracepoint_path *next;
> > > +};
> > >
> > >  int bigendian(void)
> > >  {
> > > @@ -400,6 +408,92 @@ put_tracepoints_path(struct tracepoint_path *tps)
> > >       }
> > >  }
> > >
> > > +static struct tracepoint_path *tracepoint_id_to_path(u64 config)
> > > +{
> > > +     struct tracepoint_path *path = NULL;
> > > +     DIR *sys_dir, *evt_dir;
> > > +     struct dirent *sys_dirent, *evt_dirent;
> > > +     char id_buf[24];
> > > +     int fd;
> > > +     u64 id;
> > > +     char evt_path[MAXPATHLEN];
> > > +     char *dir_path;
> > > +
> > > +     sys_dir = tracing_events__opendir();
> > > +     if (!sys_dir)
> > > +             return NULL;
> > > +
> > > +     for_each_subsystem(sys_dir, sys_dirent) {
> > > +             dir_path = get_events_file(sys_dirent->d_name);
> > > +             if (!dir_path)
> > > +                     continue;
> > > +             evt_dir = opendir(dir_path);
> > > +             if (!evt_dir)
> > > +                     goto next;
> > > +
> > > +             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > +
> > > +                     scnprintf(evt_path, MAXPATHLEN, "%s/%s/id",
> > dir_path,
> > > +                               evt_dirent->d_name);
> > > +                     fd = open(evt_path, O_RDONLY);
> > > +                     if (fd < 0)
> > > +                             continue;
> > > +                     if (read(fd, id_buf, sizeof(id_buf)) < 0) {
> > > +                             close(fd);
> > > +                             continue;
> > > +                     }
> > > +                     close(fd);
> > > +                     id = atoll(id_buf);
> > > +                     if (id == config) {
> > > +                             put_events_file(dir_path);
> > > +                             closedir(evt_dir);
> > > +                             closedir(sys_dir);
> > > +                             path = zalloc(sizeof(*path));
> > > +                             if (!path)
> > > +                                     return NULL;
> > > +                             if (asprintf(&path->system, "%.*s",
> > MAX_EVENT_LENGTH, sys_dirent->d_name) < 0) {
> > > +                                     free(path);
> > > +                                     return NULL;
> > > +                             }
> > > +                             if (asprintf(&path->name, "%.*s",
> > MAX_EVENT_LENGTH, evt_dirent->d_name) < 0) {
> > > +                                     zfree(&path->system);
> > > +                                     free(path);
> > > +                                     return NULL;
> > > +                             }
> > > +                             return path;
> > > +                     }
> > > +             }
> > > +             closedir(evt_dir);
> > > +next:
> > > +             put_events_file(dir_path);
> > > +     }
> > > +
> > > +     closedir(sys_dir);
> > > +     return NULL;
> > > +}
> > > +
> > > +static struct tracepoint_path *tracepoint_name_to_path(const char *name)
> > > +{
> > > +     struct tracepoint_path *path = zalloc(sizeof(*path));
> > > +     char *str = strchr(name, ':');
> > > +
> > > +     if (path == NULL || str == NULL) {
> > > +             free(path);
> > > +             return NULL;
> > > +     }
> > > +
> > > +     path->system = strndup(name, str - name);
> > > +     path->name = strdup(str+1);
> > > +
> > > +     if (path->system == NULL || path->name == NULL) {
> > > +             zfree(&path->system);
> > > +             zfree(&path->name);
> > > +             zfree(&path);
> > > +     }
> > > +
> > > +     return path;
> > > +}
> > > +
> > >  static struct tracepoint_path *
> > >  get_tracepoints_path(struct list_head *pattrs)
> > >  {
> > > diff --git a/tools/perf/util/tracepoint.c b/tools/perf/util/tracepoint.c
> > > new file mode 100644
> > > index 000000000000..89ef56c43311
> > > --- /dev/null
> > > +++ b/tools/perf/util/tracepoint.c
> > > @@ -0,0 +1,63 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> > > +#include "tracepoint.h"
> > > +
> > > +#include <errno.h>
> > > +#include <fcntl.h>
> > > +#include <stdio.h>
> > > +#include <sys/param.h>
> > > +#include <unistd.h>
> > > +
> > > +#include <api/fs/tracing_path.h>
> > > +
> > > +int tp_event_has_id(const char *dir_path, struct dirent *evt_dir)
> > > +{
> > > +     char evt_path[MAXPATHLEN];
> > > +     int fd;
> > > +
> > > +     snprintf(evt_path, MAXPATHLEN, "%s/%s/id", dir_path,
> > evt_dir->d_name);
> > > +     fd = open(evt_path, O_RDONLY);
> > > +     if (fd < 0)
> > > +             return -EINVAL;
> > > +     close(fd);
> > > +
> > > +     return 0;
> > > +}
> > > +
> > > +/*
> > > + * Check whether event is in <debugfs_mount_point>/tracing/events
> > > + */
> > > +int is_valid_tracepoint(const char *event_string)
> > > +{
> > > +     DIR *sys_dir, *evt_dir;
> > > +     struct dirent *sys_dirent, *evt_dirent;
> > > +     char evt_path[MAXPATHLEN];
> > > +     char *dir_path;
> > > +
> > > +     sys_dir = tracing_events__opendir();
> > > +     if (!sys_dir)
> > > +             return 0;
> > > +
> > > +     for_each_subsystem(sys_dir, sys_dirent) {
> > > +             dir_path = get_events_file(sys_dirent->d_name);
> > > +             if (!dir_path)
> > > +                     continue;
> > > +             evt_dir = opendir(dir_path);
> > > +             if (!evt_dir)
> > > +                     goto next;
> > > +
> > > +             for_each_event(dir_path, evt_dir, evt_dirent) {
> > > +                     snprintf(evt_path, MAXPATHLEN, "%s:%s",
> > > +                              sys_dirent->d_name, evt_dirent->d_name);
> > > +                     if (!strcmp(evt_path, event_string)) {
> > > +                             closedir(evt_dir);
> > > +                             closedir(sys_dir);
> > > +                             return 1;
> > > +                     }
> > > +             }
> > > +             closedir(evt_dir);
> > > +next:
> > > +             put_events_file(dir_path);
> > > +     }
> > > +     closedir(sys_dir);
> > > +     return 0;
> > > +}
> > > diff --git a/tools/perf/util/tracepoint.h b/tools/perf/util/tracepoint.h
> > > new file mode 100644
> > > index 000000000000..c4a110fe87d7
> > > --- /dev/null
> > > +++ b/tools/perf/util/tracepoint.h
> > > @@ -0,0 +1,25 @@
> > > +/* SPDX-License-Identifier: GPL-2.0 */
> > > +#ifndef __PERF_TRACEPOINT_H
> > > +#define __PERF_TRACEPOINT_H
> > > +
> > > +#include <dirent.h>
> > > +#include <string.h>
> > > +
> > > +int tp_event_has_id(const char *dir_path, struct dirent *evt_dir);
> > > +
> > > +#define for_each_event(dir_path, evt_dir, evt_dirent)                \
> > > +     while ((evt_dirent = readdir(evt_dir)) != NULL)         \
> > > +             if (evt_dirent->d_type == DT_DIR &&             \
> > > +                 (strcmp(evt_dirent->d_name, ".")) &&        \
> > > +                 (strcmp(evt_dirent->d_name, "..")) &&       \
> > > +                 (!tp_event_has_id(dir_path, evt_dirent)))
> > > +
> > > +#define for_each_subsystem(sys_dir, sys_dirent)                      \
> > > +     while ((sys_dirent = readdir(sys_dir)) != NULL)         \
> > > +             if (sys_dirent->d_type == DT_DIR &&             \
> > > +                 (strcmp(sys_dirent->d_name, ".")) &&        \
> > > +                 (strcmp(sys_dirent->d_name, "..")))
> > > +
> > > +int is_valid_tracepoint(const char *event_string);
> > > +
> > > +#endif /* __PERF_TRACEPOINT_H */
> > > --
> > > 2.30.0.365.g02bc693789-goog
> > >
> >
> > --
> >
> > - Arnaldo
> >

-- 

- Arnaldo

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ