lkml.org 
[lkml]   [2013]   [Oct]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2 0/3] perf,x86: add Intel RAPL PMU support
From
Arnaldo,

On Wed, Oct 23, 2013 at 4:22 PM, Arnaldo Carvalho de Melo
<acme@redhat.com> wrote:
> Em Wed, Oct 23, 2013 at 11:34:42AM +0200, Stephane Eranian escreveu:
>> On Wed, Oct 23, 2013 at 12:18 AM, Arnaldo Carvalho de Melo
>> > What about:
>
>> > # perf stat -a -e power/energy-cores/,power/energy-pkg/,ref-cycles -I 1000 sleep 1000
>> > # time events
>> > 1.000264953 2.09 Joules power/energy-cores/
>> > 1.000264953 5.94 Joules power/energy-pkg/
>> > 1.000264953 160,530,320 ref-cycles
>> > 2.000640422 2.07 Joules power/energy-cores/
>> > 2.000640422 5.94 Joules power/energy-pkg/
>> > 2.000640422 152,673,056 ref-cycles
>> > 3.000964416 2.08 Joules power/energy-cores/
>> > 3.000964416 5.93 Joules power/energy-pkg/
>> > 3.000964416 158,779,184 ref-cycles
>
>> > ?
>> > Or even 2.09J power/energy-cores/?
>
>> I can try that.
>
>> > I.e. a perf_evsel__fprintf_value(evsel) would append a unit string, if
>> > available.
>
>> I don't have this function is my tree yet (tip.git).
>
> That would be a new one :-)
>
> At some point I'll study the %pM, etc things in the kernel printk code
> to come up with something like perf_evsel__{f,scn}printf that allows us
> to use just one string format and then pick things like units as a
> modifier, but till then having these fprintf variants seems good enough.
>
Having the printf() would only be good to print the value but the problem is
that you'd need to synchronize with the column headers and width. So
if you say fprintf_value() print the count + unit, then you need to line up
also with the column header which comes from somwhere else. I am
talking about the interval printing mode here.


\
 
 \ /
  Last update: 2013-10-23 17:01    [W:0.063 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site