Re: [patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor()... | linux-mm

[patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor() to callsite · Johannes Weiner <hannes@cmpxchg.org> · 2012-02-28
[patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Johannes Weiner <hannes@cmpxchg.org> · 2012-02-28
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Konstantin Khlebnikov <hidden> · 2012-02-28
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · KAMEZAWA Hiroyuki <hidden> · 2012-02-29
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Johannes Weiner <hannes@cmpxchg.org> · 2012-02-29
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · KAMEZAWA Hiroyuki <hidden> · 2012-02-29
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Michal Hocko <hidden> · 2012-03-14
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Johannes Weiner <hannes@cmpxchg.org> · 2012-03-14
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Michal Hocko <hidden> · 2012-03-14
Re: [patch 2/2] mm: memcg: count pte references from every member of the reclaimed hierarchy · Michal Hocko <hidden> · 2012-03-14
Re: [patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor() to callsite · KAMEZAWA Hiroyuki <hidden> · 2012-02-29
Re: [patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor() to callsite · Li Zefan <hidden> · 2012-03-01
Re: [patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor() to callsite · Michal Hocko <hidden> · 2012-03-14

Re: [patch 1/2] kernel: cgroup: push rcu read locking from css_is_ancestor() to callsite

From: KAMEZAWA Hiroyuki <hidden>
Date: 2012-02-29 00:01:36
Also in: lkml

On Tue, 28 Feb 2012 15:14:48 +0100
Johannes Weiner [off-list ref] wrote:

Library functions should not grab locks when the callsites can do it,
even if the lock nests like the rcu read-side lock does.

Push the rcu_read_lock() from css_is_ancestor() to its single user,
mem_cgroup_same_or_subtree() in preparation for another user that may
already hold the rcu read-side lock.

Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>

Acked-by: KAMEZAWA Hiroyuki <redacted>


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  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@kvack.org"> email@kvack.org </a>

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help