Re: [PATCH 14/16] x86: Unify CPU -> NUMA node mapping between 32and 64bit

From: David Rientjes
Date: Tue Dec 28 2010 - 15:40:19 EST


On Tue, 28 Dec 2010, Tejun Heo wrote:

> diff --git a/arch/x86/mm/numa_64.c b/arch/x86/mm/numa_64.c
> index 989f23b..ce33f69 100644
> --- a/arch/x86/mm/numa_64.c
> +++ b/arch/x86/mm/numa_64.c
> @@ -31,12 +31,6 @@ static unsigned long __initdata nodemap_addr;
> static unsigned long __initdata nodemap_size;
>
> /*
> - * Map cpu index to node index
> - */
> -DEFINE_EARLY_PER_CPU(int, x86_cpu_to_node_map, NUMA_NO_NODE);
> -EXPORT_EARLY_PER_CPU_SYMBOL(x86_cpu_to_node_map);
> -
> -/*
> * Given a shift value, try to populate memnodemap[]
> * Returns :
> * 1 if OK
> @@ -737,34 +731,6 @@ int __cpuinit numa_cpu_node(int cpu)
> return NUMA_NO_NODE;
> }
>
> -void __cpuinit numa_set_node(int cpu, int node)
> -{
> - int *cpu_to_node_map = early_per_cpu_ptr(x86_cpu_to_node_map);
> -
> - /* early setting, no percpu area yet */
> - if (cpu_to_node_map) {
> - cpu_to_node_map[cpu] = node;
> - return;
> - }
> -
> -#ifdef CONFIG_DEBUG_PER_CPU_MAPS
> - if (cpu >= nr_cpu_ids || !cpu_possible(cpu)) {
> - printk(KERN_ERR "numa_set_node: invalid cpu# (%d)\n", cpu);
> - dump_stack();
> - return;
> - }
> -#endif
> - per_cpu(x86_cpu_to_node_map, cpu) = node;
> -
> - if (node != NUMA_NO_NODE)
> - set_cpu_numa_node(cpu, node);
> -}
> -
> -void __cpuinit numa_clear_node(int cpu)
> -{
> - numa_set_node(cpu, NUMA_NO_NODE);
> -}
> -
> #ifndef CONFIG_DEBUG_PER_CPU_MAPS
>
> void __cpuinit numa_add_cpu(int cpu)

This is also going to conflict with c1c3443c ("x86, numa: Fake
node-to-cpumask for NUMA emulation") in x86/numa, but it's trival to merge
because there's no actual implementation change.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/