From nobody Sat Jun 20 20:01:34 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4F4FE39526B; Fri, 10 Apr 2026 10:48:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775818108; cv=none; b=i0mM+VdKA6GoOQsRjfiDHZytzHB6nyZeGjl+O4j+QTXWR3FwTBUm2xGJnDWEgCPWAUx0Z2NTfOvHH3v1kKHzPGDOcoYQLbUHIFA7kroqLa2X+Fkufq6T7cTMuj8FT30xw7Ug0haRzQwIwcmk9VQ3iwyc7uEtVLmuPSoB7tQJo2c= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775818108; c=relaxed/simple; bh=hbXKO00OJuPDH9R1tHVKV31KSgItAizLb5IlUYtL0jg=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=Zm4r9kzOfXAkx+tLG21+LswGE4bm9jfu35y3xhpg4GbF64rj4ZCMfdmzb+zhmege8rG5cayaYANYxgCUqakqSwuEva8NvfvKIWT8kakBbL2zSLW2Nosu2RNZLrrWECkyzdV2X1zAPMCXVRFI7voNiwloLr0wTAtCHL5riOsTTwc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=lxroWHh5; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="lxroWHh5" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C962E2682; Fri, 10 Apr 2026 03:48:18 -0700 (PDT) Received: from e132581.arm.com (e132581.arm.com [10.1.196.87]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 474173FAF5; Fri, 10 Apr 2026 03:48:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1775818104; bh=hbXKO00OJuPDH9R1tHVKV31KSgItAizLb5IlUYtL0jg=; h=From:Date:Subject:To:Cc:From; b=lxroWHh5S7XF9QcfZxly93UYxg0htbGizDAw91W+YJpRIqJKQW7c8K1RPtwBkt9Xf fhlrabkcK0SCG21cmI5OOIfPl2sVJyHcbUCGJ3TGYLk1bor65FUM3fZyoe91gi1Dt3 JrS3gdCE1c4VfbjDRjn2hRo9FOLX/iBJiD47s6ls= From: Leo Yan Date: Fri, 10 Apr 2026 11:48:03 +0100 Subject: [PATCH] perf pmu: Skip test on Arm64 when #slots is zero Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260410-perf_fix_pmu_metrics_test-v1-1-18a5d80f71b6@arm.com> X-B4-Tracking: v=1; b=H4sIAGLV2GkC/42NQQ6CMBBFr0JmbU1bwaIr72FIg+0gsygl00o0h LtbOYHL9/P/+yskZMIE12oFxoUSxamAOlTgxn56oiBfGLTUZ1nLVszIgx3obefwsgEzk0s2Y8r CnFzt0bu2aRso+5mx9Hb3vSs8UsqRP/vVon7pP9ZFCSWkMbp5oNQX4289h6OLAbpt276nhx13w AAAAA== X-Change-ID: 20260408-perf_fix_pmu_metrics_test-73c4dedc8585 To: Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark Cc: linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, Leo Yan X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1775818103; l=3130; i=leo.yan@arm.com; s=20250604; h=from:subject:message-id; bh=hbXKO00OJuPDH9R1tHVKV31KSgItAizLb5IlUYtL0jg=; b=v4udSErekDWLBprs6+pXK45BtDpDWQL0hSQ7LUXrLXXNyIDbKXjgUfm6DsaR7jt1L1t8SCfcu fZ7LdGv36sqBhCx/J0sN2/fYWfVvjyRT3xZOON/6XzBERcppyQ9utWI X-Developer-Key: i=leo.yan@arm.com; a=ed25519; pk=k4BaDbvkCXzBFA7Nw184KHGP5thju8lKqJYIrOWxDhI= Some Arm64 PMUs expose 'caps/slots' as 0 when the slot count is not implemented, tool_pmu__read_event() currently treats that as a read failure, so metrics that reference #slots are reported as syntax error. Since the commit 3a61fd866ef9 ("perf expr: Return -EINVAL for syntax error in expr__find_ids()"), these syntax errors are populated as failures and make the PMU metric test fail: 9.3: Parsing of PMU event table metrics: --- start --- ... Found metric 'backend_bound' metric expr 100 * (stall_slot_backend / (#slots * cpu_cycles)) for back= end_bound parsing metric: 100 * (stall_slot_backend / (#slots * cpu_cycles)) Failure to read '#slots' literal: #slots =3D nan syntax error Fail to parse metric or group `backend_bound' ... ---- end(-1) ---- 9.3: Parsing of PMU event table metrics : FAILED! This commit introduces a new function is_expected_broken_metric() to identify broken metrics, and treats metrics containing "#slots" as broken when #slots =3D=3D 0 on Arm64 platforms. Fixes: 3a61fd866ef9 ("perf expr: Return -EINVAL for syntax error in expr__f= ind_ids()") Signed-off-by: Leo Yan --- tools/perf/tests/pmu-events.c | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/tools/perf/tests/pmu-events.c b/tools/perf/tests/pmu-events.c index a997168621688007c85495e2d9f6f459c2471516..6a68d46a4fcd96b7ce46272af36= fcb5590939505 100644 --- a/tools/perf/tests/pmu-events.c +++ b/tools/perf/tests/pmu-events.c @@ -15,6 +15,7 @@ #include "util/expr.h" #include "util/hashmap.h" #include "util/parse-events.h" +#include "util/tool_pmu.h" #include "metricgroup.h" #include "stat.h" =20 @@ -817,6 +818,26 @@ struct metric { struct metric_ref metric_ref; }; =20 +static bool is_expected_broken_metric(const struct pmu_metric *pm) +{ + if (!strcmp(pm->metric_name, "M1") || !strcmp(pm->metric_name, "M2") || + !strcmp(pm->metric_name, "M3")) + return true; + +#if defined(__aarch64__) + /* + * Arm64 platforms may return =E2=80=9C#slots =3D=3D 0=E2=80=9D, which is= treated as a + * syntax error by the parser. Don't test these metrics when running + * on such platforms. + */ + if (!strstr(pm->metric_name, "#slots") && + !tool_pmu__cpu_slots_per_cycle()) + return true; +#endif + + return false; +} + static int test__parsing_callback(const struct pmu_metric *pm, const struct pmu_metrics_table *table, void *data) @@ -852,8 +873,7 @@ static int test__parsing_callback(const struct pmu_metr= ic *pm, =20 err =3D metricgroup__parse_groups_test(evlist, table, pm->metric_name); if (err) { - if (!strcmp(pm->metric_name, "M1") || !strcmp(pm->metric_name, "M2") || - !strcmp(pm->metric_name, "M3")) { + if (is_expected_broken_metric(pm)) { (*failures)--; pr_debug("Expected broken metric %s skipping\n", pm->metric_name); err =3D 0; --- base-commit: 4cf1f549bbcdfea9c20df52994bb342677472dcd change-id: 20260408-perf_fix_pmu_metrics_test-73c4dedc8585 Best regards, --=20 Leo Yan