On Friday 03 January 2014, Geert Uytterhoeven wrote: > From: Geert Uytterhoeven <geert+rene...@linux-m68k.org> > > After becoming a mandatory function, boot_secondary() is no longer used > outside arch/arm/kernel/smp.c. Hence make it static. > The code is reshuffled a bit to avoid a forward declaration. > > Signed-off-by: Geert Uytterhoeven <geert+rene...@linux-m68k.org>
How about taking it one step further by removing the function entirely? Like this: @@ -92,6 +92,9 @@ int __cpu_up(unsigned int cpu, struct task_struct *idle) { int ret; + if (!smp_ops.smp_boot_secondary) + return -ENOSYS; + /* * We need to tell the secondary core where to find * its stack and the page tables. @@ -111,7 +114,7 @@ int __cpu_up(unsigned int cpu, struct task_struct *idle) /* * Now bring the CPU into our world. */ - ret = boot_secondary(cpu, idle); + ret = smp_ops.smp_boot_secondary(cpu, idle); if (ret == 0) { /* * CPU was successfully started, wait for it I guess we could do the same for some of the other operations as well if everyone likes the approach. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/