On Mon, May 07, 2018 at 04:24:08PM -0300, Arnaldo Carvalho de Melo wrote:
> 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 <jo...@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:
sry, overlooked this one.. good idea, looks ok to me, ack
jirka
>
> 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[] = {
