Re: [PATCH] mm:Make the function alloc_mem_cgroup_per_zone_info bool

2015-06-29 Thread Michal Hocko
On Mon 29-06-15 11:44:24, nick wrote: [...] > Here is a patch series I have been trying to merge for a bug in the > gma500 other the last few patches. There are other patches I have like > this lying around. I am not interested in looking at random and unrelated patches. I am also not thrilled abo

Re: [PATCH] mm:Make the function alloc_mem_cgroup_per_zone_info bool

2015-06-29 Thread Michal Hocko
On Mon 29-06-15 11:23:08, Nicholas Krause wrote: [...] > I agree with and looked into the callers about this wasn't sure if you > you wanted me to return - ENOMEM. I will rewrite this patch the other > way. I am not sure this path really needs a cleanup. > Furthermore I apologize about this and

Re: [PATCH] mm:Make the function alloc_mem_cgroup_per_zone_info bool

2015-06-29 Thread Michal Hocko
On Mon 29-06-15 10:13:53, Nicholas Krause wrote: [...] > -static int alloc_mem_cgroup_per_zone_info(struct mem_cgroup *memcg, int node) > +static bool alloc_mem_cgroup_per_zone_info(struct mem_cgroup *memcg, int > node) > { > struct mem_cgroup_per_node *pn; > struct mem_cgroup_per_zon