All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC PATCH 0/2] remove CONFIG_MOVABLE_NODE
@ 2017-05-24 12:24 ` Michal Hocko
  0 siblings, 0 replies; 32+ messages in thread
From: Michal Hocko @ 2017-05-24 12:24 UTC (permalink / raw)
  To: linux-mm
  Cc: Andrew Morton, Mel Gorman, Vlastimil Babka, Andrea Arcangeli,
	Jerome Glisse, Reza Arbab, Yasuaki Ishimatsu, qiuxishi,
	Kani Toshimitsu, slaoub, Joonsoo Kim, Andi Kleen, David Rientjes,
	Daniel Kiper, Igor Mammedov, Vitaly Kuznetsov, LKML

Hi,
I am continuing to cleanup the memory hotplug code and
CONFIG_MOVABLE_NODE seems dubious at best. The following two patches
simply removes the flag and make it de-facto always enabled.

The current semantic of the config option is twofold 1) it automatically
binds hotplugable nodes to have memory in zone_movable by default when
movable_node is enabled 2) forbids memory hotplug to online all the memory
as movable when !CONFIG_MOVABLE_NODE.

The later restriction is quite dubious because there is no clear cut of
how much normal memory do we need for a reasonable system operation. A
single memory block which is sufficient to allow further movable
onlines is far from sufficient (e.g a node with >2GB and memblocks
128MB will fill up this zone with struct pages leaving nothing for
other allocations). Removing the config option will not only reduce the
configuration space it also removes quite some code.

The semantic of the movable_node command line parameter is preserved.

The first patch removes the restriction mentioned above and the second
one simply removes all the CONFIG_MOVABLE_NODE related stuff.

Shortlog
Michal Hocko (2):
      mm, memory_hotplug: drop artificial restriction on online/offline
      mm, memory_hotplug: drop CONFIG_MOVABLE_NODE

Diffstat:
 Documentation/admin-guide/kernel-parameters.txt |  7 ++-
 drivers/base/node.c                             |  4 --
 include/linux/memblock.h                        | 18 -------
 include/linux/nodemask.h                        |  4 --
 mm/Kconfig                                      | 26 -----------
 mm/memblock.c                                   |  2 -
 mm/memory_hotplug.c                             | 62 -------------------------
 mm/page_alloc.c                                 |  2 -
 8 files changed, 5 insertions(+), 120 deletions(-)

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2017-05-25 12:03 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-24 12:24 [RFC PATCH 0/2] remove CONFIG_MOVABLE_NODE Michal Hocko
2017-05-24 12:24 ` Michal Hocko
2017-05-24 12:24 ` [RFC PATCH 1/2] mm, memory_hotplug: drop artificial restriction on online/offline Michal Hocko
2017-05-24 12:24   ` Michal Hocko
2017-05-24 12:44   ` Vlastimil Babka
2017-05-24 12:44     ` Vlastimil Babka
2017-05-24 12:55     ` Michal Hocko
2017-05-24 12:55       ` Michal Hocko
2017-05-24 21:50   ` Reza Arbab
2017-05-24 21:50     ` Reza Arbab
2017-05-25  6:28     ` Michal Hocko
2017-05-25  6:28       ` Michal Hocko
2017-05-24 12:24 ` [RFC PATCH 2/2] mm, memory_hotplug: drop CONFIG_MOVABLE_NODE Michal Hocko
2017-05-24 12:24   ` Michal Hocko
2017-05-24 12:53   ` Vlastimil Babka
2017-05-24 12:53     ` Vlastimil Babka
2017-05-24 13:42     ` Michal Hocko
2017-05-24 13:42       ` Michal Hocko
2017-05-24 15:17       ` Vlastimil Babka
2017-05-24 15:17         ` Vlastimil Babka
2017-05-25  6:27         ` Michal Hocko
2017-05-25  6:27           ` Michal Hocko
2017-05-25  8:41           ` Vlastimil Babka
2017-05-25  8:41             ` Vlastimil Babka
2017-05-24 13:30   ` Igor Mammedov
2017-05-24 13:30     ` Igor Mammedov
2017-05-24 13:50     ` Michal Hocko
2017-05-24 13:50       ` Michal Hocko
2017-05-24 21:51   ` Reza Arbab
2017-05-24 21:51     ` Reza Arbab
2017-05-25 12:03 ` [RFC PATCH 0/2] remove CONFIG_MOVABLE_NODE Michal Hocko
2017-05-25 12:03   ` Michal Hocko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.