xen/arm: smpboot: drop unneeded code in start_secondary
authorPeng Fan <van.freenix@gmail.com>
Thu, 19 May 2016 09:22:07 +0000 (17:22 +0800)
committerStefano Stabellini <sstabellini@kernel.org>
Fri, 17 Jun 2016 09:22:16 +0000 (10:22 +0100)
CPU0 boots up secondary CPUs one by one. Before booting
one secondary CPU, CPU0 will assign hwid to smp_up_cpu
and flush cache. After a secondary CPU boots up,
CPU0 will assign MPIDR_INVALID to smp_up_cpu and flush
cache.

There is no need for secondary CPUs to assign MPIDR_INVALID
to smp_up_cpu. So, drop it.

Signed-off-by: Peng Fan <van.freenix@gmail.com>
Reviewed-by: Julien Grall <julien.grall@arm.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>
xen/arch/arm/smpboot.c

index ba834068e307693c14b1665d17bf79e832f4d009..3a962f7d5db160b19fa64d8e46229d7565944ad8 100644 (file)
@@ -310,7 +310,6 @@ void start_secondary(unsigned long boot_phys_offset,
     smp_wmb();
 
     /* Now report this CPU is up */
-    smp_up_cpu = MPIDR_INVALID;
     cpumask_set_cpu(cpuid, &cpu_online_map);
     smp_wmb();