From nobody Wed Sep 10 05:46:10 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32D34C678D5 for ; Wed, 8 Mar 2023 08:20:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229808AbjCHIUU (ORCPT ); Wed, 8 Mar 2023 03:20:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34882 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230183AbjCHIT0 (ORCPT ); Wed, 8 Mar 2023 03:19:26 -0500 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6801FAFB94 for ; Wed, 8 Mar 2023 00:19:11 -0800 (PST) Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-536a5a0b6e3so164558287b3.10 for ; Wed, 08 Mar 2023 00:19:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; t=1678263550; h=cc:to:from:subject:references:mime-version:message-id:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=x5MPo35xGT8I5no97e54JFoAV8IlgxfgBAbMEiQkE8U=; b=e4ESgZ8ToAIyqrq8dv/6NSbnKuVRTmohRuk7EygZNRLdy/dMiSTxYdYKvas6dsWze6 VHQFO/oAV/sZUgNJ5NatjUIu83tNhajiPfaMjzY5PIhlw5Xiggq3Q3EQ66K1A2OhNbw5 8ABZpeAslQ4uHqXddYwS4vN7+wuw0b+/z1CcZPIUMxTEdtBsnm3jePax3mkUtUkAZOu3 wd3p4dq70Nj/oCZ5hKY1OdBRb2SbtFjwwDpLq6WSfTrs5qWr7u8OJ9zmgY+hi56E+Lvy FQ7lj0RT0C10phT40hj3IrHy4vfRqK6ArEgQVEUXjPhGra33pfdasIoaYRfseS5roSzp 5P4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678263550; h=cc:to:from:subject:references:mime-version:message-id:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=x5MPo35xGT8I5no97e54JFoAV8IlgxfgBAbMEiQkE8U=; b=6v1JZiZ1Zeet9jA95L2Xdvz6yfnQVNc6LPDU1ZlL+7bBfLvH72O+sFYfgDem0FmmyO VWl9Xo3Bql2DodD2kL78QY6XBoCDyFunqmHdiqge+vU97JYJjL1YQGQ0AtvNm/FXzXtl GbhWHuuehHHgvS4XauLfd5HuZN8PiGYsguaRl7MCWZhHIwoBU0oUQMA2LuOAPGyBuBIj TBr92KdVMnDZfhER/WgNEmB1/OMMujsdHymV7t06bS1AwK2mkX6V/qFiSwDiQmrqwosJ X7WhwaA9OTnLwx4s7IkYErBgcGTizYFqTVvjEURYO0fcYi51jw9ezX6FoplhpWNnM5Is xHpg== X-Gm-Message-State: AO0yUKXXbp0nB5+rinHGAND3CVZlkzBTVHMgD5FfRGtLG/dSSy2mPAAn OmSTKnrRThjGbIFgwsF7H0zGtNOHYiaW X-Google-Smtp-Source: AK7set8wR0JRCbOxXeyoYk1i7vpLK4VrAcyGzkcxx5TBYPyVU6eUjGqAbiFS0FlYYP2ABj6uyumbw0Ed0HOH X-Received: from irogers.svl.corp.google.com ([2620:15c:2d4:203:9a99:fbc4:7488:8b5f]) (user=irogers job=sendgmr) by 2002:a5b:c4e:0:b0:9f2:a1ba:6908 with SMTP id d14-20020a5b0c4e000000b009f2a1ba6908mr8377911ybr.12.1678263550648; Wed, 08 Mar 2023 00:19:10 -0800 (PST) Date: Wed, 8 Mar 2023 00:17:31 -0800 In-Reply-To: <20230308081731.1887278-1-irogers@google.com> Message-Id: <20230308081731.1887278-12-irogers@google.com> Mime-Version: 1.0 References: <20230308081731.1887278-1-irogers@google.com> X-Mailer: git-send-email 2.40.0.rc0.216.gc4246ad0f0-goog Subject: [PATCH v3 11/11] perf parse-events: Warn when events are regrouped From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Mark Rutland , Adrian Hunter , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Kan Liang , Zhengjun Xing , Ravi Bangoria , "Steinar H. Gunderson" , Qi Liu , Kim Phillips , Florian Fischer , James Clark , Suzuki Poulouse , Sean Christopherson , Leo Yan , John Garry , Kajol Jain , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Stephane Eranian , Ian Rogers Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use if an event is reordered or the number of groups increases to signal that regrouping has happened and warn about it. Disable the warning in the case wild card PMU names are used and for metrics. Signed-off-by: Ian Rogers --- tools/perf/tests/parse-events.c | 2 +- tools/perf/tests/pmu-events.c | 2 +- tools/perf/util/metricgroup.c | 3 ++- tools/perf/util/parse-events.c | 28 ++++++++++++++++++++-------- tools/perf/util/parse-events.h | 7 ++++--- tools/perf/util/parse-events.y | 1 + 6 files changed, 29 insertions(+), 14 deletions(-) diff --git a/tools/perf/tests/parse-events.c b/tools/perf/tests/parse-event= s.c index ffa6f0a90741..b1c2f0a20306 100644 --- a/tools/perf/tests/parse-events.c +++ b/tools/perf/tests/parse-events.c @@ -2103,7 +2103,7 @@ static int test_event_fake_pmu(const char *str) =20 parse_events_error__init(&err); perf_pmu__test_parse_init(); - ret =3D __parse_events(evlist, str, &err, &perf_pmu__fake); + ret =3D __parse_events(evlist, str, &err, &perf_pmu__fake, /*warn_if_reor= dered=3D*/true); if (ret) { pr_debug("failed to parse event '%s', err %d, str '%s'\n", str, ret, err.str); diff --git a/tools/perf/tests/pmu-events.c b/tools/perf/tests/pmu-events.c index 6ccd413b5983..7f8e86452527 100644 --- a/tools/perf/tests/pmu-events.c +++ b/tools/perf/tests/pmu-events.c @@ -785,7 +785,7 @@ static int check_parse_id(const char *id, struct parse_= events_error *error, */ perf_pmu__test_parse_init(); } - ret =3D __parse_events(evlist, dup, error, fake_pmu); + ret =3D __parse_events(evlist, dup, error, fake_pmu, /*warn_if_reordered= =3D*/true); free(dup); =20 evlist__delete(evlist); diff --git a/tools/perf/util/metricgroup.c b/tools/perf/util/metricgroup.c index de6dd527a2ba..5783f4c2d1ef 100644 --- a/tools/perf/util/metricgroup.c +++ b/tools/perf/util/metricgroup.c @@ -1441,7 +1441,8 @@ static int parse_ids(bool metric_no_merge, struct per= f_pmu *fake_pmu, } pr_debug("Parsing metric events '%s'\n", events.buf); parse_events_error__init(&parse_error); - ret =3D __parse_events(parsed_evlist, events.buf, &parse_error, fake_pmu); + ret =3D __parse_events(parsed_evlist, events.buf, &parse_error, fake_pmu, + /*warn_if_reordered=3D*/false); if (ret) { parse_events_error__print(&parse_error, events.buf); goto err_out; diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index 9ec3c1dc81e0..4bb28c32b511 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c @@ -2157,11 +2157,13 @@ static int evlist__cmp(void *state, const struct li= st_head *l, const struct list return arch_evlist__cmp(lhs, rhs); } =20 -static void parse_events__sort_events_and_fix_groups(struct list_head *lis= t) +static bool parse_events__sort_events_and_fix_groups(struct list_head *lis= t) { int idx =3D -1; struct evsel *pos, *cur_leader =3D NULL; struct perf_evsel *cur_leaders_grp =3D NULL; + bool idx_changed =3D false; + int orig_num_leaders =3D 0, num_leaders =3D 0; =20 /* * Compute index to insert ungrouped events at. Place them where the @@ -2170,11 +2172,11 @@ static void parse_events__sort_events_and_fix_group= s(struct list_head *list) list_for_each_entry(pos, list, core.node) { const struct evsel *pos_leader =3D evsel__leader(pos); =20 - if (pos !=3D pos_leader || pos->core.nr_members > 1) - continue; + if (pos =3D=3D pos_leader) + orig_num_leaders++; =20 - idx =3D pos->core.idx; - break; + if (idx =3D=3D -1 && pos =3D=3D pos_leader && pos->core.nr_members < 2) + idx =3D pos->core.idx; } =20 /* Sort events. */ @@ -2192,6 +2194,8 @@ static void parse_events__sort_events_and_fix_groups(= struct list_head *list) bool force_grouped =3D arch_evsel__must_be_in_group(pos); =20 /* Reset index and nr_members. */ + if (pos->core.idx !=3D idx) + idx_changed =3D true; pos->core.idx =3D idx++; pos->core.nr_members =3D 0; =20 @@ -2225,12 +2229,18 @@ static void parse_events__sort_events_and_fix_group= s(struct list_head *list) } } list_for_each_entry(pos, list, core.node) { - pos->core.leader->nr_members++; + struct evsel *pos_leader =3D evsel__leader(pos); + + if (pos =3D=3D pos_leader) + num_leaders++; + pos_leader->core.nr_members++; } + return idx_changed || num_leaders !=3D orig_num_leaders; } =20 int __parse_events(struct evlist *evlist, const char *str, - struct parse_events_error *err, struct perf_pmu *fake_pmu) + struct parse_events_error *err, struct perf_pmu *fake_pmu, + bool warn_if_reordered) { struct parse_events_state parse_state =3D { .list =3D LIST_HEAD_INIT(parse_state.list), @@ -2250,7 +2260,9 @@ int __parse_events(struct evlist *evlist, const char = *str, return -1; } =20 - parse_events__sort_events_and_fix_groups(&parse_state.list); + if (parse_events__sort_events_and_fix_groups(&parse_state.list) && + warn_if_reordered && !parse_state.wild_card_pmus) + pr_warning("WARNING: events were regrouped to match PMUs\n"); =20 /* * Add list to the evlist even with errors to allow callers to clean up. diff --git a/tools/perf/util/parse-events.h b/tools/perf/util/parse-events.h index 767ad1729228..46204c1a7916 100644 --- a/tools/perf/util/parse-events.h +++ b/tools/perf/util/parse-events.h @@ -26,13 +26,13 @@ int parse_events_option(const struct option *opt, const= char *str, int unset); int parse_events_option_new_evlist(const struct option *opt, const char *s= tr, int unset); __attribute__((nonnull(1, 2, 3))) int __parse_events(struct evlist *evlist, const char *str, struct parse_ev= ents_error *error, - struct perf_pmu *fake_pmu); + struct perf_pmu *fake_pmu, bool warn_if_reordered); =20 -__attribute__((nonnull)) +__attribute__((nonnull(1, 2, 3))) static inline int parse_events(struct evlist *evlist, const char *str, struct parse_events_error *err) { - return __parse_events(evlist, str, err, NULL); + return __parse_events(evlist, str, err, /*fake_pmu=3D*/NULL, /*warn_if_re= ordered=3D*/true); } =20 int parse_event(struct evlist *evlist, const char *str); @@ -128,6 +128,7 @@ struct parse_events_state { int stoken; struct perf_pmu *fake_pmu; char *hybrid_pmu_name; + bool wild_card_pmus; }; =20 void parse_events__shrink_config_terms(void); diff --git a/tools/perf/util/parse-events.y b/tools/perf/util/parse-events.y index 3a04602d2982..4488443e506e 100644 --- a/tools/perf/util/parse-events.y +++ b/tools/perf/util/parse-events.y @@ -323,6 +323,7 @@ event_pmu_name opt_pmu_config if (!parse_events_add_pmu(_parse_state, list, pmu->name, terms, /*auto_merge_stats=3D*/true)) { ok++; + parse_state->wild_card_pmus =3D true; } parse_events_terms__delete(terms); } --=20 2.40.0.rc0.216.gc4246ad0f0-goog