Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Taeung Song
Hi, Arnaldo :) On 01/31/2017 09:21 PM, Arnaldo Carvalho de Melo wrote: Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: Such as for_each_subsystem and for_each_event in util/parse-events.c, add new macros 'for_each_event' for easy iteration over the tracepoints in order to be

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Taeung Song
Hi, Arnaldo :) On 01/31/2017 09:21 PM, Arnaldo Carvalho de Melo wrote: Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: Such as for_each_subsystem and for_each_event in util/parse-events.c, add new macros 'for_each_event' for easy iteration over the tracepoints in order to be

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Steven Rostedt
On Tue, 31 Jan 2017 09:23:48 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Jan 31, 2017 at 09:21:16AM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > > > - while ((dent = readdir(dir))) { > > > - if

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Steven Rostedt
On Tue, 31 Jan 2017 09:23:48 -0300 Arnaldo Carvalho de Melo wrote: > Em Tue, Jan 31, 2017 at 09:21:16AM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > > > - while ((dent = readdir(dir))) { > > > - if (dent->d_type !=

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Arnaldo Carvalho de Melo
Em Tue, Jan 31, 2017 at 09:21:16AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > > - while ((dent = readdir(dir))) { > > - if (dent->d_type != DT_DIR || > > - strcmp(dent->d_name, ".") == 0 || > > -

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Arnaldo Carvalho de Melo
Em Tue, Jan 31, 2017 at 09:21:16AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > > - while ((dent = readdir(dir))) { > > - if (dent->d_type != DT_DIR || > > - strcmp(dent->d_name, ".") == 0 || > > -

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Arnaldo Carvalho de Melo
Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > Such as for_each_subsystem and for_each_event in util/parse-events.c, > add new macros 'for_each_event' for easy iteration over the tracepoints > in order to be more compact and readable. Looks ok, applied, but look below for some

Re: [PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Arnaldo Carvalho de Melo
Em Tue, Jan 31, 2017 at 08:38:28PM +0900, Taeung Song escreveu: > Such as for_each_subsystem and for_each_event in util/parse-events.c, > add new macros 'for_each_event' for easy iteration over the tracepoints > in order to be more compact and readable. Looks ok, applied, but look below for some

[PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Taeung Song
Such as for_each_subsystem and for_each_event in util/parse-events.c, add new macros 'for_each_event' for easy iteration over the tracepoints in order to be more compact and readable. Cc: Steven Rostedt Cc: Frederic Weisbecker Cc: Jiri Olsa

[PATCH v2 1/4] perf tools: Create for_each_event macro for tracepoints iteration

2017-01-31 Thread Taeung Song
Such as for_each_subsystem and for_each_event in util/parse-events.c, add new macros 'for_each_event' for easy iteration over the tracepoints in order to be more compact and readable. Cc: Steven Rostedt Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Namhyung Kim Signed-off-by: Taeung Song ---