From nobody Sun Feb 8 17:36:26 2026 Received: from mail-pj1-f73.google.com (mail-pj1-f73.google.com [209.85.216.73]) (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 EF96934AB12 for ; Fri, 24 Oct 2025 17:59:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.73 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761328772; cv=none; b=qnUsH4/tVobuddV4i+8mZ+Ckncgp/0EkHFzkbafvg/0IrO+LVlr+B/hlNfdhsu0U+cezUOKSVKootn3TLL56jkgjC8GkRw/b0sYOvjJSPw8R3aHLTeVYm3q+wcWu+i+6P/lj65kDAOdiRhYJzYTLHUCaXvpeRiCH/zOjC0fynCI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761328772; c=relaxed/simple; bh=0dW3EhwozZJdK9GyY7uSSqkQ3MrxnQx4/U/UjD7W3x0=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=HibfsCwcEcd/6dwMXxPWijFVyfa9SaRAzHWyh3o+aVUjJiY+UGNDeJrl6Pom6xE61ClzH0CgrACtoreOph6TyFZV20WHDMl/LEEtb7rxrEbHJCoQiA+DTCv4wEZ6w97tx47NTraeW6k95YpAYuTPTyK1/GBegIkUrIUDBqCMLus= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=T4uaIg2w; arc=none smtp.client-ip=209.85.216.73 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--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="T4uaIg2w" Received: by mail-pj1-f73.google.com with SMTP id 98e67ed59e1d1-33d75897745so4822629a91.0 for ; Fri, 24 Oct 2025 10:59:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1761328769; x=1761933569; darn=vger.kernel.org; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=Bmup4BIbkScoJHl2u6fwJbLlsa89q7qgBXlzEQqW1es=; b=T4uaIg2wrLxeRGdGaZp6X07cXDZHdzSqP/0sxkL3VBRIwD6gmgjQ7NTEis+KHeijI4 OaJebFqgQh1ln6IBRUrJwlm7TBtDk11sjPx11ohjL73aOFLnjHcQak9RzFdy1LUSY7rG 42r4jg50rFBODbx8zdgcfP2W2z5rnEOYs6whfT3XmRoJynQw+X3DDEMP9dHaZOCVRIyd Gv/lOErMOcczeyA1zG51j+s+tEYlc4aBrf65MX84bnE8i2mpG1+cDOtAVYI5BVmEnUWS I3PuCWiScx9gzNne+fxRCbfhpT5AHAqJjTPI7C51OAa8mM4kgQ1iDCBn6yRnvRYxbFNO nPzw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761328769; x=1761933569; h=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=Bmup4BIbkScoJHl2u6fwJbLlsa89q7qgBXlzEQqW1es=; b=WgfbBrq9KSPY2baXZiGu7oaPFjO3zYc/oYc/aViKQjPkkZjMy5d2EH+aJsAP1qBzCa vxAqyDvdX9rcKr2xF/XY/ckk5kbIoEjJmVRM8cShn0P2/TU8MSmZJCrvPjULo/a5xsxa Io1lPSycAlyIojyEXAiSDTLWceboqxmvzG5F2A+Vijfz5qiNh0dr+SlpfGVKrFrwF4yN gcUe7PkajEYc7E5KOCORPLEXWGpP1T4Ju0539z7pYmHP30Hv7/rIYSrDrJOes1s5ThXv CTpQgtUTNGZhm2Jh3Sek81YD3QZw5Qi5DiJ6hyD2sh6fKq7DKoi8R8PRT0jqMaVp1sxW 523A== X-Forwarded-Encrypted: i=1; AJvYcCX54NSRMKA6yhO+3WUso7qXEKMK3L+qq/KK1bnCnhbPtA5xjuVTgygb6I0SBDsxTzxR+QqDJSU58QqTi50=@vger.kernel.org X-Gm-Message-State: AOJu0Yy2fXpImQZWbZvQmreN+09e99B+i/SbAsT3cW9yY0lv1qdm1LRM yw0aAoH7DI1Ma9fxun2L3pAHi7rzwxrL6B3/GhFcFT4/aVKIYQ7R+QqpxbLlIqGLIUMDHITuvGg BuZYomdFi+A== X-Google-Smtp-Source: AGHT+IF8VrmJgUHc+N22FHVZeIKm9/dVMTRLTdCjZgNklBkTzpwDuzOlB+B1PvJ5QqZTPRwuwT2BrDja295a X-Received: from pjob9.prod.google.com ([2002:a17:90a:8c89:b0:33b:8aa1:75ed]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:3852:b0:33b:8ac4:1ac4 with SMTP id 98e67ed59e1d1-33bcf91c61dmr41786364a91.35.1761328769046; Fri, 24 Oct 2025 10:59:29 -0700 (PDT) Date: Fri, 24 Oct 2025 10:58:48 -0700 In-Reply-To: <20251024175857.808401-1-irogers@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251024175857.808401-1-irogers@google.com> X-Mailer: git-send-email 2.51.1.821.gb6fe4d2222-goog Message-ID: <20251024175857.808401-14-irogers@google.com> Subject: [PATCH v1 13/22] perf stat: Fix default metricgroup display on hybrid From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark , Xu Yang , Chun-Tse Shao , Thomas Richter , Sumanth Korikkar , Collin Funk , Thomas Falcon , Howard Chu , Dapeng Mi , Levi Yun , Yang Li , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The logic to skip output of a default metric line was firing on Alderlake and not displaying 'TopdownL1 (cpu_atom)'. Remove the need_full_name check as it is equivalent to the different PMU test in the cases we care about, merge the 'if's and flip the evsel of the PMU test. The 'if' is now basically saying, if the output matches the last printed output then skip the output. Before: ``` TopdownL1 (cpu_core) # 11.3 % tma_bad_spe= culation # 24.3 % tma_fronten= d_bound TopdownL1 (cpu_core) # 33.9 % tma_backend= _bound # 30.6 % tma_retiring # 42.2 % tma_backend= _bound # 25.0 % tma_fronten= d_bound (49.81%) # 12.8 % tma_bad_spe= culation # 20.0 % tma_retirin= g (59.46%) ``` After: ``` TopdownL1 (cpu_core) # 8.3 % tma_bad_spe= culation # 43.7 % tma_fronten= d_bound # 30.7 % tma_backend= _bound # 17.2 % tma_retiring TopdownL1 (cpu_atom) # 31.9 % tma_backend= _bound # 37.6 % tma_fronten= d_bound (49.66%) # 18.0 % tma_bad_spe= culation # 12.6 % tma_retirin= g (59.58%) ``` Signed-off-by: Ian Rogers --- tools/perf/util/stat-shadow.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tools/perf/util/stat-shadow.c b/tools/perf/util/stat-shadow.c index afbc49e8cb31..c1547128c396 100644 --- a/tools/perf/util/stat-shadow.c +++ b/tools/perf/util/stat-shadow.c @@ -256,11 +256,9 @@ static void perf_stat__print_metricgroup_header(struct= perf_stat_config *config, * event. Only align with other metics from * different metric events. */ - if (last_name && !strcmp(last_name, name)) { - if (!need_full_name || last_pmu !=3D evsel->pmu) { - out->print_metricgroup_header(config, ctxp, NULL); - return; - } + if (last_name && !strcmp(last_name, name) && last_pmu =3D=3D evsel->pmu) { + out->print_metricgroup_header(config, ctxp, NULL); + return; } =20 if (need_full_name && evsel->pmu) --=20 2.51.1.821.gb6fe4d2222-goog