Instead of copying current_set[cpu] to secondary_ti, directly index into current_set by the cpu number like head_64.S. In addition to removing the variable, a slow to respond cpu will not clobber another cpus stack.
--- grep found no other reference in arch/powerpc. compile tested but I have no 32bit smp hardware. Index: next.git/arch/powerpc/kernel/head_32.S =================================================================== --- next.git.orig/arch/powerpc/kernel/head_32.S 2008-11-16 01:36:45.000000000 -0600 +++ next.git/arch/powerpc/kernel/head_32.S 2008-11-16 01:42:02.000000000 -0600 @@ -906,10 +906,11 @@ __secondary_start: bl init_idle_6xx #endif /* CONFIG_6xx */ - /* get current_thread_info and current */ - lis r1,[EMAIL PROTECTED] + /* get current_thread_info (current_set[cpu]) and current = */ + slwi r1,r24,2 + addis r1,r1,[EMAIL PROTECTED] tophys(r1,r1) - lwz r1,[EMAIL PROTECTED](r1) + lwz r1,[EMAIL PROTECTED](r1) tophys(r2,r1) lwz r2,TI_TASK(r2) Index: next.git/arch/powerpc/kernel/smp.c =================================================================== --- next.git.orig/arch/powerpc/kernel/smp.c 2008-11-16 01:36:05.000000000 -0600 +++ next.git/arch/powerpc/kernel/smp.c 2008-11-16 01:36:22.000000000 -0600 @@ -58,7 +58,6 @@ #endif int smp_hw_index[NR_CPUS]; -struct thread_info *secondary_ti; cpumask_t cpu_possible_map = CPU_MASK_NONE; cpumask_t cpu_online_map = CPU_MASK_NONE; @@ -320,7 +319,6 @@ int __cpuinit __cpu_up(unsigned int cpu) { int c; - secondary_ti = current_set[cpu]; if (!cpu_enable(cpu)) return 0; _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev