Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
Argh, fix your mailer. That is unreadable. /me reflows... Sorry about that. Now I reconfigure the mail editor by applying "Preformat" and "Fixed Width" settings in thunderbird client. Wish it to be better. See, that's so much better.. Oh, so you _ARE_ adding a kernel feature? I

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
Argh, fix your mailer. That is unreadable. /me reflows... Sorry about that. Now I reconfigure the mail editor by applying "Preformat" and "Fixed Width" settings in thunderbird client. Wish it to be better. See, that's so much better.. Oh, so you _ARE_ adding a kernel feature? I

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Thu, Apr 06, 2017 at 10:43:19PM +0800, Jin, Yao wrote: > > > On 4/6/2017 5:25 PM, Peter Zijlstra wrote: > > On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: > > > Hi, otherwise we have to maintain 2 branch type copies between kernel and > > > user-space. > > > > > > For example,

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Thu, Apr 06, 2017 at 10:43:19PM +0800, Jin, Yao wrote: > > > On 4/6/2017 5:25 PM, Peter Zijlstra wrote: > > On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: > > > Hi, otherwise we have to maintain 2 branch type copies between kernel and > > > user-space. > > > > > > For example,

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
On 4/6/2017 5:25 PM, Peter Zijlstra wrote: On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: Hi, otherwise we have to maintain 2 branch type copies between kernel and user-space. For example, currently X86_BR_* are defined in lbr.c. To display the branch type in user-space, the

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
On 4/6/2017 5:25 PM, Peter Zijlstra wrote: On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: Hi, otherwise we have to maintain 2 branch type copies between kernel and user-space. For example, currently X86_BR_* are defined in lbr.c. To display the branch type in user-space, the

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: > Hi, otherwise we have to maintain 2 branch type copies between kernel and > user-space. > > For example, currently X86_BR_* are defined in lbr.c. To display the branch > type in user-space, the user-space has to maintain the same copy

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Thu, Apr 06, 2017 at 04:21:06PM +0800, Jin, Yao wrote: > Hi, otherwise we have to maintain 2 branch type copies between kernel and > user-space. > > For example, currently X86_BR_* are defined in lbr.c. To display the branch > type in user-space, the user-space has to maintain the same copy

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
On 4/6/2017 2:58 PM, Peter Zijlstra wrote: On Tue, Apr 04, 2017 at 11:18:05AM -0300, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Thanks. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is often useful to know the branch types while

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Jin, Yao
On 4/6/2017 2:58 PM, Peter Zijlstra wrote: On Tue, Apr 04, 2017 at 11:18:05AM -0300, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Thanks. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is often useful to know the branch types while

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Tue, Apr 04, 2017 at 11:18:05AM -0300, Arnaldo Carvalho de Melo wrote: > Adding the perf kernel maintainers to the CC list. Thanks. > Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > > It is often useful to know the branch types while analyzing branch > > data. For example, a

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-06 Thread Peter Zijlstra
On Tue, Apr 04, 2017 at 11:18:05AM -0300, Arnaldo Carvalho de Melo wrote: > Adding the perf kernel maintainers to the CC list. Thanks. > Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > > It is often useful to know the branch types while analyzing branch > > data. For example, a

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-05 Thread Jin, Yao
On 4/5/2017 12:09 AM, Arnaldo Carvalho de Melo wrote: Em Tue, Apr 04, 2017 at 11:52:53PM +0800, Jin, Yao escreveu: On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-05 Thread Jin, Yao
On 4/5/2017 12:09 AM, Arnaldo Carvalho de Melo wrote: Em Tue, Apr 04, 2017 at 11:52:53PM +0800, Jin, Yao escreveu: On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Arnaldo Carvalho de Melo
Em Tue, Apr 04, 2017 at 11:52:53PM +0800, Jin, Yao escreveu: > > > On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: > > Adding the perf kernel maintainers to the CC list. > > > > Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > > > It is often useful to know the branch types

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Arnaldo Carvalho de Melo
Em Tue, Apr 04, 2017 at 11:52:53PM +0800, Jin, Yao escreveu: > > > On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: > > Adding the perf kernel maintainers to the CC list. > > > > Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > > > It is often useful to know the branch types

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Jin, Yao
On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is often useful to know the branch types while analyzing branch data. For example, a call is very different from a

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Jin, Yao
On 4/4/2017 10:18 PM, Arnaldo Carvalho de Melo wrote: Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: It is often useful to know the branch types while analyzing branch data. For example, a call is very different from a

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Arnaldo Carvalho de Melo
Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > It is often useful to know the branch types while analyzing branch > data. For example, a call is very different from a conditional branch. > > Currently we have to look it up in

Re: [PATCH v1 1/5] perf/core: Define the common branch type classification

2017-04-04 Thread Arnaldo Carvalho de Melo
Adding the perf kernel maintainers to the CC list. Em Fri, Mar 31, 2017 at 11:18:38PM +0800, Jin Yao escreveu: > It is often useful to know the branch types while analyzing branch > data. For example, a call is very different from a conditional branch. > > Currently we have to look it up in