From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (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 ACAB830E0D9 for ; Wed, 10 Sep 2025 11:14:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757502856; cv=none; b=rpZp3AtpfcvRj2Mo+Z9fuRnKBhIjwXDyAyGCv/73bsG4d7P5jFJArfgaoutjUTprYktDfj7iKBekXKoL6AZx1998r24390gowiPft8yUogJT5wCk0e5eXV0dtc+EbA7FeUfAnn67AZKg6o2MriwQ0AY0AqCsd4PygoN8aaNH0kQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757502856; c=relaxed/simple; bh=4OBZ2ZfKsExYLPiXZhnyp4Ky5O+dJ7H0S4S55ZV3fVo=; h=Message-ID:Date:MIME-Version:Subject:To:References:From:Cc: In-Reply-To:Content-Type; b=arPywQRKQKOVMMRl/PEVLOwe7lAMwZJuJU3RA4EsS/Mv9l/tORPhD86ybqoNMSD1L+/osJita2hBZsBItbdomx6Yqq5wYE5h5Mv/9tvRLRgbA4h0CZvRP2r0hOvuuFL6xFXxtqkE//+jmhqctlza8H1u+Eb7h4AdqTe7evv3PeM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=cbhzhbfA; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="cbhzhbfA" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-45cb5492350so44095975e9.1 for ; Wed, 10 Sep 2025 04:14:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1757502853; x=1758107653; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:cc:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=U3OQu9v7444v/noKKraVi4jc91DRs3isyorMMbC0SUY=; b=cbhzhbfALaQnnWzjVxOUJel//fGecYdcGBFeUwvaOTeyowf26l8mQOS8fyyeZ6h8kO ixUxTNzm7m0VqhL70rAgSAg3fz+HMsjZY6w8eA6EHe4+kkJwhksvrXbxta3h3JLzysSY F6ROmmjY1S6Cug2sEVTTQRn9BWFF2H8ZWen0RKuD/Ch0HZ+RWnYTTNB1709cB3PwrBmp YORw7OchwmTpkiI9SijTxi25KTiwFhsY6KMAW8xHaximR7Quli3Tg/pNBxvgFKx4H1pX tNDQ4+CoYJO8+jazN817aVSsUiUyqL0uO5r9YPXYgLSZWVQFe94eXFguxegOtaiWjHPz m9Rw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757502853; x=1758107653; h=content-transfer-encoding:in-reply-to:cc:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=U3OQu9v7444v/noKKraVi4jc91DRs3isyorMMbC0SUY=; b=jKe364IOA4T3mhJjkN82HqHN0PwtbuTeoNMLQJU7f1JMFeZU2H9Jpd6IZpqqJzQbuc QM7ERH5bowuUnhZeF96btxU5lw+Gls7fr4Ri2uaW26a3IhZXTfquXpbInSAVzr9Af+KK V8BhpzRBxcU+Ra6ngU6tcHVshMg0zZLlzxujOE2uiE+6i9aJEzluKYATahXevrqBzS1X RFxA1mhgEkyNIqL/9Gc/Kq35Bf/qi73UJiFB51qkKaPMaJRFh6B/vvu5vl5SaOPf7ugW SBf/voMKY9nAYBm6HURq6ZNdA3C6xM1XQQ3Sv0D22326hRguz6FGXbe6O3pDUxRYBUuQ hBcw== X-Forwarded-Encrypted: i=1; AJvYcCWZjXu2vab8nLeSJNObwmyxmgIVktsuqOT0O1qnLmBtD+JdCshW+6Hvz8dJ7KlnugU2dFASJQ0Ku5FRbgw=@vger.kernel.org X-Gm-Message-State: AOJu0YwR17UbqXuYCizcT5l9OnKYSzihjrQd19GB2MovWWKo3INF0wQ/ 35nrWreCA+YlhbabPbiEe43IjyKPO5ZrRXoE2Y6c8C8RSt/vHT6SS+S7A20v0uskonM= X-Gm-Gg: ASbGncspAcfws8j9eY7Q7D3EKXSO9WLjNXO7VdS75abknaT60TrLKo/MT045Xb3eCiF 8jbeivzxBLPEfBvgR24UdLwdchsivdvOTHEwQYdcpMwq3steWIUAQDkkiVPK+fCDiCspYIhvx0c KXqnlLnidjoB7q+624zY8O0saQmE/rckbQVYRuJ1XEUEzFfsh7P7OgxjZ74xkTUK2BMhjNI1EVl d+RuPKRJ+Ek8uSkP2DkEX9mK+JRMqQOPJUQ+4kh3c5FWQ2BxzLmVK/WXskv8uxgPPTelPd6TPrW KpPTsKz0EqIVxst4bdBFh3o0wtSL0Jf3xI5c6d3Qq+ZBcMfiYPjeEDjWK8Ui7ckEejaYE0UbIWN QF3ZOSOsTE0cEymGAiNt6fLIrAm7bTPm3xyXZDA== X-Google-Smtp-Source: AGHT+IE9s13i7j3TkMJ++FVCyif8BAKmecMcfB9JJQFyKeNZbnaim6+3ufJHKPlruBr5wa8K36udJw== X-Received: by 2002:a05:600c:4692:b0:459:db7b:988e with SMTP id 5b1f17b1804b1-45dddeb907dmr132007665e9.13.1757502853033; Wed, 10 Sep 2025 04:14:13 -0700 (PDT) Received: from [192.168.1.3] ([185.48.76.109]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3e7521bf85esm6721413f8f.1.2025.09.10.04.14.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 10 Sep 2025 04:14:12 -0700 (PDT) Message-ID: <21d108f2-db8e-457a-bbef-89d18e8d7601@linaro.org> Date: Wed, 10 Sep 2025 12:14:11 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 00/15] Legacy hardware/cache events as json To: Ian Rogers References: <20250828205930.4007284-1-irogers@google.com> Content-Language: en-US From: James Clark Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Adrian Hunter , Kan Liang , Xu Yang , Thomas Falcon , Andi Kleen , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, bpf@vger.kernel.org, Atish Patra , Beeman Strong , Leo Yan , Vince Weaver In-Reply-To: <20250828205930.4007284-1-irogers@google.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 28/08/2025 9:59 pm, Ian Rogers wrote: > Mirroring similar work for software events in commit 6e9fa4131abb > ("perf parse-events: Remove non-json software events"). These changes > migrate the legacy hardware and cache events to json. With no hard > coded legacy hardware or cache events the wild card, case > insensitivity, etc. is consistent for events. This does, however, mean > events like cycles will wild card against all PMUs. A change doing the > same was originally posted and merged from: > https://lore.kernel.org/r/20240416061533.921723-10-irogers@google.com > and reverted by Linus in commit 4f1b067359ac ("Revert "perf > parse-events: Prefer sysfs/JSON hardware events over legacy"") due to > his dislike for the cycles behavior on ARM with perf record. Earlier > patches in this series make perf record event opening failures > non-fatal and hide the cycles event's failure to open on ARM in perf > record, so it is expected the behavior will now be transparent in perf > record on ARM. perf stat with a cycles event will wildcard open the > event on all PMUs. Hi Ian, Briefly testing perf record and perf stat seem to work now. i.e "perf record -e cycles" doesn't fail and just skips the uncore cycles event. And "perf stat" now includes the uncore cycles event which I think is harmless. But there are a few perf test failures. For example "test event parsing": evlist after sorting/fixing: 'arm_cmn_0/cycles/,{cycles,cache- misses,branch-misses}' FAILED tests/parse-events.c:1589 wrong number of entries Event test failure: test 57 '{cycles,cache-misses,branch- misses}:e'running test 58 'cycles/name=name/' The tests "Perf time to TSC" and "Use a dummy software event to keep tracking" are using libperf to open the cycles event as a sampling event which now fails. It seems like we've fixed Perf record to ignore this failure, but we didn't think about libperf until now. Thanks James