From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id E6454C43334 for ; Thu, 16 Jun 2022 00:27:45 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 57A4B6B0078; Wed, 15 Jun 2022 20:27:45 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4DC266B007B; Wed, 15 Jun 2022 20:27:45 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 354FC8D0001; Wed, 15 Jun 2022 20:27:45 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 22BB86B0078 for ; Wed, 15 Jun 2022 20:27:45 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id CF058348DD for ; Thu, 16 Jun 2022 00:27:44 +0000 (UTC) X-FDA: 79582210848.23.4291F19 Received: from mail-pj1-f50.google.com (mail-pj1-f50.google.com [209.85.216.50]) by imf26.hostedemail.com (Postfix) with ESMTP id 7F182140080 for ; Thu, 16 Jun 2022 00:27:44 +0000 (UTC) Received: by mail-pj1-f50.google.com with SMTP id 3-20020a17090a174300b001e426a02ac5so326118pjm.2 for ; Wed, 15 Jun 2022 17:27:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=v5EJlJC3fYMCJ96mwekJ8a2ISDsHOgu0y3JtizGSuls=; b=hsI/+WnqCF8zQPF/liunNLONAK1FPW0Nx6lgXU5U/glAxi3VvfNO3n+tovX2nJRaSu Jn3ic+Uf7LCk+iMH2pTxeLOq9wBtJUfEO/rE1Ck2UUiKEU1tkXzMgUZXSWfdCIODcqM7 13dYIig66mvBCX8XBH7EUEbl5FKTcwSXQvKpnvCyMBwSicp4E4Ehiag/Ae6rTlZGo58Y 6AxRYX8blgEYav6lzmPwzQQ+6MpsmE4yMe0B23P5lXkPJR2nZc6yCknHAvJ4KP9b7AFt LQ2OCi1Ff76Zwpb+SbzVkzv7BgKE9/4ueTc+T6MbxPtWAr0chI+nEkvYDC7Jff41xgPB FXsQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=v5EJlJC3fYMCJ96mwekJ8a2ISDsHOgu0y3JtizGSuls=; b=hJ7f2704Z3rDv5L96SOz4NMEUXz5vdL4r8aUtHvwby5xkmLEYhEHI1WjHUJwalkqce R9VMDVRTnplGnwfurcW609lSrihoUW2oYnysfGG67MCeUMnZcQqpU2MEREQIH6VQ9Hit LlXAUcISFHMej2nZRXy1RpYER2fWtwF74OD/0P8qBH0yCn05WqTdfilluJsTIyXp8eOD rlaBT7AuHC10Tp68P6Ofwv1AMEpWDw74/jq6BgxJzq4w4gCTRyOJSXnS2j86iFIkbsJM T1gjg7fmNj6bpVf8YrxNpGtMRgp1FgzwrDciC9iGxAOBuSiPPKaABV3Sqs6HXTc56BrF dIIA== X-Gm-Message-State: AJIora9U75i0g3kjwNKzkRvAcUzLjG0VfdV1T9slIBwGAjz5DKZyNaJp slEXYDY3GKDXnOVvgskvFqxS+g== X-Google-Smtp-Source: AGRyM1vSFqBmHWhi9P2WtAhphZrB3VnujgFpbtV/3TxwIwIE+xiZXy7eqzshUIglByRyFVy+arRKOw== X-Received: by 2002:a17:90a:f30d:b0:1ea:8070:ffba with SMTP id ca13-20020a17090af30d00b001ea8070ffbamr12930909pjb.169.1655339263102; Wed, 15 Jun 2022 17:27:43 -0700 (PDT) Received: from google.com (55.212.185.35.bc.googleusercontent.com. [35.185.212.55]) by smtp.gmail.com with ESMTPSA id nm4-20020a17090b19c400b001df93c8e737sm172162pjb.39.2022.06.15.17.27.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 15 Jun 2022 17:27:42 -0700 (PDT) Date: Wed, 15 Jun 2022 17:27:38 -0700 From: Zach O'Keefe To: Yang Shi Cc: vbabka@suse.cz, kirill.shutemov@linux.intel.com, willy@infradead.org, linmiaohe@huawei.com, akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [v4 PATCH 6/7] mm: khugepaged: reorg some khugepaged helpers Message-ID: References: <20220615172926.546974-1-shy828301@gmail.com> <20220615172926.546974-7-shy828301@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220615172926.546974-7-shy828301@gmail.com> ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1655339264; a=rsa-sha256; cv=none; b=YaPz+MmYcW5PG2ngQb0VwCfCMJTkVJ8ZQ6vlRIApsA3+r/zzZWQtOXkdE/xcDISjbDcKa0 vWxD5+gZzSSYd1KKaL9pspr5zzLlQSoAWAvH0Lhw39yvWkZLuQGBpWKFOshLb1lceXODmv R2FzE/aDfEyZ+vvlelpKpEjY15O2yoM= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b="hsI/+Wnq"; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf26.hostedemail.com: domain of zokeefe@google.com designates 209.85.216.50 as permitted sender) smtp.mailfrom=zokeefe@google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1655339264; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=v5EJlJC3fYMCJ96mwekJ8a2ISDsHOgu0y3JtizGSuls=; b=d70JtHNQVe9lI0jiv5dn3x1Fe3nVx0OiNnE1Ex7NvE9GA+/eimgKzrUd5RvLpY7E+cc+8r Pgc3x7XEOrB6JmzaY5wn8/Q5+apID91hINUDJBlZlZATGIodDlD9n+W9VbkEzhVF1WQ/3g 5iEFoUmxtd6aiW63UDXxjSDCzrtS978= Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b="hsI/+Wnq"; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf26.hostedemail.com: domain of zokeefe@google.com designates 209.85.216.50 as permitted sender) smtp.mailfrom=zokeefe@google.com X-Rspam-User: X-Stat-Signature: nnraprassb8ukntceqctiaaourmbb6th X-Rspamd-Queue-Id: 7F182140080 X-Rspamd-Server: rspam08 X-HE-Tag: 1655339264-177875 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On 15 Jun 10:29, Yang Shi wrote: > The khugepaged_{enabled|always|req_madv} are not khugepaged only > anymore, move them to huge_mm.h and rename to hugepage_flags_xxx, and > remove khugepaged_req_madv due to no users. > > Also move khugepaged_defrag to khugepaged.c since its only caller is in > that file, it doesn't have to be in a header file. > > Signed-off-by: Yang Shi > --- > include/linux/huge_mm.h | 8 ++++++++ > include/linux/khugepaged.h | 14 -------------- > mm/huge_memory.c | 4 ++-- > mm/khugepaged.c | 18 +++++++++++------- > 4 files changed, 21 insertions(+), 23 deletions(-) > > diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h > index 9d97d7ee6234..4cf546af7d97 100644 > --- a/include/linux/huge_mm.h > +++ b/include/linux/huge_mm.h > @@ -116,6 +116,14 @@ extern struct kobj_attribute shmem_enabled_attr; > > extern unsigned long transparent_hugepage_flags; > > +#define hugepage_flags_enabled() \ > + (transparent_hugepage_flags & \ > + ((1< + (1< +#define hugepage_flags_always() \ > + (transparent_hugepage_flags & \ > + (1< + > /* > * Do the below checks: > * - For file vma, check if the linear page offset of vma is > diff --git a/include/linux/khugepaged.h b/include/linux/khugepaged.h > index ea5fd4c398f7..384f034ae947 100644 > --- a/include/linux/khugepaged.h > +++ b/include/linux/khugepaged.h > @@ -24,20 +24,6 @@ static inline void collapse_pte_mapped_thp(struct mm_struct *mm, > } > #endif > > -#define khugepaged_enabled() \ > - (transparent_hugepage_flags & \ > - ((1< - (1< -#define khugepaged_always() \ > - (transparent_hugepage_flags & \ > - (1< -#define khugepaged_req_madv() \ > - (transparent_hugepage_flags & \ > - (1< -#define khugepaged_defrag() \ > - (transparent_hugepage_flags & \ > - (1< - > static inline void khugepaged_fork(struct mm_struct *mm, struct mm_struct *oldmm) > { > if (test_bit(MMF_VM_HUGEPAGE, &oldmm->flags)) > diff --git a/mm/huge_memory.c b/mm/huge_memory.c > index d0c37d99917b..0f2cce2d7041 100644 > --- a/mm/huge_memory.c > +++ b/mm/huge_memory.c > @@ -117,11 +117,11 @@ bool hugepage_vma_check(struct vm_area_struct *vma, > if (!in_pf && shmem_file(vma->vm_file)) > return shmem_huge_enabled(vma); > > - if (!khugepaged_enabled()) > + if (!hugepage_flags_enabled()) > return false; > > /* THP settings require madvise. */ > - if (!(vm_flags & VM_HUGEPAGE) && !khugepaged_always()) > + if (!(vm_flags & VM_HUGEPAGE) && !hugepage_flags_always()) > return false; > > /* Only regular file is valid */ > diff --git a/mm/khugepaged.c b/mm/khugepaged.c > index 2a676f37c921..d8ebb60aae36 100644 > --- a/mm/khugepaged.c > +++ b/mm/khugepaged.c > @@ -472,7 +472,7 @@ void khugepaged_enter_vma(struct vm_area_struct *vma, > unsigned long vm_flags) > { > if (!test_bit(MMF_VM_HUGEPAGE, &vma->vm_mm->flags) && > - khugepaged_enabled()) { > + hugepage_flags_enabled()) { > if (hugepage_vma_check(vma, vm_flags, false, false)) > __khugepaged_enter(vma->vm_mm); > } > @@ -763,6 +763,10 @@ static bool khugepaged_scan_abort(int nid) > return false; > } > > +#define khugepaged_defrag() \ > + (transparent_hugepage_flags & \ > + (1< + > /* Defrag for khugepaged will enter direct reclaim/compaction if necessary */ > static inline gfp_t alloc_hugepage_khugepaged_gfpmask(void) > { > @@ -860,7 +864,7 @@ static struct page *khugepaged_alloc_hugepage(bool *wait) > khugepaged_alloc_sleep(); > } else > count_vm_event(THP_COLLAPSE_ALLOC); > - } while (unlikely(!hpage) && likely(khugepaged_enabled())); > + } while (unlikely(!hpage) && likely(hugepage_flags_enabled())); > > return hpage; > } > @@ -2186,7 +2190,7 @@ static unsigned int khugepaged_scan_mm_slot(unsigned int pages, > static int khugepaged_has_work(void) > { > return !list_empty(&khugepaged_scan.mm_head) && > - khugepaged_enabled(); > + hugepage_flags_enabled(); > } > > static int khugepaged_wait_event(void) > @@ -2251,7 +2255,7 @@ static void khugepaged_wait_work(void) > return; > } > > - if (khugepaged_enabled()) > + if (hugepage_flags_enabled()) > wait_event_freezable(khugepaged_wait, khugepaged_wait_event()); > } > > @@ -2282,7 +2286,7 @@ static void set_recommended_min_free_kbytes(void) > int nr_zones = 0; > unsigned long recommended_min; > > - if (!khugepaged_enabled()) { > + if (!hugepage_flags_enabled()) { > calculate_min_free_kbytes(); > goto update_wmarks; > } > @@ -2332,7 +2336,7 @@ int start_stop_khugepaged(void) > int err = 0; > > mutex_lock(&khugepaged_mutex); > - if (khugepaged_enabled()) { > + if (hugepage_flags_enabled()) { > if (!khugepaged_thread) > khugepaged_thread = kthread_run(khugepaged, NULL, > "khugepaged"); > @@ -2358,7 +2362,7 @@ int start_stop_khugepaged(void) > void khugepaged_min_free_kbytes_update(void) > { > mutex_lock(&khugepaged_mutex); > - if (khugepaged_enabled() && khugepaged_thread) > + if (hugepage_flags_enabled() && khugepaged_thread) > set_recommended_min_free_kbytes(); > mutex_unlock(&khugepaged_mutex); > } > -- > 2.26.3 > Reviewed-by: Zach O'Keefe