Re: [PATCH: 004/012] Memory hotplug for new nodes v.2. (pgdatalloc caller for ia64)

From: Dave Hansen
Date: Fri Feb 17 2006 - 10:51:07 EST


On Fri, 2006-02-17 at 22:28 +0900, Yasunori Goto wrote:
>
> + if (node < 0)
> + node = 0; /* pxm is undefined in DSDT.
> + This might be non NUMA case */
> +
> + if (!node_online(node)){
> + ret = new_pgdat_init(node, start_pfn, nr_pages);
> + if (ret)
> + return ret;
> +
> + new_pgdat = 1;
> + }
> + pgdat = NODE_DATA(node);

Now that I've seen this exact code copy-and-pasted into two different
architectures, it begs the question: do we need a common function here?

-- Dave

-
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/