From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ea0-f174.google.com (mail-ea0-f174.google.com [209.85.215.174]) by kanga.kvack.org (Postfix) with ESMTP id 9FE816B00A2 for ; Mon, 2 Dec 2013 08:12:41 -0500 (EST) Received: by mail-ea0-f174.google.com with SMTP id b10so8849999eae.5 for ; Mon, 02 Dec 2013 05:12:40 -0800 (PST) Received: from mx2.suse.de (cantor2.suse.de. [195.135.220.15]) by mx.google.com with ESMTP id 45si671500eef.27.2013.12.02.05.12.40 for ; Mon, 02 Dec 2013 05:12:40 -0800 (PST) Date: Mon, 2 Dec 2013 14:12:38 +0100 From: Michal Hocko Subject: Re: [merged] mm-memcg-handle-non-error-oom-situations-more-gracefully.patch removed from -mm tree Message-ID: <20131202131238.GB18838@dhcp22.suse.cz> References: <526028bd.k5qPj2+MDOK1o6ii%akpm@linux-foundation.org> <20131127233353.GH3556@cmpxchg.org> <20131128021809.GI3556@cmpxchg.org> <20131128031313.GK3556@cmpxchg.org> <20131128100213.GE2761@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: David Rientjes Cc: Johannes Weiner , Andrew Morton , azurit@pobox.sk, mm-commits@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org On Fri 29-11-13 16:05:04, David Rientjes wrote: > On Thu, 28 Nov 2013, Michal Hocko wrote: > > > > None that I am currently aware of, > > > > Are you saing that scenarios described in 3812c8c8f395 (mm: memcg: do not > > trap chargers with full callstack on OOM) are not real or that _you_ > > haven't seen an issue like that? > > > > The later doesn't seem to be so relevant as we had at least one user who > > has seen those in the real life. > > > > I said I'm not currently aware of any additional problems with the > patchset, I have obviously misread your reply. Sorry about that. > but since Johannes said the entire series wasn't meant for that > merge window, I asked if it was still being worked on. > > > > You don't think something like this is helpful after scanning a memcg will > > > a large number of processes? > > > > It looks as a one-shot workaround for short lived processes to me. > > It has nothing to do with how long a process has been running, both racing > processes could have been running for years. It's obvious that even this > patch before calling oom_kill_process() does not catch a racing process > that has already freed its memory and is exiting but it makes the > liklihood significantly less in testing at scale. I guess we need to know how much is significantly less. oom_scan_process_thread already aborts on exiting tasks so we do not kill anything and then the charge (whole page fault actually) is retried when we check for the OOM again so my intuition would say that we gave the exiting task quite a lot of time. > It's simply better to avoid unnecessary oom killing at anytime > possible and this is not a hotpath. -- Michal Hocko SUSE Labs -- 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: email@kvack.org