Messages in this thread | | | Date | Tue, 26 Jun 2012 09:05:56 -0400 | From | Rik van Riel <> | Subject | Re: [PATCH -mm v2 01/11] mm: track free size between VMAs in VMA rbtree |
| |
On 06/26/2012 04:31 AM, Peter Zijlstra wrote:
> If you look at your patch 1, __vma_unlink has an adjust_free_gap() right > next to the rb_augment_erase(), vma_adjust() has 3 adjust_free_gap() > calls right next to each other. > > All these will do an entire path walk back to the root. I would think we > could save quite a bit of updating by not having them all walk back to > the root. No point in re-computing the top levels if you know the next > update will change them again anyway.
The problem is, unless we look at the augmented data at rotate time, we do not know when it is safe to stop iterating up the tree.
-- All rights reversed
| |