On Mon, Jun 29, 2026 at 01:23:20PM +0100, Lorenzo Stoakes wrote: > Similar to what we did with mapping_interval_tree*(), let's declare > anon_vma_interval_tree*() in terms of anon_vma rather than rb_root_cached. > > In each case the rb tree referenced is &anon_vma->rb_root, so just pass > anon_vma and the functions can figure this out themselves. > > Additionally, rename 'node' to 'avc', 'index' to 'pgoff_start', and 'last' > to 'pgoff_last' to make clear what is being passed. >
would it be possible to split the pure rename changes out from the changed function declarations? It's hard to pick out this as something that needs to be looked at as more than just a %s/x/y/ > +void anon_vma_interval_tree_insert(struct anon_vma_chain *avc, > + struct anon_vma *anon_vma) ... > - __anon_vma_interval_tree_insert(node, root); > + __anon_vma_interval_tree_insert(avc, &anon_vma->rb_root); an annoying request, sorry ~Gregory
