From patchwork Mon Feb 24 16:55:45 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Hildenbrand X-Patchwork-Id: 13988496 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 CB3D5C021BC for ; Mon, 24 Feb 2025 16:56:20 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 5E1556B008A; Mon, 24 Feb 2025 11:56:20 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 590D66B008C; Mon, 24 Feb 2025 11:56:20 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 36F30280001; Mon, 24 Feb 2025 11:56:20 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id EF0B46B008A for ; Mon, 24 Feb 2025 11:56:19 -0500 (EST) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 762261218FB for ; Mon, 24 Feb 2025 16:56:19 +0000 (UTC) X-FDA: 83155441278.30.C6C2613 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by imf17.hostedemail.com (Postfix) with ESMTP id 2BEC14000D for ; Mon, 24 Feb 2025 16:56:17 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=cDgaioi0; spf=pass (imf17.hostedemail.com: domain of dhildenb@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=dhildenb@redhat.com; dmarc=pass (policy=none) header.from=redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1740416177; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=8Z/i7SxHkRJm72w2k3iTqc1vo3DynQo4UiIchyZFPhI=; b=sARGekFYDYDwAqlDu1NVY/jaGQa5LjDc6Slp7Mv+w7fUGVAX45FXf3zhJV6U5lgJIKEAIp QUaG9x2rzIMbV0Qe104PAMur6xJceeAxtBXnPQOTJRhPDqei3Xc1JG043wkr/hCMd7CCYi WMYLwY3SLA4vKRHyFR+WAu1eQuBypAQ= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1740416177; a=rsa-sha256; cv=none; b=rZauYwwjk+hvEhAvaEikRZ1SLR9tWPnh+G3Fd1NrT4ZDNWyjXEa/jo3kk12r2X7LTfERIm 6C4orxom/6XAcKSd8qcRSrTjC2aWOv+HcZlUguMBCvOz/ZiwkhvB3J0GiV/pT6Z6JZunxY HMkttDwvyU7cmC0/dzhS/kWVZmYjQU4= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=cDgaioi0; spf=pass (imf17.hostedemail.com: domain of dhildenb@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=dhildenb@redhat.com; dmarc=pass (policy=none) header.from=redhat.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1740416176; 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: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=8Z/i7SxHkRJm72w2k3iTqc1vo3DynQo4UiIchyZFPhI=; b=cDgaioi0ZaOpqqYQ5RQ4Y6O2YTU2E1+G6Lo6koYElhFS6qevD3oDicLebdDu/mH6Yvx7QF wJfyKY6vkx6Z4DY0F7f0Qg7uQWmW6EVkSeCzliJuIa5NlP38sIbYnkgrM2kDABLkS2UzFe 7XrMF0tDHeCapndoUkCCS5XY5LqoWs0= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-531-7zETdBKUOE6f1ka6MfGZ9Q-1; Mon, 24 Feb 2025 11:56:13 -0500 X-MC-Unique: 7zETdBKUOE6f1ka6MfGZ9Q-1 X-Mimecast-MFC-AGG-ID: 7zETdBKUOE6f1ka6MfGZ9Q_1740416172 Received: by mail-wm1-f72.google.com with SMTP id 5b1f17b1804b1-4399d2a1331so22851115e9.1 for ; Mon, 24 Feb 2025 08:56:12 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740416172; x=1741020972; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=8Z/i7SxHkRJm72w2k3iTqc1vo3DynQo4UiIchyZFPhI=; b=HPwdtELH8/MfNKuT08wwpehebZtrxUzxVAJ/oGsOFlNOeUouDcIeQnFlONRXsS0xc5 zIWv/0bRNbW11iHPZPdkrKvrwsLsLWjGU6+R+KmQfTHfzC4IhMynaWdXLzvVNwfvhKh+ PCTh+aXrHlEwB1PZ5gPFmObGXft/10K2WemdR5cBXhMgsGR0MaUkRuwekTku3UWjQmOY hCgqUPTVEgmgRgG5Ipuc6CoxaRI6iX9UBu5XbnuKE8V0u9HMy5x2iM6xbsYYC8PWwUap T/iH5MOpIrWOmv6RW5aIB6j7HWzsU6D7/LWQKATUxArVTyjENRFZHZ/u37iYgvCHM0OO LRaw== X-Forwarded-Encrypted: i=1; AJvYcCVlx4l34Vqo//vsHJvKiGUC2p7eaYDuAOUjmseyWF2M0YxG3vjwLlefFQBm2MK+GG+UHCWSX1/jcg==@kvack.org X-Gm-Message-State: AOJu0Yyso8jqaYspHBQ7PXK6yvaD9sZIYGXkMu/eej/CTVT5elXn+Csg 7rtzYFCrfke+EttuGwIJ87QRpEUu3hhfzl+tSTGqrEn1fqRBxKobMBvsv6s8LIjFLDituovRVti JEozPCmxNJbbOXB2PaC+fp0bpbFoU8/0yIJygSLkYNm8MeYhH X-Gm-Gg: ASbGnctmx5n417/G9tzwdllZh5AnDPQEBxwImeeECRGZH8vWvDGcjW86S5X38n/4eWZ Wibq/Nf5N/P45LJBf3L8Zhpr2KPQDXj3z/qmyK+0PnW+8PKqBpFn6+PmPSWm5mRPNpFZcBQz7TM XW30aho29HyCzg6AXHmeH0Cee5mAe1a2f354xVlS4JwXihedfgdZDZ2s5qX+nQFx9zoYTV2naV2 I1OOCgBKIVGxaE5rLOMLx1zdHZu4M+64fE8aHaNFtgkegBVPtIFuJaMWLJa4iXoQnC9ugA5TLkD ZK20oz/6X2crc/pBWwIY1Msrd03hCqbs2acYs83g0A== X-Received: by 2002:a05:600c:1c85:b0:439:968b:6669 with SMTP id 5b1f17b1804b1-439ae2d254bmr112029005e9.1.1740416171872; Mon, 24 Feb 2025 08:56:11 -0800 (PST) X-Google-Smtp-Source: AGHT+IGPiVGKd002kgHq/C+YhiMRT6ZEt5tkRc6eny1qWEWThiWAlrLvHc+EEKxBr1Q0DjH+goSmcg== X-Received: by 2002:a05:600c:1c85:b0:439:968b:6669 with SMTP id 5b1f17b1804b1-439ae2d254bmr112028725e9.1.1740416171409; Mon, 24 Feb 2025 08:56:11 -0800 (PST) Received: from localhost (p4ff234b6.dip0.t-ipconnect.de. [79.242.52.182]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-439b0371b7bsm112170735e9.33.2025.02.24.08.56.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 24 Feb 2025 08:56:10 -0800 (PST) From: David Hildenbrand To: linux-kernel@vger.kernel.org Cc: linux-doc@vger.kernel.org, cgroups@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org, David Hildenbrand , Andrew Morton , "Matthew Wilcox (Oracle)" , Tejun Heo , Zefan Li , Johannes Weiner , =?utf-8?q?Michal_Koutn=C3=BD?= , Jonathan Corbet , Andy Lutomirski , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Muchun Song , "Liam R. Howlett" , Lorenzo Stoakes , Vlastimil Babka , Jann Horn , "Kirill A. Shutemov" Subject: [PATCH v2 03/20] mm: let _folio_nr_pages overlay memcg_data in first tail page Date: Mon, 24 Feb 2025 17:55:45 +0100 Message-ID: <20250224165603.1434404-4-david@redhat.com> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250224165603.1434404-1-david@redhat.com> References: <20250224165603.1434404-1-david@redhat.com> MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: gojpNXwnuMKRVlv2luA32i6RL28uWH4lT6-nKOC6Z0o_1740416172 X-Mimecast-Originator: redhat.com content-type: text/plain; charset="US-ASCII"; x-default=true X-Rspam-User: X-Rspamd-Queue-Id: 2BEC14000D X-Rspamd-Server: rspam07 X-Stat-Signature: 9cn86z4qbew59ckqagtj3ruaf4jiwns4 X-HE-Tag: 1740416177-186968 X-HE-Meta: U2FsdGVkX18ZZBwvXRZXKcg+uqpgc4oARZTdCEBWbtBA1bfETnsmYBH7c9Dnf4g6lHCbOu1XkU7n+aOBav41+CpGbCBAdW6lfqUDeV7hBphzCyFEg2qZsiDgf40vhb0hJy+6ahCbmas4xQW7UHUfBq/jLqaZfLtB8xQDBUzkMocyexhQsnzUQRXJ4gRbUDiCZG1a2ew6ceJyJy4YQ7fA9Tn1ni0YzC3SljxQzPzmmuyM0e2BaB4kslXjFgc1VA4LK0bJPZ8V0mFGQ4kzNx1y1Hk0Fyzj22X+RBqBogt6pybgAZBhwk/+NOQRIoTQ8J8YJCZDEIMseaNmkde4KGmrlD42QEdRS6iOWyImUl1SOp8lTJMu4u4tRR4Xc8OECnRucK1bbP+3C6t02zi+3oAfSYW0TfOZTegJ42pzjnaizM6v2IrUWEx33lDh3RjL+xn/bqYV81JK1fzOy9JyIiyqwpuI6MaO4XySYJQSxCaxQGRg/leOXRxcj89lMHF2/U0AVygAXNyl42jlcRVEiAf6xfGes5ZMq9abmr3GqXYSCTe5cs5VlCrLUXB/7LVYGg0Z/jDFFbFIKaGPgBsTtcybzWyQpq/ZmjHITTUdPSYdx6zDGqNOND5M//UKFLa5xx3U2OVYIP4hicFYZE136uzuKJdU6WGtittCVdlz1sZ8ayv5OrXYG9VUVUqhjqmvHqOld1YJjkRCRwZNlD1fAz+P3LSKgNoz0FClup3tdEm2oBT76TLX7dnMS8FkJ+jiUGavidg3OPx5Gk11mzTqcfz2jXwRl2lFeZkHdPAYpaxsYy6JkyXL7xvBLlFfJOjKQJ/atpzmkwTpoSakizjHtSsk9EXJ29NqyxA27vmQ0bijLvhtSWBmpZtdHh8IIJYymRwAJXfIYd1KdoHvwUUWE1ej+lKbH7UunaBjF9VzlqMWanfNDKpw8geY3n4qostyqpRjnlfj+Jk5uf3BMk/9qYO 9bYoEN13 uV1z6HS0J4kSTYxQVMhimK+lWlWi5FMfsCToit2Zk8CwIG6eMat/OJT8BQ0WWLZJpQ3gaIreOGbt+4y3QHwymZH57U8r1Nt7JvTozH2A6Gr4XOMDYVXvuYjryzlCRiWWok+40E7W36ScmGq+XYm7ndNU3xkg/oGXpOlCIOCe8h+atlX7cXShQC9NBhNNEGLswp1Nf6e9dI5vqoJ/hlqJRxT77Qza9r7ylem7/gRPpBAdbopULY7m2PujofSOjtYofzTQ/brSq8LaXvuU/ZchbfIhKH3qEDzWNvN9l+J00NZF+HjplPjPuMeE9sZASm+wY7kt4mlhbcMZpJd+/IbGTii2X03N3c6bXB4dkQi60Y/V3L9ta1B9tfd+HDkkimUjmJhUKCQbkHeTpDcaLTyF0z6WRTZG4J8ZiWJHvqd9h4ug5zAS1NljnOzzQuftz6G1p4y1RSk5LUeFNvSJGz7Hp+fFDrqSocOhlQMwDZUY5s4Z6iWMBTryy+BH+Xcq0FOiYzvs+efwOto9HxWEjU8SCFWGP6G8wzI7uQrkG2BcEOnFvABuvClCCF5XPwAjIVBzM8v/H 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: List-Subscribe: List-Unsubscribe: Let's free up some more of the "unconditionally available on 64BIT" space in order-1 folios by letting _folio_nr_pages overlay memcg_data in the first tail page (second folio page). Consequently, we have the optimization now whenever we have CONFIG_MEMCG, independent of 64BIT. We have to make sure that page->memcg on tail pages does not return "surprises". page_memcg_check() already properly refuses PageTail(). Let's do that earlier in print_page_owner_memcg() to avoid printing wrong "Slab cache page" information. No other code should touch that field on tail pages of compound pages. Reset the "_nr_pages" to 0 when splitting folios, or when freeing them back to the buddy (to avoid false page->memcg_data "bad page" reports). Note that in __split_huge_page(), folio_nr_pages() would stop working already as soon as we start messing with the subpages. Most kernel configs should have at least CONFIG_MEMCG enabled, even if disabled at runtime. 64byte "struct memmap" is what we usually have on 64BIT. While at it, rename "_folio_nr_pages" to "_nr_pages". Hopefully memdescs / dynamically allocating "strut folio" in the future will further clean this up, e.g., making _nr_pages available in all configs and maybe even in small folios. Doing that should be fairly easy on top of this change. Reviewed-by: Kirill A. Shutemov Signed-off-by: David Hildenbrand --- include/linux/mm.h | 4 ++-- include/linux/mm_types.h | 30 ++++++++++++++++++++++-------- mm/huge_memory.c | 16 +++++++++++++--- mm/internal.h | 4 ++-- mm/page_alloc.c | 6 +++++- mm/page_owner.c | 2 +- 6 files changed, 45 insertions(+), 17 deletions(-) diff --git a/include/linux/mm.h b/include/linux/mm.h index 7c5a8fd29cfcd..f6b6373a864dd 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -1199,10 +1199,10 @@ static inline unsigned int folio_large_order(const struct folio *folio) return folio->_flags_1 & 0xff; } -#ifdef CONFIG_64BIT +#ifdef NR_PAGES_IN_LARGE_FOLIO static inline long folio_large_nr_pages(const struct folio *folio) { - return folio->_folio_nr_pages; + return folio->_nr_pages; } #else static inline long folio_large_nr_pages(const struct folio *folio) diff --git a/include/linux/mm_types.h b/include/linux/mm_types.h index 689b2a7461892..e81be20bbabc6 100644 --- a/include/linux/mm_types.h +++ b/include/linux/mm_types.h @@ -287,6 +287,11 @@ typedef struct { unsigned long val; } swp_entry_t; +#if defined(CONFIG_MEMCG) || defined(CONFIG_SLAB_OBJ_EXT) +/* We have some extra room after the refcount in tail pages. */ +#define NR_PAGES_IN_LARGE_FOLIO +#endif + /** * struct folio - Represents a contiguous set of bytes. * @flags: Identical to the page flags. @@ -312,7 +317,7 @@ typedef struct { * @_large_mapcount: Do not use directly, call folio_mapcount(). * @_nr_pages_mapped: Do not use outside of rmap and debug code. * @_pincount: Do not use directly, call folio_maybe_dma_pinned(). - * @_folio_nr_pages: Do not use directly, call folio_nr_pages(). + * @_nr_pages: Do not use directly, call folio_nr_pages(). * @_hugetlb_subpool: Do not use directly, use accessor in hugetlb.h. * @_hugetlb_cgroup: Do not use directly, use accessor in hugetlb_cgroup.h. * @_hugetlb_cgroup_rsvd: Do not use directly, use accessor in hugetlb_cgroup.h. @@ -377,13 +382,20 @@ struct folio { unsigned long _flags_1; unsigned long _head_1; /* public: */ - atomic_t _large_mapcount; - atomic_t _entire_mapcount; - atomic_t _nr_pages_mapped; - atomic_t _pincount; -#ifdef CONFIG_64BIT - unsigned int _folio_nr_pages; -#endif + union { + struct { + atomic_t _large_mapcount; + atomic_t _entire_mapcount; + atomic_t _nr_pages_mapped; + atomic_t _pincount; + }; + unsigned long _usable_1[4]; + }; + atomic_t _mapcount_1; + atomic_t _refcount_1; +#ifdef NR_PAGES_IN_LARGE_FOLIO + unsigned int _nr_pages; +#endif /* NR_PAGES_IN_LARGE_FOLIO */ /* private: the union with struct page is transitional */ }; struct page __page_1; @@ -435,6 +447,8 @@ FOLIO_MATCH(_last_cpupid, _last_cpupid); offsetof(struct page, pg) + sizeof(struct page)) FOLIO_MATCH(flags, _flags_1); FOLIO_MATCH(compound_head, _head_1); +FOLIO_MATCH(_mapcount, _mapcount_1); +FOLIO_MATCH(_refcount, _refcount_1); #undef FOLIO_MATCH #define FOLIO_MATCH(pg, fl) \ static_assert(offsetof(struct folio, fl) == \ diff --git a/mm/huge_memory.c b/mm/huge_memory.c index 1e45064046a0f..13152313e77ce 100644 --- a/mm/huge_memory.c +++ b/mm/huge_memory.c @@ -3307,9 +3307,10 @@ bool can_split_folio(struct folio *folio, int caller_pins, int *pextra_pins) * It splits @folio into @new_order folios and copies the @folio metadata to * all the resulting folios. */ -static void __split_folio_to_order(struct folio *folio, int new_order) +static void __split_folio_to_order(struct folio *folio, int old_order, + int new_order) { - long nr_pages = folio_nr_pages(folio); + long nr_pages = 1 << old_order; long new_nr_pages = 1 << new_order; long index; @@ -3529,12 +3530,21 @@ static int __split_unmapped_folio(struct folio *folio, int new_order, } } + /* + * Reset any memcg data overlay in the tail pages. + * folio_nr_pages() is unreliable until prep_compound_page() + * was called again. + */ +#ifdef NR_PAGES_IN_LARGE_FOLIO + folio->_nr_pages = 0; +#endif + /* complete memcg works before add pages to LRU */ split_page_memcg(&folio->page, old_order, split_order); split_page_owner(&folio->page, old_order, split_order); pgalloc_tag_split(folio, old_order, split_order); - __split_folio_to_order(folio, split_order); + __split_folio_to_order(folio, old_order, split_order); after_split: /* diff --git a/mm/internal.h b/mm/internal.h index b07550db2bfd1..7f6d5def00fa0 100644 --- a/mm/internal.h +++ b/mm/internal.h @@ -684,8 +684,8 @@ static inline void folio_set_order(struct folio *folio, unsigned int order) return; folio->_flags_1 = (folio->_flags_1 & ~0xffUL) | order; -#ifdef CONFIG_64BIT - folio->_folio_nr_pages = 1U << order; +#ifdef NR_PAGES_IN_LARGE_FOLIO + folio->_nr_pages = 1U << order; #endif } diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 16dfcf7ade74a..bdfc954dab9aa 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -1174,8 +1174,12 @@ __always_inline bool free_pages_prepare(struct page *page, if (unlikely(order)) { int i; - if (compound) + if (compound) { page[1].flags &= ~PAGE_FLAGS_SECOND; +#ifdef NR_PAGES_IN_LARGE_FOLIO + folio->_nr_pages = 0; +#endif + } for (i = 1; i < (1 << order); i++) { if (compound) bad += free_tail_page_prepare(page, page + i); diff --git a/mm/page_owner.c b/mm/page_owner.c index 2d6360eaccbb6..a409e2561a8fd 100644 --- a/mm/page_owner.c +++ b/mm/page_owner.c @@ -507,7 +507,7 @@ static inline int print_page_owner_memcg(char *kbuf, size_t count, int ret, rcu_read_lock(); memcg_data = READ_ONCE(page->memcg_data); - if (!memcg_data) + if (!memcg_data || PageTail(page)) goto out_unlock; if (memcg_data & MEMCG_DATA_OBJEXTS)