From nobody Sun May 24 19:33:16 2026 Received: from mail-pl1-f202.google.com (mail-pl1-f202.google.com [209.85.214.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 34E71335066 for ; Thu, 21 May 2026 20:15:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779394513; cv=none; b=RndJVIfuJCWnkUTBQQA9Z8wjQVMZpcJS0vo+fW4lUq5HC88yGNaV0MYJ/RVNehwayNrGpolOIHPEmoc6jEjRUZUFdafm2YE6EW7Yajx3sKs7d/Mylm6jqwzANe/HQ2Di8/4i5wh6pcY0ubh8ZDWGPgVMMDl++exdTYA005smnfI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779394513; c=relaxed/simple; bh=tWsiwu15VnarFDruHYWrHHWkVFpU9VGKDkGj9dx/iWs=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=FS0S1v9LAP6OttIEroHC2sPNtinWkLRnsnTKHbpACo9Eeyn912yzUBzgL3ZmIbKDbycArZlZl2ZV1Qzyh1+AKNcovyM2M1A7TDqv4pIH/kUPV2SpJgV/kuOIUgyBE+zxmat/nvB5eBIedNni2E343GYRIa1CLxH7NH24UYCMrwI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--ctshao.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=udBehjyZ; arc=none smtp.client-ip=209.85.214.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--ctshao.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="udBehjyZ" Received: by mail-pl1-f202.google.com with SMTP id d9443c01a7336-2b2ecc96a9aso73315755ad.1 for ; Thu, 21 May 2026 13:15:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1779394511; x=1779999311; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=2SJnEbRx6oHTv348uZbIC/z7mpNIK5u7l+aOQXx1vq8=; b=udBehjyZtXH2EHROtdllw3IzEviaFFoF+m6kIy3e+OMkyx+TEUUYTBGxPUL/pE4IHV rvAM7D3+LrwiS7FBdXX1QCKjJaUklDUtg4T3pzXg6Edk65lSPLnaVqcXa2IaVkfaKS/j S/xXc/BHTNIds2ZQ+ApraBl8P8GwitF3W6jT81z1EBdQHpzNvfSqck40tTSjsL/0hqrL 96fysogrfFaHR5al6L9vsdLWdciTG5qe8SyCSHdeUh99vjx2BcKb6Sktz7okhJZAOdsv lZ1jhOnk7ERL8SDGe003FT2PCYrSJ5wdXDQJL9lI4GhqmHFnrpjcXBSLL149C9MJGEcN Dy1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779394511; x=1779999311; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=2SJnEbRx6oHTv348uZbIC/z7mpNIK5u7l+aOQXx1vq8=; b=UII3bqOdfbE/yrkCSSj+qlrwP2BLN6cKu+59Sw2o5vDG31bACWT/MiKOZqtIHnC4ss 83lqf7oApCKCfOgQOGxICO4QuH7n6BcMoUEm27DKpm8GaLBf7dAOgElyJIbcc6NTz74T 41ouankV38xUChe4DESWYMVXDIxKhpysUpvKOWQOpZvbVaHBzMH522silLMiQEnpXzJN ALxCT+duGTBfcTaj8y6qf/ePg3JDEX5dYw7Goqy6b3PdHI8TUl4EopnNTK3m23q7FDhH X8WLMKT6G+5oNfSTyWHCaqnyQAZKPeBdMCSAFW89AgmDB6dMEnk4FzF96LB8xvCHRWoc emkw== X-Forwarded-Encrypted: i=1; AFNElJ+XIqOM4+SQk/dRp8q1LsNom3JaDT3I1fAgllCj7r3x8DkH8NxdRVVwELWqiAMWBtRSK4Kj9CE3JWukU1U=@vger.kernel.org X-Gm-Message-State: AOJu0Yy9CW5ut8OxDePBSfUVZ7yq0hjMTDPdIE1CvmifrfqevrVCSB4i N01qj0YeJnbeVhdfu5klqnv0aoPWw5m1QOZcAtqIoROz/xnBr6oAUWGifDcCSpUkby24TTXxxcv d3dLx1g== X-Received: from pls13.prod.google.com ([2002:a17:903:448d:b0:2bd:93e0:bc08]) (user=ctshao job=prod-delivery.src-stubby-dispatcher) by 2002:a17:903:1245:b0:2bd:8c9a:a684 with SMTP id d9443c01a7336-2beb069c63cmr4169555ad.4.1779394511286; Thu, 21 May 2026 13:15:11 -0700 (PDT) Date: Thu, 21 May 2026 13:15:04 -0700 In-Reply-To: <20260521201505.124690-1-ctshao@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260521201505.124690-1-ctshao@google.com> X-Mailer: git-send-email 2.54.0.746.g67dd491aae-goog Message-ID: <20260521201505.124690-2-ctshao@google.com> Subject: [PATCH v3 1/2] perf stat: Add aggr_nr metric parser support From: Chun-Tse Shao To: peterz@infradead.org, mingo@redhat.com, acme@kernel.org, namhyung@kernel.org Cc: mark.rutland@arm.com, alexander.shishkin@linux.intel.com, jolsa@kernel.org, irogers@google.com, adrian.hunter@intel.com, james.clark@linaro.org, sandipan.das@amd.com, leo.yan@arm.com, thomas.falcon@intel.com, yang.lee@linux.alibaba.com, linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, Chun-Tse Shao Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Introduce the `aggr_nr` function to the metric expression parser. `aggr_nr` allows metric formulas to dynamically utilize the number of aggregated targets (`aggr->nr`) instead of relying on the static `source_count` (which represents the static socket or node count). This adds the `AGGR_NR` token to the lexer and parser, updates the expression parsing context helpers to store `aggr_nr`, and feeds `aggr->nr` from the aggregation structure in `prepare_metric`. Signed-off-by: Chun-Tse Shao Assisted-by: Gemini:gemini-3.1-pro-preview --- tools/perf/util/expr.c | 26 ++++++++++++++++++++++---- tools/perf/util/expr.h | 6 +++++- tools/perf/util/expr.l | 1 + tools/perf/util/expr.y | 24 +++++++++++++++++------- tools/perf/util/stat-shadow.c | 6 +++++- 5 files changed, 50 insertions(+), 13 deletions(-) diff --git a/tools/perf/util/expr.c b/tools/perf/util/expr.c index 644769e92708..232998fef72b 100644 --- a/tools/perf/util/expr.c +++ b/tools/perf/util/expr.c @@ -27,6 +27,7 @@ struct expr_id_data { struct { double val; int source_count; + int aggr_nr; } val; struct { double val; @@ -151,8 +152,8 @@ int expr__add_id_val(struct expr_parse_ctx *ctx, const = char *id, double val) } /* Caller must make sure id is allocated */ -int expr__add_id_val_source_count(struct expr_parse_ctx *ctx, const char *= id, - double val, int source_count) +int expr__add_id_val_source_count_aggr_nr(struct expr_parse_ctx *ctx, cons= t char *id, + double val, int source_count, int aggr_nr) { struct expr_id_data *data_ptr =3D NULL, *old_data =3D NULL; char *old_key =3D NULL; @@ -163,6 +164,7 @@ int expr__add_id_val_source_count(struct expr_parse_ctx= *ctx, const char *id, return -ENOMEM; data_ptr->val.val =3D val; data_ptr->val.source_count =3D source_count; + data_ptr->val.aggr_nr =3D aggr_nr; data_ptr->kind =3D EXPR_ID_DATA__VALUE; ret =3D hashmap__set(ctx->ids, id, data_ptr, &old_key, &old_data); @@ -171,12 +173,20 @@ int expr__add_id_val_source_count(struct expr_parse_c= tx *ctx, const char *id, } else if (old_data) { data_ptr->val.val +=3D old_data->val.val; data_ptr->val.source_count +=3D old_data->val.source_count; + data_ptr->val.aggr_nr +=3D old_data->val.aggr_nr; } free(old_key); free(old_data); return ret; } +/* Caller must make sure id is allocated */ +int expr__add_id_val_source_count(struct expr_parse_ctx *ctx, const char *= id, + double val, int source_count) +{ + return expr__add_id_val_source_count_aggr_nr(ctx, id, val, source_count, = 1); +} + int expr__add_ref(struct expr_parse_ctx *ctx, struct metric_ref *ref) { struct expr_id_data *data_ptr =3D NULL, *old_data =3D NULL; @@ -390,8 +400,16 @@ double expr_id_data__value(const struct expr_id_data *= data) double expr_id_data__source_count(const struct expr_id_data *data) { - assert(data->kind =3D=3D EXPR_ID_DATA__VALUE); - return data->val.source_count; + if (data->kind =3D=3D EXPR_ID_DATA__VALUE) + return data->val.source_count; + return 1.0; +} + +double expr_id_data__aggr_nr(const struct expr_id_data *data) +{ + if (data->kind =3D=3D EXPR_ID_DATA__VALUE) + return data->val.aggr_nr; + return 1.0; } double expr__get_literal(const char *literal, const struct expr_scanner_ct= x *ctx) diff --git a/tools/perf/util/expr.h b/tools/perf/util/expr.h index c0cec29ddc29..ed12e4007d2d 100644 --- a/tools/perf/util/expr.h +++ b/tools/perf/util/expr.h @@ -36,7 +36,9 @@ void expr__del_id(struct expr_parse_ctx *ctx, const char = *id); int expr__add_id(struct expr_parse_ctx *ctx, const char *id); int expr__add_id_val(struct expr_parse_ctx *ctx, const char *id, double va= l); int expr__add_id_val_source_count(struct expr_parse_ctx *ctx, const char *= id, - double val, int source_count); + double val, int source_count); +int expr__add_id_val_source_count_aggr_nr(struct expr_parse_ctx *ctx, cons= t char *id, + double val, int source_count, int aggr_nr); int expr__add_ref(struct expr_parse_ctx *ctx, struct metric_ref *ref); int expr__get_id(struct expr_parse_ctx *ctx, const char *id, struct expr_id_data **data); @@ -53,6 +55,8 @@ int expr__find_ids(const char *expr, const char *one, double expr_id_data__value(const struct expr_id_data *data); double expr_id_data__source_count(const struct expr_id_data *data); +double expr_id_data__aggr_nr(const struct expr_id_data *data); + double expr__get_literal(const char *literal, const struct expr_scanner_ct= x *ctx); double expr__has_event(const struct expr_parse_ctx *ctx, bool compute_ids,= const char *id); double expr__strcmp_cpuid_str(const struct expr_parse_ctx *ctx, bool compu= te_ids, const char *id); diff --git a/tools/perf/util/expr.l b/tools/perf/util/expr.l index a2fc43159ee9..f16ccff278d0 100644 --- a/tools/perf/util/expr.l +++ b/tools/perf/util/expr.l @@ -121,6 +121,7 @@ min { return MIN; } if { return IF; } else { return ELSE; } source_count { return SOURCE_COUNT; } +aggr_nr { return AGGR_NR; } has_event { return HAS_EVENT; } strcmp_cpuid_str { return STRCMP_CPUID_STR; } NaN { return nan_value(yyscanner); } diff --git a/tools/perf/util/expr.y b/tools/perf/util/expr.y index e364790babb5..e20f649354cf 100644 --- a/tools/perf/util/expr.y +++ b/tools/perf/util/expr.y @@ -41,7 +41,7 @@ int expr_lex(YYSTYPE * yylval_param , void *yyscanner); } ids; } -%token ID NUMBER MIN MAX IF ELSE LITERAL D_RATIO SOURCE_COUNT HAS_EVENT ST= RCMP_CPUID_STR EXPR_ERROR +%token ID NUMBER MIN MAX IF ELSE LITERAL D_RATIO SOURCE_COUNT AGGR_NR HAS_= EVENT STRCMP_CPUID_STR EXPR_ERROR %left MIN MAX IF %left '|' %left '^' @@ -87,8 +87,14 @@ static struct ids union_expr(struct ids ids1, struct ids= ids2) return result; } +enum expr_id_kind { + EXPR_ID_KIND__VALUE, + EXPR_ID_KIND__SOURCE_COUNT, + EXPR_ID_KIND__AGGR_NR, +}; + static struct ids handle_id(struct expr_parse_ctx *ctx, char *id, - bool compute_ids, bool source_count) + bool compute_ids, enum expr_id_kind kind) { struct ids result; @@ -101,9 +107,12 @@ static struct ids handle_id(struct expr_parse_ctx *ctx= , char *id, result.val =3D NAN; if (expr__resolve_id(ctx, id, &data) =3D=3D 0) { - result.val =3D source_count - ? expr_id_data__source_count(data) - : expr_id_data__value(data); + if (kind =3D=3D EXPR_ID_KIND__SOURCE_COUNT) + result.val =3D expr_id_data__source_count(data); + else if (kind =3D=3D EXPR_ID_KIND__AGGR_NR) + result.val =3D expr_id_data__aggr_nr(data); + else + result.val =3D expr_id_data__value(data); } result.ids =3D NULL; free(id); @@ -201,8 +210,9 @@ expr: NUMBER $$.val =3D $1; $$.ids =3D NULL; } -| ID { $$ =3D handle_id(ctx, $1, compute_ids, /*source_count=3D*/false)= ; } -| SOURCE_COUNT '(' ID ')' { $$ =3D handle_id(ctx, $3, compute_ids, /*sourc= e_count=3D*/true); } +| ID { $$ =3D handle_id(ctx, $1, compute_ids, EXPR_ID_KIND__VALUE); } +| SOURCE_COUNT '(' ID ')' { $$ =3D handle_id(ctx, $3, compute_ids, EXPR_ID= _KIND__SOURCE_COUNT); } +| AGGR_NR '(' ID ')' { $$ =3D handle_id(ctx, $3, compute_ids, EXPR_ID_KIN= D__AGGR_NR); } | HAS_EVENT '(' ID ')' { $$.val =3D expr__has_event(ctx, compute_ids, $3); diff --git a/tools/perf/util/stat-shadow.c b/tools/perf/util/stat-shadow.c index bc2d44df7baf..c17373bb0e1e 100644 --- a/tools/perf/util/stat-shadow.c +++ b/tools/perf/util/stat-shadow.c @@ -53,6 +53,7 @@ static int prepare_metric(struct perf_stat_config *config, for (i =3D 0; metric_events[i]; i++) { int source_count =3D 0, tool_aggr_idx; + int aggr_nr =3D 1; bool is_tool_time =3D tool_pmu__is_time_event(config, metric_events[i], &tool_aggr_idx); struct perf_stat_evsel *ps =3D metric_events[i]->stats; @@ -89,6 +90,7 @@ static int prepare_metric(struct perf_stat_config *config, */ val =3D NAN; source_count =3D 0; + aggr_nr =3D 0; } else { struct perf_stat_aggr *aggr =3D &ps->aggr[is_tool_time ? tool_aggr_idx : aggr_idx]; @@ -96,6 +98,7 @@ static int prepare_metric(struct perf_stat_config *config, if (aggr->counts.run =3D=3D 0) { val =3D NAN; source_count =3D 0; + aggr_nr =3D 0; } else { val =3D aggr->counts.val; if (is_tool_time) { @@ -104,13 +107,14 @@ static int prepare_metric(struct perf_stat_config *co= nfig, } if (!source_count) source_count =3D evsel__source_count(metric_events[i]); + aggr_nr =3D aggr->nr ?: 1; } } n =3D strdup(evsel__metric_id(metric_events[i])); if (!n) return -ENOMEM; - expr__add_id_val_source_count(pctx, n, val, source_count); + expr__add_id_val_source_count_aggr_nr(pctx, n, val, source_count, aggr_n= r); } for (int j =3D 0; metric_refs && metric_refs[j].metric_name; j++) { -- 2.54.0.746.g67dd491aae-goog From nobody Sun May 24 19:33:16 2026 Received: from mail-pl1-f201.google.com (mail-pl1-f201.google.com [209.85.214.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 286C130567F for ; Thu, 21 May 2026 20:15:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779394514; cv=none; b=G3NiyrVMG2nkzyHac5jLsIc/XdGQO0xSeJn4xDb0DBXJJUDzq0tFBXqxEHl9NYHK2QgqogW7XrxanfebA8dncqZbj87r6M+m/wn6V9M0v2na2fbSTXGou3yGuS6Ee+W72heXasbzhhYUirwiaiK5IDBQZwIjdeTnBA5d3O/07r0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779394514; c=relaxed/simple; bh=1nopkE7ABcqxCFr7tFm0Jk9AQ2twaOaENXt9n5Rfr/M=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=Va4n00+hOIWmQ9+L7mDYRAOx8Usg8Hp7hP9IG/T9CWcf2li10+BYA2D9ol687ULzwj9oDYyajppKuGEmdv0dJeExpmpGN1OiPBVbPWfII8gai4HJENZV1WQIJiFum+SbC9rBS4etaFI9RvpMgaKcGD/8F7xJ2vfGfiSnZNRYm2U= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--ctshao.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=R4/TGgOB; arc=none smtp.client-ip=209.85.214.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--ctshao.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="R4/TGgOB" Received: by mail-pl1-f201.google.com with SMTP id d9443c01a7336-2ba224c3ffdso157108545ad.0 for ; Thu, 21 May 2026 13:15:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1779394512; x=1779999312; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=CBnOYBVXmHtY2VWhsrZ8/NjRHULzNCf4oOIydFbNlGM=; b=R4/TGgOByuMfCaA0C9Q+NUKLOHFtV4JK9c6R8phxqbTq9Wn6GTFsD9bWmiN7YR9NL+ 1w/RvmoI6ft7mekEX7AhMwGBEUl91clTycyn9mUS+72j8fwmKQL1DrRPeuXGIxY8FETz zHcYIYVIZZh4Sdcjcv2uYNsWs+alD4sgVmB7ZaMkUHL/IYU5p0agv4NszACx+GTg615T A7B0eG4Mz8ZB1zIOHBHPjj9qafL85AEfPOeCf/pR5nHjA7fj6SqFvB9E6jb8oTMZ0A3r uIOI2UT2ggTdmiBi5z1+0H67k3RIsQfJxOu6pVsoaGaPm2zGZLHpcXGDE+45iC+kt1OW k8Pw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779394512; x=1779999312; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=CBnOYBVXmHtY2VWhsrZ8/NjRHULzNCf4oOIydFbNlGM=; b=InJYJsjNsKjgHYbsIRXRDTnMUVUOu0lfwUTNVU2tbspMRO4N/CdOchyXX8n9j+wpXU aeKMo/TMxXpllA8ONmz86zOGs3Ym2YBuJzEcovlFw8gJityTFNX1HF3/a2T7B1OeEdHM qDlJZCywikAzUWqBceOQwUnUPnB22KABJPKw9/lRAfX/kQuCAhjszeE65oeJVDINHF5w VABh9tdGfz9aI5XYlPO1c/foY4ReZEQBUrfzejQkIyxxBW31jCLA2I64AVUlU9s4bFZ9 gCUivOeDLyQUuw+3jsfHAaykih060BqtT0khFKqVP8M81zSXktrp7mI3nm3Wj+lh9p27 4PJg== X-Forwarded-Encrypted: i=1; AFNElJ9UR9Et4ZrHcRrBu9GYn8JBNyWitTwbBjWJYyrOqXSWZBcg9o+9bkrLagGLw0cKXjY5wTOb081abxx3lng=@vger.kernel.org X-Gm-Message-State: AOJu0YzKet1g8ZdJemNU6Ph/8YHXUV18mkA5p4nLPbdV1r6MrNu6QDv8 wV8ADCi9rdiCzg5LozKcVIHvGZZfTh99BUq4b3HkuUYNg8Xi4OfWskyj7unPRivB4KNUo2KcY73 4HZxbWg== X-Received: from plbjc8.prod.google.com ([2002:a17:903:25c8:b0:2bc:db13:463a]) (user=ctshao job=prod-delivery.src-stubby-dispatcher) by 2002:a17:902:f64e:b0:2b2:50bd:83b3 with SMTP id d9443c01a7336-2beb035c826mr4541675ad.10.1779394512273; Thu, 21 May 2026 13:15:12 -0700 (PDT) Date: Thu, 21 May 2026 13:15:05 -0700 In-Reply-To: <20260521201505.124690-1-ctshao@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260521201505.124690-1-ctshao@google.com> X-Mailer: git-send-email 2.54.0.746.g67dd491aae-goog Message-ID: <20260521201505.124690-3-ctshao@google.com> Subject: [PATCH v3 2/2] perf stat: Use aggr_nr scaling for Intel uncore miss latency metrics From: Chun-Tse Shao To: peterz@infradead.org, mingo@redhat.com, acme@kernel.org, namhyung@kernel.org Cc: mark.rutland@arm.com, alexander.shishkin@linux.intel.com, jolsa@kernel.org, irogers@google.com, adrian.hunter@intel.com, james.clark@linaro.org, sandipan.das@amd.com, leo.yan@arm.com, thomas.falcon@intel.com, yang.lee@linux.alibaba.com, linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, Chun-Tse Shao Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Update `metric.py` to support the new `aggr_nr` keyword in the python metric generator. Replace the usage of `source_count` with `aggr_nr` in `IntelMissLat` inside `intel_metrics.py` so that uncore latency metrics (like `lpm_miss_lat`) scale correctly on multi-socket and SNC systems when aggregated globally. Additionally, update the validation bypass logic in `CheckEveryEvent()` inside `metric.py` to whitelist 'cha' and 'uncore' events. This prevents validation failures when compiling metrics referencing these PMU-specific uncore events. Signed-off-by: Chun-Tse Shao Assisted-by: Gemini:gemini-3.1-pro-preview --- tools/perf/pmu-events/intel_metrics.py | 6 +++--- tools/perf/pmu-events/metric.py | 9 +++++++-- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/tools/perf/pmu-events/intel_metrics.py b/tools/perf/pmu-events= /intel_metrics.py index 52035433b505..d99c7dd43797 100755 --- a/tools/perf/pmu-events/intel_metrics.py +++ b/tools/perf/pmu-events/intel_metrics.py @@ -7,7 +7,7 @@ import os import re from typing import Optional from common_metrics import Cycles -from metric import (d_ratio, has_event, max, source_count, CheckPmu, Event, +from metric import (d_ratio, has_event, max, aggr_nr, CheckPmu, Event, JsonEncodeMetric, JsonEncodeMetricGroupDescriptions, Literal, LoadEvents, Metric, MetricConstraint, MetricG= roup, MetricRef, Select) @@ -674,10 +674,10 @@ def IntelMissLat() -> Optional[MetricGroup]: else: assert data_rd_loc_occ.name =3D=3D "UNC_CHA_TOR_OCCUPANCY.IA_MISS_= DRD_LOCAL", data_rd_loc_occ =20 - ticks_per_cha =3D ticks / source_count(data_rd_loc_ins) + ticks_per_cha =3D ticks / aggr_nr(data_rd_loc_ins) loc_lat =3D interval_sec * 1e9 * data_rd_loc_occ / \ (ticks_per_cha * data_rd_loc_ins) - ticks_per_cha =3D ticks / source_count(data_rd_rem_ins) + ticks_per_cha =3D ticks / aggr_nr(data_rd_rem_ins) rem_lat =3D interval_sec * 1e9 * data_rd_rem_occ / \ (ticks_per_cha * data_rd_rem_ins) return MetricGroup("lpm_miss_lat", [ diff --git a/tools/perf/pmu-events/metric.py b/tools/perf/pmu-events/metric= .py index ac582db785fc..a91ccb5977f0 100644 --- a/tools/perf/pmu-events/metric.py +++ b/tools/perf/pmu-events/metric.py @@ -93,7 +93,7 @@ def CheckEveryEvent(*names: str) -> None: name =3D name[:name.find(':')] elif '/' in name: name =3D name[:name.find('/')] - if any([name.startswith(x) for x in ['amd', 'arm', 'cpu', 'msr', 'po= wer']]): + if any([name.startswith(x) for x in ['amd', 'arm', 'cpu', 'msr', 'po= wer', 'cha', 'uncore']]): continue if name not in all_events_all_models: raise Exception(f"Is {name} a named json event?") @@ -576,6 +576,11 @@ def source_count(event: Event) -> Function: return Function('source_count', event) =20 =20 +def aggr_nr(event: Event) -> Function: + # pylint: disable=3Dinvalid-name + return Function('aggr_nr', event) + + def has_event(event: Event) -> Function: # pylint: disable=3Dredefined-builtin # pylint: disable=3Dinvalid-name @@ -762,7 +767,7 @@ def ParsePerfJson(orig: str) -> Expression: # Convert accidentally converted scientific notation constants back py =3D re.sub(r'([0-9]+)Event\(r"(e[0-9]*)"\)', r'\1\2', py) # Convert all the known keywords back from events to just the keyword - keywords =3D ['if', 'else', 'min', 'max', 'd_ratio', 'source_count', 'ha= s_event', 'strcmp_cpuid_str'] + keywords =3D ['if', 'else', 'min', 'max', 'd_ratio', 'source_count', 'ag= gr_nr', 'has_event', 'strcmp_cpuid_str'] for kw in keywords: py =3D re.sub(rf'Event\(r"{kw}"\)', kw, py) try: --=20 2.54.0.746.g67dd491aae-goog