Messages in this thread | | | Date | Thu, 10 Jun 2021 10:14:36 -0400 | From | Peter Xu <> | Subject | Re: [PATCH 01/11] mm: page_vma_mapped_walk(): use page for pvmw->page |
| |
On Thu, Jun 10, 2021 at 11:55:22AM +0300, Kirill A. Shutemov wrote: > > @@ -234,9 +233,9 @@ bool page_vma_mapped_walk(struct page_vma_mapped_walk *pvmw) > > return true; > > next_pte: > > /* Seek to next pte only makes sense for THP */ > > - if (!PageTransHuge(pvmw->page) || PageHuge(pvmw->page)) > > + if (!PageTransHuge(page) || PageHuge(page)) > > return not_found(pvmw); > > - end = vma_address_end(pvmw->page, pvmw->vma); > > + end = vma_address_end(page, pvmw->vma); > > do { > > pvmw->address += PAGE_SIZE; > > if (pvmw->address >= end) > > I see two more pvmw->page in this loop. Do you leave them here as the code > will be rewritten later in the patchset?
I think they've got removed in previous series ("[PATCH v2 04/10] mm/thp: fix vma_address() if virtual address below file offset").
Reviewed-by: Peter Xu <peterx@redhat.com>
-- Peter Xu
| |