All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vladimir Murzin <murzin.v@gmail.com>
To: David Rientjes <rientjes@google.com>
Cc: Michal Hocko <mhocko@suse.cz>,
	linux-mm@kvack.org, Greg Thelen <gthelen@google.com>,
	Glauber Costa <glommer@gmail.com>, Mel Gorman <mgorman@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Johannes Weiner <hannes@cmpxchg.org>,
	KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>,
	Rik van Riel <riel@redhat.com>, Joern Engel <joern@logfs.org>,
	Hugh Dickins <hughd@google.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: user defined OOM policies
Date: Wed, 20 Nov 2013 18:25:16 +0100	[thread overview]
Message-ID: <20131120172511.GB1848@hp530> (raw)
In-Reply-To: <alpine.DEB.2.02.1311192352070.20752@chino.kir.corp.google.com>

Hi David

On Wed, Nov 20, 2013 at 12:02:20AM -0800, David Rientjes wrote:
> On Tue, 19 Nov 2013, Michal Hocko wrote:
> 
> > > We have basically ended up with 3 options AFAIR:
> > > 	1) allow memcg approach (memcg.oom_control) on the root level
> > >            for both OOM notification and blocking OOM killer and handle
> > >            the situation from the userspace same as we can for other
> > > 	   memcgs.
> > 
> > This looks like a straightforward approach as the similar thing is done
> > on the local (memcg) level. There are several problems though.
> > Running userspace from within OOM context is terribly hard to do
> > right.
> 
> Not sure it's hard if you have per-memcg memory reserves which I've 
> brought up in the past with true and complete kmem accounting.  Even if 
> you don't allocate slab, it guarantees that there will be at least a 
> little excess memory available so that the userspace oom handler isn't oom 
> itself.
> 
> This involves treating processes waiting on memory.oom_control to be 
> treated as a special class so that they are allowed to allocate an 
> additional pre-configured amount of memory.  For non-root memcgs, this 
> would simply be a dummy usage that would be charged to the memcg when the 
> oom notification is registered and actually accessible only by the oom 
> handler itself while memcg->under_oom.  For root memcgs, this would simply 
> be a PF_MEMALLOC type behavior that dips into per-zone memory reserves.
> 
> > This is true even in the memcg case and we strongly discurage
> > users from doing that. The global case has nothing like outside of OOM
> > context though. So any hang would blocking the whole machine.
> 
> Why would there be a hang if the userspace oom handlers aren't actually 
> oom themselves as described above?
> 
> I'd suggest against the other two suggestions because hierarchical 
> per-memcg userspace oom handlers are very powerful and can be useful 
> without actually killing anything at all, and parent oom handlers can 
> signal child oom handlers to free memory in oom conditions (in other 
> words, defer a parent oom condition to a child's oom handler upon 

Is not vmpressure notifications was designed for that purpose?

Vladimir

> notification).  I was planning on writing a liboom library that would lay 
> the foundation for how this was supposed to work and some generic 
> functions that make use of the per-memcg memory reserves.
> 
> So my plan for the complete solution was:
> 
>  - allow userspace notification from the root memcg on system oom 
>    conditions,
> 
>  - implement a memory.oom_delay_millisecs timeout so that the kernel 
>    eventually intervenes if userspace fails to respond, including for
>    system oom conditions, for whatever reason which would be set to 0
>    if no userspace oom handler is registered for the notification, and
> 
>  - implement per-memcg reserves as described above so that userspace oom 
>    handlers have access to memory even in oom conditions as an upfront
>    charge and have the ability to free memory as necessary.
> 
> We already have the ability to do the actual kill from userspace, both the 
> system oom killer and the memcg oom killer grants access to memory 
> reserves for any process needing to allocate memory if it has a pending 
> SIGKILL which we can send from userspace.
> 
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@kvack.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Vladimir Murzin <murzin.v@gmail.com>
To: David Rientjes <rientjes@google.com>
Cc: Michal Hocko <mhocko@suse.cz>,
	linux-mm@kvack.org, Greg Thelen <gthelen@google.com>,
	Glauber Costa <glommer@gmail.com>, Mel Gorman <mgorman@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Johannes Weiner <hannes@cmpxchg.org>,
	KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>,
	Rik van Riel <riel@redhat.com>, Joern Engel <joern@logfs.org>,
	Hugh Dickins <hughd@google.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: user defined OOM policies
