diff options
author | Konstantin Khlebnikov <khlebnikov@openvz.org> | 2013-02-22 19:34:40 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-23 20:50:17 -0500 |
commit | 08b52706d505658eac0962d215ff697f898bbc13 (patch) | |
tree | b732ff7986179ab4e767ccbf5f6909a67cdb03c3 /mm/rmap.c | |
parent | ec8acf20afb8534ed511f6613dd2226b9e301010 (diff) |
mm/rmap: rename anon_vma_unlock() => anon_vma_unlock_write()
The comment in commit 4fc3f1d66b1e ("mm/rmap, migration: Make
rmap_walk_anon() and try_to_unmap_anon() more scalable") says:
| Rename anon_vma_[un]lock() => anon_vma_[un]lock_write(),
| to make it clearer that it's an exclusive write-lock in
| that case - suggested by Rik van Riel.
But that commit renames only anon_vma_lock()
Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org>
Cc: Ingo Molnar <mingo@kernel.org>
Reviewed-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/rmap.c')
-rw-r--r-- | mm/rmap.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -105,7 +105,7 @@ static inline void anon_vma_free(struct anon_vma *anon_vma) | |||
105 | */ | 105 | */ |
106 | if (rwsem_is_locked(&anon_vma->root->rwsem)) { | 106 | if (rwsem_is_locked(&anon_vma->root->rwsem)) { |
107 | anon_vma_lock_write(anon_vma); | 107 | anon_vma_lock_write(anon_vma); |
108 | anon_vma_unlock(anon_vma); | 108 | anon_vma_unlock_write(anon_vma); |
109 | } | 109 | } |
110 | 110 | ||
111 | kmem_cache_free(anon_vma_cachep, anon_vma); | 111 | kmem_cache_free(anon_vma_cachep, anon_vma); |
@@ -191,7 +191,7 @@ int anon_vma_prepare(struct vm_area_struct *vma) | |||
191 | avc = NULL; | 191 | avc = NULL; |
192 | } | 192 | } |
193 | spin_unlock(&mm->page_table_lock); | 193 | spin_unlock(&mm->page_table_lock); |
194 | anon_vma_unlock(anon_vma); | 194 | anon_vma_unlock_write(anon_vma); |
195 | 195 | ||
196 | if (unlikely(allocated)) | 196 | if (unlikely(allocated)) |
197 | put_anon_vma(allocated); | 197 | put_anon_vma(allocated); |
@@ -308,7 +308,7 @@ int anon_vma_fork(struct vm_area_struct *vma, struct vm_area_struct *pvma) | |||
308 | vma->anon_vma = anon_vma; | 308 | vma->anon_vma = anon_vma; |
309 | anon_vma_lock_write(anon_vma); | 309 | anon_vma_lock_write(anon_vma); |
310 | anon_vma_chain_link(vma, avc, anon_vma); | 310 | anon_vma_chain_link(vma, avc, anon_vma); |
311 | anon_vma_unlock(anon_vma); | 311 | anon_vma_unlock_write(anon_vma); |
312 | 312 | ||
313 | return 0; | 313 | return 0; |
314 | 314 | ||