Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-03-01 Thread Naveen N. Rao
On 2017/02/25 01:46AM, Masami Hiramatsu wrote: > On Thu, 23 Feb 2017 17:07:23 +0530 > "Naveen N. Rao" wrote: > > > ...into a generic function for opening trace files. > > Even if it repeats subject, please write complete description... Agh, ok sure. I will try

Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-03-01 Thread Naveen N. Rao
On 2017/02/25 01:46AM, Masami Hiramatsu wrote: > On Thu, 23 Feb 2017 17:07:23 +0530 > "Naveen N. Rao" wrote: > > > ...into a generic function for opening trace files. > > Even if it repeats subject, please write complete description... Agh, ok sure. I will try to curb my urge to not type too

Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-24 Thread Arnaldo Carvalho de Melo
Em Sat, Feb 25, 2017 at 01:46:01AM +0900, Masami Hiramatsu escreveu: > On Thu, 23 Feb 2017 17:07:23 +0530 > "Naveen N. Rao" wrote: > > > ...into a generic function for opening trace files. > > Even if it repeats subject, please write complete description... > >

Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-24 Thread Arnaldo Carvalho de Melo
Em Sat, Feb 25, 2017 at 01:46:01AM +0900, Masami Hiramatsu escreveu: > On Thu, 23 Feb 2017 17:07:23 +0530 > "Naveen N. Rao" wrote: > > > ...into a generic function for opening trace files. > > Even if it repeats subject, please write complete description... > > Patch itself is OK to me. Did

Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-24 Thread Masami Hiramatsu
On Thu, 23 Feb 2017 17:07:23 +0530 "Naveen N. Rao" wrote: > ...into a generic function for opening trace files. Even if it repeats subject, please write complete description... Patch itself is OK to me. Thanks, > > Signed-off-by: Naveen N. Rao

Re: [PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-24 Thread Masami Hiramatsu
On Thu, 23 Feb 2017 17:07:23 +0530 "Naveen N. Rao" wrote: > ...into a generic function for opening trace files. Even if it repeats subject, please write complete description... Patch itself is OK to me. Thanks, > > Signed-off-by: Naveen N. Rao > --- > tools/perf/util/probe-file.c | 20

[PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-23 Thread Naveen N. Rao
...into a generic function for opening trace files. Signed-off-by: Naveen N. Rao --- tools/perf/util/probe-file.c | 20 +++- tools/perf/util/probe-file.h | 1 + 2 files changed, 12 insertions(+), 9 deletions(-) diff --git

[PATCH v3 1/2] perf: probe: generalize probe event file open routine

2017-02-23 Thread Naveen N. Rao
...into a generic function for opening trace files. Signed-off-by: Naveen N. Rao --- tools/perf/util/probe-file.c | 20 +++- tools/perf/util/probe-file.h | 1 + 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/tools/perf/util/probe-file.c