diff options
| author | Suzuki K Poulose <Suzuki.Poulose@arm.com> | 2018-01-15 10:51:16 +0000 |
|---|---|---|
| committer | Catalin Marinas <catalin.marinas@arm.com> | 2018-01-15 18:02:17 +0000 |
| commit | a22fde8e979494f326e88ef0728367e009260d75 (patch) | |
| tree | bb0d3c26f09d67eb46c1dbbb59ac969fcebd97d1 /drivers/perf | |
| parent | 67948af41f2e6818edeeba5182811c704d484949 (diff) | |
| download | linux-a22fde8e979494f326e88ef0728367e009260d75.tar.bz2 | |
perf: dsu: Use signed field for dsu_pmu->num_counters
We set dsu_pmu->num_counters to -1, when the DSU is allocated
but not initialised when none of the CPUs are active in the DSU.
However, we use an unsigned field for num_counters. Switch this
to a signed field.
Fixes: 7520fa99246d ("perf: ARM DynamIQ Shared Unit PMU support")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Diffstat (limited to 'drivers/perf')
| -rw-r--r-- | drivers/perf/arm_dsu_pmu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/perf/arm_dsu_pmu.c b/drivers/perf/arm_dsu_pmu.c index 37c0526c93d5..93c50e377507 100644 --- a/drivers/perf/arm_dsu_pmu.c +++ b/drivers/perf/arm_dsu_pmu.c @@ -120,7 +120,7 @@ struct dsu_pmu { cpumask_t associated_cpus; cpumask_t active_cpu; struct hlist_node cpuhp_node; - u8 num_counters; + s8 num_counters; int irq; DECLARE_BITMAP(cpmceid_bitmap, DSU_PMU_MAX_COMMON_EVENTS); }; |