diff options
author | Andrea Arcangeli <aarcange@redhat.com> | 2010-08-09 20:19:07 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-08-09 23:44:58 -0400 |
commit | 5e549e989f94de0596b8149a90e0088e7d4d7c97 (patch) | |
tree | 2a20fdb475ea0320310a4a770da72cd54ea9347f /mm/mmap.c | |
parent | ff36b801624d02a876bb7deded6ab860ea3503f2 (diff) |
mmap: remove unnecessary lock from __vma_link
There's no anon-vma related mangling happening inside __vma_link anymore
so no need of anon_vma locking there.
Signed-off-by: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Rik van Riel <riel@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/mmap.c')
-rw-r--r-- | mm/mmap.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -452,12 +452,10 @@ static void vma_link(struct mm_struct *mm, struct vm_area_struct *vma, | |||
452 | spin_lock(&mapping->i_mmap_lock); | 452 | spin_lock(&mapping->i_mmap_lock); |
453 | vma->vm_truncate_count = mapping->truncate_count; | 453 | vma->vm_truncate_count = mapping->truncate_count; |
454 | } | 454 | } |
455 | vma_lock_anon_vma(vma); | ||
456 | 455 | ||
457 | __vma_link(mm, vma, prev, rb_link, rb_parent); | 456 | __vma_link(mm, vma, prev, rb_link, rb_parent); |
458 | __vma_link_file(vma); | 457 | __vma_link_file(vma); |
459 | 458 | ||
460 | vma_unlock_anon_vma(vma); | ||
461 | if (mapping) | 459 | if (mapping) |
462 | spin_unlock(&mapping->i_mmap_lock); | 460 | spin_unlock(&mapping->i_mmap_lock); |
463 | 461 | ||