Re: [PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-19 Thread Taeung Song
Hi Namhyung, I'm late. Thanks for your review. On 07/19/2017 01:07 AM, Namhyung Kim wrote: Hi Taeung, On Fri, Jul 14, 2017 at 02:45:44AM +0900, Taeung Song wrote: Hello, Currently the --show-total-period option of perf-annotate is different from perf-report's. It has two problem like

Re: [PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-19 Thread Taeung Song
Hi Namhyung, I'm late. Thanks for your review. On 07/19/2017 01:07 AM, Namhyung Kim wrote: Hi Taeung, On Fri, Jul 14, 2017 at 02:45:44AM +0900, Taeung Song wrote: Hello, Currently the --show-total-period option of perf-annotate is different from perf-report's. It has two problem like

Re: [PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-18 Thread Namhyung Kim
Hi Taeung, On Fri, Jul 14, 2017 at 02:45:44AM +0900, Taeung Song wrote: > Hello, > > Currently the --show-total-period option of perf-annotate > is different from perf-report's. > > It has two problem like below: > (Reported by Namhyung Kim and Milian Wolff) > > 1) Wrong column i.e.

Re: [PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-18 Thread Namhyung Kim
Hi Taeung, On Fri, Jul 14, 2017 at 02:45:44AM +0900, Taeung Song wrote: > Hello, > > Currently the --show-total-period option of perf-annotate > is different from perf-report's. > > It has two problem like below: > (Reported by Namhyung Kim and Milian Wolff) > > 1) Wrong column i.e.

[PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-13 Thread Taeung Song
Hello, Currently the --show-total-period option of perf-annotate is different from perf-report's. It has two problem like below: (Reported by Namhyung Kim and Milian Wolff) 1) Wrong column i.e. 'Percent' (even though using --show-total-period) 2) Show number of samples, not period So fix

[PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-13 Thread Taeung Song
Hello, Currently the --show-total-period option of perf-annotate is different from perf-report's. It has two problem like below: (Reported by Namhyung Kim and Milian Wolff) 1) Wrong column i.e. 'Percent' (even though using --show-total-period) 2) Show number of samples, not period So fix