Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
MIME-Version: 1.0
References: <20220330145400.18565-1-songmuchun@bytedance.com> <20220330194147.000c80ef35345397756e096e@linux-foundation.org>
In-Reply-To: <20220330194147.000c80ef35345397756e096e@linux-foundation.org>
From:   Muchun Song <songmuchun@bytedance.com>
Date:   Thu, 31 Mar 2022 10:46:25 +0800
Message-ID: <CAMZfGtWqsQsyr6-R1CtA9Uzswrb7MRXbBZqa4VKA9NCYyVvKOA@mail.gmail.com>
Subject: Re: [PATCH v3] arm64: mm: hugetlb: Enable HUGETLB_PAGE_FREE_VMEMMAP
 for arm64
To:     Andrew Morton <akpm@linux-foundation.org>
Cc:     Will Deacon <will@kernel.org>,
        David Hildenbrand <david@redhat.com>,
        "Bodeddula, Balasubramaniam" <bodeddub@amazon.com>,
        Oscar Salvador <osalvador@suse.de>,
        Mike Kravetz <mike.kravetz@oracle.com>,
        David Rientjes <rientjes@google.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Catalin Marinas <catalin.marinas@arm.com>, james.morse@arm.com,
        Barry Song <21cnbao@gmail.com>,
        LAK <linux-arm-kernel@lists.infradead.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux Memory Management List <linux-mm@kvack.org>,
        Xiongchun duan <duanxiongchun@bytedance.com>,
        Fam Zheng <fam.zheng@bytedance.com>,
        Muchun Song <smuchun@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, Mar 31, 2022 at 10:41 AM Andrew Morton
<akpm@linux-foundation.org> wrote:
>
> On Wed, 30 Mar 2022 22:54:00 +0800 Muchun Song <songmuchun@bytedance.com> wrote:
>
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -247,7 +247,7 @@ config HUGETLB_PAGE
> >
> >  config HUGETLB_PAGE_FREE_VMEMMAP
> >       def_bool HUGETLB_PAGE
> > -     depends on X86_64
> > +     depends on X86_64 || ARM64
>
> This list is just going to get longer, until someone goes and switches
> it to use ARCH_WANT_HUGETLB_PAGE_FREE_VMEMMAP or whatever.  Why not do
> that now?

Good suggestion. Yep, the line is going to be long if someone
wants to enable it for other architectures in the future.  Will do.
Thanks.