Tixy, Could you like to send the bad config to reproduce the fixing issue?
On 09/06/2013 11:25 PM, Mark Hambleton wrote:
diff --git a/arch/arm/kernel/head.S b/arch/arm/kernel/head.S index 8bac553..2725c87 100644 --- a/arch/arm/kernel/head.S +++ b/arch/arm/kernel/head.S @@ -342,7 +342,6 @@ __turn_mmu_on_loc: .long __turn_mmu_on_end #if defined(CONFIG_SMP)
- __CPUINIT ENTRY(secondary_startup)
I just has few concern of the solution of upstream kernel. the commit 2449189bb7c73b5fe55a18bc0d289e39bdcd4998 in Linus' tree did similar fixing.
#if defined(CONFIG_SMP) + .text ENTRY(secondary_startup) /*
/* * Common entry point for secondary CPUs. diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c index 84d55da..a666911 100644 --- a/arch/arm/kernel/smp.c +++ b/arch/arm/kernel/smp.c @@ -57,7 +57,7 @@ struct secondary_data secondary_data;
- control for which core is the next to come out of the secondary
- boot "holding pen"
*/ -volatile int __cpuinitdata pen_release = -1; +volatile int pen_release = -1;
and commit 8bd26e3a7e49af2697449bbc, includes this change on Linus' tree too. I saw Greg also picked up the commit.
Tixy, I appreciate your great effort on this bug. But as my understanding, if we pick up this patch, linaro stable tree will be ugly when merge the Greg's stable tree again.
Mark, what's your comments of this?
Regards Alex
enum ipi_msg_type { IPI_WAKEUP, -- 1.7.10.4