Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751339AbdIFEh7 (ORCPT ); Wed, 6 Sep 2017 00:37:59 -0400 Received: from mail-pg0-f67.google.com ([74.125.83.67]:34119 "EHLO mail-pg0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750752AbdIFEh6 (ORCPT ); Wed, 6 Sep 2017 00:37:58 -0400 X-Google-Smtp-Source: ADKCNb7TE43YIVY8j3c0Alav/m5+AAafjI8LffCyI54x+w12VZmZg2aLXdXIVJTvM27q/rbatZOHog== From: js1304@gmail.com X-Google-Original-From: iamjoonsoo.kim@lge.com To: Andrew Morton Cc: Christoph Lameter , Pekka Enberg , David Rientjes , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Mel Gorman , Vlastimil Babka , Michal Hocko , Joonsoo Kim Subject: [PATCH 1/2] mm/slub: wake up kswapd for initial high order allocation Date: Wed, 6 Sep 2017 13:37:45 +0900 Message-Id: <1504672666-19682-1-git-send-email-iamjoonsoo.kim@lge.com> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2075 Lines: 50 From: Joonsoo Kim slub uses higher order allocation than it actually needs. In this case, we don't want to do direct reclaim to make such a high order page since it causes a big latency to the user. Instead, we would like to fallback lower order allocation that it actually needs. However, we also want to get this higher order page in the next time in order to get the best performance and it would be a role of the background thread like as kswapd and kcompactd. To wake up them, we should not clear __GFP_KSWAPD_RECLAIM. Unlike this intention, current code clears __GFP_KSWAPD_RECLAIM so fix it. Current unintended code is done by Mel's commit 444eb2a449ef ("mm: thp: set THP defrag by default to madvise and add a stall-free defrag option") for slub part. It removes a special case in __alloc_page_slowpath() where including __GFP_THISNODE and lacking ~__GFP_DIRECT_RECLAIM effectively means also lacking __GFP_KSWAPD_RECLAIM. However, slub doesn't use __GFP_THISNODE so it is not the case for this purpose. So, partially reverting this code in slub doesn't hurt Mel's intention. Note that this patch does some clean up, too. __GFP_NOFAIL is cleared twice so remove one. Signed-off-by: Joonsoo Kim --- mm/slub.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/mm/slub.c b/mm/slub.c index 163352c..45f4a4b 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -1578,8 +1578,12 @@ static struct page *allocate_slab(struct kmem_cache *s, gfp_t flags, int node) * so we fall-back to the minimum order allocation. */ alloc_gfp = (flags | __GFP_NOWARN | __GFP_NORETRY) & ~__GFP_NOFAIL; - if ((alloc_gfp & __GFP_DIRECT_RECLAIM) && oo_order(oo) > oo_order(s->min)) - alloc_gfp = (alloc_gfp | __GFP_NOMEMALLOC) & ~(__GFP_RECLAIM|__GFP_NOFAIL); + if (oo_order(oo) > oo_order(s->min)) { + if (alloc_gfp & __GFP_DIRECT_RECLAIM) { + alloc_gfp |= __GFP_NOMEMALLOC; + alloc_gfp &= ~__GFP_DIRECT_RECLAIM; + } + } page = alloc_slab_page(s, alloc_gfp, node, oo); if (unlikely(!page)) { -- 2.7.4