Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-18 Thread One Thousand Gnomes
> And accessing the emergency reserves means we are definitely no longer > A-OK, this is not comparable to the first direct reclaim invocation. > > We exhausted our options and we got really lucky. It should not be > considered the baseline and a user listening for "OOM conditions" > should be in

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-14 Thread David Rientjes
On Wed, 13 Nov 2013, Johannes Weiner wrote: > > > Somebody called out_of_memory() after they > > > failed reclaim, the machine is OOM. > > > > While momentarily oom, the oom notifiers in powerpc and s390 have the > > ability to free memory without requiring a kill. > > So either > > 1) they sh

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-13 Thread Johannes Weiner
On Wed, Nov 13, 2013 at 04:56:09PM -0800, David Rientjes wrote: > On Wed, 13 Nov 2013, Johannes Weiner wrote: > > > > > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > > > > > --- a/include/linux/memcontrol.h > > > > > +++ b/include/linux/memcontrol.h > > > > > @@ -155,6 +

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-13 Thread David Rientjes
On Wed, 13 Nov 2013, Johannes Weiner wrote: > > > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > > > > --- a/include/linux/memcontrol.h > > > > +++ b/include/linux/memcontrol.h > > > > @@ -155,6 +155,7 @@ static inline bool task_in_memcg_oom(struct > > > > task_struct *p

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-13 Thread Johannes Weiner
On Wed, Nov 13, 2013 at 02:19:00PM -0800, David Rientjes wrote: > On Thu, 31 Oct 2013, Johannes Weiner wrote: > > > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > > > --- a/include/linux/memcontrol.h > > > +++ b/include/linux/memcontrol.h > > > @@ -155,6 +155,7 @@ static

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-11-13 Thread David Rientjes
On Thu, 31 Oct 2013, Johannes Weiner wrote: > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > > --- a/include/linux/memcontrol.h > > +++ b/include/linux/memcontrol.h > > @@ -155,6 +155,7 @@ static inline bool task_in_memcg_oom(struct task_struct > > *p) > > } > > > >

Re: [patch] mm, memcg: add memory.oom_control notification for system oom

2013-10-30 Thread Johannes Weiner
On Wed, Oct 30, 2013 at 06:39:16PM -0700, David Rientjes wrote: > A subset of applications that wait on memory.oom_control don't disable > the oom killer for that memcg and simply log or cleanup after the kernel > oom killer kills a process to free memory. > > We need the ability to do this for sy