Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-07-19 Thread Michal Hocko
On Mon 15-07-19 12:54:20, David Hildenbrand wrote: [...] > So I'm leaving it like it is. arch_remove_memory() will be mandatory for > architectures implementing arch_add_memory(). I do agree that removing CONFIG_MEMORY_HOTREMOVE makes some sense. But this patch being a mid step should be simpler

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-07-15 Thread David Hildenbrand
On 01.07.19 14:51, Michal Hocko wrote: > On Mon 01-07-19 10:01:41, Michal Hocko wrote: >> On Mon 27-05-19 13:11:47, David Hildenbrand wrote: >>> We want to improve error handling while adding memory by allowing >>> to use arch_remove_memory() and __remove_pages() even if >>>

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-07-01 Thread Michal Hocko
On Mon 01-07-19 10:01:41, Michal Hocko wrote: > On Mon 27-05-19 13:11:47, David Hildenbrand wrote: > > We want to improve error handling while adding memory by allowing > > to use arch_remove_memory() and __remove_pages() even if > > CONFIG_MEMORY_HOTREMOVE is not set to e.g., implement something

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-07-01 Thread Michal Hocko
On Mon 27-05-19 13:11:47, David Hildenbrand wrote: > We want to improve error handling while adding memory by allowing > to use arch_remove_memory() and __remove_pages() even if > CONFIG_MEMORY_HOTREMOVE is not set to e.g., implement something like: > > arch_add_memory() > rc =

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-06-04 Thread Wei Yang
On Tue, Jun 04, 2019 at 08:59:43AM +0200, David Hildenbrand wrote: >On 04.06.19 00:15, Wei Yang wrote: >> Allow arch_remove_pages() or arch_remove_memory()? > >Looks like I merged __remove_pages() and arch_remove_memory(). > >@Andrew, can you fix this up to > >"mm/memory_hotplug: Allow

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-06-04 Thread David Hildenbrand
On 04.06.19 10:31, Wei Yang wrote: > On Tue, Jun 04, 2019 at 08:59:43AM +0200, David Hildenbrand wrote: >> On 04.06.19 00:15, Wei Yang wrote: >>> Allow arch_remove_pages() or arch_remove_memory()? >> >> Looks like I merged __remove_pages() and arch_remove_memory(). >> >> @Andrew, can you fix this

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-06-04 Thread David Hildenbrand
On 04.06.19 00:15, Wei Yang wrote: > Allow arch_remove_pages() or arch_remove_memory()? Looks like I merged __remove_pages() and arch_remove_memory(). @Andrew, can you fix this up to "mm/memory_hotplug: Allow arch_remove_memory() without CONFIG_MEMORY_HOTREMOVE" ? Thanks! > > And want to

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-06-03 Thread Wei Yang
Allow arch_remove_pages() or arch_remove_memory()? And want to confirm the kernel build on affected arch succeed? On Mon, May 27, 2019 at 01:11:47PM +0200, David Hildenbrand wrote: >We want to improve error handling while adding memory by allowing >to use arch_remove_memory() and

Re: [PATCH v3 06/11] mm/memory_hotplug: Allow arch_remove_pages() without CONFIG_MEMORY_HOTREMOVE

2019-05-30 Thread Pavel Tatashin
On Mon, May 27, 2019 at 7:12 AM David Hildenbrand wrote: > > We want to improve error handling while adding memory by allowing > to use arch_remove_memory() and __remove_pages() even if > CONFIG_MEMORY_HOTREMOVE is not set to e.g., implement something like: > > arch_add_memory() >