From nobody Thu Dec 18 08:36:22 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 04630C71153 for ; Thu, 24 Aug 2023 04:15:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240125AbjHXEPc (ORCPT ); Thu, 24 Aug 2023 00:15:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60304 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240195AbjHXEOk (ORCPT ); Thu, 24 Aug 2023 00:14:40 -0400 Received: from mail-pj1-x1049.google.com (mail-pj1-x1049.google.com [IPv6:2607:f8b0:4864:20::1049]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C5C7D173F for ; Wed, 23 Aug 2023 21:14:31 -0700 (PDT) Received: by mail-pj1-x1049.google.com with SMTP id 98e67ed59e1d1-26f9107479bso3050629a91.1 for ; Wed, 23 Aug 2023 21:14:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20221208; t=1692850471; x=1693455271; h=to:from:subject:references:mime-version:message-id:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=TvLE3Spqr0cSOEv2xXiI7FCQdeqKmdObep26XyHFaEA=; b=0vO41xS+LjkPKHDjpVXscNfFqCSMB7ITBrNVNeAk4Paz3gQXB5fvmRpMFfRSHETaP2 aWU5eGIqYfFxKw+xRhKJowSWPm4if1XXtvaK8jHr3Gl2D0gCNqpCyRlphUWMW4iADo1T u7xoBk9UE9lhEz+7eAVJA1J3k8gyH0RBLnGFbPdHrsWFlfdSW1atQLY2nUSwXYTL6FsI 5cMZ65p7NE8Og8x8tBSXL5p8EoNthY/nwicaGIYjkFKWquqYC8FdAyc6wiGsTvUP8NDo 99j0fBniQII3hw9CiE9JObznBCYs2OstumIbMBXTR7I+QJFLl6ixvJ76b6bYEs0R4Kk0 XNnQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692850471; x=1693455271; h=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=TvLE3Spqr0cSOEv2xXiI7FCQdeqKmdObep26XyHFaEA=; b=ID42gAwngfGZMWDwXo5symXjD62rbZX0f9XDkKAjd+qtuwTtotCRVkb9OMcypln7+D gd97m61Xu5y4YlRCeT5XNdcFXCrUNkNuY2ReC1e0wkfVDPojlfnpDNdlDuc6C+RTCfxp 2UhrETSCNIZjl2vzNe8fvpWOWVhyoTZQk1avp0kHsmWed2RuvTypNKIGmp3Sw2graxc7 JFffZqtd0Eub4m7oXFphUtREVgD1UEx7Jy/V2pqtHhH+ulEk+nAfM2yaC8XJVl2/Vh2B b2pkfPItcuJFo0WCGftjv/BBt2khuau2DIC0dUU0iAyRf8VbRaFsrFHwGH9R1Fi67sKv locw== X-Gm-Message-State: AOJu0YyhneRRy6hCiYqasKhxezUTcbEBOFes+QIB363xys/gytuk/7qs KlPeDgs6vM/L1cxywO/kh/+a8a3TcyC9 X-Google-Smtp-Source: AGHT+IGPxDqYXAjpEC8kRChbVf9bd4e24xHLGbDbs7LLn6nrJmUfFHbxUsGqfUXPUGgYxtzWEGqbel03lsJ6 X-Received: from irogers.svl.corp.google.com ([2620:15c:2a3:200:fbb9:d9e7:7405:2651]) (user=irogers job=sendgmr) by 2002:a17:90a:d80c:b0:26c:fab1:9e23 with SMTP id a12-20020a17090ad80c00b0026cfab19e23mr3745874pjv.0.1692850471230; Wed, 23 Aug 2023 21:14:31 -0700 (PDT) Date: Wed, 23 Aug 2023 21:13:30 -0700 In-Reply-To: <20230824041330.266337-1-irogers@google.com> Message-Id: <20230824041330.266337-19-irogers@google.com> Mime-Version: 1.0 References: <20230824041330.266337-1-irogers@google.com> X-Mailer: git-send-email 2.42.0.rc1.204.g551eb34607-goog Subject: [PATCH v2 18/18] perf jevents: Don't append Unit to desc From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Ian Rogers , Adrian Hunter , James Clark , Kan Liang , John Garry , Kajol Jain , Jing Zhang , Ravi Bangoria , Rob Herring , Gaosheng Cui , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Unit with the PMU name is appended to desc in jevents.py, but on hybrid platforms it causes the desc to differ from the regular non-hybrid system with a PMU of 'cpu'. Having differing descs means the events don't deduplicate. To make the perf list output not differ, append the Unit on again in the perf list printing code. On x86 reduces the binary size by 409,600 bytes or about 4%. Signed-off-by: Ian Rogers --- tools/perf/builtin-list.c | 13 ++++++++++++- tools/perf/pmu-events/jevents.py | 7 ------- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c index 7fec2cca759f..d8b9f606e734 100644 --- a/tools/perf/builtin-list.c +++ b/tools/perf/builtin-list.c @@ -145,9 +145,20 @@ static void default_print_event(void *ps, const char *= pmu_name, const char *topi putchar('\n'); =20 if (desc && print_state->desc) { + char *desc_with_unit =3D NULL; + int desc_len =3D -1; + + if (pmu_name && strcmp(pmu_name, "cpu")) { + desc_len =3D strlen(desc); + desc_len =3D asprintf(&desc_with_unit, + desc[desc_len - 1] !=3D '.' + ? "%s. Unit: %s" : "%s Unit: %s", + desc, pmu_name); + } printf("%*s", 8, "["); - wordwrap(desc, 8, pager_get_columns(), 0); + wordwrap(desc_len > 0 ? desc_with_unit : desc, 8, pager_get_columns(), 0= ); printf("]\n"); + free(desc_with_unit); } long_desc =3D long_desc ?: desc; if (long_desc && print_state->long_desc) { diff --git a/tools/perf/pmu-events/jevents.py b/tools/perf/pmu-events/jeven= ts.py index e5bce57f5688..712f80d7d071 100755 --- a/tools/perf/pmu-events/jevents.py +++ b/tools/perf/pmu-events/jevents.py @@ -357,13 +357,6 @@ class JsonEvent: self.desc +=3D extra_desc if self.long_desc and extra_desc: self.long_desc +=3D extra_desc - if self.pmu and self.pmu !=3D 'cpu': - if not self.desc: - self.desc =3D 'Unit: ' + self.pmu - else: - if not self.desc.endswith('. '): - self.desc +=3D '. ' - self.desc +=3D 'Unit: ' + self.pmu if arch_std: if arch_std.lower() in _arch_std_events: event =3D _arch_std_events[arch_std.lower()].event --=20 2.42.0.rc1.204.g551eb34607-goog