Re: [PATCH 09/11] perf top: error handling for counter creationshould parallel perf-record

From: Arnaldo Carvalho de Melo
Date: Tue Jul 24 2012 - 10:39:24 EST


Em Tue, Jul 24, 2012 at 08:01:38AM -0600, David Ahern escreveu:
> On 7/23/12 12:15 PM, Arnaldo Carvalho de Melo wrote:
> >Em Fri, Jul 20, 2012 at 05:25:54PM -0600, David Ahern escreveu:
> >>+ if ((err == ENOENT || err == ENXIO)
> >>+ && attr->type == PERF_TYPE_HARDWARE
> >>+ && attr->config == PERF_COUNT_HW_CPU_CYCLES) {

> >usual style:
> > if ((err == ENOENT || err == ENXIO) &&
> > attr->type == PERF_TYPE_HARDWARE &&
> > attr->config == PERF_COUNT_HW_CPU_CYCLES) {

> Right. I literally made it the same as builtin-record.c. I'll resubmit.

Ok, at some point we should fix that too, lowest prio tho, just trying
to avoid adding more.

- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/