Re: [RFC PATCH v2 0/6] mm, memory_hotplug: allocate memmap from hotadded memory

From: Michal Hocko <mhocko@kernel.org>
To: Dan Williams <dan.j.williams@intel.com>
Cc: Linux MM <linux-mm@kvack.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Mel Gorman <mgorman@suse.de>, Vlastimil Babka <vbabka@suse.cz>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Jerome Glisse <jglisse@redhat.com>,
	Reza Arbab <arbab@linux.vnet.ibm.com>,
	Yasuaki Ishimatsu <yasu.isimatu@gmail.com>,
	qiuxishi@huawei.com, Kani Toshimitsu <toshi.kani@hpe.com>,
	slaoub@gmail.com, Joonsoo Kim <js1304@gmail.com>,
	Andi Kleen <ak@linux.intel.com>,
	Daniel Kiper <daniel.kiper@oracle.com>,
	Igor Mammedov <imammedo@redhat.com>,
	Vitaly Kuznetsov <vkuznets@redhat.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Fenghua Yu <fenghua.yu@intel.com>,
	Gerald Schaefer <gerald.schaefer@de.ibm.com>,
	Heiko Carstens <heiko.carstens@de.ibm.com>,
	"H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@redhat.com>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>,
	Michael Ellerman <mpe@ellerman.id.au>,
	Paul Mackerras <paulus@samba.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Tony Luck <tony.luck@intel.com>,
	Will Deacon <will.deacon@arm.com>, X86 ML <x86@kernel.org>
Subject: Re: [RFC PATCH v2 0/6] mm, memory_hotplug: allocate memmap from hotadded memory
Date: Thu, 10 Aug 2017 17:33:01 +0200	[thread overview]
Message-ID: <20170810153300.GA23863@dhcp22.suse.cz> (raw)
In-Reply-To: <CAPcyv4jxdhJ0VSViqKxBzD0-H+3dvrWh3OOmM1diX7i_go_4Mg@mail.gmail.com>

On Thu 10-08-17 08:27:56, Dan Williams wrote:
> On Thu, Aug 10, 2017 at 4:40 AM, Michal Hocko <mhocko@kernel.org> wrote:
> > On Tue 08-08-17 13:01:36, Dan Williams wrote:
> >> On Mon, Aug 7, 2017 at 12:00 AM, Michal Hocko <mhocko@kernel.org> wrote:
> >> > Any comments? Especially for the arch specific? Has anybody had a chance
> >> > to test this? I do not want to rush this but I would be really glag if
> >> > we could push this work in 4.14 merge window.
> >>
> >> Hi Michal,
> >>
> >> I'm interested in taking a look at this especially if we might be able
> >> to get rid of vmem_altmap, but this is currently stuck behind some
> >> other work in my queue. I'll try to circle back in the next couple
> >> weeks.
> >
> > Well, vmem_altmap was there and easy to reuse. Replacing with something
> > else is certainly possible but I really need something to hook a
> > dedicated allocator into vmemmap code.
> 
> Oh, you're reusing it, that's great. Then I definitely got the wrong
> impression from the first glance at the patch set, I'll dig deeper.

yeah, reusing and extending a bit so I would highly appreciate if you
could have a look I am doing that in a sane way.

-- 
Michal Hocko
SUSE Labs