Commit 3cf61110 authored by Mark Rutland's avatar Mark Rutland Committed by Will Deacon

drivers/perf: arm_pmu: rename irq request/free functions

For historical reasons, portions of the arm_pmu code use a cpu_pmu_
prefix rather than an armpmu_ prefix. While a minor annoyance, this
hasn't been a problem thusfar.

However, to enable ACPI support, we'll need to expose a few things in
header files, and we should aim to keep those consistently namespaced.
In preparation for exporting our IRQ request/free functions, rename
these to have an armpmu_ prefix. For consistency, the 'cpu_pmu'
parameter is also renamed to 'armpmu'.
Signed-off-by: default avatarMark Rutland <mark.rutland@arm.com>
Tested-by: default avatarJeremy Linton <jeremy.linton@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: default avatarWill Deacon <will.deacon@arm.com>
parent 76541370
...@@ -527,12 +527,12 @@ int perf_num_counters(void) ...@@ -527,12 +527,12 @@ int perf_num_counters(void)
} }
EXPORT_SYMBOL_GPL(perf_num_counters); EXPORT_SYMBOL_GPL(perf_num_counters);
static void cpu_pmu_free_irqs(struct arm_pmu *cpu_pmu) static void armpmu_free_irqs(struct arm_pmu *armpmu)
{ {
int cpu; int cpu;
struct pmu_hw_events __percpu *hw_events = cpu_pmu->hw_events; struct pmu_hw_events __percpu *hw_events = armpmu->hw_events;
for_each_cpu(cpu, &cpu_pmu->supported_cpus) { for_each_cpu(cpu, &armpmu->supported_cpus) {
int irq = per_cpu(hw_events->irq, cpu); int irq = per_cpu(hw_events->irq, cpu);
if (!irq) if (!irq)
continue; continue;
...@@ -542,20 +542,20 @@ static void cpu_pmu_free_irqs(struct arm_pmu *cpu_pmu) ...@@ -542,20 +542,20 @@ static void cpu_pmu_free_irqs(struct arm_pmu *cpu_pmu)
break; break;
} }
if (!cpumask_test_and_clear_cpu(cpu, &cpu_pmu->active_irqs)) if (!cpumask_test_and_clear_cpu(cpu, &armpmu->active_irqs))
continue; continue;
free_irq(irq, per_cpu_ptr(&hw_events->percpu_pmu, cpu)); free_irq(irq, per_cpu_ptr(&hw_events->percpu_pmu, cpu));
} }
} }
static int cpu_pmu_request_irqs(struct arm_pmu *cpu_pmu) static int armpmu_request_irqs(struct arm_pmu *armpmu)
{ {
int cpu, err; int cpu, err;
struct pmu_hw_events __percpu *hw_events = cpu_pmu->hw_events; struct pmu_hw_events __percpu *hw_events = armpmu->hw_events;
const irq_handler_t handler = armpmu_dispatch_irq; const irq_handler_t handler = armpmu_dispatch_irq;
for_each_cpu(cpu, &cpu_pmu->supported_cpus) { for_each_cpu(cpu, &armpmu->supported_cpus) {
int irq = per_cpu(hw_events->irq, cpu); int irq = per_cpu(hw_events->irq, cpu);
if (!irq) if (!irq)
continue; continue;
...@@ -580,7 +580,7 @@ static int cpu_pmu_request_irqs(struct arm_pmu *cpu_pmu) ...@@ -580,7 +580,7 @@ static int cpu_pmu_request_irqs(struct arm_pmu *cpu_pmu)
return err; return err;
} }
cpumask_set_cpu(cpu, &cpu_pmu->active_irqs); cpumask_set_cpu(cpu, &armpmu->active_irqs);
} }
return 0; return 0;
...@@ -742,7 +742,7 @@ static int cpu_pmu_init(struct arm_pmu *cpu_pmu) ...@@ -742,7 +742,7 @@ static int cpu_pmu_init(struct arm_pmu *cpu_pmu)
{ {
int err; int err;
err = cpu_pmu_request_irqs(cpu_pmu); err = armpmu_request_irqs(cpu_pmu);
if (err) if (err)
goto out; goto out;
...@@ -761,7 +761,7 @@ static int cpu_pmu_init(struct arm_pmu *cpu_pmu) ...@@ -761,7 +761,7 @@ static int cpu_pmu_init(struct arm_pmu *cpu_pmu)
cpuhp_state_remove_instance_nocalls(CPUHP_AP_PERF_ARM_STARTING, cpuhp_state_remove_instance_nocalls(CPUHP_AP_PERF_ARM_STARTING,
&cpu_pmu->node); &cpu_pmu->node);
out: out:
cpu_pmu_free_irqs(cpu_pmu); armpmu_free_irqs(cpu_pmu);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment