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 <[email protected]>
>> Cc: Andrew Morton <[email protected]>
>> Cc: Vlastimil Babka <[email protected]>
>> Cc: Michal Hocko <[email protected]>
>> Cc: David Rientjes <[email protected]>
>> Cc: Andrea Arcangeli <[email protected]>
>> Cc: Oscar Salvador <[email protected]>
>> Cc: Mel Gorman <[email protected]>
>> Cc: Mike Rapoport <[email protected]>
>> Cc: Dan Williams <[email protected]>
>> Cc: Pavel Tatashin <[email protected]>
>> Cc: Matthew Wilcox <[email protected]>
>> Cc: David Hildenbrand <[email protected]>
>> Cc: [email protected]
>> Acked-by: Michal Hocko <[email protected]>
>> Signed-off-by: Anshuman Khandual <[email protected]>
>> ---
>> 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
