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 36058C00140 for ; Wed, 10 Aug 2022 13:23:21 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7BACB8E0002; Wed, 10 Aug 2022 09:23:20 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 769938E0001; Wed, 10 Aug 2022 09:23:20 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 60A768E0002; Wed, 10 Aug 2022 09:23:20 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 4B9938E0001 for ; Wed, 10 Aug 2022 09:23:20 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 277D6A052D for ; Wed, 10 Aug 2022 13:23:20 +0000 (UTC) X-FDA: 79783749360.26.2CA7E7F Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by imf16.hostedemail.com (Postfix) with ESMTP id 892D4180043 for ; Wed, 10 Aug 2022 13:23:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1660137798; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=UHiMxplfNJQ6RhacgZ+xBSTgZSfx+YMHv5xB6BH8Lo8=; b=bnfUjL5GWi+ATacUETn7sKzkNjke2FwhB3DXnYtbJ/ByjJEDXtwL5gJiym4ZjwMzHsgNsr +WLyzwSltFWRJFDJoyrMUQgKCytRgV5s5E+B01MatY7DvAyRMFsEaBzAD1pVVML1g8NW0M bXOxKX659bVLa76T07c0reWR6p9knVI= Received: from mail-il1-f199.google.com (mail-il1-f199.google.com [209.85.166.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-120-O3flJlfiPOCy8c4HAPp71A-1; Wed, 10 Aug 2022 09:23:17 -0400 X-MC-Unique: O3flJlfiPOCy8c4HAPp71A-1 Received: by mail-il1-f199.google.com with SMTP id w6-20020a056e021a6600b002dea6904708so10455499ilv.6 for ; Wed, 10 Aug 2022 06:23:17 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=UHiMxplfNJQ6RhacgZ+xBSTgZSfx+YMHv5xB6BH8Lo8=; b=nuNjxEFSo2FzdTbzS/0Tg/2R51ODT5E6lxz34WrEs1PqSk4uTCMLyAUgyFUDrv1f9B FYyd3xRjpAZgMzC8GeTVpYB62XBIn3ysv2VgVVikpezS07PwcqEwRUPVWfbg4pM4Jz0b eNUO/FqM8K5g+4vAPP31/0EpWSQZCXQFgStn1/W4oKIJcZuupZqgW/HvBz5IJ8sgkiTf PmR/8oGYNXn6tskziSk0fBSKcBD8FXyE5yy7pvDp2b6vnMv+8hOhRKqKeBbe2OYqQqVH sHPb6xkNeSKblfqR4fxv4NFk4nWir/LPQYuR5pG4iP5T+GkpiU1/TEqMV5BSQpvYfSuq U2aw== X-Gm-Message-State: ACgBeo36Fso3Q6rsYUMIBNF8pwG4wbUqKN47WLBzSNcG4kCR4XAamd1Q 3LpcXQ2Dm58v/qHqz0cBaiDh6Tq4KxJJxwe4K3223zCHt3To7VEWbA6g0lUNJUWM1jaDJ4MKwSd M0eShZE/eUg8= X-Received: by 2002:a92:8748:0:b0:2d9:3f81:d0b7 with SMTP id d8-20020a928748000000b002d93f81d0b7mr12270749ilm.310.1660137796531; Wed, 10 Aug 2022 06:23:16 -0700 (PDT) X-Google-Smtp-Source: AA6agR4CFc0fNM1RWLMb7NxUuoRwLTkzJPTVqhqiOzuypVus+LgkAyKMstesr/EDCaOwx4Fr4WdnaA== X-Received: by 2002:a92:8748:0:b0:2d9:3f81:d0b7 with SMTP id d8-20020a928748000000b002d93f81d0b7mr12270727ilm.310.1660137796239; Wed, 10 Aug 2022 06:23:16 -0700 (PDT) Received: from xz-m1.local (bras-base-aurron9127w-grc-35-70-27-3-10.dsl.bell.ca. [70.27.3.10]) by smtp.gmail.com with ESMTPSA id o16-20020a0566022e1000b0067bcb28e036sm2314721iow.49.2022.08.10.06.23.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Aug 2022 06:23:15 -0700 (PDT) Date: Wed, 10 Aug 2022 09:23:14 -0400 From: Peter Xu To: "Huang, Ying" Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Minchan Kim , David Hildenbrand , Nadav Amit , Andrew Morton , Hugh Dickins , Vlastimil Babka , Andrea Arcangeli , Andi Kleen , "Kirill A . Shutemov" Subject: Re: [PATCH v3 6/7] mm/swap: Cache maximum swapfile size when init swap Message-ID: References: <20220809220100.20033-1-peterx@redhat.com> <20220809220100.20033-7-peterx@redhat.com> <87y1vwehgh.fsf@yhuang6-desk2.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: <87y1vwehgh.fsf@yhuang6-desk2.ccr.corp.intel.com> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Disposition: inline ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1660137799; a=rsa-sha256; cv=none; b=vhm0TJoKqOCEUY/ccW7X0qdAgBz6CkkjBhYaHxxhklCcv+6XKrvtB3UHskiVj+J5YbfULw WhifK7Asik9bmBnRlOYslI6Jwh4fPSm2UyJhu/cYQz9CGde/eOQeQbkQHs79wcfMDMZaCP NWZ3IS/NLcFo1rSVHJt3f2aHIuM2IJA= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=bnfUjL5G; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf16.hostedemail.com: domain of peterx@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=peterx@redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1660137799; 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=UHiMxplfNJQ6RhacgZ+xBSTgZSfx+YMHv5xB6BH8Lo8=; b=PK6XQ0IvtN1pUiVfcxDFq/f8FcVrfniWFCTT2RVcO/Kk+3zACvyh5Uang+fiS/XUrX3LKM 6g4AzpaJNJjZC6LsV8wGeJPza6k3hEI0Hl/HSKINatWN690/VTQMbmDovaZiDRmhBDYGVN AUSo7IVGaDy10mGkBWd5WK2/thKOXwM= X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 892D4180043 Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=bnfUjL5G; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf16.hostedemail.com: domain of peterx@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=peterx@redhat.com X-Rspam-User: X-Stat-Signature: quxzg7977id9mhg5h3ddkwdrrcp6sptu X-HE-Tag: 1660137799-747835 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 Wed, Aug 10, 2022 at 02:33:02PM +0800, Huang, Ying wrote: > Peter Xu writes: > > > We used to have swapfile_maximum_size() fetching a maximum value of > > swapfile size per-arch. > > > > As the caller of max_swapfile_size() grows, this patch introduce a variable > > "swapfile_maximum_size" and cache the value of old max_swapfile_size(), so > > that we don't need to calculate the value every time. > > > > Caching the value in swapfile_init() is safe because when reaching the > > phase we should have initialized all the relevant information. Here the > > major arch to look after is x86, which defines the max size based on L1TF > > mitigation. > > > > Here both X86_BUG_L1TF or l1tf_mitigation should have been setup properly > > when reaching swapfile_init(). As a reference, the code path looks like > > this for x86: > > > > - start_kernel > > - setup_arch > > - early_cpu_init > > - early_identify_cpu --> setup X86_BUG_L1TF > > - parse_early_param > > - l1tf_cmdline --> set l1tf_mitigation > > - check_bugs > > - l1tf_select_mitigation --> set l1tf_mitigation > > - arch_call_rest_init > > - rest_init > > - kernel_init > > - kernel_init_freeable > > - do_basic_setup > > - do_initcalls --> calls swapfile_init() (initcall level 4) > > > > The swapfile size only depends on swp pte format on non-x86 archs, so > > caching it is safe too. > > > > Since at it, rename max_swapfile_size() to arch_max_swapfile_size() because > > arch can define its own function, so it's more straightforward to have > > "arch_" as its prefix. At the meantime, keep the swapfile_maximum_size() > > function to fetch the value from the cache initialized in swapfile_init(). > > > > Signed-off-by: Peter Xu > > --- > > arch/x86/mm/init.c | 2 +- > > mm/swapfile.c | 10 +++++++++- > > 2 files changed, 10 insertions(+), 2 deletions(-) > > > > diff --git a/arch/x86/mm/init.c b/arch/x86/mm/init.c > > index 82a042c03824..9121bc1b9453 100644 > > --- a/arch/x86/mm/init.c > > +++ b/arch/x86/mm/init.c > > @@ -1054,7 +1054,7 @@ void update_cache_mode_entry(unsigned entry, enum page_cache_mode cache) > > } > > > > #ifdef CONFIG_SWAP > > -unsigned long max_swapfile_size(void) > > +unsigned long arch_max_swapfile_size(void) > > { > > unsigned long pages; > > > > diff --git a/mm/swapfile.c b/mm/swapfile.c > > index 1fdccd2f1422..794fa37bd0c3 100644 > > --- a/mm/swapfile.c > > +++ b/mm/swapfile.c > > @@ -63,6 +63,7 @@ EXPORT_SYMBOL_GPL(nr_swap_pages); > > /* protected with swap_lock. reading in vm_swap_full() doesn't need lock */ > > long total_swap_pages; > > static int least_priority = -1; > > +static unsigned long swapfile_maximum_size; > > > > static const char Bad_file[] = "Bad swap file entry "; > > static const char Unused_file[] = "Unused swap file entry "; > > @@ -2816,11 +2817,16 @@ unsigned long generic_max_swapfile_size(void) > > } > > > > /* Can be overridden by an architecture for additional checks. */ > > -__weak unsigned long max_swapfile_size(void) > > +__weak unsigned long arch_max_swapfile_size(void) > > { > > return generic_max_swapfile_size(); > > } > > > > +unsigned long max_swapfile_size(void) > > +{ > > + return swapfile_maximum_size; > > +} > > + > > It appears unnecessary to hide a variable with a function. Why not just > use the variable directly. Sure. -- Peter Xu