Messages in this thread |  | | Date | Sun, 21 Feb 2016 17:39:40 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 2/6] perf, tools, stat: Implement CSV metrics output |
| |
On Thu, Feb 18, 2016 at 06:39:21PM +0100, Andi Kleen wrote: > > Where before it wasn't there: > > > > cat /tmp/before > > 0.282628,,task-clock,282628,100.00 > > 1,,context-switches,282628,100.00 > > 0,,cpu-migrations,282628,100.00 > > 52,,page-faults,282628,100.00 > > 861213,,cycles,285354,100.00 > > <not supported>,,stalled-cycles-frontend,0,100.00 > > <not supported>,,stalled-cycles-backend,0,100.00 > > 686082,,instructions,285354,100.00 > > 137846,,branches,285354,100.00 > > 7142,,branch-misses,285354,100.00 > > This is intentional. See the standard perf output: > > > 521,232 instructions # 0.63 insns per cycle > # 1.13 stalled cycles per insn > > So this line has multiple metrics. In CSV this is expressed as a mostly empty line.
it's intentional if you have data from stalled cycles counter on cpu where this one is non supported you get blank line:
0.186177,,task-clock,186177,100.00,0.448,CPUs utilized 0,,context-switches,186177,100.00,0.000,K/sec 0,,cpu-migrations,186177,100.00,0.000,K/sec 43,,page-faults,186177,100.00,0.231,M/sec 567286,,cycles,187628,100.00,3.047,GHz <not supported>,,stalled-cycles-frontend,0,100.00,,,, <not supported>,,stalled-cycles-backend,0,100.00,,,, 456664,,instructions,187628,100.00,0.80,insn per cycle ,,,,,,,, 89069,,branches,187628,100.00,478.410,M/sec 3360,,branch-misses,187628,100.00,3.77,of all branches
which I think is wrong and we should not print
jirka
|  |