LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> To: mhocko@suse.cz, hannes@cmpxchg.org Cc: akpm@linux-foundation.org, david@fromorbit.com, mgorman@suse.de, riel@redhat.com, fengguang.wu@intel.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2 v2] mm: Allow small allocations to fail Date: Wed, 18 Mar 2015 20:35:49 +0900 [thread overview] Message-ID: <201503182035.FJD69716.FMOHLOOFQFtVSJ@I-love.SAKURA.ne.jp> (raw) In-Reply-To: <20150317194136.GA31691@dhcp22.suse.cz> I'm not opposing to have fundamental solutions. As you know the fundamental solution will need many years to complete, I'm asking for interim workaround which we can use now. Michal Hocko wrote: > The problem, as I see it, is that such a change cannot be pushed to > Linus tree without extensive testing because there are thousands of code > paths which never got exercised. We have basically two options here. Your options are based on your proposal. We can have different options based on Johannes's and my proposal. > Either have a non-upstream patch (e.g. sitting in mmotm and linux-next) > and have developers to do their testing. This will surely help to > catch a lot of fallouts and fix them right away. But we will miss those > who are using Linus based trees and would be willing to help to test > in their loads which we never dreamed of. > The other option would be pushing an experimental code to the Linus > tree (and distribution kernels) and allow people to turn it on to help > testing. The third option is to purge majority of code paths which never got exercised, by replacing kmalloc() with kmalloc_nofail() where amount of requested size is known to be <= PAGE_SIZE bytes. The third option becomes possible if we "allow triggering the OOM killer for both __GFP_FS allocations and !__GFP_FS allocations" and "introduce the OOM-killer timeout" so that OOM-deadlock which we are already observing can be handled.
next prev parent reply other threads:[~2015-03-18 11:36 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-03-11 20:54 [PATCH 0/2] Move away from non-failing small allocations Michal Hocko 2015-03-11 20:54 ` [PATCH 1/2] mm: Allow small allocations to fail Michal Hocko 2015-03-12 12:54 ` Tetsuo Handa 2015-03-12 13:12 ` Michal Hocko 2015-03-15 5:43 ` Tetsuo Handa 2015-03-15 12:13 ` Michal Hocko 2015-03-15 13:06 ` Tetsuo Handa 2015-03-16 7:46 ` [PATCH 1/2 v2] " Michal Hocko 2015-03-16 21:11 ` Johannes Weiner 2015-03-17 10:25 ` Michal Hocko 2015-03-17 13:29 ` Johannes Weiner 2015-03-17 14:17 ` Michal Hocko 2015-03-17 17:26 ` Johannes Weiner 2015-03-17 19:41 ` Michal Hocko 2015-03-18 9:10 ` Vlastimil Babka 2015-03-18 12:04 ` Michal Hocko 2015-03-18 12:36 ` Tetsuo Handa 2015-03-18 11:35 ` Tetsuo Handa [this message] 2015-03-17 11:13 ` Tetsuo Handa 2015-03-17 13:15 ` Michal Hocko 2015-03-18 11:33 ` Tetsuo Handa 2015-03-18 12:23 ` Michal Hocko 2015-03-19 11:03 ` Tetsuo Handa 2015-03-11 20:54 ` [PATCH 2/2] mmotm: Enable small allocation " Michal Hocko 2015-03-11 22:36 ` [PATCH 0/2] Move away from non-failing small allocations Sasha Levin 2015-03-16 22:38 ` Andrew Morton 2015-03-17 9:07 ` Michal Hocko 2015-03-17 14:06 ` Tetsuo Handa 2015-04-02 11:53 ` Tetsuo Handa
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=201503182035.FJD69716.FMOHLOOFQFtVSJ@I-love.SAKURA.ne.jp \ --to=penguin-kernel@i-love.sakura.ne.jp \ --cc=akpm@linux-foundation.org \ --cc=david@fromorbit.com \ --cc=fengguang.wu@intel.com \ --cc=hannes@cmpxchg.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ --cc=mhocko@suse.cz \ --cc=riel@redhat.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).