From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f200.google.com (mail-pf0-f200.google.com [209.85.192.200]) by kanga.kvack.org (Postfix) with ESMTP id 6F5636B0390 for ; Fri, 31 Mar 2017 23:29:44 -0400 (EDT) Received: by mail-pf0-f200.google.com with SMTP id f63so12329665pfc.23 for ; Fri, 31 Mar 2017 20:29:44 -0700 (PDT) Received: from mga09.intel.com (mga09.intel.com. [134.134.136.24]) by mx.google.com with ESMTPS id y69si6905888plh.168.2017.03.31.20.29.43 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 31 Mar 2017 20:29:43 -0700 (PDT) From: "Huang\, Ying" Subject: Re: [PATCH -mm -v7 1/9] mm, swap: Make swap cluster size same of THP size on x86_64 References: <20170328053209.25876-1-ying.huang@intel.com> <20170328053209.25876-2-ying.huang@intel.com> <20170329165522.GA31821@cmpxchg.org> <87o9wjtwvv.fsf@yhuang-dev.intel.com> <20170331145617.GB6408@cmpxchg.org> Date: Sat, 01 Apr 2017 11:29:41 +0800 In-Reply-To: <20170331145617.GB6408@cmpxchg.org> (Johannes Weiner's message of "Fri, 31 Mar 2017 10:56:17 -0400") Message-ID: <87pogwreje.fsf@yhuang-dev.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ascii Sender: owner-linux-mm@kvack.org List-ID: To: Johannes Weiner Cc: "Huang, Ying" , Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Hugh Dickins , Shaohua Li , Minchan Kim , Rik van Riel Johannes Weiner writes: > On Thu, Mar 30, 2017 at 08:45:56AM +0800, Huang, Ying wrote: >> Johannes Weiner writes: >> >> > On Tue, Mar 28, 2017 at 01:32:01PM +0800, Huang, Ying wrote: >> >> @@ -499,6 +499,19 @@ config FRONTSWAP >> >> >> >> If unsure, say Y to enable frontswap. >> >> >> >> +config ARCH_USES_THP_SWAP_CLUSTER >> >> + bool >> >> + default n >> > >> > This is fine. >> > >> >> +config THP_SWAP_CLUSTER >> >> + bool >> >> + depends on SWAP && TRANSPARENT_HUGEPAGE && ARCH_USES_THP_SWAP_CLUSTER >> >> + default y >> >> + help >> >> + Use one swap cluster to hold the contents of the THP >> >> + (Transparent Huge Page) swapped out. The size of the swap >> >> + cluster will be same as that of THP. >> > >> > But this is a super weird thing to ask the user. How would they know >> > what to say, if we don't know? I don't think this should be a config >> > knob at all. Merge the two config items into a simple >> >> The user will not see this, because there is no string after "bool" to >> let user to select it. The help here is for document only, so that >> architecture developers could know what this is for. > > Oh, I missed that. My bad! > >> > config THP_SWAP_CLUSTER >> > bool >> > default n >> > >> > and let the archs with reasonable THP sizes select it. >> >> This will have same effect as the original solution except the document >> is removed. > > Then I still don't understand why we need two config symbols. Can't > archs select the documented THP_SWAP_CLUSTER directly? > > The #ifdef in swapfile.c could check THP && THP_SWAP_CLUSTER. > > Am I missing something? I use two config symbols just to save some typing, instead of #ifdef CONFIG_THP_SWAP_CLUSTER it will be, #if defined(CONFIG_TRANSPARENT_HUGEPAGE) && defined(CONFIG_THP_SWAP_CLUSTER) or #if defined(CONFIG_SWAP) && defined(CONFIG_TRANSPARENT_HUGEPAGE) && defined(CONFIG_THP_SWAP_CLUSTER) Best Regards, Huang, Ying -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org