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 X-Spam-Level: X-Spam-Status: No, score=-13.3 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7EDC1C433E0 for ; Mon, 25 Jan 2021 18:47:36 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1B301230FA for ; Mon, 25 Jan 2021 18:47:35 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1B301230FA Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 658606B000A; Mon, 25 Jan 2021 13:47:35 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 6088D6B000C; Mon, 25 Jan 2021 13:47:35 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4F6C26B002C; Mon, 25 Jan 2021 13:47:35 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0173.hostedemail.com [216.40.44.173]) by kanga.kvack.org (Postfix) with ESMTP id 35CE56B000A for ; Mon, 25 Jan 2021 13:47:35 -0500 (EST) Received: from smtpin10.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 00E47181AEF3C for ; Mon, 25 Jan 2021 18:47:34 +0000 (UTC) X-FDA: 77745180870.10.robin44_5207b0827588 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin10.hostedemail.com (Postfix) with ESMTP id D29EE16A0AE for ; Mon, 25 Jan 2021 18:47:34 +0000 (UTC) X-HE-Tag: robin44_5207b0827588 X-Filterd-Recvd-Size: 5273 Received: from mail-pl1-f179.google.com (mail-pl1-f179.google.com [209.85.214.179]) by imf30.hostedemail.com (Postfix) with ESMTP for ; Mon, 25 Jan 2021 18:47:34 +0000 (UTC) Received: by mail-pl1-f179.google.com with SMTP id d4so8226767plh.5 for ; Mon, 25 Jan 2021 10:47:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:from:to:cc:subject:in-reply-to:message-id:references :mime-version; bh=ETq2ktZkANjUC7Me/RyH9n1tJakbH50bRmVCx0xOmwg=; b=OwEBYQmaBsKbc7iDfMlqm2AWtZvRyFNZfh0RZHpJvnSTDzlpuc8InF121vKQkdAM9I o48u7t/O1pHkbeuCyGvD7PBJxbfgQ1ZpGAs1P93++HXiembtNowP79kdpPL71MazE2Js MOjKH7BVJPSFVTSk2iMDaDNbJ66ZVC/Oyudj3q0Ek98fFGw/iu9n0JPoBDNWgkY6JayK ptMBDCpsyrA7UZQzBm4vtKw6TLFOGYqAjPF1awu+O3RoDAwgGvyeZMIY1Km951sUFv19 kr02qfCVRCvnVUjfbxLlg0eG0NaOULqteXBvwUnwOULESMD4hbmUWrAGjdwvS8W2jeE7 +Rfw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:in-reply-to:message-id :references:mime-version; bh=ETq2ktZkANjUC7Me/RyH9n1tJakbH50bRmVCx0xOmwg=; b=Twh/UxnGXNbLqOeqE55f17TaX/kpag21H8ZwHbZ10exyBPbnMZh9DuDFiVaXrxZ7vG npPrg/5wsMU4utW3WnyNP7GkUwyHHkFJ7avVkEMXnPRYSQt3GRnh1zEp1/jcQMEBq19g LtRG28Vh4F2dLx4MSXPCJD61YGrkR0sZELxxHA9RuYCQkKqwOTvc94xx206XupKbG3is gFk7xpzcgq1cwtTlWTFGXpfhgO45Ah2D8mwGabfGYZ9q+zDPQA7xXUfc0kfeNskUVjm+ JazBN9Fu7AaTf1fFXyUdNEaU3piZSrML50v6K6SVJu6ASeQ1EVmsB1h8lCxjIa+RYoa9 l9FA== X-Gm-Message-State: AOAM532eQALXGV06OvQPylcFKSSejsXt7E3hzXTmPHlZi8Du+S/51To9 Bzw8f8EII3hKZoovUY0rThJwmg== X-Google-Smtp-Source: ABdhPJwcJvhoNQhB4A5BKUQVkatkkHcE3W30AKdVI3lYpo5ilXK0mRFQaoESZtN23pj62Dq7kfxK8w== X-Received: by 2002:a17:90a:a394:: with SMTP id x20mr1593054pjp.24.1611600453220; Mon, 25 Jan 2021 10:47:33 -0800 (PST) Received: from [2620:15c:17:3:4a0f:cfff:fe51:6667] ([2620:15c:17:3:4a0f:cfff:fe51:6667]) by smtp.gmail.com with ESMTPSA id z16sm17086131pgj.51.2021.01.25.10.47.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Jan 2021 10:47:32 -0800 (PST) Date: Mon, 25 Jan 2021 10:47:31 -0800 (PST) From: David Rientjes To: Muchun Song cc: Randy Dunlap , Jonathan Corbet , Mike Kravetz , Thomas Gleixner , mingo@redhat.com, bp@alien8.de, x86@kernel.org, hpa@zytor.com, dave.hansen@linux.intel.com, luto@kernel.org, Peter Zijlstra , viro@zeniv.linux.org.uk, Andrew Morton , paulmck@kernel.org, mchehab+huawei@kernel.org, pawan.kumar.gupta@linux.intel.com, oneukum@suse.com, anshuman.khandual@arm.com, jroedel@suse.de, Mina Almasry , Matthew Wilcox , Oscar Salvador , Michal Hocko , "Song Bao Hua (Barry Song)" , David Hildenbrand , =?UTF-8?Q?HORIGUCHI_NAOYA=28=E5=A0=80=E5=8F=A3_=E7=9B=B4=E4=B9=9F=29?= , Xiongchun duan , linux-doc@vger.kernel.org, LKML , Linux Memory Management List , linux-fsdevel Subject: Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP In-Reply-To: Message-ID: <2d9bfd8d-a77f-6470-807c-1a71ffeac3ff@google.com> References: <20210117151053.24600-1-songmuchun@bytedance.com> <20210117151053.24600-3-songmuchun@bytedance.com> <472a58b9-12cb-3c3-d132-13dbae5174f0@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII 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 Mon, 25 Jan 2021, Muchun Song wrote: > > >> I'm not sure I understand the rationale for providing this help text if > > >> this is def_bool depending on CONFIG_HUGETLB_PAGE. Are you intending that > > >> this is actually configurable and we want to provide guidance to the admin > > >> on when to disable it (which it currently doesn't)? If not, why have the > > >> help text? > > > > > > This is __not__ configurable. Seems like a comment to help others > > > understand this option. Like Randy said. > > > > Yes, it could be written with '#' (or "comment") comment syntax instead of as help text. > > Got it. I will update in the next version. Thanks. > I'm not sure that Kconfig is the right place to document functional behavior of the kernel, especially for non-configurable options. Seems like this is already served by existing comments added by this patch series in the files where the description is helpful.