On Fri Jul 10, 2026 at 4:16 PM EDT, Lorenzo Stoakes wrote: > vma_last_pgoff() already lives there, so it's a bit odd to keep > vma_start_pgoff() in mm/interval_tree.c. Move them together. > > These each return unsigned long, which pgoff_t is typedef'd to. Make this > consistent and have these functions return pgoff_t instead. > > Additionally, express vma_last_pgoff() in terms of vma_start_pgoff(), since > we wrap the vma->vm_pgoff access, we may as well use it here. > > Also while we're here, const-ify the VMA and cleanup a bit. > > Also update the VMA userland tests to reflect the change. > > No functional change intended. > > Reviewed-by: Gregory Price <[email protected]> > Acked-by: David Hildenbrand (Arm) <[email protected]> > Reviewed-by: Pedro Falcato <[email protected]> > Reviewed-by: Vlastimil Babka (SUSE) <[email protected]> > Signed-off-by: Lorenzo Stoakes <[email protected]> > --- > include/linux/mm.h | 9 +++++++-- > mm/interval_tree.c | 5 ----- > tools/testing/vma/include/dup.h | 5 +++++ > 3 files changed, 12 insertions(+), 7 deletions(-) > LGTM.
Reviewed-by: Zi Yan <[email protected]> -- Best Regards, Yan, Zi
