From nobody Sat Feb 7 12:57:11 2026 Received: from mail-yb1-f201.google.com (mail-yb1-f201.google.com [209.85.219.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 929F91A7249 for ; Wed, 8 Jan 2025 05:34:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736314493; cv=none; b=lDp1hhyULy9w3Td/bxll+6P+t4+ghOgFYtlw48Zq9+ai7GQ7BzNOgiLxKUOFBiKZcMh/8BwHFUagxbPAPgZ+EG2ZDaOJ1LR5XzDYFo/HuCTxMASuV0Y01tZt0gJKfgLc8UzXSnkoiM9uFCQkevSvrNIUAcNIEyiBgG4nO2gF/4M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736314493; c=relaxed/simple; bh=pU3q7gcfHF5YJFJO82z2GpY1D+Ad/Z/0AoIS1WPrJj4=; h=Date:In-Reply-To:Message-Id:Mime-Version:References:Subject:From: To:Content-Type; b=eOXtp7HarLyLomMH8cQNGyExPLcc1ntarZb3O6HbLpagcBU7vSlzjMt6K+wKDDkpiyoorcSyCJiflXVtnFRYuNREl7GpCcg+iLq9/3XHYLjDJedSMB3bvLZ1GPRD2lViEeq1kLFQNN/8V/AY/fiH415r4Tps7avwvUWoV9aAmAg= 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=HOpGT0Ov; arc=none smtp.client-ip=209.85.219.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--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="HOpGT0Ov" Received: by mail-yb1-f201.google.com with SMTP id 3f1490d57ef6-e3a0f608b88so32912022276.0 for ; Tue, 07 Jan 2025 21:34:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1736314490; x=1736919290; darn=vger.kernel.org; h=to:from:subject:references:mime-version:message-id:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=0FF1+dR5Lo7oMmZmNU3Ip/TUs1XkJ0WZAvmFs9ECk2g=; b=HOpGT0OvHQT9L23a61RD+HaqCJ8bwWC4uAb4zQMtLWMWqB9t/ZF8BxufgVg7xsazy5 4A6y65BvOam+yaumHaYrH4WzIgaDh/YQ+pLt3oHzNwLEYRV4s7bFCCBd2MzJL1qZw079 eJVVEKQuyUUl5sHCeisjoECFxfSOcF9BqS3sYKuSEWRaSzEVcXZIHMNcgH0MhQVR/VZD PrHJ2o7xv/6ITxVGqUxzyKVzHPNdT0fKEJyDkYMe6hjlPtAirLXsZtOZtLSVuRi+zKA6 vXYhVhLYrWoefF1IXYoOTLJw+JjC3qc5W2YITyPaemT9ANk2rINvHy01DWhAVyiAQRS+ zI4w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736314490; x=1736919290; 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=0FF1+dR5Lo7oMmZmNU3Ip/TUs1XkJ0WZAvmFs9ECk2g=; b=jW4GyoJ+Rq1rmVbM3TYQuFfoUw4W3Ak4X25F2IWn0khrtwnV12KCWbffpfvtp0xoCc kYKI8dv74fEMJDOr43tqdd+wd/ulDybeVqMDW4UEOXoOIQBowGjI0OSGGfOI3mm/vtp1 BLl4Vg7seTF+ZdiILElr5o4qhvC1XLz+rf1rkjomhV8F70mj71roLVInPrkssmljhRkU CGTsI0tm1NgARBRaNmkcGAL2Elr2ZurtBwztI0S4EWpCGbUZlUUvhSkz7cVKvG+Zv8Me FCI7HeSAtH9A0iuwiLGz+PKuccXpMJy7fL9H/8LXwrpGaF8WJzBEKZtbrrAM7GDQCaR+ 0/jA== X-Forwarded-Encrypted: i=1; AJvYcCWZXK+u2lZQgcvX1eKWgBD1RZdr/SW1gTZzEPPFwzJ7suCXjU383IXh5OdOrmcP9hcV/lZUOHxRtTVkYyc=@vger.kernel.org X-Gm-Message-State: AOJu0YxW2ZhlTCEliTQ8NRXz6RjTilXpjUWR8S3mJj3yqNEYpYROXpqd 1L7HBtMMNmJDw5B3UDqb5X9UYJGjnwfqdt1oitCS/pcPwe9ga92YmizICeEM/qiCzRTidNy13TI U30FvMA== X-Google-Smtp-Source: AGHT+IEUKIlCLJcb4EXajotMSIYDCcsHbs9Nh5LAlws/3NJnHcyVabzgEL04vNJh/CrGAixRszW7Ydk2FO36 X-Received: from irogers.svl.corp.google.com ([2620:15c:2c5:11:7ae:a887:5504:e3ca]) (user=irogers job=sendgmr) by 2002:a0d:dc46:0:b0:6ee:3fb4:45b9 with SMTP id 00721157ae682-6f5312b4c37mr41297b3.4.1736314490615; Tue, 07 Jan 2025 21:34:50 -0800 (PST) Date: Tue, 7 Jan 2025 21:34:27 -0800 In-Reply-To: <20250108053428.1724490-1-irogers@google.com> Message-Id: <20250108053428.1724490-4-irogers@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20250108053428.1724490-1-irogers@google.com> X-Mailer: git-send-email 2.47.1.613.gc27f4b7a9f-goog Subject: [PATCH v4 3/4] perf parse-events: Set is_pmu_core for legacy hardware events From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , Kan Liang , Leo Yan , Yoshihiro Furudera , Weilin Wang , Andi Kleen , James Clark , Dominique Martinet , Yicong Yang , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Also set the CPU map to all online CPU maps. This is done so the behavior of legacy hardware and hardware cache events better matches that of sysfs and json events during __perf_evlist__propagate_maps. Fix missing cpumap put in "Synthesize attr update" test. Signed-off-by: Ian Rogers --- tools/perf/tests/event_update.c | 1 + tools/perf/util/parse-events.c | 37 ++++++++++++++++++++------------- 2 files changed, 24 insertions(+), 14 deletions(-) diff --git a/tools/perf/tests/event_update.c b/tools/perf/tests/event_updat= e.c index d6b4ce3ef4ee..9301fde11366 100644 --- a/tools/perf/tests/event_update.c +++ b/tools/perf/tests/event_update.c @@ -109,6 +109,7 @@ static int test__event_update(struct test_suite *test _= _maybe_unused, int subtes TEST_ASSERT_VAL("failed to synthesize attr update name", !perf_event__synthesize_event_update_name(&tmp.tool, evsel, process_eve= nt_name)); =20 + perf_cpu_map__put(evsel->core.own_cpus); evsel->core.own_cpus =3D perf_cpu_map__new("1,2,3"); =20 TEST_ASSERT_VAL("failed to synthesize attr update cpus", diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index 1e23faa364b1..4bda4141e9e7 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c @@ -231,21 +231,30 @@ __add_event(struct list_head *list, int *idx, struct perf_cpu_map *cpu_list, u64 alternate_hw_config) { struct evsel *evsel; - struct perf_cpu_map *cpus =3D perf_cpu_map__is_empty(cpu_list) && pmu ? p= mu->cpus : cpu_list; + bool is_pmu_core; + struct perf_cpu_map *cpus; =20 - cpus =3D perf_cpu_map__get(cpus); - if (pmu) + if (pmu) { + is_pmu_core =3D pmu->is_core; + cpus =3D perf_cpu_map__get(perf_cpu_map__is_empty(cpu_list) ? pmu->cpus = : cpu_list); perf_pmu__warn_invalid_formats(pmu); - - if (pmu && (attr->type =3D=3D PERF_TYPE_RAW || attr->type >=3D PERF_TYPE_= MAX)) { - perf_pmu__warn_invalid_config(pmu, attr->config, name, - PERF_PMU_FORMAT_VALUE_CONFIG, "config"); - perf_pmu__warn_invalid_config(pmu, attr->config1, name, - PERF_PMU_FORMAT_VALUE_CONFIG1, "config1"); - perf_pmu__warn_invalid_config(pmu, attr->config2, name, - PERF_PMU_FORMAT_VALUE_CONFIG2, "config2"); - perf_pmu__warn_invalid_config(pmu, attr->config3, name, - PERF_PMU_FORMAT_VALUE_CONFIG3, "config3"); + if (attr->type =3D=3D PERF_TYPE_RAW || attr->type >=3D PERF_TYPE_MAX) { + perf_pmu__warn_invalid_config(pmu, attr->config, name, + PERF_PMU_FORMAT_VALUE_CONFIG, "config"); + perf_pmu__warn_invalid_config(pmu, attr->config1, name, + PERF_PMU_FORMAT_VALUE_CONFIG1, "config1"); + perf_pmu__warn_invalid_config(pmu, attr->config2, name, + PERF_PMU_FORMAT_VALUE_CONFIG2, "config2"); + perf_pmu__warn_invalid_config(pmu, attr->config3, name, + PERF_PMU_FORMAT_VALUE_CONFIG3, "config3"); + } + } else { + is_pmu_core =3D (attr->type =3D=3D PERF_TYPE_HARDWARE || + attr->type =3D=3D PERF_TYPE_HW_CACHE); + if (perf_cpu_map__is_empty(cpu_list)) + cpus =3D is_pmu_core ? perf_cpu_map__new_online_cpus() : NULL; + else + cpus =3D perf_cpu_map__get(cpu_list); } if (init_attr) event_attr_init(attr); @@ -260,7 +269,7 @@ __add_event(struct list_head *list, int *idx, evsel->core.cpus =3D cpus; evsel->core.own_cpus =3D perf_cpu_map__get(cpus); evsel->core.requires_cpu =3D pmu ? pmu->is_uncore : false; - evsel->core.is_pmu_core =3D pmu ? pmu->is_core : false; + evsel->core.is_pmu_core =3D is_pmu_core; evsel->auto_merge_stats =3D auto_merge_stats; evsel->pmu =3D pmu; evsel->alternate_hw_config =3D alternate_hw_config; --=20 2.47.1.613.gc27f4b7a9f-goog