[PATCH] Help fix SMP boot issues with L2 cache

[catalin.marinas@arm.com (Catalin Marinas)]

Russell King - ARM Linux wrote:
>     ARM: Add L2 cache handling to smp boot support
>     
>     The page table and secondary data which we're asking the secondary CPU
>     to make use of has to hit RAM to ensure that the secondary CPU can see
>     it since it may not be taking part in coherency or cache searches at
>     this point.
>     
>     Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
> 
>  arch/arm/kernel/smp.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c
> index 57162af..577543f 100644
> --- a/arch/arm/kernel/smp.c
> +++ b/arch/arm/kernel/smp.c
> @@ -99,6 +99,7 @@ int __cpuinit __cpu_up(unsigned int cpu)
>  	*pmd = __pmd((PHYS_OFFSET & PGDIR_MASK) |
>  		     PMD_TYPE_SECT | PMD_SECT_AP_WRITE);
>  	flush_pmd_entry(pmd);
> +	outer_clean_range(__pa(pmd), __pa(pmd + 1));
>  
>  	/*
>  	 * We need to tell the secondary core where to find
> @@ -106,7 +107,8 @@ int __cpuinit __cpu_up(unsigned int cpu)
>  	 */
>  	secondary_data.stack = task_stack_page(idle) + THREAD_START_SP;
>  	secondary_data.pgdir = virt_to_phys(pgd);
> -	wmb();
> +	__cpuc_flush_dcache_area(&secondary_data, sizeof(secondary_data));
> +	outer_clean_range(__pa(&secondary_data), __pa(&secondary_data + 1));
>  
>  	/*
>  	 * Now bring the CPU into our world.

But is the outer_cache initialised at this point? I recall for RealView 
I had to change the l2x0_init to an early_initcall:

http://linux-arm.org/git?p=linux-2.6-stable.git;a=commitdiff;h=70e1b8d308f635eaeb7dad14fc5c79f082a147ef;hp=f034a4ab31b0663a5b68c65503122bed1eaca349

I think I have a patch for 2.6.33, I'll dig it out on Monday. But this 
was only needed when running the kernel in non-secure mode and the L2 
was pre-initialised by the secure monitor (it may be the case on some 
Omap boards).

-- 
Catalin