Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-31 Thread Oleg Nesterov
On 05/31, Michal Hocko wrote: > > On Mon 30-05-16 20:18:16, Oleg Nesterov wrote: > > > > perhaps the is_global_init() == T case needs a warning too? the previous > > changes > > take care about vfork() from /sbin/init, so the only reason we can see it > > true > > is that /sbin/init shares the

Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-31 Thread Oleg Nesterov
On 05/31, Michal Hocko wrote: > > On Mon 30-05-16 20:18:16, Oleg Nesterov wrote: > > > > perhaps the is_global_init() == T case needs a warning too? the previous > > changes > > take care about vfork() from /sbin/init, so the only reason we can see it > > true > > is that /sbin/init shares the

Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-31 Thread Michal Hocko
On Mon 30-05-16 20:18:16, Oleg Nesterov wrote: > On 05/30, Michal Hocko wrote: > > > > @@ -852,8 +852,7 @@ void oom_kill_process(struct oom_control *oc, struct > > task_struct *p, > > continue; > > if (same_thread_group(p, victim)) > > continue;

Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-31 Thread Michal Hocko
On Mon 30-05-16 20:18:16, Oleg Nesterov wrote: > On 05/30, Michal Hocko wrote: > > > > @@ -852,8 +852,7 @@ void oom_kill_process(struct oom_control *oc, struct > > task_struct *p, > > continue; > > if (same_thread_group(p, victim)) > > continue;

Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-30 Thread Oleg Nesterov
On 05/30, Michal Hocko wrote: > > @@ -852,8 +852,7 @@ void oom_kill_process(struct oom_control *oc, struct > task_struct *p, > continue; > if (same_thread_group(p, victim)) > continue; > - if (unlikely(p->flags & PF_KTHREAD) ||

Re: [PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-30 Thread Oleg Nesterov
On 05/30, Michal Hocko wrote: > > @@ -852,8 +852,7 @@ void oom_kill_process(struct oom_control *oc, struct > task_struct *p, > continue; > if (same_thread_group(p, victim)) > continue; > - if (unlikely(p->flags & PF_KTHREAD) ||

[PATCH 5/6] mm, oom: kill all tasks sharing the mm

2016-05-30 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 5/6] mm, oom: kill all tasks sharing the mm

2016-05-30 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 5/6] mm, oom: kill all tasks sharing the mm

2016-05-26 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 5/6] mm, oom: kill all tasks sharing the mm

2016-05-26 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