Re: [PATCH 2/2] mm: Make alloc_contig_range handle in-use hugetlb pages

From: Oscar Salvador
Date: Wed Feb 17 2021 - 08:47:49 EST


On Wed, Feb 17, 2021 at 02:36:31PM +0100, Michal Hocko wrote:
> On Wed 17-02-21 11:08:16, Oscar Salvador wrote:
> > In-use hugetlb pages can be migrated as any other page (LRU
> > and Movable), so let alloc_contig_range handle them.
> >
> > All we need is to succesfully isolate such page.
>
> Again, this is missing a problem statement and a justification why we
> want/need this.

Heh, I was poor in words.

"alloc_contig_range() will fail miserably if it finds a HugeTLB page within
the range without a chance to handle them. Since HugeTLB pages can be migrated
as any other page (LRU and Movable), it does not make sense to bail out.
Enable the interface to recognize in-use HugeTLB pages and have a chance
to migrate them"

What about something along those lines?


--
Oscar Salvador
SUSE L3