perf stat: Add rbtree node_delete op
authorJin Yao <yao.jin@linux.intel.com>
Fri, 1 Dec 2017 10:57:28 +0000 (18:57 +0800)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 5 Dec 2017 13:24:31 +0000 (10:24 -0300)
In current stat-shadow.c, the rbtree deleting is ignored.

The patch adds the implementation to node_delete method of rblist.

Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Kan Liang <kan.liang@intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1512125856-22056-5-git-send-email-yao.jin@linux.intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/stat-shadow.c

index 855e35cbb1dcec3bf65bcfe1ab4a2afd614b4141..57ec22513971f0707714ff9c10efbd6da26fc30d 100644 (file)
@@ -87,6 +87,16 @@ static struct rb_node *saved_value_new(struct rblist *rblist __maybe_unused,
        return &nd->rb_node;
 }
 
+static void saved_value_delete(struct rblist *rblist __maybe_unused,
+                              struct rb_node *rb_node)
+{
+       struct saved_value *v;
+
+       BUG_ON(!rb_node);
+       v = container_of(rb_node, struct saved_value, rb_node);
+       free(v);
+}
+
 static struct saved_value *saved_value_lookup(struct perf_evsel *evsel,
                                              int cpu,
                                              bool create)
@@ -114,7 +124,7 @@ void perf_stat__init_shadow_stats(void)
        rblist__init(&runtime_saved_values);
        runtime_saved_values.node_cmp = saved_value_cmp;
        runtime_saved_values.node_new = saved_value_new;
-       /* No delete for now */
+       runtime_saved_values.node_delete = saved_value_delete;
 }
 
 static int evsel_context(struct perf_evsel *evsel)