From: Taeung Song <treeze.taeung@gmail.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: linux-kernel@vger.kernel.org, Jiri Olsa <jolsa@kernel.org>,
Namhyung Kim <namhyung@kernel.org>,
Ingo Molnar <mingo@kernel.org>,
Taeung Song <treeze.taeung@gmail.com>
Subject: [PATCHSET v15 0/9] perf config: Add several options into perf-config
Date: Thu, 11 Feb 2016 02:51:16 +0900 [thread overview]
Message-ID: <1455126685-32367-1-git-send-email-treeze.taeung@gmail.com> (raw)
Add options (user, system, list-all, skel, verbose, remove) and
getting and setting features into perf-config subcommand to more useful.
Changes in v15:
- Separate out perf-config documention from this patchset
- 'list' and 'list-all' options can also work with --verbose
- 'get' functionality can also work with --verbose
Changes in v14:
- Change default value for report.group config [Namhyung]
- Make a distinction between user and system config file name
in error message when nothing configured
- Bug fix, segmentation fault of '--skel' option
of which reason is that a variable wasn't initialized
- Rename colors.code to colors.jump_arrows among config variables
Changes in v13:
- Add record.build-id config info to perf-config doc [Namhyung]
- Fill missing options info ('verbose', 'skel') on perf-config doc
Changes in v12:
- Modify perf-config documentation for each section to be more proper
and correct small typing errors [Namhyung]
- Add missing config variables that are group, show_linenr and
show_total_period [Namhyung]
Changes in v11:
- Add --skel option to produce an skeleton with the possible config
variables [Arnaldo]
- Add exception handling to --list option when nothing configured [Arnaldo]
- Default action without option is to show an usage message [Arnaldo]
Changes in v10:
- Split perf-config.txt into several patches for each section and replace some
paragraphes with better proper things [Arnaldo, Namhyung]
- Correct wrong default values for each variable [Namhyung]
- Remove the compare name function
Changes in v9:
- Add the compare name functionality treating '-' and '-' as being
the same thing for usability
Changes in v8:
- Correct small typing errors in a perf-config documention
- Split the collecting configs part into a separate patch
- Use new enum and struct for default configs instead of
hard-coded value [Namhyung]
Changes in v7:
- Modify explanations of syntax and options(color, gtk, tui, buildid, annotate)
to be better proper descriptions [Arnaldo]
Changes in v6:
- Split a 'set' feature patch into two patch to separate normalize_value() from it
- Bug fix : 'remove' and 'set' malfunctions when without a specific file-option
(If file-option isn't used, 'remove' feature had to use both user and system
config file and 'set' feature had to only handle user config file.)
Changes in v5:
- Simplify the switch statement in cmd_config()
- Set a config file path with '--system' or '--user'
instead of '--global' or '--system' [Namhyung]
- The patch about 'get' and 'set 'split into two patchs [Namhyung]
Changes in v4:
- If some config value is default value, notice it is '(default)' [Jirka]
- If there wasn't any perfconfig file, perf-config malfunctioned
So add exception routine with '--global' and '--system'
option which can select perf config file path [Jirka]
Changes in v3:
- Add a config variable 'kmem.default' with a default value [Namhyung]
Changes in v2:
- Change option name of listing all configs as '--list-all'
instead of '--all' [Namhyung]
- Correct small infelicities or typing errors in a perf-config documention [Arnaldo]
- Declaration a global variable 'static struct default_configsets' has config
variables with default values instead of using a 'util/PERFCONFIG-DEFAULT' file.
- Add a function to normalize a value and check data type of it.
- Simplify parsing arguments as arguments is just divided by '=' and then
in front of '.' is a section, between '.' and '=' is a name,
and behind '=' is a value.
- If run perf-config command without any option, perf-config work as '--list'.
Taeung Song (9):
perf config: Add '--system' and '--user' options to select which
config file is used
perf config: Collect configs to handle config variables
perf config: Add 'list-all' option to perf-config
perf config: Add a option 'skel' to perf-config
perf config: Add --verbose option for showing config description
perf config: Add 'get' functionality
perf config: Add 'set' feature
perf config: normalize a value depending on default type of it
perf config: Add a option 'remove' to perf-config
tools/perf/Documentation/perf-config.txt | 38 +-
tools/perf/builtin-config.c | 739 ++++++++++++++++++++++++++++++-
tools/perf/util/cache.h | 20 +
tools/perf/util/config.c | 31 +-
4 files changed, 812 insertions(+), 16 deletions(-)
--
2.5.0
next reply other threads:[~2016-02-10 17:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-10 17:51 Taeung Song [this message]
2016-02-10 17:51 ` [PATCH v15 1/9] perf config: Add '--system' and '--user' options to select which config file is used Taeung Song
2016-02-16 7:51 ` [tip:perf/core] " tip-bot for Taeung Song
2016-02-10 17:51 ` [PATCH v15 2/9] perf config: Collect configs to handle config variables Taeung Song
2016-02-10 17:51 ` [PATCH v15 3/9] perf config: Add 'list-all' option to perf-config Taeung Song
2016-02-10 17:51 ` [PATCH v15 4/9] perf config: Add a option 'skel' " Taeung Song
2016-02-10 17:51 ` [PATCH v15 5/9] perf config: Add --verbose option for showing config description Taeung Song
2016-02-10 17:51 ` [PATCH v15 6/9] perf config: Add 'get' functionality Taeung Song
2016-02-10 17:51 ` [PATCH v15 7/9] perf config: Add 'set' feature Taeung Song
2016-02-10 17:51 ` [PATCH v15 8/9] perf config: normalize a value depending on default type of it Taeung Song
2016-02-10 17:51 ` [PATCH v15 9/9] perf config: Add a option 'remove' to perf-config Taeung Song
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1455126685-32367-1-git-send-email-treeze.taeung@gmail.com \
--to=treeze.taeung@gmail.com \
--cc=acme@kernel.org \
--cc=jolsa@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=namhyung@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).