Commit 0c7c6414 authored by Yury Norov (NVIDIA)'s avatar Yury Norov (NVIDIA) Committed by Will Deacon
Browse files

drivers: perf: use bitmap_empty() where appropriate 



bitmap_empty() is more verbose and efficient, as it stops traversing
bitmaps as soon as the 1st set bit found.

Switch perf code to using bitmap_empty() where appropriate, and
correspondingly use boolean types.

Signed-off-by: default avatarYury Norov (NVIDIA) <yury.norov@gmail.com>
Signed-off-by: default avatarWill Deacon <will@kernel.org>
parent 12a94953

drivers/perf/riscv_pmu_sbi.c

+1 −1
Original line number Diff line number Diff line
@@ -1244,7 +1244,7 @@ static int riscv_pm_pmu_notify(struct notifier_block *b, unsigned long cmd, 
{

	struct riscv_pmu *rvpmu = container_of(b, struct riscv_pmu, riscv_pm_nb);

	struct cpu_hw_events *cpuc = this_cpu_ptr(rvpmu->hw_events);

	int enabled = bitmap_weight(cpuc->used_hw_ctrs, RISCV_MAX_COUNTERS);

	bool enabled = !bitmap_empty(cpuc->used_hw_ctrs, RISCV_MAX_COUNTERS);

	struct perf_event *event;

	int idx;

drivers/perf/starfive_starlink_pmu.c

+1 −2
Original line number Diff line number Diff line
@@ -450,8 +450,7 @@ static int starlink_pmu_pm_notify(struct notifier_block *b, 
							 starlink_pmu_pm_nb);

	struct starlink_hw_events *hw_events =

					this_cpu_ptr(starlink_pmu->hw_events);

	int enabled = bitmap_weight(hw_events->used_mask,

				    STARLINK_PMU_MAX_COUNTERS);

	bool enabled = !bitmap_empty(hw_events->used_mask, STARLINK_PMU_MAX_COUNTERS);

	struct perf_event *event;

	int idx;