core/cpu: refactor IPI sending
Pull the IPI sending code into its own function where it is used in
two places.
cpu_wake() already checks in_idle, so its caller does not need to
check pm_enabled.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
diff --git a/core/cpu.c b/core/cpu.c
index 20b3f6a..e915d44 100644
--- a/core/cpu.c
+++ b/core/cpu.c
@@ -96,13 +96,8 @@
barrier();
}
-static void cpu_wake(struct cpu_thread *cpu)
+static void cpu_send_ipi(struct cpu_thread *cpu)
{
- /* Is it idle ? If not, no need to wake */
- sync();
- if (!cpu->in_idle)
- return;
-
if (proc_gen == proc_gen_p8) {
/* Poke IPI */
icp_kick_cpu(cpu);
@@ -111,6 +106,14 @@
}
}
+static void cpu_wake(struct cpu_thread *cpu)
+{
+ /* Is it idle ? If not, no need to wake */
+ sync();
+ if (cpu->in_idle)
+ cpu_send_ipi(cpu);
+}
+
/*
* If chip_id is >= 0, schedule the job on that node.
* Otherwise schedule the job anywhere.
@@ -195,8 +198,7 @@
cpu->job_has_no_return = true;
else
cpu->job_count++;
- if (pm_enabled)
- cpu_wake(cpu);
+ cpu_wake(cpu);
unlock(&cpu->job_lock);
}
@@ -607,16 +609,9 @@
*/
sync();
- if (proc_gen == proc_gen_p8) {
- for_each_available_cpu(cpu) {
- if (cpu->in_sleep || cpu->in_idle)
- icp_kick_cpu(cpu);
- }
- } else if (proc_gen == proc_gen_p9 || proc_gen == proc_gen_p10) {
- for_each_available_cpu(cpu) {
- if (cpu->in_sleep || cpu->in_idle)
- p9_dbell_send(cpu->pir);
- }
+ for_each_available_cpu(cpu) {
+ if (cpu->in_sleep || cpu->in_idle)
+ cpu_send_ipi(cpu);
}
/*