Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Taeung Song
On 08/15/2017 01:31 AM, Arnaldo Carvalho de Melo wrote: Em Mon, Aug 14, 2017 at 07:42:07PM +0900, Taeung Song escreveu: On 08/01/2017 03:24 PM, Taeung Song wrote: On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Taeung Song
On 08/15/2017 01:31 AM, Arnaldo Carvalho de Melo wrote: Em Mon, Aug 14, 2017 at 07:42:07PM +0900, Taeung Song escreveu: On 08/01/2017 03:24 PM, Taeung Song wrote: On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 14, 2017 at 07:42:07PM +0900, Taeung Song escreveu: > On 08/01/2017 03:24 PM, Taeung Song wrote: > > On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: > > > > Add --show-nr-samples option to perf-annotate > > >

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 14, 2017 at 07:42:07PM +0900, Taeung Song escreveu: > On 08/01/2017 03:24 PM, Taeung Song wrote: > > On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: > > > > Add --show-nr-samples option to perf-annotate > > >

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Taeung Song
Hi Arnaldo, On 08/01/2017 03:24 PM, Taeung Song wrote: On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-14 Thread Taeung Song
Hi Arnaldo, On 08/01/2017 03:24 PM, Taeung Song wrote: On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-01 Thread Taeung Song
On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-01 Thread Taeung Song
On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-01 Thread Taeung Song
On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-08-01 Thread Taeung Song
On 07/29/2017 01:26 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-07-28 Thread Arnaldo Carvalho de Melo
Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: > Add --show-nr-samples option to perf-annotate > so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate --show-total-period --show-nr-samples then both should appear, no? -

Re: [PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-07-28 Thread Arnaldo Carvalho de Melo
Em Fri, Jul 28, 2017 at 01:16:16AM +0900, Taeung Song escreveu: > Add --show-nr-samples option to perf-annotate > so that it corresponds with perf-report. So this is not that intuitive, i.e. if one ask for: perf annotate --show-total-period --show-nr-samples then both should appear, no? -

[PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-07-27 Thread Taeung Song
Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. Cc: Namhyung Kim Cc: Milian Wolff Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/builtin-annotate.c | 2 ++

[PATCH v4 5/9] perf annotate stdio: Support --show-nr-samples option

2017-07-27 Thread Taeung Song
Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. Cc: Namhyung Kim Cc: Milian Wolff Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/builtin-annotate.c | 2 ++ tools/perf/util/annotate.c| 6 +- 2 files changed, 7 insertions(+), 1