Re: [RFC PATCH v3 22/24] ARM: NUMA: Initialize ACPI NUMA

From: Stefano Stabellini <sstabellini@kernel.org>
To: vijay.kilari@gmail.com
Cc: tim@xen.org, kevin.tian@intel.com, sstabellini@kernel.org,
	wei.liu2@citrix.com, George.Dunlap@eu.citrix.com,
	andrew.cooper3@citrix.com, dario.faggioli@citrix.com,
	ian.jackson@eu.citrix.com, xen-devel@lists.xen.org,
	julien.grall@arm.com, jbeulich@suse.com,
	Vijaya Kumar K <Vijaya.Kumar@cavium.com>
Subject: Re: [RFC PATCH v3 22/24] ARM: NUMA: Initialize ACPI NUMA
Date: Mon, 24 Jul 2017 15:11:44 -0700 (PDT)	[thread overview]
Message-ID: <alpine.DEB.2.10.1707241440480.22381@sstabellini-ThinkPad-X260> (raw)
In-Reply-To: <1500378106-2620-23-git-send-email-vijay.kilari@gmail.com>

On Tue, 18 Jul 2017, vijay.kilari@gmail.com wrote:
> From: Vijaya Kumar K <Vijaya.Kumar@cavium.com>
> 
> Call ACPI NUMA initialization under CONFIG_ACPI_NUMA.
> 
> Signed-off-by: Vijaya Kumar <Vijaya.Kumar@cavium.com>
> ---
>  xen/arch/arm/numa/acpi_numa.c | 27 ++++++++++++++++++++++++++-
>  xen/arch/arm/numa/numa.c      | 15 +++++++++++++--
>  xen/common/numa.c             | 14 ++++++++++++++
>  xen/include/asm-arm/numa.h    |  1 +
>  xen/include/xen/numa.h        |  1 +
>  5 files changed, 55 insertions(+), 3 deletions(-)
> 
> diff --git a/xen/arch/arm/numa/acpi_numa.c b/xen/arch/arm/numa/acpi_numa.c
> index 95617f9..68fff95 100644
> --- a/xen/arch/arm/numa/acpi_numa.c
> +++ b/xen/arch/arm/numa/acpi_numa.c
> @@ -181,7 +181,7 @@ acpi_numa_gicc_affinity_init(const struct acpi_srat_gicc_affinity *pa)
>             pxm, mpidr, node);
>  }
>  
> -void __init acpi_map_uid_to_mpidr(void)
> +static void __init acpi_map_uid_to_mpidr(void)

Can we introduce it as static since the beginning?

>  {
>      acpi_table_parse_madt(ACPI_MADT_TYPE_GENERIC_INTERRUPT,
>                      acpi_parse_madt_handler, NR_CPUS);
> @@ -209,6 +209,31 @@ void __init arch_table_parse_srat(void)
>                            acpi_parse_gicc_affinity, NR_CPUS);
>  }
>  
> +bool_t __init arch_acpi_numa_init(void)
> +{
> +    int ret;
> +
> +    if ( !acpi_disabled )

  if ( acpi_disabled )
    return false;

> +    {
> +        /*
> +         * If firmware has DT, process_memory_node() call
> +         * would have added memory blocks. So reset it before
> +         * ACPI numa init.
> +         */
> +        numa_clear_memblks();
> +        nodes_clear(memory_nodes_parsed);
> +        acpi_map_uid_to_mpidr();
> +        ret = acpi_numa_init();
> +        if ( ret || srat_disabled() )

I don't think we need to check again for srat_disabled: that check is
already done by acpi_numa_init

> +            return 1;

return true

> +
> +        /* Register acpi node_distance handler */
> +        register_node_distance(&acpi_node_distance);
> +    }
> +
> +    return 0;
> +}
> +
>  void __init acpi_numa_arch_fixup(void) {}
>  
>  /*
> diff --git a/xen/arch/arm/numa/numa.c b/xen/arch/arm/numa/numa.c
> index 26aa4c0..68599c4 100644
> --- a/xen/arch/arm/numa/numa.c
> +++ b/xen/arch/arm/numa/numa.c
> @@ -139,11 +139,22 @@ void __init numa_init(void)
>      if ( numa_off )
>          goto no_numa;
>  
> -    ret = dt_numa_init();
> +#ifdef CONFIG_ACPI_NUMA
> +    ret = arch_acpi_numa_init();
>      if ( ret )
>      {
>          numa_off = true;
> -        printk(XENLOG_WARNING "DT NUMA init failed\n");
> +        printk(XENLOG_WARNING "ACPI NUMA init failed\n");
> +    }
> +#endif
> +    if ( acpi_disabled )
> +    {
> +        ret = dt_numa_init();
> +        if ( ret )
> +        {
> +            numa_off = true;
> +            printk(XENLOG_WARNING "DT NUMA init failed\n");
> +        }
>      }
>  
>  no_numa:
> diff --git a/xen/common/numa.c b/xen/common/numa.c
> index 0f79a07..020bc19 100644
> --- a/xen/common/numa.c
> +++ b/xen/common/numa.c
> @@ -76,6 +76,20 @@ nodeid_t get_memblk_nodeid(unsigned int id)
>      return memblk_nodeid[id];
>  }
>  
> +void __init numa_clear_memblks(void)
> +{
> +    unsigned int i;
> +
> +    for ( i = 0; i < get_num_node_memblks(); i++ )
> +    {
> +        node_memblk_range[i].start = 0;
> +        node_memblk_range[i].end = 0;
> +        memblk_nodeid[i] = NUMA_NO_NODE;
> +    }
> +
> +    num_node_memblks = 0;
> +}
> +
>  int __init get_mem_nodeid(paddr_t start, paddr_t end)
>  {
>      unsigned int i;
> diff --git a/xen/include/asm-arm/numa.h b/xen/include/asm-arm/numa.h
> index f0a50bd..ff10b31 100644
> --- a/xen/include/asm-arm/numa.h
> +++ b/xen/include/asm-arm/numa.h
> @@ -20,6 +20,7 @@ static inline nodeid_t acpi_get_nodeid(uint64_t hwid)
>  void numa_init(void);
>  int dt_numa_init(void);
>  void numa_set_cpu_node(int cpu, unsigned int nid);
> +bool_t arch_acpi_numa_init(void);
>  
>  #else
>  static inline void numa_init(void)
> diff --git a/xen/include/xen/numa.h b/xen/include/xen/numa.h
> index a541eb7..14a7a0c 100644
> --- a/xen/include/xen/numa.h
> +++ b/xen/include/xen/numa.h
> @@ -75,6 +75,7 @@ int get_num_node_memblks(void);
>  bool arch_sanitize_nodes_memory(void);
>  void numa_failed(void);
>  uint8_t __node_distance(nodeid_t a, nodeid_t b);
> +void numa_clear_memblks(void);
>  #else
>  static inline void numa_add_cpu(int cpu) { }
>  static inline void numa_set_node(int cpu, nodeid_t node) { }
> -- 
> 2.7.4
> 

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