Re: [PATCH] ia64_cleanup_topology_init

[Dave Hansen <haveblue@us.ibm.com>]

On Thu, 2004-06-10 at 16:00, Keshavamurthy, Anil S wrote:
> Side-effects with this patch:
> With this patch register_cpu() is now called only for _online_ cpu's and
> not for all _possible_ cpu's as the code was doing in
> arch/ia64/dig/topology.c. The reason for this is when the new cpu or
> node gets hot added at that time one can dynamically call these register
> functions.

I personally like this a lot better than the current ppc64 scheme where
all possible cpus are brought online.  It's just confusing.  

> +static int __init topology_init(void)
> +{
> +	int i, err = 0;
> +
> +#ifdef CONFIG_NUMA
> +	sysfs_nodes = kmalloc(sizeof(struct node) * numnodes,
> GFP_KERNEL);
> +	if (!sysfs_nodes) {
> +		err = -ENOMEM;
> +		goto out;
> +	}
> +	memset(sysfs_nodes, 0, sizeof(struct node) * numnodes);
> +
> +	for (i = 0; i < numnodes; i++)
> +		if ((err = register_node(&sysfs_nodes[i], i, 0)))
> +			goto out;
> +#endif
...

I wonder there should be a numa_topology_init() function to do this
part, instead if ifdef'ing it here.  In fact, it might even be able to
be a completely generic function.

-- Dave