On 10/16/2019 04:39 PM, David Hildenbrand wrote:
> On 16.10.19 13:02, Anshuman Khandual wrote:
>> HugeTLB helper alloc_gigantic_page() implements fairly generic allocation
>> method where it scans over various zones looking for a large contiguous pfn
>> range before trying to allocate it with alloc_contig_range(). Other than
>> deriving the requested order from 'struct hstate', there is nothing HugeTLB
>> specific in there. This can be made available for general use to allocate
>> contiguous memory which could not have been allocated through the buddy
>> allocator.
>>
>> alloc_gigantic_page() has been split carving out actual allocation method
>> which is then made available via new alloc_contig_pages() helper wrapped
>> under CONFIG_CONTIG_ALLOC. All references to 'gigantic' have been replaced
>> with more generic term 'contig'. Allocated pages here should be freed with
>> free_contig_range() or by calling __free_page() on each allocated page.
>>
>> Cc: Mike Kravetz <mike.krav...@oracle.com>
>> Cc: Andrew Morton <a...@linux-foundation.org>
>> Cc: Vlastimil Babka <vba...@suse.cz>
>> Cc: Michal Hocko <mho...@suse.com>
>> Cc: David Rientjes <rient...@google.com>
>> Cc: Andrea Arcangeli <aarca...@redhat.com>
>> Cc: Oscar Salvador <osalva...@suse.de>
>> Cc: Mel Gorman <mgor...@techsingularity.net>
>> Cc: Mike Rapoport <r...@linux.ibm.com>
>> Cc: Dan Williams <dan.j.willi...@intel.com>
>> Cc: Pavel Tatashin <pavel.tatas...@microsoft.com>
>> Cc: Matthew Wilcox <wi...@infradead.org>
>> Cc: David Hildenbrand <da...@redhat.com>
>> Cc: linux-kernel@vger.kernel.org
>> Acked-by: Michal Hocko <mho...@suse.com>
>> Signed-off-by: Anshuman Khandual <anshuman.khand...@arm.com>
>> ---
>> This is based on https://patchwork.kernel.org/patch/11190213/
>>
>> Changes in V2:
>>
>> - Rephrased patch subject per David
>> - Fixed all typos per David
>> - s/order/contiguous
>
> Just to make sure, you ignored my comment regarding alignment although I
> explicitly mentioned it a second time? Thanks.
I had asked Michal explicitly what to be included for the respin. Anyways
seems like the previous thread is active again. I am happy to incorporate
anything new getting agreed on there.
- Anshuman