Date: Wed, 20 Nov 2013 18:25:16 +0100	[thread overview]
Message-ID: <20131120172511.GB1848@hp530> (raw)
In-Reply-To: <alpine.DEB.2.02.1311192352070.20752@chino.kir.corp.google.com>

Hi David

On Wed, Nov 20, 2013 at 12:02:20AM -0800, David Rientjes wrote:
> On Tue, 19 Nov 2013, Michal Hocko wrote:
> 
> > > We have basically ended up with 3 options AFAIR:
> > > 	1) allow memcg approach (memcg.oom_control) on the root level
> > >            for both OOM notification and blocking OOM killer and handle
> > >            the situation from the userspace same as we can for other
> > > 	   memcgs.
> > 
> > This looks like a straightforward approach as the similar thing is done
> > on the local (memcg) level. There are several problems though.
> > Running userspace from within OOM context is terribly hard to do
> > right.
> 
> Not sure it's hard if you have per-memcg memory reserves which I've 
> brought up in the past with true and complete kmem accounting.  Even if 
> you don't allocate slab, it guarantees that there will be at least a 
> little excess memory available so that the userspace oom handler isn't oom 
> itself.
> 
> This involves treating processes waiting on memory.oom_control to be 
> treated as a special class so that they are allowed to allocate an 
> additional pre-configured amount of memory.  For non-root memcgs, this 
> would simply be a dummy usage that would be charged to the memcg when the 
> oom notification is registered and actually accessible only by the oom 
> handler itself while memcg->under_oom.  For root memcgs, this would simply 
> be a PF_MEMALLOC type behavior that dips into per-zone memory reserves.
> 
> > This is true even in the memcg case and we strongly discurage
> > users from doing that. The global case has nothing like outside of OOM
> > context though. So any hang would blocking the whole machine.
> 
> Why would there be a hang if the userspace oom handlers aren't actually 
> oom themselves as described above?
> 
> I'd suggest against the other two suggestions because hierarchical 
> per-memcg userspace oom handlers are very powerful and can be useful 
> without actually killing anything at all, and parent oom handlers can 
> signal child oom handlers to free memory in oom conditions (in other 
> words, defer a parent oom condition to a child's oom handler upon 

Is not vmpressure notifications was designed for that purpose?

Vladimir

> notification).  I was planning on writing a liboom library that would lay 
> the foundation for how this was supposed to work and some generic 
> functions that make use of the per-memcg memory reserves.
> 
> So my plan for the complete solution was:
> 
>  - allow userspace notification from the root memcg on system oom 
>    conditions,
> 
>  - implement a memory.oom_delay_millisecs timeout so that the kernel 
>    eventually intervenes if userspace fails to respond, including for
>    system oom conditions, for whatever reason which would be set to 0
>    if no userspace oom handler is registered for the notification, and
> 
>  - implement per-memcg reserves as described above so that userspace oom 
>    handlers have access to memory even in oom conditions as an upfront
>    charge and have the ability to free memory as necessary.
> 
> We already have the ability to do the actual kill from userspace, both the 
> system oom killer and the memcg oom killer grants access to memory 
> reserves for any process needing to allocate memory if it has a pending 
> SIGKILL which we can send from userspace.
> 
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@kvack.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  parent reply	other threads:[~2013-11-20 17:25 UTC|newest]

