Re: [PATCH v1 07/11] mm/sparse-vmemmap: populate compound pagemaps

Dan Williams <[email protected]> Mon, 10 May 2021 12:19:56 -0700
Newsgroups org.01.lists.linux-nvdimm,org.kvack.linux-mm
Message-ID <CAPcyv4guVWx0TdEe_Z+CQEbRPDk9LGcgb87pb=g8XdRyeUaPcQ@mail.gmail.com>
On Thu, May 6, 2021 at 4:02 AM Joao Martins <[email protected]> wrote:
[..]
> >> +static pte_t * __meminit vmemmap_lookup_address(unsigned long addr)
> >
> > I think this can be replaced with a call to follow_pte(&init_mm...).
> >
>
> Ah, of course! That would shorten things up too.

Now that I look closely, I notice that function disclaims being
suitable as a general purpose pte lookup utility. If it works for you,
great, but if not I think it's past time to create such a helper. I
know Ira is looking for one, and I contributed to the proliferation
when I added dev_pagemap_mapping_shift().
_______________________________________________
Linux-nvdimm mailing list -- [email protected]
To unsubscribe send an email to [email protected]