[PATCH 4/4] perf tools: Add a option 'remove' to perf-config.

[Taeung Song]

A option 'remove' is to remove specific config variables.
For the syntax examples,

    # perf config -r | --remove [section.subkey ...]

Signed-off-by: Taeung Song <treeze.tae...@gmail.com>
---
 tools/perf/Documentation/perf-config.txt |  6 ++++
 tools/perf/builtin-config.c              | 48 +++++++++++++++++++++++---------
 2 files changed, 41 insertions(+), 13 deletions(-)

diff --git a/tools/perf/Documentation/perf-config.txt 
b/tools/perf/Documentation/perf-config.txt
index 1c9027e..fb8ca74 100644
--- a/tools/perf/Documentation/perf-config.txt
+++ b/tools/perf/Documentation/perf-config.txt
@@ -13,6 +13,8 @@ or
 'perf config' -l | --list
 or
 'perf config' -a | --all
+or
+'perf config' -r | --remove [section.subkey ...]
 
 DESCRIPTION
 -----------
@@ -29,6 +31,10 @@ OPTIONS
 --all::
        Show current and all possible config variables with default values.
 
+-r::
+--remove::
+       Remove specific config variables.
+
 CONFIGURATION FILE
 ------------------
 
diff --git a/tools/perf/builtin-config.c b/tools/perf/builtin-config.c
index a67aea3..ff49c22 100644
--- a/tools/perf/builtin-config.c
+++ b/tools/perf/builtin-config.c
@@ -18,6 +18,7 @@ static struct {
        bool get_action;
        bool set_action;
        bool all_action;
+       bool remove_action;
 } params;
 
 struct list_head *sections;
@@ -31,6 +32,7 @@ static const struct option config_options[] = {
        OPT_BOOLEAN('l', "list", &params.list_action, "show current config 
variables"),
        OPT_BOOLEAN('a', "all", &params.all_action,
                    "show current and all possible config variables with 
default values"),
+       OPT_BOOLEAN('r', "remove", &params.remove_action, "remove specific 
variables: [section.subkey ...]"),
        OPT_END()
 };
 
@@ -150,19 +152,26 @@ static int set_config(const char *section_name, const 
char *subkey,
 
        find_config(&section_node, &element_node, section_name, subkey);
 
-       /* if there isn't existent section, add a new section */
-       if (!section_node) {
-               section_node = init_config_section(section_name);
-               if (!section_node)
-                       return -1;
-               list_add_tail(&section_node->list, sections);
+       if (!value) {
+               /* value == NULL means remove the variable */
+               if (section_node && element_node)
+                       element_node->value = NULL;
+               else
+                       pr_err("Error: Failed to find the variable.\n");
+       } else {
+               /* if there isn't existent section, add a new section */
+               if (!section_node) {
+                       section_node = init_config_section(section_name);
+                       if (!section_node)
+                               return -1;
+                       list_add_tail(&section_node->list, sections);
+               }
+               /* if nothing to replace, add a new element which contains 
key-value pair. */
+               if (!element_node)
+                       return add_config_element(&section_node->element_head, 
subkey, value);
+               else
+                       element_node->value = (char *)value;;
        }
-       /* if nothing to replace, add a new element which contains key-value 
pair. */
-       if (!element_node)
-               return add_config_element(&section_node->element_head, subkey, 
value);
-       else
-               element_node->value = (char *)value;
-
        return 0;
 }
 
@@ -367,7 +376,20 @@ int cmd_config(int argc, const char **argv, const char 
*prefix __maybe_unused)
                ret = perf_config(show_config, NULL);
        else if (params.all_action && argc == 0)
                ret = show_all_config();
-       else {
+       else if (params.remove_action) {
+               for (i = 0; argv[i]; i++) {
+                       value = strrchr(argv[i], '=');
+                       if (value == NULL || value == argv[0])
+                               ret = 
perf_configset_with_option(set_spec_config, argv[i]);
+                       else {
+                               pr_err("invalid key: %s", argv[i]);
+                               return -1;
+                       }
+                       if (ret < 0)
+                               goto out;
+               }
+       } else {
+
                pr_warning("Error: Unknown argument.\n");
                usage_with_options(config_usage, config_options);
        }
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/