Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754085AbcDNHxq (ORCPT ); Thu, 14 Apr 2016 03:53:46 -0400 Received: from mail-pf0-f193.google.com ([209.85.192.193]:34360 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752606AbcDNHxo (ORCPT ); Thu, 14 Apr 2016 03:53:44 -0400 From: Taeung Song To: Arnaldo Carvalho de Melo Cc: linux-kernel@vger.kernel.org, Jiri Olsa , Namhyung Kim , Ingo Molnar , Peter Zijlstra , Masami Hiramatsu , Taeung Song Subject: [PATCH v8 4/4] perf config: Initialize perf_config_set with all default configs Date: Thu, 14 Apr 2016 16:53:21 +0900 Message-Id: <1460620401-23430-5-git-send-email-treeze.taeung@gmail.com> X-Mailer: git-send-email 2.5.0 In-Reply-To: <1460620401-23430-1-git-send-email-treeze.taeung@gmail.com> References: <1460620401-23430-1-git-send-email-treeze.taeung@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4702 Lines: 174 To avoid duplicated config variables and use perf_config_set classifying between standard perf config variables and unknown or new config variables other than them, initialize perf_config_set with all default configs. And this will be needed when showing all configs with default value or checking correct type of a config variable in the near future. Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/builtin-config.c | 11 +++++++---- tools/perf/util/config.c | 39 ++++++++++++++++++++++++++++++++++----- tools/perf/util/config.h | 7 +++++++ 3 files changed, 48 insertions(+), 9 deletions(-) diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c index fe1b77f..0fe9bc5 100644 --- a/tools/perf/builtin-config.c +++ b/tools/perf/builtin-config.c @@ -35,6 +35,7 @@ static struct option config_options[] = { static int show_config(struct perf_config_set *set) { + bool has_value = false; struct perf_config_section *section; struct perf_config_item *item; struct list_head *sections; @@ -43,19 +44,21 @@ static int show_config(struct perf_config_set *set) return -1; sections = &set->sections; - if (list_empty(sections)) - return -1; - list_for_each_entry(section, sections, node) { list_for_each_entry(item, §ion->items, node) { char *value = item->value; - if (value) + if (value) { printf("%s.%s=%s\n", section->name, item->name, value); + has_value = true; + } } } + if (!has_value) + return -1; + return 0; } diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c index 5604392..61af657 100644 --- a/tools/perf/util/config.c +++ b/tools/perf/util/config.c @@ -663,6 +663,7 @@ static struct perf_config_section *add_section(struct list_head *sections, if (!section) return NULL; + section->is_allocated = true; INIT_LIST_HEAD(§ion->items); section->name = strdup(section_name); if (!section->name) { @@ -683,6 +684,7 @@ static struct perf_config_item *add_config_item(struct perf_config_section *sect if (!item) return NULL; + item->is_allocated = true; item->name = strdup(name); if (!item->name) { pr_debug("%s: strdup failed\n", __func__); @@ -751,12 +753,35 @@ out_free: return -1; } +static struct perf_config_set *perf_config_set__init(struct perf_config_set *set) +{ + int i, j; + struct perf_config_section *section; + struct perf_config_item *items; + struct list_head *sections = &set->sections; + + INIT_LIST_HEAD(&set->sections); + + for (i = 0; i != CONFIG_END; i++) { + section = &default_sections[i]; + INIT_LIST_HEAD(§ion->items); + + items = default_config_items[i]; + for (j = 0; items[j].name != NULL; j++) + list_add_tail(&items[j].node, §ion->items); + + list_add_tail(§ion->node, sections); + } + + return set; +} + struct perf_config_set *perf_config_set__new(void) { struct perf_config_set *set = zalloc(sizeof(*set)); if (set) { - INIT_LIST_HEAD(&set->sections); + perf_config_set__init(set); perf_config(collect_config, set); } @@ -765,9 +790,11 @@ struct perf_config_set *perf_config_set__new(void) static void perf_config_item__delete(struct perf_config_item *item) { - zfree((char **)&item->name); zfree(&item->value); - free(item); + if (item->is_allocated) { + zfree((char **)&item->name); + free(item); + } } static void perf_config_section__purge(struct perf_config_section *section) @@ -783,8 +810,10 @@ static void perf_config_section__purge(struct perf_config_section *section) static void perf_config_section__delete(struct perf_config_section *section) { perf_config_section__purge(section); - zfree((char **)§ion->name); - free(section); + if (section->is_allocated) { + zfree((char **)§ion->name); + free(section); + } } static void perf_config_set__purge(struct perf_config_set *set) diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h index 84dcc1d..35c0075 100644 --- a/tools/perf/util/config.h +++ b/tools/perf/util/config.h @@ -14,6 +14,11 @@ enum perf_config_type { CONFIG_TYPE_STRING }; +/** + * struct perf_config_item - element of perf's configs + * + * @is_allocated: unknown or new config other than default config + */ struct perf_config_item { const char *name; char *value; @@ -27,11 +32,13 @@ struct perf_config_item { const char *s; } default_value; enum perf_config_type type; + bool is_allocated; struct list_head node; }; struct perf_config_section { const char *name; + bool is_allocated; struct list_head items; struct list_head node; }; -- 2.5.0