Date: Thu, 21 Apr 2016 10:27:16 +0900
From: Namhyung Kim <namhyung@kernel.org>
To: Taeung Song <treeze.taeung@gmail.com>
CC: Arnaldo Carvalho de Melo <acme@kernel.org>, <linux-kernel@vger.kernel.org>,
        Jiri Olsa <jolsa@kernel.org>, Ingo Molnar <mingo@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Masami Hiramatsu <mhiramat@kernel.org>
Subject: Re: [RFC][PATCH v9 0/2] Infrastructure code for perf-config
Message-ID: <20160421012716.GA4475@sejong>
References: <1461174157-7323-1-git-send-email-treeze.taeung@gmail.com>
MIME-Version: 1.0
In-Reply-To: <1461174157-7323-1-git-send-email-treeze.taeung@gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2929
Lines: 85

On Thu, Apr 21, 2016 at 02:42:35AM +0900, Taeung Song wrote:
> Hi,
> 
> We can use the config files (i.e user wide ~/.perfconfig
> and system wide $(sysconfdir)/perfconfig)
> to configure perf tools. perf-config help user
> manage the config files, not manually look into or edit them.
> 
> Introduce new infrastructure code for config
> management features of perf-config subcommand.
> 
> This pathset contains basic code for various purposes of configuration management
> showing current configs, in the near future,
> showing all configs with default value,
> getting current configs from the config files
> or writing configs that user type on the config files, etc.
> 
> IMHO, I think this infrastructure code is needed
> to add new funcationalities for config management of perf-config.
> 
> If anyone reviews this, I'd appreciate it.

For both patches,

Acked-by: Namhyung Kim <namhyung@kernel.org>

Thanks,
Namhyung


> 
> v9:
> - don't use the arbitrary maximum 'MAX_CONFIGS' (Arnaldo, Namhyung)
> - change two-dimensinal arrays 'default_config_items' to array of pointers (Namhyung)
> - remove needless 'enum perf_config_secion_idx'
> - add sections 'intel-pt','convert' and their items
> - modify perf_config_set__init() in accordance with new default config sections and items
> - (applied two previous patches 860b8d4 and 20105ca from this patchset)
> 
> v8:
> - rebased onto the current acme/perf/core
> 
> v7:
> - rename 'is_custom' to 'is_allocated' to be proper (Masami)
> - fix the code about free() or zfree() in perf_config_*__delete() (Masami)
> - check set == NULL or not in show_config() (Masami)
> 
> v6:
> - don't use goto in add_config_item() (Masami)
> 
> v5:
> - departmentalize perf_config_set__delete() (Arnaldo)
> - remove confusing find_config() (Arnaldo)
> - use pr_debug() instead of pr_err() (Arnaldo)
> - use zfree() instead of free() (Arnaldo)
> - more compact in perf_config_set__new() (Arnaldo)
> - rename variables 'perf_configs', 'config_items', etc. (Arnaldo)
> 
> v4:
> - fill perf_config_set__delete() in collect_config() for state of error
> - fill the code setting is_custom value in add_config_item() (Namhyung)
> 
> v3:
> - use the section list that contains configs each section
>   instead of the single config list (Namhyung)
> - exclude a patch for '--list-all' option from this patchset
> 
> v2:
> - remove perf_config_kind (user, system or both config files)
>   and needless at this time, etc. (Namhyung)
> - separate this patch as several patches (Namhyung)
> - fix typing errors, etc.
> 
> Taeung Song (2):
>   perf config: Prepare all default configs
>   perf config: Initialize perf_config_set with all default configs
> 
>  tools/perf/builtin-config.c |  11 ++-
>  tools/perf/util/config.c    | 187 ++++++++++++++++++++++++++++++++++++++++++--
>  tools/perf/util/config.h    |  51 +++++++++++-
>  3 files changed, 238 insertions(+), 11 deletions(-)
> 
> -- 
> 2.5.0
>