Re: [PATCH] perf, tools: Improve error messages for missing PMUs

[Arnaldo Carvalho de Melo]

Em Fri, Sep 15, 2017 at 08:45:10PM +0200, Jiri Olsa escreveu:
> On Wed, Sep 13, 2017 at 02:50:06PM -0700, Andi Kleen wrote:
> > +++ b/tools/perf/util/metricgroup.c
> > @@ -477,7 +477,7 @@ int metricgroup__parse_groups(const struct option *opt,
> >  	memset(&parse_error, 0, sizeof(struct parse_events_error));
> >  	ret = parse_events(perf_evlist, extra_events.buf, &parse_error);
> >  	if (ret) {
> > -		pr_err("Cannot set up events %s\n", extra_events.buf);
> > +                parse_events_print_error(&parse_error, extra_events.buf);
> 
> nit, mixed tabs/spaces indent, other than that:

Fixed that.
 
> Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

Thanks, applied.

- Arnaldo