[PATCH v7 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-20 Thread Taeung Song
c: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 25

[PATCH v7 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-20 Thread Taeung Song
amatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index 31e2028..598f434 100644 --- a/tools/perf/ui/brows

[PATCH v7 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-20 Thread Taeung Song
org> Cc: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com&g

[PATCH v7 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-20 Thread Taeung Song
: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 53 +++-- 1 file changed, 25 insertions(+), 28 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index 3eb3edb

[PATCH v7 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-20 Thread Taeung Song
: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browsers/annotate.c | 16 ++

[PATCH v7 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-20 Thread Taeung Song
Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 16 tools/perf/util/config.h | 3 +++ 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui

[PATCH v7 6/7] perf config: Add default section and item arrays for 'annotate' config

2016-07-20 Thread Taeung Song
of setting default values of actual variables for 'annotate' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com&

[PATCH v7 6/7] perf config: Add default section and item arrays for 'annotate' config

2016-07-20 Thread Taeung Song
of setting default values of actual variables for 'annotate' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 11 +++ tools/perf/util/config.h | 11 +++ 2 files changed, 22

[PATCH v7 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-20 Thread Taeung Song
: Jiri Olsa <jo...@redhat.com> Cc: Wang Nan <wangn...@huawei.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 20 +++

[PATCH v7 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-20 Thread Taeung Song
;pet...@infradead.org> Cc: Ingo Molnar <mi...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 29 +

[RFC PATCH v7 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-20 Thread Taeung Song
'ui_browser__config_gcolors' to 'ui_browser__config_colors' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config

[PATCH v7 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-20 Thread Taeung Song
: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 20 1 file changed, 20 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 1fd8e4c..613900f 100644 --- a/tools/perf/util/config.h +++ b/tools

[PATCH v7 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-20 Thread Taeung Song
-by: Taeung Song --- tools/perf/util/config.h | 29 + 1 file changed, 29 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 6f813d4..1fd8e4c 100644 --- a/tools/perf/util/config.h +++ b/tools/perf/util/config.h @@ -63,4 +63,33 @@ void

[RFC PATCH v7 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-20 Thread Taeung Song
'ui_browser__config_gcolors' to 'ui_browser__config_colors' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config

[PATCH v7 3/7] perf config: Add default section and item arrays for 'colors' config

2016-07-20 Thread Taeung Song
ause of setting default values of actual variables for 'colors' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish..

[PATCH v7 3/7] perf config: Add default section and item arrays for 'colors' config

2016-07-20 Thread Taeung Song
ause of setting default values of actual variables for 'colors' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 15 +++ tools/perf/util/config.h | 17 +

[PATCH v6 6/7] perf config: Add default section and item arrays for 'annotate' config

2016-07-12 Thread Taeung Song
of setting default values of actual variables for 'annotate' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com&

[PATCH v6 6/7] perf config: Add default section and item arrays for 'annotate' config

2016-07-12 Thread Taeung Song
of setting default values of actual variables for 'annotate' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 11 +++ tools/perf/util/config.h | 11 +++ 2 files changed, 22

[PATCH v6 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-12 Thread Taeung Song
c: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 25

[PATCH v6 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-12 Thread Taeung Song
: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browsers/annotate.c | 16 ++

[PATCH v6 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-12 Thread Taeung Song
amatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index 31e2028..598f434 100644 --- a/tools/perf/ui/brows

[PATCH v6 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-12 Thread Taeung Song
Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 16 tools/perf/util/config.h | 3 +++ 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui

[PATCH v6 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-12 Thread Taeung Song
;pet...@infradead.org> Cc: Ingo Molnar <mi...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 29 +

[PATCH v6 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-12 Thread Taeung Song
-by: Taeung Song --- tools/perf/util/config.h | 29 + 1 file changed, 29 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 6f813d4..1fd8e4c 100644 --- a/tools/perf/util/config.h +++ b/tools/perf/util/config.h @@ -63,4 +63,33 @@ void

[RFC PATCH v6 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-12 Thread Taeung Song
' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf config: Add macros

[PATCH v6 3/7] perf config: Add default section and item arrays for 'colors' config

2016-07-12 Thread Taeung Song
ause of setting default values of actual variables for 'colors' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish..

[PATCH v6 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-12 Thread Taeung Song
org> Cc: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com&g

[RFC PATCH v6 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-12 Thread Taeung Song
' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf config: Add macros

[PATCH v6 3/7] perf config: Add default section and item arrays for 'colors' config

2016-07-12 Thread Taeung Song
ause of setting default values of actual variables for 'colors' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 15 +++ tools/perf/util/config.h | 17 +

[PATCH v6 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-12 Thread Taeung Song
: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 53 +++-- 1 file changed, 25 insertions(+), 28 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index 3eb3edb

[PATCH v6 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-12 Thread Taeung Song
: Jiri Olsa <jo...@redhat.com> Cc: Wang Nan <wangn...@huawei.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 20 +++

[PATCH v6 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-12 Thread Taeung Song
: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 20 1 file changed, 20 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 1fd8e4c..613900f 100644 --- a/tools/perf/util/config.h +++ b/tools

Re: [PATCH v5 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-12 Thread Taeung Song
On 07/12/2016 02:47 PM, Namhyung Kim wrote: On Wed, Jul 06, 2016 at 02:20:23PM +0900, Taeung Song wrote: Set default config values for 'annotate' section with 'annotate_config_items[]' instead of actual bool type values. (e.g. using annotate_config_items[CONFIG_ANNOTATE_USE_OFFSET].value

Re: [PATCH v5 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-12 Thread Taeung Song
On 07/12/2016 02:47 PM, Namhyung Kim wrote: On Wed, Jul 06, 2016 at 02:20:23PM +0900, Taeung Song wrote: Set default config values for 'annotate' section with 'annotate_config_items[]' instead of actual bool type values. (e.g. using annotate_config_items[CONFIG_ANNOTATE_USE_OFFSET].value

Re: [PATCH v5 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-12 Thread Taeung Song
On 07/12/2016 02:39 PM, Namhyung Kim wrote: On Wed, Jul 06, 2016 at 02:20:21PM +0900, Taeung Song wrote: Set default config values for 'colors' section with 'colors_config_items[]' instead of actual const char * type values. (e.g. using colors_config_item[CONFIG_COLORS_TOP].value instead

Re: [PATCH v5 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-12 Thread Taeung Song
On 07/12/2016 02:39 PM, Namhyung Kim wrote: On Wed, Jul 06, 2016 at 02:20:21PM +0900, Taeung Song wrote: Set default config values for 'colors' section with 'colors_config_items[]' instead of actual const char * type values. (e.g. using colors_config_item[CONFIG_COLORS_TOP].value instead

Re: [RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-12 Thread Taeung Song
On 07/12/2016 02:37 PM, Namhyung Kim wrote: Hi Taeung, On Tue, Jul 12, 2016 at 01:20:35PM +0900, Taeung Song wrote: Hi, Namhyung :) As you know, the patch work from v3 to v4 took too long time because of a patchset refactoring perf_config(). So I guess it is hard for you to recall

Re: [RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-12 Thread Taeung Song
On 07/12/2016 02:37 PM, Namhyung Kim wrote: Hi Taeung, On Tue, Jul 12, 2016 at 01:20:35PM +0900, Taeung Song wrote: Hi, Namhyung :) As you know, the patch work from v3 to v4 took too long time because of a patchset refactoring perf_config(). So I guess it is hard for you to recall

Re: [RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-11 Thread Taeung Song
') could you agree this way ? Thanks, Taeung On 07/06/2016 02:20 PM, Taeung Song wrote: Hello, :) When initializing default perf config values, we currently use values of actual type(int, bool, char *, etc.). But I suggest using default config key-value pairs arrays. For example

Re: [RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-11 Thread Taeung Song
') could you agree this way ? Thanks, Taeung On 07/06/2016 02:20 PM, Taeung Song wrote: Hello, :) When initializing default perf config values, we currently use values of actual type(int, bool, char *, etc.). But I suggest using default config key-value pairs arrays. For example

[PATCH v5 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-05 Thread Taeung Song
: Jiri Olsa <jo...@redhat.com> Cc: Wang Nan <wangn...@huawei.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 20 +++

[PATCH v5 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-05 Thread Taeung Song
: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 20 1 file changed, 20 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 1fd8e4c..613900f 100644 --- a/tools/perf/util/config.h +++ b/tools

[PATCH v5 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-05 Thread Taeung Song
: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browsers/annotate.c | 16 ++

[PATCH v5 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-05 Thread Taeung Song
Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 16 tools/perf/util/config.h | 6 ++ 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui

[PATCH v5 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-05 Thread Taeung Song
c: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 81 - 1 file changed, 39 insertions(+), 42 deletions(-) di

[PATCH v5 6/7] perf config: Add 'annotate' section default configs arrrays

2016-07-05 Thread Taeung Song
actual variable for 'annotate' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-o

[PATCH v5 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-05 Thread Taeung Song
c: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 53 +++

[PATCH v5 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-05 Thread Taeung Song
that have default values at util/config.c because both actual variable and config item of 'colors_config_items' are equal in the number of values (as just one). Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui

[PATCH v5 6/7] perf config: Add 'annotate' section default configs arrrays

2016-07-05 Thread Taeung Song
actual variable for 'annotate' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 11 +++ tools/perf/util/config.h | 11 +++ 2 files changed, 22 insertions(+) diff --git

[PATCH v5 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-05 Thread Taeung Song
Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 53 + 1 file changed, 32 insertions(+), 21 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index b4e21d1..380abab 100644 -

[PATCH v5 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-05 Thread Taeung Song
i...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 29 + 1 file changed, 29 insertions(+) diff --

[RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-05 Thread Taeung Song
' to 'ui_browser__config_colors' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf

[PATCH v5 3/7] perf config: Add 'colors' section default configs arrrays

2016-07-05 Thread Taeung Song
ause of setting default actual variable for 'colors' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Sig

[PATCH v5 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-05 Thread Taeung Song
) with default config arrays and it could be easy and simple to modify default config values or add new configs. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Wang Nan Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 29

[RFC PATCH v5 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-05 Thread Taeung Song
' to 'ui_browser__config_colors' (Arnaldo) - change 'ground colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf

[PATCH v5 3/7] perf config: Add 'colors' section default configs arrrays

2016-07-05 Thread Taeung Song
ause of setting default actual variable for 'colors' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 15 +++ tools/perf/util/config.h | 16 2 files changed, 31 ins

[PATCH v4 3/7] perf config: Add 'colors' section default configs arrrays

2016-07-03 Thread Taeung Song
ause of setting default actual variable for 'colors' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Sig

[PATCH v4 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-03 Thread Taeung Song
i...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 29 + 1 file changed, 29 insertions(+) diff --

[PATCH v4 3/7] perf config: Add 'colors' section default configs arrrays

2016-07-03 Thread Taeung Song
ause of setting default actual variable for 'colors' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 15 +++ tools/perf/util/config.h | 16 2 files changed, 31 ins

[PATCH v4 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-07-03 Thread Taeung Song
) with default config arrays and it could be easy and simple to modify default config values or add new configs. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Wang Nan Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 29

[PATCH v4 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-03 Thread Taeung Song
c: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 81 - 1 file changed, 39 insertions(+), 42 deletions(-) di

[PATCH v4 6/7] perf config: Add 'annotate' section default configs arrrays

2016-07-03 Thread Taeung Song
actual variable for 'annotate' config. Cc: Namhyung Kim <namhy...@kernel.org> Cc: Jiri Olsa <jo...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-o

[PATCH v4 4/7] perf config: Use combined {fore,back}ground colors value instead of each two color

2016-07-03 Thread Taeung Song
that have default values at util/config.c because both actual variable and config item of 'colors_config_items' are equal in the number of values (as just one). Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui

[PATCH v4 6/7] perf config: Add 'annotate' section default configs arrrays

2016-07-03 Thread Taeung Song
actual variable for 'annotate' config. Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.c | 11 +++ tools/perf/util/config.h | 11 +++ 2 files changed, 22 insertions(+) diff --git

[PATCH v4 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-03 Thread Taeung Song
c: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browser.c | 53 +++

[PATCH v4 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-03 Thread Taeung Song
: Jiri Olsa <jo...@kernel.org> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Wang Nan <wangn...@huawei.com> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/ui/browsers/annotate.c | 16 ++

[PATCH v4 5/7] perf config: Initialize ui_browser__colorsets with default config items

2016-07-03 Thread Taeung Song
Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browser.c | 53 + 1 file changed, 32 insertions(+), 21 deletions(-) diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c index b4e21d1..380abab 100644 -

[PATCH v4 7/7] perf config: Initialize annotate_browser__opts with default config items

2016-07-03 Thread Taeung Song
Hiramatsu Cc: Wang Nan Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 16 tools/perf/util/config.h | 6 ++ 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui

[PATCH v4 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-03 Thread Taeung Song
: Jiri Olsa <jo...@redhat.com> Cc: Wang Nan <wangn...@huawei.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/util/config.h | 20 +++

[RFC PATCH v4 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-03 Thread Taeung Song
colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf config: Add macros assigning key-value

[PATCH v4 2/7] perf config: Add macros assigning key-value pairs to default_config_item

2016-07-03 Thread Taeung Song
: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools/perf/util/config.h | 20 1 file changed, 20 insertions(+) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 1fd8e4c..613900f 100644 --- a/tools/perf/util/config.h +++ b/tools

[RFC PATCH v4 0/7] perf config: Introduce default config key-value pairs arrays

2016-07-03 Thread Taeung Song
colors' to '{back, fore}ground colors' (Arnaldo) - use strtok + ltrim instead of strchr and while (isspace(*++bg)); (Arnaldo) Taeung Song (7): perf config: Introduce default_config_section and default_config_item for default config key-value pairs perf config: Add macros assigning key-value

[tip:perf/core] perf config: Introduce new init() and exit()

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 8a0a9c7e9146781defc96f6743e7ee14ccc9ab23 Gitweb: http://git.kernel.org/tip/8a0a9c7e9146781defc96f6743e7ee14ccc9ab23 Author: Taeung Song <treeze.tae...@gmail.com> AuthorDate: Thu, 23 Jun 2016 23:14:31 +0900 Committer: Arnaldo Carvalho de Melo <a...@redhat.com> Com

[tip:perf/core] perf config: Reimplement show_config() using config_set__for_each

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 4a35b3497c413de8b409f9d75700eeb4772b21b8 Gitweb: http://git.kernel.org/tip/4a35b3497c413de8b409f9d75700eeb4772b21b8 Author: Taeung Song <treeze.tae...@gmail.com> AuthorDate: Thu, 23 Jun 2016 23:14:32 +0900 Committer: Arnaldo Carvalho de Melo <a...@redhat.com> Com

[tip:perf/core] perf config: Introduce new init() and exit()

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 8a0a9c7e9146781defc96f6743e7ee14ccc9ab23 Gitweb: http://git.kernel.org/tip/8a0a9c7e9146781defc96f6743e7ee14ccc9ab23 Author: Taeung Song AuthorDate: Thu, 23 Jun 2016 23:14:31 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 23 Jun 2016 17:20:04 -0300 perf config

[tip:perf/core] perf config: Reimplement show_config() using config_set__for_each

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 4a35b3497c413de8b409f9d75700eeb4772b21b8 Gitweb: http://git.kernel.org/tip/4a35b3497c413de8b409f9d75700eeb4772b21b8 Author: Taeung Song AuthorDate: Thu, 23 Jun 2016 23:14:32 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 23 Jun 2016 17:23:00 -0300 perf config

[tip:perf/core] perf config: Move config declarations from util/cache.h to util/config.h

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 41840d211c518e6af6e327b03e09323824e563bf Gitweb: http://git.kernel.org/tip/41840d211c518e6af6e327b03e09323824e563bf Author: Taeung Song <treeze.tae...@gmail.com> AuthorDate: Thu, 23 Jun 2016 17:55:17 +0900 Committer: Arnaldo Carvalho de Melo <a...@redhat.com> Com

[tip:perf/core] perf config: Move config declarations from util/cache.h to util/config.h

2016-06-26 Thread tip-bot for Taeung Song
Commit-ID: 41840d211c518e6af6e327b03e09323824e563bf Gitweb: http://git.kernel.org/tip/41840d211c518e6af6e327b03e09323824e563bf Author: Taeung Song AuthorDate: Thu, 23 Jun 2016 17:55:17 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 23 Jun 2016 08:51:41 -0300 perf config

Re: [PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
On 06/23/2016 11:30 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 11:15:49PM +0900, Taeung Song escreveu: On 06/23/2016 10:38 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 10:03:39PM +0900, Taeung Song escreveu: +/** + * perf_config_sections__for_each - iterate

Re: [PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
On 06/23/2016 11:30 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 11:15:49PM +0900, Taeung Song escreveu: On 06/23/2016 10:38 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 10:03:39PM +0900, Taeung Song escreveu: +/** + * perf_config_sections__for_each - iterate

[PATCH v12 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
> Cc: Ingo Molnar <mi...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-config.c | 4 ++ tools/perf/perf.c

Re: [PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
On 06/23/2016 10:38 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 10:03:39PM +0900, Taeung Song escreveu: +/** + * perf_config_sections__for_each - iterate thru all the sections + * @list: list_head instance to iterate + * @section: struct perf_config_section iterator

[PATCH v12 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
config set perf_config__finish() # destroy existing config set and initialize a new config set. perf_config__refresh() Cc: Namhyung Kim Cc: Jiri Olsa Cc: Wang Nan Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools

Re: [PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
On 06/23/2016 10:38 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 10:03:39PM +0900, Taeung Song escreveu: +/** + * perf_config_sections__for_each - iterate thru all the sections + * @list: list_head instance to iterate + * @section: struct perf_config_section iterator

[PATCH v12 0/3] perf config: Reimplement perf_config()

2016-06-23 Thread Taeung Song
'config_set' - reset config set when only 'config' sub-commaned work because of options for config file location Taeung Song (3): perf config: Bring declarations about config from util/cache.h to util/config.h perf config: Reimplement perf_config() introducing new perf_config__init

[PATCH v12fg 3/3] perf config: Reimplement show_config() using config_set__for_each

2016-06-23 Thread Taeung Song
n <alexander.shish...@linux.intel.com> Cc: Wang Nan <wangn...@huawei.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-config.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/tools/perf/builtin-config.c b/tools

[PATCH v12 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
n...@huawei.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-help.c | 2 +- tools/perf/builtin-kmem.c | 2 +- tools/perf/builtin-record.c| 1 + tools/perf/builtin-report.c| 2 +- tools/perf/builtin-top.c | 2 +- too

[PATCH v12 0/3] perf config: Reimplement perf_config()

2016-06-23 Thread Taeung Song
'config_set' - reset config set when only 'config' sub-commaned work because of options for config file location Taeung Song (3): perf config: Bring declarations about config from util/cache.h to util/config.h perf config: Reimplement perf_config() introducing new perf_config__init

[PATCH v12fg 3/3] perf config: Reimplement show_config() using config_set__for_each

2016-06-23 Thread Taeung Song
Lately config_set__for_each is added. In order to let show_config() be short and clear, remake this function using config_set__for_each macro Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Alexander Shishkin Cc: Wang Nan Signed-off-by: Taeung Song --- tools/perf/builtin-config.c

[PATCH v12 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
that included previous cache.h need only config.h, remove including cache.h. Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Wang Nan Signed-off-by: Taeung Song --- tools/perf/builtin-help.c | 2 +- tools/perf/builtin-kmem.c | 2 +- tools/perf

[PATCH v11 3/3] perf config: Reimplement show_config() using config_set__for_each

2016-06-23 Thread Taeung Song
n <alexander.shish...@linux.intel.com> Cc: Wang Nan <wangn...@huawei.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-config.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/tools/perf/builtin-config.c b/tools

[PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
> Cc: Ingo Molnar <mi...@redhat.com> Cc: Masami Hiramatsu <mhira...@kernel.org> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-config.c | 4 ++ tools/perf/perf.c

[PATCH v11 3/3] perf config: Reimplement show_config() using config_set__for_each

2016-06-23 Thread Taeung Song
Lately config_set__for_each is added. In order to let show_config() be short and clear, remake this function using config_set__for_each macro Cc: Namhyung Kim Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Alexander Shishkin Cc: Wang Nan Signed-off-by: Taeung Song --- tools/perf/builtin-config.c

[PATCH v11 2/3] perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish()

2016-06-23 Thread Taeung Song
config set perf_config__finish() # destroy existing config set and initialize a new config set. perf_config__refresh() Cc: Namhyung Kim Cc: Jiri Olsa Cc: Wang Nan Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Masami Hiramatsu Cc: Alexander Shishkin Signed-off-by: Taeung Song --- tools

[PATCH v11 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
n...@huawei.com> Signed-off-by: Taeung Song <treeze.tae...@gmail.com> --- tools/perf/builtin-help.c | 2 +- tools/perf/builtin-kmem.c | 2 +- tools/perf/builtin-record.c| 1 + tools/perf/builtin-report.c| 2 +- tools/perf/builtin-top.c | 2 +- too

[PATCH v11 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
that included previous cache.h need only config.h, remove including cache.h. Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Wang Nan Signed-off-by: Taeung Song --- tools/perf/builtin-help.c | 2 +- tools/perf/builtin-kmem.c | 2 +- tools/perf

[PATCH v11 0/3] perf config: Reimplement perf_config()

2016-06-23 Thread Taeung Song
because of options for config file location Taeung Song (3): perf config: Bring declarations about config from util/cache.h to util/config.h perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish() perf config: Reimplement show_config

[PATCH v11 0/3] perf config: Reimplement perf_config()

2016-06-23 Thread Taeung Song
because of options for config file location Taeung Song (3): perf config: Bring declarations about config from util/cache.h to util/config.h perf config: Reimplement perf_config() introducing new perf_config__init() and perf_config__finish() perf config: Reimplement show_config

Re: [PATCH v10 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
Thank you !! :-D On 06/23/2016 08:57 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 05:55:17PM +0900, Taeung Song escreveu: Lately util/config.h has been added but util/cache.h has declarations of functions and a global variable for config features. To manage codes about

Re: [PATCH v10 1/3] perf config: Bring declarations about config from util/cache.h to util/config.h

2016-06-23 Thread Taeung Song
Thank you !! :-D On 06/23/2016 08:57 PM, Arnaldo Carvalho de Melo wrote: Em Thu, Jun 23, 2016 at 05:55:17PM +0900, Taeung Song escreveu: Lately util/config.h has been added but util/cache.h has declarations of functions and a global variable for config features. To manage codes about

<    3   4   5   6   7   8   9   10   11   12   >