Thread overview: 135+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-19 13:14 user defined OOM policies Michal Hocko
2013-11-19 13:14 ` Michal Hocko
2013-11-19 13:40 ` Michal Hocko
2013-11-19 13:40   ` Michal Hocko
2013-11-20  8:02   ` David Rientjes
2013-11-20  8:02     ` David Rientjes
2013-11-20 15:22     ` Michal Hocko
2013-11-20 15:22       ` Michal Hocko
2013-11-20 17:14       ` Luigi Semenzato
2013-11-20 17:14         ` Luigi Semenzato
2013-11-21  3:36         ` David Rientjes
2013-11-21  3:36           ` David Rientjes
2013-11-21  7:03           ` Luigi Semenzato
2013-11-21  7:03             ` Luigi Semenzato
2013-11-22 18:08             ` Johannes Weiner
2013-11-22 18:08               ` Johannes Weiner
2013-11-28 11:36               ` Michal Hocko
2013-11-28 11:36                 ` Michal Hocko
2013-11-26  1:29             ` David Rientjes
2013-11-26  1:29               ` David Rientjes
2013-11-28 11:42               ` Michal Hocko
2013-11-28 11:42                 ` Michal Hocko
2013-12-02 23:09                 ` David Rientjes
2013-12-02 23:09                   ` David Rientjes
2013-11-21  3:33       ` David Rientjes
2013-11-21  3:33         ` David Rientjes
2013-11-28 11:54         ` Michal Hocko
2013-11-28 11:54           ` Michal Hocko
2013-12-02 23:07           ` David Rientjes
2013-12-02 23:07             ` David Rientjes
2013-12-04  5:19             ` [patch 1/8] fork: collapse copy_flags into copy_process David Rientjes
2013-12-04  5:19               ` David Rientjes
2013-12-04  5:19               ` [patch 2/8] mm, mempolicy: rename slab_node for clarity David Rientjes
2013-12-04  5:19                 ` David Rientjes
2013-12-04 15:21                 ` Christoph Lameter
2013-12-04 15:21                   ` Christoph Lameter
2013-12-04 15:21                   ` Christoph Lameter
2013-12-04  5:20               ` [patch 3/8] mm, mempolicy: remove per-process flag David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-12-04 15:24                 ` Christoph Lameter
2013-12-04 15:24                   ` Christoph Lameter
2013-12-05  0:53                   ` David Rientjes
2013-12-05  0:53                     ` David Rientjes
2013-12-05 19:05                     ` Christoph Lameter
2013-12-05 19:05                       ` Christoph Lameter
2013-12-05 23:53                       ` David Rientjes
2013-12-05 23:53                         ` David Rientjes
2013-12-06 14:46                         ` Christoph Lameter
2013-12-06 14:46                           ` Christoph Lameter
2013-12-04  5:20               ` [patch 4/8] mm, memcg: add tunable for oom reserves David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-12-04  5:20               ` [patch 5/8] res_counter: remove interface for locked charging and uncharging David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-12-04  5:20               ` [patch 6/8] res_counter: add interface for maximum nofail charge David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-12-04  5:20               ` [patch 7/8] mm, memcg: allow processes handling oom notifications to access reserves David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-12-04  5:45                 ` Johannes Weiner
2013-12-04  5:45                   ` Johannes Weiner
2013-12-05  1:49                   ` David Rientjes
2013-12-05  1:49                     ` David Rientjes
2013-12-05  2:50                     ` Tejun Heo
2013-12-05  2:50                       ` Tejun Heo
2013-12-05 23:49                       ` David Rientjes
2013-12-05 23:49                         ` David Rientjes
2013-12-05 23:49                         ` David Rientjes
2013-12-06 17:34                         ` Johannes Weiner
2013-12-06 17:34                           ` Johannes Weiner
2013-12-06 17:34                           ` Johannes Weiner
2013-12-07 16:38                           ` Tim Hockin
2013-12-07 17:40                             ` Johannes Weiner
2013-12-07 17:40                               ` Johannes Weiner
2013-12-07 18:12                               ` Tim Hockin
2013-12-07 19:06                                 ` Johannes Weiner
2013-12-07 19:06                                   ` Johannes Weiner
2013-12-07 19:06                                   ` Johannes Weiner
2013-12-07 21:04                                   ` Tim Hockin
2013-12-06 19:01                         ` Tejun Heo
2013-12-06 19:01                           ` Tejun Heo
2013-12-06 19:01                           ` Tejun Heo
2013-12-09 20:10                           ` David Rientjes
2013-12-09 20:10                             ` David Rientjes
2013-12-09 22:37                             ` Johannes Weiner
2013-12-09 22:37                               ` Johannes Weiner
2013-12-09 22:37                               ` Johannes Weiner
2013-12-10 21:50                             ` Tejun Heo
2013-12-10 21:50                               ` Tejun Heo
2013-12-10 21:50                               ` Tejun Heo
2013-12-10 23:55                               ` David Rientjes
2013-12-10 23:55                                 ` David Rientjes
2013-12-11  9:49                                 ` Mel Gorman
2013-12-11  9:49                                   ` Mel Gorman
2013-12-11 12:42                                 ` Tejun Heo
2013-12-11 12:42                                   ` Tejun Heo
2013-12-12  5:37                                   ` Tim Hockin
2013-12-12  5:37                                     ` Tim Hockin
2013-12-12 14:21                                     ` Tejun Heo
2013-12-12 14:21                                       ` Tejun Heo
2013-12-12 16:32                                       ` Michal Hocko
2013-12-12 16:32                                         ` Michal Hocko
2013-12-12 16:37                                         ` Tejun Heo
2013-12-12 16:37                                           ` Tejun Heo
2013-12-12 18:42                                       ` Tim Hockin
2013-12-12 18:42                                         ` Tim Hockin
2013-12-12 18:42                                         ` Tim Hockin
2013-12-12 19:23                                         ` Tejun Heo
2013-12-12 19:23                                           ` Tejun Heo
2013-12-13  0:23                                           ` Tim Hockin
2013-12-13  0:23                                             ` Tim Hockin
2013-12-13 11:47                                             ` Tejun Heo
2013-12-13 11:47                                               ` Tejun Heo
2013-12-04  5:20               ` [patch 8/8] mm, memcg: add memcg oom reserve documentation David Rientjes
2013-12-04  5:20                 ` David Rientjes
2013-11-20 17:25     ` Vladimir Murzin [this message]
2013-11-20 17:25       ` user defined OOM policies Vladimir Murzin
2013-11-20 17:21   ` Vladimir Murzin
2013-11-20 17:21     ` Vladimir Murzin
2013-11-20 17:33     ` Michal Hocko
2013-11-20 17:33       ` Michal Hocko
2013-11-21  3:38       ` David Rientjes
2013-11-21  3:38         ` David Rientjes
2013-11-21 17:13         ` Michal Hocko
2013-11-21 17:13           ` Michal Hocko
2013-11-26  1:36           ` David Rientjes
2013-11-26  1:36             ` David Rientjes
2013-11-22  7:28       ` Vladimir Murzin
2013-11-22  7:28         ` Vladimir Murzin
2013-11-22 13:18         ` Michal Hocko
2013-11-22 13:18           ` Michal Hocko
2013-11-20  7:50 ` David Rientjes
2013-11-20  7:50   ` David Rientjes
2013-11-22  0:19 ` Jörn Engel
2013-11-22  0:19   ` Jörn Engel
2013-11-26  1:31   ` David Rientjes
2013-11-26  1:31     ` David Rientjes

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20131120172511.GB1848@hp530 \
    --to=murzin.v@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=glommer@gmail.com \
    --cc=gthelen@google.com \
    --cc=hannes@cmpxchg.org \
    --cc=hughd@google.com \
    --cc=joern@logfs.org \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@suse.de \
    --cc=mhocko@suse.cz \
    --cc=riel@redhat.com \
    --cc=rientjes@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.