Re: [PATCH] perf evsel: Correct clock unit to nanosecond

2018-12-02 Thread leo . yan
On Fri, Nov 30, 2018 at 12:25:21PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Nov 30, 2018 at 10:21:40PM +0800, leo@linaro.org escreveu: > > On Fri, Nov 30, 2018 at 11:21:54AM +0100, Jiri Olsa wrote: > > > On Fri, Nov 30, 2018 at 06:06:05PM +0800, Leo Yan wrote: > > > > Since commit 0aa80

Re: [PATCH] perf evsel: Correct clock unit to nanosecond

2018-11-30 Thread Arnaldo Carvalho de Melo
Em Fri, Nov 30, 2018 at 10:21:40PM +0800, leo@linaro.org escreveu: > On Fri, Nov 30, 2018 at 11:21:54AM +0100, Jiri Olsa wrote: > > On Fri, Nov 30, 2018 at 06:06:05PM +0800, Leo Yan wrote: > > > Since commit 0aa802a79469 ("perf stat: Get rid of extra clock display > > > function"), the cpu and

Re: [PATCH] perf evsel: Correct clock unit to nanosecond

2018-11-30 Thread leo . yan
On Fri, Nov 30, 2018 at 11:21:54AM +0100, Jiri Olsa wrote: > On Fri, Nov 30, 2018 at 06:06:05PM +0800, Leo Yan wrote: > > Since commit 0aa802a79469 ("perf stat: Get rid of extra clock display > > function"), the cpu and task clock unit has been changed from > > nanosecond value to millisecond value

Re: [PATCH] perf evsel: Correct clock unit to nanosecond

2018-11-30 Thread Jiri Olsa
On Fri, Nov 30, 2018 at 06:06:05PM +0800, Leo Yan wrote: > Since commit 0aa802a79469 ("perf stat: Get rid of extra clock display > function"), the cpu and task clock unit has been changed from > nanosecond value to millisecond value. This introduces confusion for > CPU run time statistics, we can

[PATCH] perf evsel: Correct clock unit to nanosecond

2018-11-30 Thread Leo Yan
Since commit 0aa802a79469 ("perf stat: Get rid of extra clock display function"), the cpu and task clock unit has been changed from nanosecond value to millisecond value. This introduces confusion for CPU run time statistics, we can see in below flow the clock value is scaled from nanosecond value