lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <1464074252-21991-1-git-send-email-treeze.taeung@gmail.com> Date: Tue, 24 May 2016 16:17:27 +0900 From: Taeung Song <treeze.taeung@...il.com> To: Arnaldo Carvalho de Melo <acme@...nel.org> Cc: linux-kernel@...r.kernel.org, Jiri Olsa <jolsa@...nel.org>, Namhyung Kim <namhyung@...nel.org>, Ingo Molnar <mingo@...nel.org>, Peter Zijlstra <peterz@...radead.org>, Alexander Shishkin <alexander.shishkin@...ux.intel.com>, Masami Hiramatsu <mhiramat@...nel.org>, Taeung Song <treeze.taeung@...il.com> Subject: [RFC][PATCH v2 0/5] perf config: Reimplement perf_config() using perf_config_set__inter() Everytime perf_config() is called, perf_config() always read config files. (i.e. user config '~/.perfconfig' and system config '$(sysconfdir)/perfconfig') But we need to use 'struct perf_config_set config_set' variable that already contains all config key-value pairs to avoid this repetitive work in perf_config(). In other words, if new perf_config() is called, only first time 'config_set' is initialized collecting all configs from config files and it work with perf_config_set__iter(). If we do, what old perf_config() handle is the same as new perf_config() work without the repetitive work that read config files. IMHO, I think this patchset is needed because not only the repetitive work should be avoided but also in near future, it would be smooth to manage perf configs. If you give me any feedback, I'd apprecicated it. :) Thanks, Taeung v2: - 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 (5): perf config: Use new perf_config_set__init() to initialize config set perf config: Reimplement perf_config() using perf_config_set__iter() perf config: Modify perf_config_set__delete() using global variable 'config_set' perf config: Reimplement show_config() using perf_config() perf config: Reset config set at only 'config' sub-command tools/perf/builtin-config.c | 43 ++++-------- tools/perf/util/config.c | 156 +++++++++++++++++++++++++++++--------------- tools/perf/util/config.h | 2 +- 3 files changed, 117 insertions(+), 84 deletions(-) -- 2.5.0
Powered by blists - more mailing lists