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=-16.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham 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 14516C433B4 for ; Wed, 5 May 2021 16:02:10 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id A0DA8613C0 for ; Wed, 5 May 2021 16:02:09 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A0DA8613C0 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 1BD376B0071; Wed, 5 May 2021 12:02:09 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 194356B0073; Wed, 5 May 2021 12:02:09 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 05C076B0074; Wed, 5 May 2021 12:02:09 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0210.hostedemail.com [216.40.44.210]) by kanga.kvack.org (Postfix) with ESMTP id E1A196B0071 for ; Wed, 5 May 2021 12:02:08 -0400 (EDT) Received: from smtpin15.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 9D8A18249980 for ; Wed, 5 May 2021 16:02:08 +0000 (UTC) X-FDA: 78107643936.15.DF10C37 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf21.hostedemail.com (Postfix) with ESMTP id B925BE002013 for ; Wed, 5 May 2021 16:02:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To: Content-Type:Content-ID:Content-Description; bh=/PxgZtEeisM5jqWmkSg4Ioma+K7rS43fQciK8cAPhf0=; b=a/vtSicz95XOLmNDnt6OS4EKSv ZUrUE8N1la8aehppG2HjAeAfBYJzDCqnfW+AJX2Onhag1q6pDc0t90WvciX/+AkDLX6rOEYqzMdyL wP9Orlv1sWSUOM51birljYdx87m9uFxwwrfuVuaV0wvJTLhC61cuvvm3sORW74Sb+zY0s+GvvLIM6 X1b1OlBnLtOOmcJtEwZaHXoP8RfqyrZm0ZcyqBAkaCFB2U1Kbwd/BC3jVj8HpczILtDoEowgMQX/4 HTV/dd2vO6Tt8ayhBGH3dIvjuwssHlDQl5dgFMBCqW24OjM28aeMPiaU05aLN9dGu+AVhqHlCQK1v vBEoh1Iw==; Received: from willy by casper.infradead.org with local (Exim 4.94 #2 (Red Hat Linux)) id 1leJwe-000Xip-J3; Wed, 05 May 2021 16:00:32 +0000 From: "Matthew Wilcox (Oracle)" To: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Cc: "Matthew Wilcox (Oracle)" , linux-kernel@vger.kernel.org Subject: [PATCH v9 43/96] mm/swap: Add folio_mark_accessed Date: Wed, 5 May 2021 16:05:35 +0100 Message-Id: <20210505150628.111735-44-willy@infradead.org> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210505150628.111735-1-willy@infradead.org> References: <20210505150628.111735-1-willy@infradead.org> MIME-Version: 1.0 Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b="a/vtSicz"; dmarc=none; spf=none (imf21.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org X-Rspamd-Server: rspam03 X-Stat-Signature: 49f856kyxjwxppe7kwuggq633s178uiu X-Rspamd-Queue-Id: B925BE002013 Received-SPF: none (infradead.org>: No applicable sender policy available) receiver=imf21; identity=mailfrom; envelope-from=""; helo=casper.infradead.org; client-ip=90.155.50.34 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1620230522-691633 Content-Transfer-Encoding: quoted-printable 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: Convert mark_page_accessed() to folio_mark_accessed(). It already operated on the entire compound page, but now we can avoid calling compound_head quite so many times. Shrinks the function from 424 bytes to 295 bytes (shrinking by 129 bytes). The compatibility wrapper is 30 bytes, plus the 8 bytes for the exported symbol means the kernel shrinks by 91 bytes. Signed-off-by: Matthew Wilcox (Oracle) --- include/linux/swap.h | 3 ++- mm/folio-compat.c | 7 +++++++ mm/swap.c | 34 ++++++++++++++++------------------ 3 files changed, 25 insertions(+), 19 deletions(-) diff --git a/include/linux/swap.h b/include/linux/swap.h index 8e0118b25bdc..d1cb67cdb476 100644 --- a/include/linux/swap.h +++ b/include/linux/swap.h @@ -346,7 +346,8 @@ extern void lru_note_cost(struct lruvec *lruvec, bool= file, unsigned int nr_pages); extern void lru_note_cost_page(struct page *); extern void lru_cache_add(struct page *); -extern void mark_page_accessed(struct page *); +void mark_page_accessed(struct page *); +void folio_mark_accessed(struct folio *); =20 extern atomic_t lru_disable_count; =20 diff --git a/mm/folio-compat.c b/mm/folio-compat.c index 7044fcc8a8aa..a374747ae1c6 100644 --- a/mm/folio-compat.c +++ b/mm/folio-compat.c @@ -5,6 +5,7 @@ */ =20 #include +#include =20 struct address_space *page_mapping(struct page *page) { @@ -41,3 +42,9 @@ bool page_mapped(struct page *page) return folio_mapped(page_folio(page)); } EXPORT_SYMBOL(page_mapped); + +void mark_page_accessed(struct page *page) +{ + folio_mark_accessed(page_folio(page)); +} +EXPORT_SYMBOL(mark_page_accessed); diff --git a/mm/swap.c b/mm/swap.c index 07244999bcc6..8e7f92be2f6f 100644 --- a/mm/swap.c +++ b/mm/swap.c @@ -380,7 +380,7 @@ static void folio_activate(struct folio *folio) } #endif =20 -static void __lru_cache_activate_page(struct page *page) +static void __lru_cache_activate_folio(struct folio *folio) { struct pagevec *pvec; int i; @@ -401,8 +401,8 @@ static void __lru_cache_activate_page(struct page *pa= ge) for (i =3D pagevec_count(pvec) - 1; i >=3D 0; i--) { struct page *pagevec_page =3D pvec->pages[i]; =20 - if (pagevec_page =3D=3D page) { - SetPageActive(page); + if (pagevec_page =3D=3D &folio->page) { + folio_set_active_flag(folio); break; } } @@ -420,36 +420,34 @@ static void __lru_cache_activate_page(struct page *= page) * When a newly allocated page is not yet visible, so safe for non-atomi= c ops, * __SetPageReferenced(page) may be substituted for mark_page_accessed(p= age). */ -void mark_page_accessed(struct page *page) +void folio_mark_accessed(struct folio *folio) { - page =3D compound_head(page); - - if (!PageReferenced(page)) { - SetPageReferenced(page); - } else if (PageUnevictable(page)) { + if (!folio_referenced(folio)) { + folio_set_referenced_flag(folio); + } else if (folio_unevictable(folio)) { /* * Unevictable pages are on the "LRU_UNEVICTABLE" list. But, * this list is never rotated or maintained, so marking an * evictable page accessed has no effect. */ - } else if (!PageActive(page)) { + } else if (!folio_active(folio)) { /* * If the page is on the LRU, queue it for activation via * lru_pvecs.activate_page. Otherwise, assume the page is on a * pagevec, mark it active and it'll be moved to the active * LRU on the next drain. */ - if (PageLRU(page)) - folio_activate(page_folio(page)); + if (folio_lru(folio)) + folio_activate(folio); else - __lru_cache_activate_page(page); - ClearPageReferenced(page); - workingset_activation(page_folio(page)); + __lru_cache_activate_folio(folio); + folio_clear_referenced_flag(folio); + workingset_activation(folio); } - if (page_is_idle(page)) - clear_page_idle(page); + if (folio_idle(folio)) + folio_clear_idle_flag(folio); } -EXPORT_SYMBOL(mark_page_accessed); +EXPORT_SYMBOL(folio_mark_accessed); =20 /** * lru_cache_add - add a page to a page list --=20 2.30.2