Re: [7/8,v3] NUMA Hotplug Emulator: extend memory probe interface to support NUMA

Dave Hansen
2010-11-17T19:04:20Z

On Wed, 2010-11-17 at 10:08 +0800, shaohui.zheng@intel.com wrote:
> And more we make it friendly, it is possible to add memory to do
>
> echo 3g > memory/probe
> echo 1024m,3 > memory/probe
>
> It maintains backwards compatibility.
>
> Another format suggested by Dave Hansen:
>
> echo physical_address=0x40000000 numa_node=3 > memory/probe
>
> it is more explicit to show meaning of the parameters.

The other thing that Greg suggested was to use configfs. Looking back
on it, that makes a lot of sense. We can do better than these "probe"
files.

In your case, it might be useful to tell the kernel to be able to add
memory in a node and add the node all in one go. That'll probably be
closer to what the hardware will do, and will exercise different code
paths that the separate "add node", "then add memory" steps that you're
using here.

For the emulator, I also have to wonder if using debugfs is the right
was since its ABI is a bit more, well, _flexible_ over time. :)

> + depends on NUMA_HOTPLUG_EMU
> + ---help---
> + Enable memory hotplug emulation. Reserve memory with grub parameter
> + "mem=N"(such as mem=1024M), where N is the initial memory size, the
> + rest physical memory will be removed from e820 table; the memory probe
> + interface is for memory hot-add to specified node in software method.
> + This is for debuging and testing purpose

mem= actually sets the largest physical address that we're trying to
use. If you have a 256MB hole at 768MB, then mem=1G will only get you
768MB of memory. We probably get this wrong in a number of other places
in the documentation, but we might as well get it right here.

Maybe something like:

Enable emulation of hotplug of NUMA nodes. To use this, you
must also boot with the kernel command-line parameter
"mem=N"(such as mem=1024M), where N is the highest physical
address you would like to use at boot. The rest of physical
memory will be removed from firmware tables and may be then be
hotplugged with this feature. This is for debuging and testing
purposes.

Note that you can still examine the original, non-modified
firmware tables in: /sys/firmware/memmap

-- Dave