Thread (39 messages) 39 messages, 6 authors, 2012-12-05
STALE4956d

[PATCH 2/2, v2] mm/migration: Make rmap_walk_anon() and try_to_unmap_anon() more scalable

From: Ingo Molnar <mingo@kernel.org>
Date: 2012-12-02 15:12:41
Also in: lkml
Subsystem: hwpoison memory failure handling, memory management, memory management - ksm (kernel samepage merging), memory management - memory policy and migration, memory management - rmap (reverse mapping), memory management - thp (transparent huge page), memory mapping, the rest · Maintainers: Miaohe Lin, Andrew Morton, David Hildenbrand, Lorenzo Stoakes, Liam R. Howlett, Linus Torvalds

* Rik van Riel [off-list ref] wrote:
quoted
+static inline void anon_vma_lock_read(struct anon_vma *anon_vma)
+{
+	down_read(&anon_vma->root->rwsem);
+}
I see you did not rename anon_vma_lock and anon_vma_unlock to 
anon_vma_lock_write and anon_vma_unlock_write.

That could get confusing to people touching that code in the 
future.
Agreed, doing that rename makes perfect sense - I've done that 
in the v2 version attached below.

Thanks,

	Ingo

----------------------->
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help