[lkml]   [2016]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 08/10] perf, tools: Expand PMU events by prefix match
> so there's a special treatment for uncore events,
> what if user says 'uncore_box/..' then?

It should work. There's nothing special for uncore later, this
is just for convenience so that I have less to type.

> > + if (!strncmp($1, name, strlen($1))) {
> > + if (parse_events_copy_term_list(orig_terms, &terms))
> > + YYABORT;
> > + if (!parse_events_add_pmu(data, list, pmu->name, terms))
> > + ok++;
> so we're ok if some of the events is not added?
> do we warn at least?

It would warn a lot because most PMUs don't have a given aliases.
So you would get an warning for every extra PMU.

Trying to warn only for those that have the alias would need a lot
of extra tracking, and it didn't seem worth the complexity.


 \ /
  Last update: 2016-10-17 19:11    [W:0.086 / U:1.304 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site