Re: [PATCH: 003/017]Memory hotplug for new nodes v.4.(get node id at probe memory)

On Friday 17 March 2006 09:20, Yasunori Goto wrote:
> When CONFIG_NUMA && CONFIG_ARCH_MEMORY_PROBE, nid should be defined
> before calling add_memory(nid, start, size).

Seems a bit weird to have the node number assignment somewhere
hidden in memory hotadd. It would be probably cleaner to have
a separate step for this.

> +#if defined(CONFIG_NUMA) && defined(CONFIG_ARCH_MEMORY_PROBE)
> +extern int arch_nid_probe(u64 start);

Instead of adding such ugly ifdefs better just add stubs to all the
architectures that support memory hotplug. There are not that many
anyways.

-Andi

> +#else
> +static inline int arch_nid_probe(u64 start)
> +{
> +	return 0;
> +}
> +#endif
> +
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH: 003/017]Memory hotplug for new nodes v.4.(get node id at probe memory)
    • From: Yasunori Goto <[email protected]>

• Prev by Date: writing to a device through driver entrypoint or directly from user space after mapping it
• Next by Date: Re: [PATCH] i386: run BIOS PCI detection before direct
• Previous by thread: [PATCH: 003/017]Memory hotplug for new nodes v.4.(get node id at probe memory)
• Next by thread: [PATCH: 006/017]Memory hotplug for new nodes v.4.(move out pgdat array from mem_data for ia64)
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]