Re: [PATCH 2/3] perf stat: Don't require a workload when usingsystem wide or CPU options

From: Arnaldo Carvalho de Melo
Date: Tue Oct 08 2013 - 09:33:37 EST


Em Tue, Oct 08, 2013 at 07:25:39AM -0600, David Ahern escreveu:
> On 10/8/13 6:39 AM, Arnaldo Carvalho de Melo wrote:
> >While trying it noticed this, that should also be fixed
> >eventually:

> >[root@zoo ~]# perf stat -c C 0
> >C: No such file or directory

> you are missing '-' before the C: perf stat -C 0. The -C = cpus. -c
> for stat is scale.

Nah, I mean:

[acme@zoo linux]$ perf stat non-existent-proggie
non-existent-proggie: No such file or directory

Performance counter stats for 'non-existent-proggie':

<not counted> task-clock
<not counted> context-switches
<not counted> cpu-migrations
<not counted> page-faults
<not counted> cycles
<not counted> stalled-cycles-frontend
<not supported> stalled-cycles-backend
<not counted> instructions
<not counted> branches
<not counted> branch-misses

0.001823577 seconds time elapsed

[acme@zoo linux]$

Why should we report all those counters for something that wasn't found
in PATH?

We either report just the first line, with the errno, or that and the
last, with how long it took to try to execute it, but that probably
shouldn't be printed, as it is supposed to be how long the workload ran,
right?

- 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/