On 5/5/21 11:43 PM, Dan Williams wrote: > I suspect it's a good sign I'm only finding cosmetic and changelog > changes in the review... Hopefully it continues that way, but the meat of the series is located in patches 4, 6, 7 and 11. *Specially* 6 and 7. I very strongly suspect I am gonna get non-cosmetic comments there. > I have some more: > > A year for now if I'm tracking down a problem and looking through mm > commits I would appreciate a subject line like the following: > "refactor core of vmemmap_populate_basepages() to helper" that gives > an idea of the impact and side effects of the change. > Fixed. > On Thu, Mar 25, 2021 at 4:10 PM Joao Martins <joao.m.mart...@oracle.com> > wrote: >> > > I would add a lead in phrase like: "In preparation for describing a > memmap with compound pages, move the actual..." > >> Move the actual pte population logic into a separate function >> vmemmap_populate_address() and have vmemmap_populate_basepages() >> walk through all base pages it needs to populate. > > Aside from the above, looks good. > Cool! _______________________________________________ Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org To unsubscribe send an email to linux-nvdimm-le...@lists.01.org
- [PATCH v1 00/11] mm, sparse-vmemmap: Introduce compound pag... Joao Martins
- [PATCH v1 07/11] mm/sparse-vmemmap: populate compound ... Joao Martins
- Re: [PATCH v1 07/11] mm/sparse-vmemmap: populate c... Dan Williams
- Re: [PATCH v1 07/11] mm/sparse-vmemmap: popula... Joao Martins
- Re: [PATCH v1 07/11] mm/sparse-vmemmap: po... Dan Williams
- Re: [PATCH v1 07/11] mm/sparse-vmemma... Joao Martins
- [PATCH v1 09/11] mm/page_alloc: reuse tail struct page... Joao Martins
- [PATCH v1 06/11] mm/sparse-vmemmap: refactor vmemmap_p... Joao Martins
- Re: [PATCH v1 06/11] mm/sparse-vmemmap: refactor v... Dan Williams
- Re: [PATCH v1 06/11] mm/sparse-vmemmap: refact... Joao Martins
- Re: [PATCH v1 06/11] mm/sparse-vmemmap: re... Joao Martins
- [PATCH v1 04/11] mm/memremap: add ZONE_DEVICE support ... Joao Martins
- Re: [PATCH v1 04/11] mm/memremap: add ZONE_DEVICE ... Dan Williams
- Re: [PATCH v1 04/11] mm/memremap: add ZONE_DEV... Matthew Wilcox
- Re: [PATCH v1 04/11] mm/memremap: add ZONE_DEV... Joao Martins
- Re: [PATCH v1 04/11] mm/memremap: add ZONE... Dan Williams
- Re: [PATCH v1 04/11] mm/memremap: add... Joao Martins
- Re: [PATCH v1 04/11] mm/memremap:... Dan Williams
- Re: [PATCH v1 04/11] mm/memre... Joao Martins
- Re: [PATCH v1 04/11] mm/memremap:... Joao Martins