Re: [PATCH 8/8] memcg: get rid of mem_cgroup_from_task

2015-07-09 Thread Michal Hocko
On Thu 09-07-15 17:32:47, Vladimir Davydov wrote: > On Thu, Jul 09, 2015 at 04:13:21PM +0200, Michal Hocko wrote: > > On Wed 08-07-15 20:43:31, Vladimir Davydov wrote: > > > On Wed, Jul 08, 2015 at 02:27:52PM +0200, Michal Hocko wrote: > > [...] > > > > @@ -1091,12 +1079,14 @@ bool task_in_mem_cgro

Re: [PATCH 8/8] memcg: get rid of mem_cgroup_from_task

2015-07-09 Thread Vladimir Davydov
On Thu, Jul 09, 2015 at 04:13:21PM +0200, Michal Hocko wrote: > On Wed 08-07-15 20:43:31, Vladimir Davydov wrote: > > On Wed, Jul 08, 2015 at 02:27:52PM +0200, Michal Hocko wrote: > [...] > > > @@ -1091,12 +1079,14 @@ bool task_in_mem_cgroup(struct task_struct *task, > > > struct mem_cgroup *memcg

Re: [PATCH 8/8] memcg: get rid of mem_cgroup_from_task

2015-07-09 Thread Michal Hocko
On Wed 08-07-15 20:43:31, Vladimir Davydov wrote: > On Wed, Jul 08, 2015 at 02:27:52PM +0200, Michal Hocko wrote: [...] > > @@ -1091,12 +1079,14 @@ bool task_in_mem_cgroup(struct task_struct *task, > > struct mem_cgroup *memcg) > > task_unlock(p); > > } else { > > /* >

Re: [PATCH 8/8] memcg: get rid of mem_cgroup_from_task

2015-07-08 Thread Vladimir Davydov
On Wed, Jul 08, 2015 at 02:27:52PM +0200, Michal Hocko wrote: > From: Michal Hocko > > mem_cgroup_from_task has always been a tricky API. It was added > by 78fb74669e80 ("Memory controller: accounting setup") for > mm_struct::mem_cgroup initialization. Later on it gained new callers > mostly due