MIME-Version: 1.0
In-Reply-To: <20131023070748.GB7756@tassilo.jf.intel.com>
References: <1381416608-2741-1-git-send-email-eranian@google.com>
	<20131010180049.GD9929@pd.tnic>
	<20131016124627.GA2611@gmail.com>
	<CABPqkBRQcisi8ukhD=6fKMq+iqwM+Qgxd3=jo82EUJB5coXs1A@mail.gmail.com>
	<20131016175302.GB4100@infradead.org>
	<CABPqkBSs1YwTiy_nkyXc89vg8-Tg=ONqCk930A+ksoLKsGLdHw@mail.gmail.com>
	<20131017081420.GB22705@gmail.com>
	<CABPqkBTBqEUU9p3SwNTXwwJ9j7VQXdofA5DzyoDmw+xJBE-uUw@mail.gmail.com>
	<20131023070748.GB7756@tassilo.jf.intel.com>
Date: Wed, 23 Oct 2013 11:24:51 +0200
Message-ID: <CABPqkBRa49QtPUn2An=vhS3NVK2HL0wc7jRiTtptEejv5V6zBw@mail.gmail.com>
Subject: Re: [PATCH v2 0/3] perf,x86: add Intel RAPL PMU support
From: Stephane Eranian <eranian@google.com>
To: Andi Kleen <ak@linux.intel.com>
Cc: Ingo Molnar <mingo@kernel.org>, Arnaldo Carvalho de Melo <acme@redhat.com>,
        Borislav Petkov <bp@alien8.de>, LKML <linux-kernel@vger.kernel.org>,
        Peter Zijlstra <peterz@infradead.org>, "mingo@elte.hu" <mingo@elte.hu>,
        Jiri Olsa <jolsa@redhat.com>, "Yan, Zheng" <zheng.z.yan@intel.com>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1379
Lines: 29

Andi,

On Wed, Oct 23, 2013 at 9:07 AM, Andi Kleen <ak@linux.intel.com> wrote:
>> # perf stat -a -e power/energy-cores/,power/energy-pkg/,ref-cycles -I
>> 1000 sleep 1000
>> #           time        unit            counts events
>>      1.000264953      Joules              2.09 power/energy-cores/
>>   [100.00%]
>>      1.000264953      Joules              5.94 power/energy-pkg/
>>      1.000264953                   160,530,320 ref-cycles
>>      2.000640422      Joules              2.07 power/energy-cores/
>>      2.000640422      Joules              5.94 power/energy-pkg/
>>      2.000640422                   152,673,056 ref-cycles
>>      3.000964416      Joules              2.08 power/energy-cores/
>>      3.000964416      Joules              5.93 power/energy-pkg/
>>      3.000964416                   158,779,184 ref-cycles
>
> Can you add some column marker that there is no unit (like -) ?
>
> This is just in case someone wants to parse this with a tool. Yes they
> should be using -x, but it is still better to be always parseable.
>
It is parseable, it's just that you get an empty field: ,,
But I can add a "?".
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/