Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-06-01 Thread Michal Hocko
On Thu 31-05-18 13:41:38, Eric W. Biederman wrote: > Michal Hocko writes: > > > On Thu 24-05-18 14:16:35, Andrew Morton wrote: > >> On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: > >> > >> > I would really prefer and appreciate a repost with all the fixes folded > >> > in. > >> > >> [1

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-31 Thread Eric W. Biederman
Michal Hocko writes: > On Thu 24-05-18 14:16:35, Andrew Morton wrote: >> On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: >> >> > I would really prefer and appreciate a repost with all the fixes folded >> > in. >> >> [1/2] > > Thanks Andrew and sorry it took so long! This seems to be mis

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-31 Thread Eric W. Biederman
Michal Hocko writes: > On Wed 23-05-18 14:46:43, Eric W. Biederman wrote: > [...] >> As two processes sharing an mm is useless and highly unlikely there is >> no need to handle this case well, it just needs to be handled well >> enough to prevent an indefinite loop. So when css_tryget_online fai

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-30 Thread Michal Hocko
On Thu 24-05-18 14:16:35, Andrew Morton wrote: > On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: > > > I would really prefer and appreciate a repost with all the fixes folded > > in. > > [1/2] Thanks Andrew and sorry it took so long! This seems to be missing the fix for the issue I've me

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-30 Thread Michal Hocko
On Wed 23-05-18 14:46:43, Eric W. Biederman wrote: [...] > As two processes sharing an mm is useless and highly unlikely there is > no need to handle this case well, it just needs to be handled well > enough to prevent an indefinite loop. So when css_tryget_online fails > just treat the mm as belo

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-24 Thread Andrea Parri
On Thu, May 24, 2018 at 02:16:35PM -0700, Andrew Morton wrote: > On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: > > > I would really prefer and appreciate a repost with all the fixes folded > > in. > > [1/2] > > From: "Eric W. Biederman" > Subject: memcg: replace mm->owner with mm->mem

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-24 Thread Andrew Morton
On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: > I would really prefer and appreciate a repost with all the fixes folded > in. [2/2] From: Andrew Morton Subject: mm/memcontrol.c: add mem_cgroup_from_task() as a local helper Factor out some commonly-occurring code. Cc: "Eric W. Bieder

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-24 Thread Andrew Morton
On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote: > I would really prefer and appreciate a repost with all the fixes folded > in. [1/2] From: "Eric W. Biederman" Subject: memcg: replace mm->owner with mm->memcg Recently it was reported that mm_update_next_owner could get into cases where

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-24 Thread Michal Hocko
On Wed 23-05-18 14:46:43, Eric W. Biederman wrote: > Michal Hocko writes: > > > On Thu 10-05-18 14:14:18, Michal Hocko wrote: > >> On Fri 04-05-18 12:26:03, Eric W. Biederman wrote: > >> > > >> > Andrew can you pick up these two fixes as well. > >> > > >> > These address the issues Michal Hocko

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-23 Thread Eric W. Biederman
Michal Hocko writes: > On Thu 10-05-18 14:14:18, Michal Hocko wrote: >> On Fri 04-05-18 12:26:03, Eric W. Biederman wrote: >> > >> > Andrew can you pick up these two fixes as well. >> > >> > These address the issues Michal Hocko and Oleg Nesterov noticed. >> >> I completely got lost in this th

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-22 Thread Michal Hocko
On Thu 10-05-18 14:14:18, Michal Hocko wrote: > On Fri 04-05-18 12:26:03, Eric W. Biederman wrote: > > > > Andrew can you pick up these two fixes as well. > > > > These address the issues Michal Hocko and Oleg Nesterov noticed. > > I completely got lost in this thread. There are way too many thi

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-10 Thread Michal Hocko
On Thu 10-05-18 14:14:18, Michal Hocko wrote: > On Fri 04-05-18 12:26:03, Eric W. Biederman wrote: > > > > Andrew can you pick up these two fixes as well. > > > > These address the issues Michal Hocko and Oleg Nesterov noticed. > > I completely got lost in this thread. There are way too many thi

Re: [PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-10 Thread Michal Hocko
On Fri 04-05-18 12:26:03, Eric W. Biederman wrote: > > Andrew can you pick up these two fixes as well. > > These address the issues Michal Hocko and Oleg Nesterov noticed. I completely got lost in this thread. There are way too many things discussed at once. Could you repost the full series for

[PATCH 0/2] mm->owner to mm->memcg fixes

2018-05-04 Thread Eric W. Biederman
Andrew can you pick up these two fixes as well. These address the issues Michal Hocko and Oleg Nesterov noticed. Eric W. Biederman (2): memcg: Update the mm->memcg maintenance to work when !CONFIG_MMU memcg: Close the race between migration and installing bprm->mm as mm fs/exec.c