lkml.org 
[lkml]   [2020]   [May]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip: perf/core] perf parse-events: Add parse_events_option() variant that creates evlist
    The following commit has been merged into the perf/core branch of tip:

    Commit-ID: d0abbc3ce695437fe83446aef44b2f5ef65a80b9
    Gitweb: https://git.kernel.org/tip/d0abbc3ce695437fe83446aef44b2f5ef65a80b9
    Author: Arnaldo Carvalho de Melo <acme@redhat.com>
    AuthorDate: Mon, 27 Apr 2020 13:58:11 -03:00
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitterDate: Tue, 05 May 2020 16:35:29 -03:00

    perf parse-events: Add parse_events_option() variant that creates evlist

    For the upcoming --switch-output-event option we want to create the side
    band event, populate it with the specified events and then, if it is
    present multiple times, go on adding to it, then, if the BPF tracking is
    required, use the first event to set its attr.bpf_event to get those
    PERF_RECORD_BPF_EVENT metadata events too.

    Acked-by: Jiri Olsa <jolsa@redhat.com>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Song Liu <songliubraving@fb.com>
    Link: http://lore.kernel.org/lkml/20200429131106.27974-5-acme@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/util/parse-events.c | 23 +++++++++++++++++++++++
    tools/perf/util/parse-events.h | 1 +
    2 files changed, 24 insertions(+)

    diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c
    index 1010774..5795f3a 100644
    --- a/tools/perf/util/parse-events.c
    +++ b/tools/perf/util/parse-events.c
    @@ -2190,6 +2190,29 @@ int parse_events_option(const struct option *opt, const char *str,
    return ret;
    }

    +int parse_events_option_new_evlist(const struct option *opt, const char *str, int unset)
    +{
    + struct evlist **evlistp = opt->value;
    + int ret;
    +
    + if (*evlistp == NULL) {
    + *evlistp = evlist__new();
    +
    + if (*evlistp == NULL) {
    + fprintf(stderr, "Not enough memory to create evlist\n");
    + return -1;
    + }
    + }
    +
    + ret = parse_events_option(opt, str, unset);
    + if (ret) {
    + evlist__delete(*evlistp);
    + *evlistp = NULL;
    + }
    +
    + return ret;
    +}
    +
    static int
    foreach_evsel_in_last_glob(struct evlist *evlist,
    int (*func)(struct evsel *evsel,
    diff --git a/tools/perf/util/parse-events.h b/tools/perf/util/parse-events.h
    index 27596cb..6ead966 100644
    --- a/tools/perf/util/parse-events.h
    +++ b/tools/perf/util/parse-events.h
    @@ -31,6 +31,7 @@ bool have_tracepoints(struct list_head *evlist);
    const char *event_type(int type);

    int parse_events_option(const struct option *opt, const char *str, int unset);
    +int parse_events_option_new_evlist(const struct option *opt, const char *str, int unset);
    int parse_events(struct evlist *evlist, const char *str,
    struct parse_events_error *error);
    int parse_events_terms(struct list_head *terms, const char *str);
    \
     
     \ /
      Last update: 2020-05-08 15:07    [W:2.224 / U:0.752 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site