Re: [Xen-devel] [PATCH v2] xen/arm: Clean up identify processor call for secondary cpus

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-07-29 at 14:42 +0100, Julien Grall wrote:
> The smp_processor_id() is set at the beginning of start_secondary. We don't
> need to compute ourself the offset of the cpu data.
> 
> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>

Applied, thanks.

> 
> ---
>     Changes in v2:
>         - Initalize the current cpu data with boot cpu data
> ---
>  xen/arch/arm/smpboot.c |    6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c
> index 5895178..b6aea63 100644
> --- a/xen/arch/arm/smpboot.c
> +++ b/xen/arch/arm/smpboot.c
> @@ -137,15 +137,13 @@ void __cpuinit start_secondary(unsigned long 
> boot_phys_offset,
>                                 unsigned long fdt_paddr,
>                                 unsigned long cpuid)
>  {
> -    struct cpuinfo_arm *c = cpu_data + cpuid;
> -
>      memset(get_cpu_info(), 0, sizeof (struct cpu_info));
>  
>      /* TODO: handle boards where CPUIDs are not contiguous */
>      set_processor_id(cpuid);
>  
> -    *c = boot_cpu_data;
> -    identify_cpu(c);
> +    current_cpu_data = boot_cpu_data;
> +    identify_cpu(&current_cpu_data);
>  
>      init_traps();
>  




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel