![Taeung Song](/assets/img/avatar_default.png)
Currently show_config() has a problem when user and system config files have the same config variables i.e.: # cat ~/.perfconfig [top] children = false When $(sysconfdir) is /usr/local/etc # cat /usr/local/etc/perfconfig [top] children = true Before: # perf config --user --list top.children=false # perf config --system --list top.children=true # perf config --list top.children=true top.children=false Because perf_config() can call show_config() each the config file (user and system). Fix it. After: # perf config --user --list top.children=false # perf config --system --list top.children=true # perf config --list top.children=false Signed-off-by: Taeung Song <treeze.taeung@gmail.com> Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Jiri Olsa <jolsa@kernel.org> Cc: Masami Hiramatsu <mhiramat@kernel.org> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/r/1460620401-23430-3-git-send-email-treeze.taeung@gmail.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
113 lines
2.5 KiB
C
113 lines
2.5 KiB
C
/*
|
|
* builtin-config.c
|
|
*
|
|
* Copyright (C) 2015, Taeung Song <treeze.taeung@gmail.com>
|
|
*
|
|
*/
|
|
#include "builtin.h"
|
|
|
|
#include "perf.h"
|
|
|
|
#include "util/cache.h"
|
|
#include <subcmd/parse-options.h>
|
|
#include "util/util.h"
|
|
#include "util/debug.h"
|
|
#include "util/config.h"
|
|
|
|
static bool use_system_config, use_user_config;
|
|
|
|
static const char * const config_usage[] = {
|
|
"perf config [<file-option>] [options]",
|
|
NULL
|
|
};
|
|
|
|
enum actions {
|
|
ACTION_LIST = 1
|
|
} actions;
|
|
|
|
static struct option config_options[] = {
|
|
OPT_SET_UINT('l', "list", &actions,
|
|
"show current config variables", ACTION_LIST),
|
|
OPT_BOOLEAN(0, "system", &use_system_config, "use system config file"),
|
|
OPT_BOOLEAN(0, "user", &use_user_config, "use user config file"),
|
|
OPT_END()
|
|
};
|
|
|
|
static int show_config(struct perf_config_set *set)
|
|
{
|
|
struct perf_config_section *section;
|
|
struct perf_config_item *item;
|
|
struct list_head *sections;
|
|
|
|
if (set == NULL)
|
|
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)
|
|
printf("%s.%s=%s\n", section->name,
|
|
item->name, value);
|
|
}
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
int cmd_config(int argc, const char **argv, const char *prefix __maybe_unused)
|
|
{
|
|
int ret = 0;
|
|
struct perf_config_set *set;
|
|
char *user_config = mkpath("%s/.perfconfig", getenv("HOME"));
|
|
|
|
argc = parse_options(argc, argv, config_options, config_usage,
|
|
PARSE_OPT_STOP_AT_NON_OPTION);
|
|
|
|
if (use_system_config && use_user_config) {
|
|
pr_err("Error: only one config file at a time\n");
|
|
parse_options_usage(config_usage, config_options, "user", 0);
|
|
parse_options_usage(NULL, config_options, "system", 0);
|
|
return -1;
|
|
}
|
|
|
|
if (use_system_config)
|
|
config_exclusive_filename = perf_etc_perfconfig();
|
|
else if (use_user_config)
|
|
config_exclusive_filename = user_config;
|
|
|
|
set = perf_config_set__new();
|
|
if (!set) {
|
|
ret = -1;
|
|
goto out_err;
|
|
}
|
|
|
|
switch (actions) {
|
|
case ACTION_LIST:
|
|
if (argc) {
|
|
pr_err("Error: takes no arguments\n");
|
|
parse_options_usage(config_usage, config_options, "l", 1);
|
|
} else {
|
|
ret = show_config(set);
|
|
if (ret < 0) {
|
|
const char * config_filename = config_exclusive_filename;
|
|
if (!config_exclusive_filename)
|
|
config_filename = user_config;
|
|
pr_err("Nothing configured, "
|
|
"please check your %s \n", config_filename);
|
|
}
|
|
}
|
|
break;
|
|
default:
|
|
usage_with_options(config_usage, config_options);
|
|
}
|
|
|
|
perf_config_set__delete(set);
|
|
out_err:
|
|
return ret;
|
|
}
|