Hi, Arnaldo :)
On 06/23/2016 08:56 PM, Arnaldo Carvalho de Melo wrote:
Em Thu, Jun 23, 2016 at 05:55:18PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() but
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system
Hi, Arnaldo :)
On 06/23/2016 08:56 PM, Arnaldo Carvalho de Melo wrote:
Em Thu, Jun 23, 2016 at 05:55:18PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() but
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system
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
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
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
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
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() and perf_config_finish()
perf config
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() and perf_config_finish()
perf config
>
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
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/perf
that is made
by perf_config_set__new() would be reused
when we initialize all actual config variables
and then if using is is finished, we could free the config set.
Thanks,
Taeung
On 06/10/2016 07:58 PM, Taeung Song wrote:
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016
that is made
by perf_config_set__new() would be reused
when we initialize all actual config variables
and then if using is is finished, we could free the config set.
Thanks,
Taeung
On 06/10/2016 07:58 PM, Taeung Song wrote:
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016
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
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
:
- split a patch into several patches
- reimplement show_config() using new perf_config()
- modify perf_config_set__delete using global variable 'config_set'
- reset config set when only 'config' sub-commaned work
because of options for config file location
Taeung Song (3):
perf config: Bring
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
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
:
- split a patch into several patches
- reimplement show_config() using new perf_config()
- modify perf_config_set__delete using global variable 'config_set'
- reset config set when only 'config' sub-commaned work
because of options for config file location
Taeung Song (3):
perf config: Bring
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/builtin-config.c | 4 ++
tools/perf/perf.c | 2 +
tools/perf/util/config.c| 92
()
# 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/perf/builtin-config.c | 4
Commit-ID: 826424cc919529d6d234af12c6ba975b63528a74
Gitweb: http://git.kernel.org/tip/826424cc919529d6d234af12c6ba975b63528a74
Author: Taeung Song <treeze.tae...@gmail.com>
AuthorDate: Wed, 8 Jun 2016 21:36:49 +0900
Committer: Arnaldo Carvalho de Melo <a...@redhat.com>
Com
Commit-ID: 826424cc919529d6d234af12c6ba975b63528a74
Gitweb: http://git.kernel.org/tip/826424cc919529d6d234af12c6ba975b63528a74
Author: Taeung Song
AuthorDate: Wed, 8 Jun 2016 21:36:49 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Tue, 14 Jun 2016 09:29:53 -0300
perf config
by perf_config_set__new() would be reused
when we initialize all actual config variables
and then if using is is finished, we could free the config set.
Thanks,
Taeung
On 06/10/2016 07:58 PM, Taeung Song wrote:
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09
by perf_config_set__new() would be reused
when we initialize all actual config variables
and then if using is is finished, we could free the config set.
Thanks,
Taeung
On 06/10/2016 07:58 PM, Taeung Song wrote:
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09
ually send patches for this problem.)
Thanks,
Taeung
On 06/11/2016 09:59 AM, Taeung Song wrote:
Good evening :)
On 06/11/2016 04:06 AM, Arnaldo Carvalho de Melo wrote:
Em Fri, Jun 10, 2016 at 03:20:43PM +0900, Taeung Song escreveu:
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
+
ually send patches for this problem.)
Thanks,
Taeung
On 06/11/2016 09:59 AM, Taeung Song wrote:
Good evening :)
On 06/11/2016 04:06 AM, Arnaldo Carvalho de Melo wrote:
Em Fri, Jun 10, 2016 at 03:20:43PM +0900, Taeung Song escreveu:
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
+
Good evening :)
On 06/11/2016 04:06 AM, Arnaldo Carvalho de Melo wrote:
Em Fri, Jun 10, 2016 at 03:20:43PM +0900, Taeung Song escreveu:
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
+++ b/tools/perf/util/cache.h
@@ -7,6 +7,7 @@
#include
+#include "config.h"
Why have
Good evening :)
On 06/11/2016 04:06 AM, Arnaldo Carvalho de Melo wrote:
Em Fri, Jun 10, 2016 at 03:20:43PM +0900, Taeung Song escreveu:
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
+++ b/tools/perf/util/cache.h
@@ -7,6 +7,7 @@
#include
+#include "config.h"
Why have
On 06/09/2016 10:41 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:51PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() so
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system config
On 06/09/2016 10:41 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:51PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() so
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system config
On 06/09/2016 10:37 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:52PM +0900, Taeung Song escreveu:
perf_config_set__delete() delete allocated the config set
but the global variable 'config_set' is used all around.
So purge and zfree by an address of the global variable
On 06/09/2016 10:37 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:52PM +0900, Taeung Song escreveu:
perf_config_set__delete() delete allocated the config set
but the global variable 'config_set' is used all around.
So purge and zfree by an address of the global variable
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:51PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() so
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system config
On 06/09/2016 10:34 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:51PM +0900, Taeung Song escreveu:
Many sub-commands use perf_config() so
everytime perf_config() is called, perf_config() always read config files.
(i.e. user config '~/.perfconfig' and system config
Hi, Arnaldo :)
Thank you for your review.
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:50PM +0900, Taeung Song escreveu:
Lately util/config.h has been added
but util/cache.h has declarations of functions
and extern variable for config features
Hi, Arnaldo :)
Thank you for your review.
On 06/09/2016 10:29 PM, Arnaldo Carvalho de Melo wrote:
Em Wed, Jun 08, 2016 at 09:36:50PM +0900, Taeung Song escreveu:
Lately util/config.h has been added
but util/cache.h has declarations of functions
and extern variable for config features
, IMHO we need to use 'config_set' as a global variable.
Anyway I sent v8 recasting this patchset with other way.
(But 'config_set' is still a global variable.)
Please, reconsider it again.
Thanks,
Taeung
On 06/08/2016 12:14 AM, Taeung Song wrote:
Good afternoon :)
On 06/07/2016 11:05 PM, Arnaldo
, IMHO we need to use 'config_set' as a global variable.
Anyway I sent v8 recasting this patchset with other way.
(But 'config_set' is still a global variable.)
Please, reconsider it again.
Thanks,
Taeung
On 06/08/2016 12:14 AM, Taeung Song wrote:
Good afternoon :)
On 06/07/2016 11:05 PM, Arnaldo
config set when only 'config' sub-commaned work
because of options for config file location
Taeung Song (5):
perf config: Handle the error about NULL at perf_config_set__delete()
perf config: Bring declarations about config from util/cache.h to
util/config.h
perf config: Reimplement
config set when only 'config' sub-commaned work
because of options for config file location
Taeung Song (5):
perf config: Handle the error about NULL at perf_config_set__delete()
perf config: Bring declarations about config from util/cache.h to
util/config.h
perf config: Reimplement
: Jiri Olsa <jo...@redhat.com>
Cc: Wang Nan <wangn...@huawei.com>
Cc: Peter Zijlstra <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
ish...@linux.intel.com>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Wang Nan <wangn...@huawei.com>
Signed-off-by: Taeung Song <treeze.tae...@gmail.com>
---
tools/perf/util/cache.h | 13 +---
sa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <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 | 29 +++
: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <alexander.shish...@linux.intel.com>
Cc: Wang Nan <wangn...@huawei.com>
Signed-off-by: Taeung Song <treeze.tae...@gmail.com>
-
: Peter Zijlstra
Cc: Ingo Molnar
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 4 +++
tools/perf/perf.c | 1 +
tools/perf/util/config.c| 87 ++---
tools/perf/util/config.h| 1
Hiramatsu
Cc: Namhyung Kim
Cc: Wang Nan
Signed-off-by: Taeung Song
---
tools/perf/util/cache.h | 13 +
tools/perf/util/config.h | 12
2 files changed, 13 insertions(+), 12 deletions(-)
diff --git a/tools/perf/util/cache.h b/tools/perf/util/cache.h
index 0d814bb..cd8cd4d
: Alexander Shishkin
Cc: Wang Nan
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 29 +++--
tools/perf/util/config.c| 2 +-
tools/perf/util/config.h| 1 +
3 files changed, 9 insertions(+), 23 deletions(-)
diff --git a/tools/perf/builtin-config.c b
: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Cc: Wang Nan
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 2 +-
tools/perf/perf.c | 2 +-
tools/perf/util/config.c| 10 +-
tools/perf/util/config.h| 2 +-
4 files changed, 8
t.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Wang Nan <wangn...@huawei.com>
Signed-off-by: Taeung Song <treeze.tae...@gmail.com>
---
tools/perf/util/config.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/perf/util
-off-by: Taeung Song
---
tools/perf/util/config.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index 8749eca..31e09a4 100644
--- a/tools/perf/util/config.c
+++ b/tools/perf/util/config.c
@@ -742,6 +742,9 @@ static void
Commit-ID: 25d8f48f78f37dd6af08bd11212ab3d53a4c8cc6
Gitweb: http://git.kernel.org/tip/25d8f48f78f37dd6af08bd11212ab3d53a4c8cc6
Author: Taeung Song <treeze.tae...@gmail.com>
AuthorDate: Tue, 7 Jun 2016 18:26:11 +0900
Committer: Arnaldo Carvalho de Melo <a...@redhat.com>
Commit
Commit-ID: 25d8f48f78f37dd6af08bd11212ab3d53a4c8cc6
Gitweb: http://git.kernel.org/tip/25d8f48f78f37dd6af08bd11212ab3d53a4c8cc6
Author: Taeung Song
AuthorDate: Tue, 7 Jun 2016 18:26:11 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Tue, 7 Jun 2016 10:58:55 -0300
perf config
Commit-ID: 8beeb00f2c8498686eee02b8edcd1488b903c90b
Gitweb: http://git.kernel.org/tip/8beeb00f2c8498686eee02b8edcd1488b903c90b
Author: Taeung Song <treeze.tae...@gmail.com>
AuthorDate: Tue, 7 Jun 2016 18:26:12 +0900
Committer: Arnaldo Carvalho de Melo <a...@redhat.com>
Commit
Commit-ID: 8beeb00f2c8498686eee02b8edcd1488b903c90b
Gitweb: http://git.kernel.org/tip/8beeb00f2c8498686eee02b8edcd1488b903c90b
Author: Taeung Song
AuthorDate: Tue, 7 Jun 2016 18:26:12 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Tue, 7 Jun 2016 11:01:25 -0300
perf config: Use
Commit-ID: 7db91f251056f90fec4121f028680ab3153a0f3c
Gitweb: http://git.kernel.org/tip/7db91f251056f90fec4121f028680ab3153a0f3c
Author: Taeung Song <treeze.tae...@gmail.com>
AuthorDate: Mon, 6 Jun 2016 19:52:54 +0900
Committer: Arnaldo Carvalho de Melo <a...@redhat.com>
Commit
Commit-ID: 7db91f251056f90fec4121f028680ab3153a0f3c
Gitweb: http://git.kernel.org/tip/7db91f251056f90fec4121f028680ab3153a0f3c
Author: Taeung Song
AuthorDate: Mon, 6 Jun 2016 19:52:54 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Mon, 6 Jun 2016 17:43:19 -0300
perf config
Commit-ID: 78f71c996fb92d129ec75d572e2f5367a4f4c757
Gitweb: http://git.kernel.org/tip/78f71c996fb92d129ec75d572e2f5367a4f4c757
Author: Taeung Song <treeze.tae...@gmail.com>
AuthorDate: Mon, 6 Jun 2016 19:52:52 +0900
Committer: Arnaldo Carvalho de Melo <a...@redhat.com>
Commit
Commit-ID: 78f71c996fb92d129ec75d572e2f5367a4f4c757
Gitweb: http://git.kernel.org/tip/78f71c996fb92d129ec75d572e2f5367a4f4c757
Author: Taeung Song
AuthorDate: Mon, 6 Jun 2016 19:52:52 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Mon, 6 Jun 2016 17:43:17 -0300
perf config: Fix
consistently and also stop
using the _get_ term for non-reference count methods (Arnaldo Carvalho de
Melo)
- 'perf config' refactorings (Taeung Song)
Signed-off-by: Arnaldo Carvalho de Melo <a...@redhat.com>
Arnaldo Carvalho
consistently and also stop
using the _get_ term for non-reference count methods (Arnaldo Carvalho de
Melo)
- 'perf config' refactorings (Taeung Song)
Signed-off-by: Arnaldo Carvalho de Melo
Arnaldo Carvalho de Melo (7):
Good afternoon :)
On 06/07/2016 11:05 PM, Arnaldo Carvalho de Melo wrote:
Em Tue, Jun 07, 2016 at 06:26:13PM +0900, Taeung Song escreveu:
The config set is prepared by collecting
all configs from config files (i.e. user config
~/.perfconfig and system config $(sysconfdir)/perfconfig)
so
Good afternoon :)
On 06/07/2016 11:05 PM, Arnaldo Carvalho de Melo wrote:
Em Tue, Jun 07, 2016 at 06:26:13PM +0900, Taeung Song escreveu:
The config set is prepared by collecting
all configs from config files (i.e. user config
~/.perfconfig and system config $(sysconfdir)/perfconfig)
so
config.
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@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>
---
config.
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/util/config.c | 47 ++-
1 file changed, 46 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/config.c b/tools/perf
: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@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 | 2 +-
to
: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 2 +-
tools/perf/util/config.c| 11 +++
tools/perf/util/config.h| 2 +-
3 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/tools
set in collect_config().
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@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/pe
set in collect_config().
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/util/config.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index
will be handled in perf_config()
and other functions at util/config.c
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <alexander.shish...@linux.intel.com>
Signed-off-by: Taeung Song <treeze.t
will be handled in perf_config()
and other functions at util/config.c
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 9 -
tools/perf/util/config.c| 1 +
tools/perf/util/config.h| 2 ++
3 files changed
sa <jo...@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 | 29 +++--
1 file changed, 7 insertions(+
: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 29 +++--
1 file changed, 7 insertions(+), 22 deletions(-)
diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c
index 4dab41e..b6ae8ea 100644
--- a/tools/perf/builtin
g Nan <wangn...@huawei.com>
Cc: Peter Zijlstra <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>
be reset
at the very beginning at cmd_config()
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <alexander.shish...@linux.intel.com>
Signed-off-by: Taeung Song <treeze.tae...@gmail.c
: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/perf.c| 1 +
tools/perf/util/cache.h | 1 +
tools/perf/util/config.c | 86 +++-
3 files changed, 43 insertions(+), 45 deletions(-)
diff --git a/tools/perf/perf.c
be reset
at the very beginning at cmd_config()
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c
Taeung Song (7):
perf config: If collect_config() is failed, finally free a config set
after it is done
perf config: Use new perf_config_set__init() to initialize config set
perf config: Add global variable 'config_set'
perf config: Use zfree() instead of free
Taeung Song (7):
perf config: If collect_config() is failed, finally free a config set
after it is done
perf config: Use new perf_config_set__init() to initialize config set
perf config: Add global variable 'config_set'
perf config: Use zfree() instead of free
On 06/07/2016 06:37 AM, Taeung Song wrote:
On 06/07/2016 05:23 AM, Arnaldo Carvalho de Melo wrote:
Em Mon, Jun 06, 2016 at 07:52:53PM +0900, Taeung Song escreveu:
Because of die() at perf_parse_file() a config set was freed
in collect_config(), if failed.
But it is natural to free a config
On 06/07/2016 06:37 AM, Taeung Song wrote:
On 06/07/2016 05:23 AM, Arnaldo Carvalho de Melo wrote:
Em Mon, Jun 06, 2016 at 07:52:53PM +0900, Taeung Song escreveu:
Because of die() at perf_parse_file() a config set was freed
in collect_config(), if failed.
But it is natural to free a config
set in collect_config().
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@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/pe
set in collect_config().
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/util/config.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index
On 06/07/2016 05:23 AM, Arnaldo Carvalho de Melo wrote:
Em Mon, Jun 06, 2016 at 07:52:53PM +0900, Taeung Song escreveu:
Because of die() at perf_parse_file() a config set was freed
in collect_config(), if failed.
But it is natural to free a config set after collect_config() is done
when some
On 06/07/2016 05:23 AM, Arnaldo Carvalho de Melo wrote:
Em Mon, Jun 06, 2016 at 07:52:53PM +0900, Taeung Song escreveu:
Because of die() at perf_parse_file() a config set was freed
in collect_config(), if failed.
But it is natural to free a config set after collect_config() is done
when some
Hi, Arnaldo :)
Did you have a nice weekend?
I sent this mail for nothing else but to tell the reason of v6 to you.
On 06/01/2016 01:52 AM, Taeung Song wrote:
On 05/31/2016 10:43 PM, Arnaldo Carvalho de Melo wrote:
Em Tue, May 31, 2016 at 10:13:43AM +0900, Taeung Song escreveu:
Instead
Hi, Arnaldo :)
Did you have a nice weekend?
I sent this mail for nothing else but to tell the reason of v6 to you.
On 06/01/2016 01:52 AM, Taeung Song wrote:
On 05/31/2016 10:43 PM, Arnaldo Carvalho de Melo wrote:
Em Tue, May 31, 2016 at 10:13:43AM +0900, Taeung Song escreveu:
Instead
set in collect_config().
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@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/
set in collect_config().
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/util/config.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index b500737
will be handled in perf_config()
and other functions at util/config.c
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <alexander.shish...@linux.intel.com>
Signed-off-by: Taeung Song <treeze.t
will be handled in perf_config()
and other functions at util/config.c
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 9 -
tools/perf/util/config.c| 1 +
tools/perf/util/config.h| 2 ++
3 files changed
sa <jo...@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 | 29 +++--
1 file changed, 7 insertions(+
;
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.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/config.c b/
g Nan <wangn...@huawei.com>
Cc: Peter Zijlstra <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>
be reset
at the very beginning at cmd_config()
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Jiri Olsa <jo...@redhat.com>
Cc: Masami Hiramatsu <mhira...@kernel.org>
Cc: Alexander Shishkin <alexander.shish...@linux.intel.com>
Signed-off-by: Taeung Song <treeze.tae...@gmail.c
: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/perf.c| 1 +
tools/perf/util/cache.h | 1 +
tools/perf/util/config.c | 86 +++-
3 files changed, 43 insertions(+), 45 deletions(-)
diff --git a/tools/perf/perf.c
be reset
at the very beginning at cmd_config()
Cc: Namhyung Kim
Cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c
: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/builtin-config.c | 29 +++--
1 file changed, 7 insertions(+), 22 deletions(-)
diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c
index 4dab41e..b6ae8ea 100644
--- a/tools/perf/builtin
-by: Taeung Song
---
tools/perf/util/config.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index d013f90..062eeb8 100644
--- a/tools/perf/util/config.c
+++ b/tools/perf/util/config.c
@@ -607,8 +607,12 @@ static int
mhy...@kernel.org>
Cc: Jiri Olsa <jo...@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.c | 16 +++-
1 file changed
Olsa
Cc: Masami Hiramatsu
Cc: Alexander Shishkin
Signed-off-by: Taeung Song
---
tools/perf/util/config.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c
index dad7d82..b500737 100644
--- a/tools/perf/util
801 - 900 of 1768 matches
Mail list logo