@@ -56,7 +56,7 @@ struct cpu_operations {
};
int __init init_cpu_ops(int cpu);
-extern const struct cpu_operations *get_cpu_ops(int cpu);
+extern const struct cpu_operations *get_cpu_ops(void);
static inline void __init init_bootcpu_ops(void)
{
@@ -106,7 +106,7 @@ int __init init_cpu_ops(int cpu)
return 0;
}
-const struct cpu_operations *get_cpu_ops(int cpu)
+const struct cpu_operations *get_cpu_ops(void)
{
return cpu_ops;
}
@@ -18,7 +18,7 @@
int arm_cpuidle_init(unsigned int cpu)
{
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
int ret = -EOPNOTSUPP;
if (ops && ops->cpu_suspend && ops->cpu_init_idle)
@@ -36,8 +36,7 @@ int arm_cpuidle_init(unsigned int cpu)
*/
int arm_cpuidle_suspend(int index)
{
- int cpu = smp_processor_id();
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
return ops->cpu_suspend(index);
}
@@ -371,7 +371,7 @@ void __init setup_arch(char **cmdline_p)
static inline bool cpu_can_disable(unsigned int cpu)
{
#ifdef CONFIG_HOTPLUG_CPU
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
if (ops && ops->cpu_can_disable)
return ops->cpu_can_disable(cpu);
@@ -93,7 +93,7 @@ static inline int op_cpu_kill(unsigned int cpu)
*/
static int boot_secondary(unsigned int cpu, struct task_struct *idle)
{
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
if (ops->cpu_boot)
return ops->cpu_boot(cpu);
@@ -230,7 +230,7 @@ asmlinkage notrace void secondary_start_kernel(void)
*/
check_local_cpu_capabilities();
- ops = get_cpu_ops(cpu);
+ ops = get_cpu_ops();
if (ops->cpu_postboot)
ops->cpu_postboot();
@@ -270,7 +270,7 @@ asmlinkage notrace void secondary_start_kernel(void)
#ifdef CONFIG_HOTPLUG_CPU
static int op_cpu_disable(unsigned int cpu)
{
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
/*
* If we don't have a cpu_die method, abort before we reach the point
@@ -320,7 +320,7 @@ int __cpu_disable(void)
static int op_cpu_kill(unsigned int cpu)
{
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
/*
* If we have no means of synchronising with the dying CPU, then assume
@@ -365,7 +365,7 @@ void __cpu_die(unsigned int cpu)
void cpu_die(void)
{
unsigned int cpu = smp_processor_id();
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
idle_task_exit();
@@ -392,7 +392,7 @@ void cpu_die(void)
void cpu_die_early(void)
{
int cpu = smp_processor_id();
- const struct cpu_operations *ops = get_cpu_ops(cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
pr_crit("CPU%d: will not boot\n", cpu);
@@ -503,7 +503,7 @@ static int __init smp_cpu_setup(int cpu)
if (init_cpu_ops(cpu))
return -ENODEV;
- ops = get_cpu_ops(cpu);
+ ops = get_cpu_ops();
if (ops->cpu_init(cpu))
return -ENODEV;
@@ -758,7 +758,7 @@ void __init smp_prepare_cpus(unsigned int max_cpus)
if (cpu == smp_processor_id())
continue;
- ops = get_cpu_ops(cpu);
+ ops = get_cpu_ops();
if (!ops)
continue;
@@ -883,7 +883,7 @@ static void ipi_cpu_crash_stop(unsigned int cpu, struct pt_regs *regs)
sdei_mask_local_cpu();
#ifdef CONFIG_HOTPLUG_CPU
- ops = get_cpu_ops(cpu);
+ ops = get_cpu_ops();
if (ops->cpu_die)
ops->cpu_die(cpu);
#endif
@@ -1063,8 +1063,7 @@ int setup_profiling_timer(unsigned int multiplier)
static bool have_cpu_die(void)
{
#ifdef CONFIG_HOTPLUG_CPU
- int any_cpu = raw_smp_processor_id();
- const struct cpu_operations *ops = get_cpu_ops(any_cpu);
+ const struct cpu_operations *ops = get_cpu_ops();
if (ops && ops->cpu_die)
return true;
This removes the argument (@cpu) of get_cpu_ops() as it's not used. Signed-off-by: Gavin Shan <gshan@redhat.com> --- arch/arm64/include/asm/cpu_ops.h | 2 +- arch/arm64/kernel/cpu_ops.c | 2 +- arch/arm64/kernel/cpuidle.c | 5 ++--- arch/arm64/kernel/setup.c | 2 +- arch/arm64/kernel/smp.c | 21 ++++++++++----------- 5 files changed, 15 insertions(+), 17 deletions(-)