Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-27 Thread Jiri Olsa
On Tue, Nov 27, 2018 at 11:42:12AM +0800, Jin, Yao wrote: > > > On 11/26/2018 5:55 PM, Jiri Olsa wrote: > > On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: > > > > SNIP > > > > > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > > > index f96c005..94f62c8 100644 > > > ---

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-27 Thread Jiri Olsa
On Tue, Nov 27, 2018 at 11:42:12AM +0800, Jin, Yao wrote: > > > On 11/26/2018 5:55 PM, Jiri Olsa wrote: > > On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: > > > > SNIP > > > > > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > > > index f96c005..94f62c8 100644 > > > ---

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:53 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index f96c005..94f62c8 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -13,6 +13,7 @@ #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:53 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index f96c005..94f62c8 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -13,6 +13,7 @@ #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:52 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: Support displaying the average IPC and IPC coverage for symbol in perf report TUI browser. We create a new sort-key 'ipc' for that. For example, $ perf record -g -b ... $ perf report -s

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:52 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: Support displaying the average IPC and IPC coverage for symbol in perf report TUI browser. We create a new sort-key 'ipc' for that. For example, $ perf record -g -b ... $ perf report -s

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:55 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index f96c005..94f62c8 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -13,6 +13,7 @@ #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jin, Yao
On 11/26/2018 5:55 PM, Jiri Olsa wrote: On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index f96c005..94f62c8 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -13,6 +13,7 @@ #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index f96c005..94f62c8 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -13,6 +13,7 @@ > #include "strlist.h" > #include > #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index f96c005..94f62c8 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -13,6 +13,7 @@ > #include "strlist.h" > #include > #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index f96c005..94f62c8 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -13,6 +13,7 @@ > #include "strlist.h" > #include > #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: SNIP > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index f96c005..94f62c8 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -13,6 +13,7 @@ > #include "strlist.h" > #include > #include

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: > Support displaying the average IPC and IPC coverage for symbol > in perf report TUI browser. We create a new sort-key 'ipc' for > that. > > For example, > > $ perf record -g -b ... > $ perf report -s symbol,ipc or > perf report -s ipc

Re: [PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-26 Thread Jiri Olsa
On Mon, Nov 26, 2018 at 05:40:54PM +0800, Jin Yao wrote: > Support displaying the average IPC and IPC coverage for symbol > in perf report TUI browser. We create a new sort-key 'ipc' for > that. > > For example, > > $ perf record -g -b ... > $ perf report -s symbol,ipc or > perf report -s ipc

[PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-25 Thread Jin Yao
Support displaying the average IPC and IPC coverage for symbol in perf report TUI browser. We create a new sort-key 'ipc' for that. For example, $ perf record -g -b ... $ perf report -s symbol,ipc or perf report -s ipc Overhead Symbol IPC [IPC Coverage] 39.60%

[PATCH 2/2] perf report: Display average IPC and IPC coverage per symbol

2018-11-25 Thread Jin Yao
Support displaying the average IPC and IPC coverage for symbol in perf report TUI browser. We create a new sort-key 'ipc' for that. For example, $ perf record -g -b ... $ perf report -s symbol,ipc or perf report -s ipc Overhead Symbol IPC [IPC Coverage] 39.60%