On Mon, Jun 29, 2026 at 01:23:16PM +0100, Lorenzo Stoakes wrote: > Update the file comment to clarify that both file-backed and anonymous > interval trees are provided, referencing the relevant data types for > clarity. > > Also add comments to indicate which parts of the file apply to each. > > While we're here, convert the VM_BUG_ON_VMA() to VM_WARN_ON_ONCE_VMA(). > > Signed-off-by: Lorenzo Stoakes <[email protected]>
Reviewed-by: Pedro Falcato <[email protected]> This is fine for now, but I'm wondering if it doesn't make sense to, in the long term, have: mm/rmap.c - common rmap mechanisms mm/anon_rmap.c - anon rmap gunk mm/file_rmap.c - file rmap gunk or even something like mm/rmap/{core,anon,file,ksm??}.c While working on my file rmap patches I noticed there's so much stuff just splurged all over rmap.c - interval_tree.c - fs.h - fs/inode.c. It's a little silly. -- Pedro
