linux-kernel - [PATCH 0/7] arm_pmu: Counter enabling clean-ups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20250107-arm-pmu-cleanups-v1-v1-0-313951346a25@kernel.org>
Date: Tue, 07 Jan 2025 11:57:18 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Will Deacon <will@...nel.org>, Mark Rutland <mark.rutland@....com>
Cc: Marc Zyngier <maz@...nel.org>, linux-arm-kernel@...ts.infradead.org, 
 linux-kernel@...r.kernel.org
Subject: [PATCH 0/7] arm_pmu: Counter enabling clean-ups

This series of clean-ups is in preparation for BRBE support, but are all 
valid on their own. Mark started on these and I have since picked up 
working on BRBE. I've pulled these out of that series as it is not 
looking like BRBE series is going to be ready to send out again before 
next cycle. Hopefully these can go into 6.14.

The first patch reorders kvm_vcpu_pmu_resync_el0() call to avoid 
counting it. In testing BRBE, the call to it showed up in the branch 
recordings.

The next 5 patches drop disabling the counter before enabling it as 
that's redundant. Mark did that for arm_pmu and pmuv3. I found a couple 
more cases in armv7_pmu and apple_m1. 

The last patch simply reorders struct arm_pmu to group pmuv3 specific 
fields.

Tested pmuv3 on FVP and armv7pmu on qemu. 

Signed-off-by: Rob Herring (Arm) <robh@...nel.org>
---
Mark Rutland (3):
      perf: arm_pmu: Don't disable counter in armpmu_add()
      perf: arm_pmuv3: Don't disable counter in armv8pmu_enable_event()
      perf: arm_pmu: Move PMUv3-specific data

Rob Herring (Arm) (4):
      perf: arm_pmuv3: Call kvm_vcpu_pmu_resync_el0() before enabling counters
      perf: arm_v7_pmu: Drop obvious comments for enabling/disabling counters and interrupts
      perf: arm_v7_pmu: Don't disable counter in (armv7|krait_|scorpion_)pmu_enable_event()
      perf: apple_m1: Don't disable counter in m1_pmu_enable_event()

 drivers/perf/apple_m1_cpu_pmu.c |  4 ----
 drivers/perf/arm_pmu.c          |  8 +++----
 drivers/perf/arm_pmuv3.c        |  9 ++------
 drivers/perf/arm_v7_pmu.c       | 50 -----------------------------------------
 include/linux/perf/arm_pmu.h    | 13 ++++++-----
 5 files changed, 12 insertions(+), 72 deletions(-)
---
base-commit: 40384c840ea1944d7c5a392e8975ed088ecf0b37
change-id: 20250107-arm-pmu-cleanups-v1-36485dec867d

Best regards,
-- 
Rob Herring (Arm) <robh@...nel.org>