Re: [patch 2/2] mm: memcg: clean up mm_match_cgroup() signature

2012-10-05 Thread Andrew Morton
On Thu, 4 Oct 2012 14:09:17 -0400 Johannes Weiner wrote: > It really should return a boolean for match/no match. And since it > takes a memcg, not a cgroup, fix that parameter name as well. > > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -84,14 +84,14 @@ extern

Re: [patch 2/2] mm: memcg: clean up mm_match_cgroup() signature

2012-10-05 Thread Andrew Morton
On Thu, 4 Oct 2012 14:09:17 -0400 Johannes Weiner han...@cmpxchg.org wrote: It really should return a boolean for match/no match. And since it takes a memcg, not a cgroup, fix that parameter name as well. --- a/include/linux/memcontrol.h +++ b/include/linux/memcontrol.h @@ -84,14 +84,14

[patch 2/2] mm: memcg: clean up mm_match_cgroup() signature

2012-10-04 Thread Johannes Weiner
It really should return a boolean for match/no match. And since it takes a memcg, not a cgroup, fix that parameter name as well. Signed-off-by: Johannes Weiner Acked-by: Michal Hocko --- include/linux/memcontrol.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff

[patch 2/2] mm: memcg: clean up mm_match_cgroup() signature

2012-10-04 Thread Johannes Weiner
It really should return a boolean for match/no match. And since it takes a memcg, not a cgroup, fix that parameter name as well. Signed-off-by: Johannes Weiner han...@cmpxchg.org Acked-by: Michal Hocko mho...@suse.cz --- include/linux/memcontrol.h | 14 +++--- 1 file changed, 7