linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] mm,page_alloc: don't call __node_reclaim() without scoped allocation constraints.
@ 2017-09-01 12:40 Tetsuo Handa
  2017-09-01 12:40 ` [RFC PATCH 2/2] mm,sched: include memalloc info when printing debug dump of a task Tetsuo Handa
  2017-09-01 12:55 ` [PATCH 1/2] mm,page_alloc: don't call __node_reclaim() without scoped allocation constraints Michal Hocko
  0 siblings, 2 replies; 4+ messages in thread
From: Tetsuo Handa @ 2017-09-01 12:40 UTC (permalink / raw)
  To: akpm
  Cc: linux-kernel, linux-mm, Tetsuo Handa, David Rientjes, Mel Gorman,
	Michal Hocko, Vlastimil Babka

We are doing the first allocation attempt before calling
current_gfp_context(). But since slab shrinker functions might depend on
__GFP_FS and/or __GFP_IO masking, calling slab shrinker functions from
node_reclaim() from get_page_from_freelist() without calling
current_gfp_context() has possibility of deadlock. Therefore, make sure
that the first memory allocation attempt does not call slab shrinker
functions.

Well, do we want to call node_reclaim() on the first allocation attempt?

If yes, I guess this patch will not be acceptable. But what is correct
flags passed to the first allocation attempt, for currently we ignore
gfp_allowed_mask masking for the first allocation attempt?

Maybe we can tolerate not calling node_reclaim() on the first allocation
attempt, for commit 31a6c1909f51dbe9 ("mm, page_alloc: set alloc_flags
only once in slowpath") says that the fastpath is trying to avoid the
cost of setting up alloc_flags precisely which sounds to me that falling
back to slowpath if node_reclaim() is needed is acceptable?

Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: David Rientjes <rientjes@google.com>
---
 mm/page_alloc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 6dbc49e..20af138 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -4189,7 +4189,8 @@ struct page *
 	finalise_ac(gfp_mask, order, &ac);
 
 	/* First allocation attempt */
-	page = get_page_from_freelist(alloc_mask, order, alloc_flags, &ac);
+	page = get_page_from_freelist(alloc_mask & ~__GFP_DIRECT_RECLAIM,
+				      order, alloc_flags, &ac);
 	if (likely(page))
 		goto out;
 
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-09-01 13:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-01 12:40 [PATCH 1/2] mm,page_alloc: don't call __node_reclaim() without scoped allocation constraints Tetsuo Handa
2017-09-01 12:40 ` [RFC PATCH 2/2] mm,sched: include memalloc info when printing debug dump of a task Tetsuo Handa
2017-09-01 12:55 ` [PATCH 1/2] mm,page_alloc: don't call __node_reclaim() without scoped allocation constraints Michal Hocko
2017-09-01 13:15   ` Tetsuo Handa

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).