Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-22 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 22, 2017 at 08:00:18AM +0900, Masami Hiramatsu escreveu: > On Tue, 21 Mar 2017 11:10:49 -0300 > Arnaldo Carvalho de Melo wrote: > > > Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > > > Thanks Masami for the review. > > > > > > On Tuesday 07

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-22 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 22, 2017 at 08:00:18AM +0900, Masami Hiramatsu escreveu: > On Tue, 21 Mar 2017 11:10:49 -0300 > Arnaldo Carvalho de Melo wrote: > > > Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > > > Thanks Masami for the review. > > > > > > On Tuesday 07 February 2017 08:41

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Masami Hiramatsu
On Tue, 21 Mar 2017 11:10:49 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > > Thanks Masami for the review. > > > > On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > > > On Thu, 2 Feb 2017 16:41:41

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Masami Hiramatsu
On Tue, 21 Mar 2017 11:10:49 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > > Thanks Masami for the review. > > > > On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > > > On Thu, 2 Feb 2017 16:41:41 +0530 > > > Ravi

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Masami Hiramatsu
On Thu, 2 Feb 2017 16:41:41 +0530 Ravi Bangoria wrote: > SDT marker argument is in N@OP format. N is the size of argument and > OP is the actual assembly operand. OP is arch dependent component and > hence it's parsing logic also should be placed under

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Masami Hiramatsu
On Thu, 2 Feb 2017 16:41:41 +0530 Ravi Bangoria wrote: > SDT marker argument is in N@OP format. N is the size of argument and > OP is the actual assembly operand. OP is arch dependent component and > hence it's parsing logic also should be placed under tools/perf/arch/. > > Signed-off-by: Ravi

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Arnaldo Carvalho de Melo
Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > Thanks Masami for the review. > > On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > > On Thu, 2 Feb 2017 16:41:41 +0530 > > Ravi Bangoria wrote: > > > >> SDT marker argument is in

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-03-21 Thread Arnaldo Carvalho de Melo
Em Tue, Feb 07, 2017 at 10:52:17AM +0530, Ravi Bangoria escreveu: > Thanks Masami for the review. > > On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > > On Thu, 2 Feb 2017 16:41:41 +0530 > > Ravi Bangoria wrote: > > > >> SDT marker argument is in N@OP format. N is the size of

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-06 Thread Ravi Bangoria
Thanks Masami for the review. On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > On Thu, 2 Feb 2017 16:41:41 +0530 > Ravi Bangoria wrote: > >> SDT marker argument is in N@OP format. N is the size of argument and >> OP is the actual assembly

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-06 Thread Ravi Bangoria
Thanks Masami for the review. On Tuesday 07 February 2017 08:41 AM, Masami Hiramatsu wrote: > On Thu, 2 Feb 2017 16:41:41 +0530 > Ravi Bangoria wrote: > >> SDT marker argument is in N@OP format. N is the size of argument and >> OP is the actual assembly operand. OP is arch dependent component

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-06 Thread Masami Hiramatsu
On Thu, 2 Feb 2017 16:41:41 +0530 Ravi Bangoria wrote: > SDT marker argument is in N@OP format. N is the size of argument and > OP is the actual assembly operand. OP is arch dependent component and > hence it's parsing logic also should be placed under

Re: [PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-06 Thread Masami Hiramatsu
On Thu, 2 Feb 2017 16:41:41 +0530 Ravi Bangoria wrote: > SDT marker argument is in N@OP format. N is the size of argument and > OP is the actual assembly operand. OP is arch dependent component and > hence it's parsing logic also should be placed under tools/perf/arch/. > Ok, I have one

[PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-02 Thread Ravi Bangoria
SDT marker argument is in N@OP format. N is the size of argument and OP is the actual assembly operand. OP is arch dependent component and hence it's parsing logic also should be placed under tools/perf/arch/. Signed-off-by: Ravi Bangoria ---

[PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-02 Thread Ravi Bangoria
SDT marker argument is in N@OP format. N is the size of argument and OP is the actual assembly operand. OP is arch dependent component and hence it's parsing logic also should be placed under tools/perf/arch/. Signed-off-by: Ravi Bangoria --- tools/perf/arch/x86/util/perf_regs.c | 93