On 23.12.24 23:00, Luiz Capitulino wrote:
The commit 387ba26fb1cb ("mm/page_alloc: add a bulk page allocator") added
__alloc_pages_bulk() along with the page_list argument. The next
commit 0f87d9d30f21 ("mm/page_alloc: add an array-based interface to the
bulk page allocator") added the array-based argument. As it turns out, the
page_list argument has no users in the current tree (if it ever had any).
Dropping it allows for a slight simplification and eliminates some
unnecessary checks, now that page_array is required.
Also, note that the removal of the page_list argument was proposed before
in the thread below, where Matthew Wilcox mentions that:
"""
Iterating a linked list is _expensive_. It is about 10x quicker to
iterate an array than a linked list.
"""
(https://lore.kernel.org/linux-mm/20231025093254.xvomlctwhcuerzky@xxxxxxxxxxxxxxxxxxx)
Signed-off-by: Luiz Capitulino <luizcap@xxxxxxxxxx>
---
Acked-by: David Hildenbrand <david@xxxxxxxxxx>
Thanks!
--
Cheers,
David / dhildenb
