Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-02-01 Thread David Rientjes
On Wed, 31 Jan 2018, Michal Hocko wrote: > > > > > root > > > > > / | \ > > > > > A B C > > > > >/ \/ \ > > > > > D E F G > > > > > > > > > > Assume A: cgroup, B: oom_group=1, C: tree, G: oom_group=1 > > > > > > > > > > > > > At each level

Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-31 Thread Michal Hocko
On Tue 30-01-18 14:38:40, David Rientjes wrote: > On Tue, 30 Jan 2018, Michal Hocko wrote: > > > > > So what is the actual semantic and scope of this policy. Does it apply > > > > only down the hierarchy. Also how do you compare cgroups with different > > > > policies? Let's say you have > > > >

Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-30 Thread David Rientjes
On Tue, 30 Jan 2018, Michal Hocko wrote: > > > So what is the actual semantic and scope of this policy. Does it apply > > > only down the hierarchy. Also how do you compare cgroups with different > > > policies? Let's say you have > > > root > > > / | \ > > > A B C

Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-30 Thread Michal Hocko
On Mon 29-01-18 14:38:02, David Rientjes wrote: > On Fri, 26 Jan 2018, Michal Hocko wrote: > > > > The cgroup aware oom killer is needlessly declared for the entire system > > > by a mount option. It's unnecessary to force the system into a single > > > oom policy: either cgroup aware, or the

Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-29 Thread David Rientjes
On Fri, 26 Jan 2018, Michal Hocko wrote: > > The cgroup aware oom killer is needlessly declared for the entire system > > by a mount option. It's unnecessary to force the system into a single > > oom policy: either cgroup aware, or the traditional process aware. > > > > This patch introduces a

Re: [patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-27 Thread Michal Hocko
On Thu 25-01-18 15:53:45, David Rientjes wrote: > The cgroup aware oom killer is needlessly declared for the entire system > by a mount option. It's unnecessary to force the system into a single > oom policy: either cgroup aware, or the traditional process aware. > > This patch introduces a

[patch -mm v2 1/3] mm, memcg: introduce per-memcg oom policy tunable

2018-01-25 Thread David Rientjes
The cgroup aware oom killer is needlessly declared for the entire system by a mount option. It's unnecessary to force the system into a single oom policy: either cgroup aware, or the traditional process aware. This patch introduces a memory.oom_policy tunable for all mem cgroups. It is currently