From: Michal Hocko <mhocko@kernel.org> To: Andrew Morton <akpm@linux-foundation.org> Cc: Vlastimil Babka <vbabka@suse.cz>, Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>, Johannes Weiner <hannes@cmpxchg.org>, Mel Gorman <mgorman@suse.de>, David Rientjes <rientjes@google.com>, <linux-mm@kvack.org>, LKML <linux-kernel@vger.kernel.org> Subject: [PATCH 0/2] GFP_NOFAIL cleanups Date: Thu, 1 Dec 2016 16:25:15 +0100 [thread overview] Message-ID: <20161201152517.27698-1-mhocko@kernel.org> (raw) Hi, I have posted this as an RFC previously [1] as there was no fundamental disagreement I would like to ask for inclusion. Tetsuo has noticed [2] that recent changes have changed GFP_NOFAIL semantic for costly order requests. I believe that the primary reason why this happened is that our GFP_NOFAIL checks are too scattered and it is really easy to forget about adding one. That's why I am proposing patch 1 which consolidates all the nofail handling at a single place. This should help to make this code better maintainable. Patch 2 on top is a further attempt to make GFP_NOFAIL semantic less surprising. As things stand currently GFP_NOFAIL overrides the oom killer prevention code which is both subtle and not really needed. The patch 2 has more details about issues this might cause. I would consider both patches more a cleanup than anything else. Any feedback is highly appreciated. [1] http://lkml.kernel.org/r/20161123064925.9716-1-mhocko@kernel.org [2] http://lkml.kernel.org/r/1479387004-5998-1-git-send-email-penguin-kernel@I-love.SAKURA.ne.jp
WARNING: multiple messages have this Message-ID (diff)
From: Michal Hocko <mhocko@kernel.org> To: Andrew Morton <akpm@linux-foundation.org> Cc: Vlastimil Babka <vbabka@suse.cz>, Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>, Johannes Weiner <hannes@cmpxchg.org>, Mel Gorman <mgorman@suse.de>, David Rientjes <rientjes@google.com>, linux-mm@kvack.org, LKML <linux-kernel@vger.kernel.org> Subject: [PATCH 0/2] GFP_NOFAIL cleanups Date: Thu, 1 Dec 2016 16:25:15 +0100 [thread overview] Message-ID: <20161201152517.27698-1-mhocko@kernel.org> (raw) Hi, I have posted this as an RFC previously [1] as there was no fundamental disagreement I would like to ask for inclusion. Tetsuo has noticed [2] that recent changes have changed GFP_NOFAIL semantic for costly order requests. I believe that the primary reason why this happened is that our GFP_NOFAIL checks are too scattered and it is really easy to forget about adding one. That's why I am proposing patch 1 which consolidates all the nofail handling at a single place. This should help to make this code better maintainable. Patch 2 on top is a further attempt to make GFP_NOFAIL semantic less surprising. As things stand currently GFP_NOFAIL overrides the oom killer prevention code which is both subtle and not really needed. The patch 2 has more details about issues this might cause. I would consider both patches more a cleanup than anything else. Any feedback is highly appreciated. [1] http://lkml.kernel.org/r/20161123064925.9716-1-mhocko@kernel.org [2] http://lkml.kernel.org/r/1479387004-5998-1-git-send-email-penguin-kernel@I-love.SAKURA.ne.jp -- 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>
next reply other threads:[~2016-12-01 15:25 UTC|newest] Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-12-01 15:25 Michal Hocko [this message] 2016-12-01 15:25 ` [PATCH 0/2] GFP_NOFAIL cleanups Michal Hocko 2016-12-01 15:25 ` [PATCH 1/2] mm: consolidate GFP_NOFAIL checks in the allocator slowpath Michal Hocko 2016-12-01 15:25 ` Michal Hocko 2016-12-01 15:25 ` [PATCH 2/2] mm, oom: do not enfore OOM killer for __GFP_NOFAIL automatically Michal Hocko 2016-12-01 15:25 ` Michal Hocko 2016-12-02 7:23 ` Vlastimil Babka 2016-12-02 7:23 ` Vlastimil Babka 2016-12-05 13:45 ` Tetsuo Handa 2016-12-05 13:45 ` Tetsuo Handa 2016-12-05 14:10 ` Michal Hocko 2016-12-05 14:10 ` Michal Hocko 2016-12-06 8:27 ` Michal Hocko 2016-12-06 8:27 ` Michal Hocko 2016-12-06 10:38 ` Tetsuo Handa 2016-12-06 10:38 ` Tetsuo Handa 2016-12-06 11:03 ` Vlastimil Babka 2016-12-06 11:03 ` Vlastimil Babka 2016-12-06 19:25 ` Michal Hocko 2016-12-06 19:25 ` Michal Hocko 2016-12-06 19:22 ` Michal Hocko 2016-12-06 19:22 ` Michal Hocko 2016-12-08 12:53 ` Tetsuo Handa 2016-12-08 12:53 ` Tetsuo Handa 2016-12-08 13:47 ` Michal Hocko 2016-12-08 13:47 ` Michal Hocko 2016-12-11 11:23 ` Tetsuo Handa 2016-12-11 11:23 ` Tetsuo Handa 2016-12-11 13:53 ` Tetsuo Handa 2016-12-11 13:53 ` Tetsuo Handa 2016-12-12 8:52 ` Michal Hocko 2016-12-12 8:52 ` Michal Hocko 2016-12-12 8:48 ` Michal Hocko 2016-12-12 8:48 ` Michal Hocko 2016-12-14 10:34 ` Michal Hocko 2016-12-14 10:34 ` Michal Hocko
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=20161201152517.27698-1-mhocko@kernel.org \ --to=mhocko@kernel.org \ --cc=akpm@linux-foundation.org \ --cc=hannes@cmpxchg.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ --cc=penguin-kernel@I-love.SAKURA.ne.jp \ --cc=rientjes@google.com \ --cc=vbabka@suse.cz \ /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 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.