Messages in this thread Patch in this message | | | Date | Mon, 7 May 2018 16:24:08 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule |
| |
Em Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > Jiri Olsa <jolsa@redhat.com> writes: > > > > Please fix this quickly, PT is currently totally non functional in Linus > > mainline. > > Ok, so I'm reverting this patch, the previous situation was just a > misleading error message, so it can wait for the discussion about the > parser fixes to come to a conclusion and a proper patch to be submitted.
... and I'm adding this to my perf/core branch, so that we notice this faster in the future:
diff --git a/tools/perf/tests/parse-events.c b/tools/perf/tests/parse-events.c index 18b06444f230..6829dd416a99 100644 --- a/tools/perf/tests/parse-events.c +++ b/tools/perf/tests/parse-events.c @@ -1309,6 +1309,14 @@ static int test__checkevent_config_cache(struct perf_evlist *evlist) return 0; } +static int test__intel_pt(struct perf_evlist *evlist) +{ + struct perf_evsel *evsel = perf_evlist__first(evlist); + + TEST_ASSERT_VAL("wrong name setting", strcmp(evsel->name, "intel_pt//u") == 0); + return 0; +} + static int count_tracepoints(void) { struct dirent *events_ent; @@ -1637,6 +1645,11 @@ static struct evlist_test test__events[] = { .check = test__checkevent_config_cache, .id = 51, }, + { + .name = "intel_pt//u", + .check = test__intel_pt, + .id = 52, + }, }; static struct evlist_test test__events_pmu[] = {
| |