[PATCH 1/2] mm: Allow architectures to request 'old' entries when prefaulting
torvalds at linux-foundation.org
Sat Dec 26 12:57:13 EST 2020
On Fri, Dec 25, 2020 at 3:31 AM Kirill A. Shutemov <kirill at shutemov.name> wrote:
> The new helper next_page() returns a stablized page, so filemap_map_pmd()
> can clearly decide if we should set up a page table or a huge page.
I really like that next_page() abstraction, my only comment is that I
think it should be renamed as "next_stable_page()" or something, and
then this part:
+ if (!page)
+ page = xas_find(xas, end_pgoff);
+ page = xas_next_entry(xas, end_pgoff);
should be in the caller.
Then just have two helper functions like 'first_map_page()' and
'next_map_page()' which just do
Because not only does that get rid of the "if (page)" test, I think it
would make things a bit clearer. When I read the patch first, the
initial "next_page()" call confused me.
But maybe I'm just grasping at straws. Even in this format, I think
it's a nice cleanup and makes for more understandable code.
More information about the linux-arm-kernel