Received: by 2002:a05:6a10:a841:0:0:0:0 with SMTP id d1csp212939pxy; Wed, 21 Apr 2021 00:52:25 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxERZxxdeO1cErpBcZuvpaC1C6HDHQqQzqXVbtBAFqY1bF+ayZ04Gk5EfZuqbjr0cSFzhwi X-Received: by 2002:aa7:87d3:0:b029:259:ff63:3500 with SMTP id i19-20020aa787d30000b0290259ff633500mr22673666pfo.35.1618991544969; Wed, 21 Apr 2021 00:52:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1618991544; cv=none; d=google.com; s=arc-20160816; b=IPKBbOTPZxedC4DHWrrILRGnCgzInvN9uPdmHEkQpC25mxKQUa99jo/AXAGYd6p9u1 zFWhucpW/57xF+zlcVz3/ACXpADQfdCGdY9grwuJVRJOl509jSrziib9KoFCD62awJMk CNqvRcS28lhbzDOPv+K6ZeWkJtWgvC3JZidX76piQ3io+f3yQn9figj48dLTKI1Dl3xa ztO0wgjLFDr1XqeN9R7bvqwx9UrM6kHajhcWm91a7XB6qGwvbZ8w2AdRUgNAwxtjVCrN 1nuKgMegs3JlSUlJSTQUs+8kohhrQ3qpEHh2idNqTlrAQxckRjFBNfBlxWRPNR0E8gIM Nmug== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from; bh=DNqYROujCPJTsu9JxGDg8G/uPwADeCd3pRenp9vKAVI=; b=pKpRI2pBMH1019KU0+54OYowu2sFJDofOd7h5v7BIlrF9hXOXc0nNSfWYmQO1Vvk/6 3HzcG5L84FN3ogGB+TfrFKRXqh6DAfENPMghbk3WsOCAwklCzcdbXXOq5hCN6ZNRyTC6 ndOOayN4giEz8GGfkmXBjuvdOjdSotHm/yPxIjvnns8gjBwJ5f7pTnLoU+ynjVqH0tJ7 OJ3plwXf7ED/k/tjwyg4MH26XaqJ7JyW5Cmz+dqWwj+JTndroZIXc+ZG+XVTtMEYl0Oh jg6U6ccOhYot4nBX+RzdeuY9EzM9u1mw/UBaAcWEj2YQIwK77QAHuh/1kemquXhdilbu dJPA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id t7si1758727plg.18.2021.04.21.00.52.12; Wed, 21 Apr 2021 00:52:24 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236812AbhDUHWa (ORCPT + 99 others); Wed, 21 Apr 2021 03:22:30 -0400 Received: from mx314.baidu.com ([180.101.52.172]:41684 "EHLO njjs-sys-mailin08.njjs.baidu.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S230343AbhDUHW3 (ORCPT ); Wed, 21 Apr 2021 03:22:29 -0400 Received: from unknown.domain.tld (bjhw-sys-rpm015653cc5.bjhw.baidu.com [10.227.53.39]) by njjs-sys-mailin08.njjs.baidu.com (Postfix) with ESMTP id 0EFDD60C0050; Wed, 21 Apr 2021 15:21:54 +0800 (CST) From: chukaiping To: mcgrof@kernel.org, keescook@chromium.org, yzaikin@google.com, akpm@linux-foundation.org, vbabka@suse.cz, nigupta@nvidia.com, bhe@redhat.com, khalid.aziz@oracle.com, iamjoonsoo.kim@lge.com, mateusznosek0@gmail.com, sh_def@163.com Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: [PATCH v2] mm/compaction:let proactive compaction order configurable Date: Wed, 21 Apr 2021 15:21:53 +0800 Message-Id: <1618989713-20962-1-git-send-email-chukaiping@baidu.com> X-Mailer: git-send-email 1.7.1 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Currently the proactive compaction order is fixed to COMPACTION_HPAGE_ORDER(9), it's OK in most machines with lots of normal 4KB memory, but it's too high for the machines with small normal memory, for example the machines with most memory configured as 1GB hugetlbfs huge pages. In these machines the max order of free pages is often below 9, and it's always below 9 even with hard compaction. This will lead to proactive compaction be triggered very frequently. In these machines we only care about order of 3 or 4. This patch export the oder to proc and let it configurable by user, and the default value is still COMPACTION_HPAGE_ORDER. Signed-off-by: chukaiping Reported-by: kernel test robot --- Changes in v2: - fix the compile error in ia64 and powerpc - change the hard coded max order number from 10 to MAX_ORDER - 1 include/linux/compaction.h | 1 + kernel/sysctl.c | 11 +++++++++++ mm/compaction.c | 14 +++++++++++--- 3 files changed, 23 insertions(+), 3 deletions(-) diff --git a/include/linux/compaction.h b/include/linux/compaction.h index ed4070e..151ccd1 100644 --- a/include/linux/compaction.h +++ b/include/linux/compaction.h @@ -83,6 +83,7 @@ static inline unsigned long compact_gap(unsigned int order) #ifdef CONFIG_COMPACTION extern int sysctl_compact_memory; extern unsigned int sysctl_compaction_proactiveness; +extern unsigned int sysctl_compaction_order; extern int sysctl_compaction_handler(struct ctl_table *table, int write, void *buffer, size_t *length, loff_t *ppos); extern int sysctl_extfrag_threshold; diff --git a/kernel/sysctl.c b/kernel/sysctl.c index 62fbd09..a607d4d 100644 --- a/kernel/sysctl.c +++ b/kernel/sysctl.c @@ -195,6 +195,8 @@ enum sysctl_writes_mode { #endif /* CONFIG_SMP */ #endif /* CONFIG_SCHED_DEBUG */ +static int max_buddy_zone = MAX_ORDER - 1; + #ifdef CONFIG_COMPACTION static int min_extfrag_threshold; static int max_extfrag_threshold = 1000; @@ -2871,6 +2873,15 @@ int proc_do_static_key(struct ctl_table *table, int write, .extra2 = &one_hundred, }, { + .procname = "compaction_order", + .data = &sysctl_compaction_order, + .maxlen = sizeof(sysctl_compaction_order), + .mode = 0644, + .proc_handler = proc_dointvec_minmax, + .extra1 = SYSCTL_ZERO, + .extra2 = &max_buddy_zone, + }, + { .procname = "extfrag_threshold", .data = &sysctl_extfrag_threshold, .maxlen = sizeof(int), diff --git a/mm/compaction.c b/mm/compaction.c index e04f447..bfd1d5e 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -1925,16 +1925,16 @@ static bool kswapd_is_running(pg_data_t *pgdat) /* * A zone's fragmentation score is the external fragmentation wrt to the - * COMPACTION_HPAGE_ORDER. It returns a value in the range [0, 100]. + * sysctl_compaction_order. It returns a value in the range [0, 100]. */ static unsigned int fragmentation_score_zone(struct zone *zone) { - return extfrag_for_order(zone, COMPACTION_HPAGE_ORDER); + return extfrag_for_order(zone, sysctl_compaction_order); } /* * A weighted zone's fragmentation score is the external fragmentation - * wrt to the COMPACTION_HPAGE_ORDER scaled by the zone's size. It + * wrt to the sysctl_compaction_order scaled by the zone's size. It * returns a value in the range [0, 100]. * * The scaling factor ensures that proactive compaction focuses on larger @@ -2666,6 +2666,7 @@ static void compact_nodes(void) * background. It takes values in the range [0, 100]. */ unsigned int __read_mostly sysctl_compaction_proactiveness = 20; +unsigned int __read_mostly sysctl_compaction_order; /* * This is the entry point for compacting all nodes via @@ -2958,6 +2959,13 @@ static int __init kcompactd_init(void) int nid; int ret; + /* + * move the initialization of sysctl_compaction_order to here to + * eliminate compile error in ia64 and powerpc architecture because + * COMPACTION_HPAGE_ORDER is a variable in this architecture + */ + sysctl_compaction_order = COMPACTION_HPAGE_ORDER; + ret = cpuhp_setup_state_nocalls(CPUHP_AP_ONLINE_DYN, "mm/compaction:online", kcompactd_cpu_online, NULL); -- 1.7.1