Re: [PATCH v2 08/33] mm/rmap: rename vma_interval_tree_*() to mapping_rmap_tree_*()

"Zi Yan" <[email protected]> Fri, 17 Jul 2026 22:54:24 -0400
Newsgroups org.kernel.vger.linux-sgx,dev.linux.lists.damon,dev.linux.lists.iommu,dev.linux.lists.nvdimm,org.freedesktop.lists.dri-devel,org.kernel.vger.kvm,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-kernel,org.kernel.vger.linux-parisc,org.kernel.vger.linux-perf-users,org.kernel.vger.linux-tegra,org.kernel.vger.linux-trace-kernel,org.kvack.linux-mm
Message-ID <[email protected]>
On Fri Jul 10, 2026 at 4:16 PM EDT, Lorenzo Stoakes wrote:
> The family of vma_interval_tree_() functions manipulate the
> address_space (which, of course, is generally referred to as 'mapping')
> reverse mapping, but are named the 'VMA' interval tree.
>
> VMAs may be mapped by an anon_vma, an address_space, or both. Therefore
> calling the mapping interval tree a 'VMA' interval tree is rather
> confusing.
>
> This is also inconsistent with the anon_vma_interval_tree_*() functions
> which explicitly reference the rmap object to which they pertain.
>
> Rename the vma_interval_tree_*() functions to mapping_rmap_tree_*() to
> correct this.
>
> We will rename the anon rmap functions similarly in a subsequent patch.
>
> No functional change intended.
>
> Reviewed-by: Gregory Price <[email protected]>
> Signed-off-by: Lorenzo Stoakes <[email protected]>
> ---
>  arch/arm/mm/fault-armv.c          |  2 +-
>  arch/arm/mm/flush.c               |  2 +-
>  arch/nios2/mm/cacheflush.c        |  2 +-
>  arch/parisc/kernel/cache.c        |  2 +-
>  fs/dax.c                          |  2 +-
>  fs/hugetlbfs/inode.c              |  4 ++--
>  include/linux/mm.h                | 16 ++++++++--------
>  kernel/events/uprobes.c           |  2 +-
>  mm/hugetlb.c                      |  4 ++--
>  mm/interval_tree.c                | 22 +++++++++++-----------
>  mm/khugepaged.c                   |  4 ++--
>  mm/memory-failure.c               |  6 +++---
>  mm/memory.c                       |  2 +-
>  mm/mmap.c                         |  2 +-
>  mm/nommu.c                        |  8 ++++----
>  mm/pagewalk.c                     |  2 +-
>  mm/rmap.c                         |  2 +-
>  mm/vma.c                          | 12 ++++++------
>  tools/testing/vma/include/stubs.h |  4 ++--
>  19 files changed, 50 insertions(+), 50 deletions(-)
>

Just curious, why not file_rmap_tree_*()?

One nice part of this renaming is that the length of each function does
not chagne. That makes the patch easy to review.

Reviewed-by: Zi Yan <[email protected]>

--=20
Best Regards,
Yan, Zi