Currently printing metricgroups for core- or uncore-based events matched
by CPUID is supported.

Extend this for system events.

Signed-off-by: John Garry <john.ga...@huawei.com>
---
 tools/perf/util/metricgroup.c | 62 +++++++++++++++++++++++++++++++++--
 1 file changed, 59 insertions(+), 3 deletions(-)

diff --git a/tools/perf/util/metricgroup.c b/tools/perf/util/metricgroup.c
index 1102391cafeb..03f811eb600b 100644
--- a/tools/perf/util/metricgroup.c
+++ b/tools/perf/util/metricgroup.c
@@ -557,6 +557,49 @@ static int metricgroup__print_pmu_event(struct pmu_event 
*pe,
        return 0;
 }
 
+struct metricgroup_print_sys_idata {
+       struct strlist *metriclist;
+       bool metricgroups;
+       char *filter;
+       bool raw;
+       bool details;
+       struct rblist *groups;
+};
+
+typedef int (*metricgroup_sys_event_iter_fn)(struct pmu_event *pe, void *);
+
+struct metricgroup_iter_data {
+       metricgroup_sys_event_iter_fn fn;
+       void *data;
+};
+
+static int metricgroup__sys_event_iter(struct pmu_event *pe, void *data)
+{
+       struct metricgroup_iter_data *d = data;
+       struct perf_pmu *pmu = NULL;
+
+       if (!pe->metric_expr || !pe->compat)
+               return 0;
+
+       while ((pmu = perf_pmu__scan(pmu))) {
+
+               if (!pmu->id || strcmp(pmu->id, pe->compat))
+                       continue;
+
+               return d->fn(pe, d->data);
+       }
+
+       return 0;
+}
+
+static int metricgroup__print_sys_event_iter(struct pmu_event *pe, void *data)
+{
+       struct metricgroup_print_sys_idata *d = data;
+
+       return metricgroup__print_pmu_event(pe, d->metricgroups, d->filter, 
d->raw,
+                                    d->details, d->groups, d->metriclist);
+}
+
 void metricgroup__print(bool metrics, bool metricgroups, char *filter,
                        bool raw, bool details)
 {
@@ -567,9 +610,6 @@ void metricgroup__print(bool metrics, bool metricgroups, 
char *filter,
        struct rb_node *node, *next;
        struct strlist *metriclist = NULL;
 
-       if (!map)
-               return;
-
        if (!metricgroups) {
                metriclist = strlist__new(NULL, NULL);
                if (!metriclist)
@@ -593,6 +633,22 @@ void metricgroup__print(bool metrics, bool metricgroups, 
char *filter,
                        return;
        }
 
+       {
+               struct metricgroup_iter_data data = {
+                       .fn = metricgroup__print_sys_event_iter,
+                       .data = (void *) &(struct metricgroup_print_sys_idata){
+                               .metriclist = metriclist,
+                               .metricgroups = metricgroups,
+                               .filter = filter,
+                               .raw = raw,
+                               .details = details,
+                               .groups = &groups,
+                       },
+               };
+
+               pmu_for_each_sys_event(metricgroup__sys_event_iter, &data);
+       }
+
        if (metricgroups && !raw)
                printf("\nMetric Groups:\n\n");
        else if (metrics && !raw)
-- 
2.26.2

Reply via email to