On Mon, May 28, 2018 at 10:13:52AM +0200, osalvador@xxxxxxxxxxxxxxxxxx wrote: > From: Oscar Salvador <osalvador@xxxxxxx> > > Hi guys, > > I wanted to give it a chance a do a small cleanup in the hotplug memory code. > A lot more could be done, but I wanted to start somewhere. > I tried to unify/remove duplicated code. > > Here I have just done three things > > 1) add_memory_resource() had code to allocate a node in case it was offline. > Since try_online_node already does that, I just made add_memory_resource() to > use that function. > This is better explained in patch 1/3. > > 2) register_mem_sect_under_node() will be called only from link_mem_sections > > 3) Get rid of link_mem_sections() in favour of walk_memory_range with a callback to > register_mem_sect_under_node() > > I am posting this as a RFC because I could not see that these patches break anything, > but expert eyes might see something that I am missing here. > > Oscar Salvador (3): > mm/memory_hotplug: Make add_memory_resource use __try_online_node > mm/memory_hotplug: Call register_mem_sect_under_node > mm/memory_hotplug: Get rid of link_mem_sections > > drivers/base/memory.c | 2 - > drivers/base/node.c | 47 +++++------------------ > include/linux/node.h | 21 +++++------ > mm/memory_hotplug.c | 101 ++++++++++++++++++++++++++------------------------ > 4 files changed, 71 insertions(+), 100 deletions(-) > > Signed-off-by: Oscar Salvador <osalvador@xxxxxxx> > > -- > 2.13.6 > Since there have not been any concerns so far, I will send v1 of this patchset. Thanks Oscar Salvador
