diff options
| author | Yury Norov <yury.norov@gmail.com> | 2022-02-10 23:48:56 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2022-02-15 15:38:57 +0100 |
| commit | 95ed57c73bbcffa02cbb1d61c19484c2ec6de094 (patch) | |
| tree | 868ab4d1a7d99ade05dc3729f5dc705d10387d87 /drivers/perf/arm-cci.c | |
| parent | perf: Replace acpi_bus_get_device() (diff) | |
| download | linux-95ed57c73bbcffa02cbb1d61c19484c2ec6de094.tar.xz linux-95ed57c73bbcffa02cbb1d61c19484c2ec6de094.zip | |
perf: replace bitmap_weight with bitmap_empty where appropriate
In some places, drivers/perf code calls bitmap_weight() to check if any
bit of a given bitmap is set. It's better to use bitmap_empty() in that
case because bitmap_empty() stops traversing the bitmap as soon as it
finds first set bit, while bitmap_weight() counts all bits unconditionally.
Signed-off-by: Yury Norov <yury.norov@gmail.com>
Acked-by: Mark Rutland <mark.rutland@arm.com>
Link: https://lore.kernel.org/r/20220210224933.379149-13-yury.norov@gmail.com
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'drivers/perf/arm-cci.c')
| -rw-r--r-- | drivers/perf/arm-cci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/perf/arm-cci.c b/drivers/perf/arm-cci.c index 54aca3a62814..96e09fa40909 100644 --- a/drivers/perf/arm-cci.c +++ b/drivers/perf/arm-cci.c @@ -1096,7 +1096,7 @@ static void cci_pmu_enable(struct pmu *pmu) { struct cci_pmu *cci_pmu = to_cci_pmu(pmu); struct cci_pmu_hw_events *hw_events = &cci_pmu->hw_events; - int enabled = bitmap_weight(hw_events->used_mask, cci_pmu->num_cntrs); + bool enabled = !bitmap_empty(hw_events->used_mask, cci_pmu->num_cntrs); unsigned long flags; if (!enabled) |