[U-Boot] [PATCH] ARM: HYP/non-sec: relocation before enable secondary cores

Peng Fan B51431 at freescale.com
Fri Feb 13 02:37:23 CET 2015


Hi, Albert and Tom

On 2/4/2015 7:02 PM, Marc Zyngier wrote:
> On 04/02/15 10:15, Peng Fan wrote:
>> If CONFIG_ARMV7_PSCI is not defined and CONFIG_ARMV7_SECURE_BASE is defined,
>> smp_kicl_all_cpus may enable secondary cores and runs into secure_ram_addr(
>> _smp_pen), before code is relocated to secure ram.
>> So need relocation to secure ram before enable secondary cores.
>>
>> Signed-off-by: Peng Fan <Peng.Fan at freescale.com>
>> ---
>>   arch/arm/cpu/armv7/virt-v7.c | 9 ++++++++-
>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/cpu/armv7/virt-v7.c b/arch/arm/cpu/armv7/virt-v7.c
>> index b69fd37..4cb8806 100644
>> --- a/arch/arm/cpu/armv7/virt-v7.c
>> +++ b/arch/arm/cpu/armv7/virt-v7.c
>> @@ -112,13 +112,20 @@ int armv7_init_nonsec(void)
>>   	for (i = 1; i <= itlinesnr; i++)
>>   		writel((unsigned)-1, gic_dist_addr + GICD_IGROUPRn + 4 * i);
>>   
>> +	/*
>> +	 * Relocate secure section before any cpu runs in secure ram.
>> +	 * smp_kick_all_cpus may enable other cores and runs into secure
>> +	 * ram, so need to relocate secure section before enabling other
>> +	 * cores.
>> +	 */
>> +	relocate_secure_section();
>> +
>>   #ifndef CONFIG_ARMV7_PSCI
>>   	smp_set_core_boot_addr((unsigned long)secure_ram_addr(_smp_pen), -1);
>>   	smp_kick_all_cpus();
>>   #endif
>>   
>>   	/* call the non-sec switching code on this CPU also */
>> -	relocate_secure_section();
>>   	secure_ram_addr(_nonsec_init)();
>>   	return 0;
>>   }
>>
> Seems like a sensible thing to do. FWIW:
>
> Acked-by: Marc Zyngier <marc.zyngier at arm.com>
Just kindly remind. Will this patch be merged into u-boot-arm git repo 
or directly into u-boot master git repo?
>
> 	M.
Thanks,
Peng.


More information about the U-Boot mailing list