[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-20 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-20 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same value so we

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-09 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-09 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same value so we

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 15:51:20, David Rientjes wrote: > On Wed, 8 Jun 2016, Michal Hocko wrote: > > > > Why is the patch asking users to report oom killing of a process that > > > raced with setting /proc/pid/oom_score_adj to OOM_SCORE_ADJ_MIN? What is > > > possibly actionable about it? > > > >

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 15:51:20, David Rientjes wrote: > On Wed, 8 Jun 2016, Michal Hocko wrote: > > > > Why is the patch asking users to report oom killing of a process that > > > raced with setting /proc/pid/oom_score_adj to OOM_SCORE_ADJ_MIN? What is > > > possibly actionable about it? > > > >

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-08 Thread David Rientjes
On Wed, 8 Jun 2016, Michal Hocko wrote: > > Why is the patch asking users to report oom killing of a process that > > raced with setting /proc/pid/oom_score_adj to OOM_SCORE_ADJ_MIN? What is > > possibly actionable about it? > > Well, the primary point is to know whether such races happen in

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-08 Thread David Rientjes
On Wed, 8 Jun 2016, Michal Hocko wrote: > > Why is the patch asking users to report oom killing of a process that > > raced with setting /proc/pid/oom_score_adj to OOM_SCORE_ADJ_MIN? What is > > possibly actionable about it? > > Well, the primary point is to know whether such races happen in

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-08 Thread Michal Hocko
On Tue 07-06-16 15:15:37, David Rientjes wrote: > On Tue, 7 Jun 2016, Oleg Nesterov wrote: > > > On 06/06, David Rientjes wrote: > > > > > > > There is a potential race where we kill the oom disabled task which is > > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > >

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-08 Thread Michal Hocko
On Tue 07-06-16 15:15:37, David Rientjes wrote: > On Tue, 7 Jun 2016, Oleg Nesterov wrote: > > > On 06/06, David Rientjes wrote: > > > > > > > There is a potential race where we kill the oom disabled task which is > > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > >

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-07 Thread David Rientjes
On Tue, 7 Jun 2016, Oleg Nesterov wrote: > On 06/06, David Rientjes wrote: > > > > > There is a potential race where we kill the oom disabled task which is > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > > with select_bad_process and then it is OK to consider the

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-07 Thread David Rientjes
On Tue, 7 Jun 2016, Oleg Nesterov wrote: > On 06/06, David Rientjes wrote: > > > > > There is a potential race where we kill the oom disabled task which is > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > > with select_bad_process and then it is OK to consider the

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-07 Thread Michal Hocko
On Tue 07-06-16 01:20:08, Oleg Nesterov wrote: > On 06/06, David Rientjes wrote: > > > > > There is a potential race where we kill the oom disabled task which is > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > > with select_bad_process and then it is OK to consider

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-07 Thread Michal Hocko
On Tue 07-06-16 01:20:08, Oleg Nesterov wrote: > On 06/06, David Rientjes wrote: > > > > > There is a potential race where we kill the oom disabled task which is > > > highly unlikely but possible. It would happen if __set_oom_adj raced > > > with select_bad_process and then it is OK to consider

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-06 Thread Oleg Nesterov
On 06/06, David Rientjes wrote: > > > There is a potential race where we kill the oom disabled task which is > > highly unlikely but possible. It would happen if __set_oom_adj raced > > with select_bad_process and then it is OK to consider the old value or > > with fork when it should be

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-06 Thread Oleg Nesterov
On 06/06, David Rientjes wrote: > > > There is a potential race where we kill the oom disabled task which is > > highly unlikely but possible. It would happen if __set_oom_adj raced > > with select_bad_process and then it is OK to consider the old value or > > with fork when it should be

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-06 Thread David Rientjes
On Fri, 3 Jun 2016, Michal Hocko wrote: > From: Michal Hocko > > Currently oom_kill_process skips both the oom reaper and SIG_KILL if a > process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, > oom_adj: make sure processes sharing mm have same view of

Re: [PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-06 Thread David Rientjes
On Fri, 3 Jun 2016, Michal Hocko wrote: > From: Michal Hocko > > Currently oom_kill_process skips both the oom reaper and SIG_KILL if a > process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, > oom_adj: make sure processes sharing mm have same view of oom_score_adj" > all

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-03 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-03 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same value so we