(2012/03/17 2:23), Kautuk Consul wrote: > The comment above __insert_vm_struct seems to suggest that this > function is also going to link the VMA with the anon_vma, but > this is not true. > This function only links the VMA to the mm->mm_rb tree and the mm->mmap linked > list. > > Signed-off-by: Kautuk Consul <consul.kautuk@xxxxxxxxx> > --- > mm/mmap.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/mm/mmap.c b/mm/mmap.c > index da15a79..6328a36 100644 > --- a/mm/mmap.c > +++ b/mm/mmap.c > @@ -452,8 +452,8 @@ static void vma_link(struct mm_struct *mm, struct vm_area_struct *vma, > > /* > * Helper for vma_adjust in the split_vma insert case: > - * insert vm structure into list and rbtree and anon_vma, > - * but it has already been inserted into prio_tree earlier. > + * insert vm structure into list and rbtree, but it has > + * already been inserted into prio_tree earlier. > */ > static void __insert_vm_struct(struct mm_struct *mm, struct vm_area_struct *vma) > { Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>