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

Tom Rini trini at ti.com
Fri Feb 13 19:09:33 CET 2015


On Fri, Feb 13, 2015 at 09:37:23AM +0800, Peng Fan wrote:
> 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?

This is a core ARM thing so I'm expecting Albert to pick it up.  Albert?
Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150213/cf4494bc/attachment.sig>


More information about the U-Boot mailing list