From nobody Mon Feb 9 04:04:18 2026 Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2C16630FC3C for ; Fri, 6 Feb 2026 13:34:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.176.79.56 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770384885; cv=none; b=Y+F4drOGvEVTlJnyJ9xG8NeOwXiCU1LY+mrbU0vQRf8J0p9bMUVwqPrWbghT8PbpEHEOA3QKkLK0IQnzfzcPA5Q3wvU0jM/S3ozexaTW6i9NxPHEIVFgCyqVadu0TqyDj5plb/6x7XbAJS4fBVSmggyQryehhRpEiwZEMO/6Nlg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770384885; c=relaxed/simple; bh=It5ttsXt48cTcbif/WNE4710YSAfTM35g5KTHZfcjgw=; h=From:To:CC:Subject:Date:Message-ID:MIME-Version:Content-Type; b=ZxTPKjBHOjMPuFtWsliLekPrXJ5bAI3KkciqNL9+mtD9QbfsmBSFLVTMSHGhMsFrAvm0cdBwvG79qZKK8DriWi4APh71kJlIogBYOn4tLZK5eUoQfQhwl4CcgU6w8Y6M2USXUjb2C4aEe2bIFtdAkvXjOiJysfj1HMJDSPKrGGo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=h-partners.com; spf=pass smtp.mailfrom=h-partners.com; arc=none smtp.client-ip=185.176.79.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=h-partners.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=h-partners.com Received: from mail.maildlp.com (unknown [172.18.224.107]) by frasgout.his.huawei.com (SkyGuard) with ESMTPS id 4f6w645B89zJ46FB; Fri, 6 Feb 2026 21:33:48 +0800 (CST) Received: from mscpeml500003.china.huawei.com (unknown [7.188.49.51]) by mail.maildlp.com (Postfix) with ESMTPS id 1402840584; Fri, 6 Feb 2026 21:34:41 +0800 (CST) Received: from mscphis00972.huawei.com (10.123.68.107) by mscpeml500003.china.huawei.com (7.188.49.51) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.11; Fri, 6 Feb 2026 16:34:40 +0300 From: Gladyshev Ilya To: CC: , , , , David Hildenbrand , Andrew Morton , Lorenzo Stoakes , "Liam R. Howlett" , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Zi Yan , Harry Yoo , "Matthew Wilcox (Oracle)" , Yu Zhao , Baolin Wang , Will Deacon , , Subject: [PATCH] mm: make ref_unless functions unless_zero only Date: Fri, 6 Feb 2026 13:33:02 +0000 Message-ID: <20260206133328.426921-1-gladyshev.ilya1@h-partners.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: mscpeml100003.china.huawei.com (10.199.174.67) To mscpeml500003.china.huawei.com (7.188.49.51) Content-Type: text/plain; charset="utf-8" There are no users of (folio/page)_ref_add_unless(page, nr, u) with u !=3D 0 [1] and all current users are "internal" for page refcounting API. This allows us to safely drop this parameter and reduce function semantics to the "unless zero" cases only, which will be optimized in the following patch. If needed, these functions for the u!=3D0 cases can be trivially reintroduced later using the same atomic_add_unless operations as before. [1]: The last user was dropped in v5.18 kernel, commit 27674ef6c73f ("mm: remove the extra ZONE_DEVICE struct page refcount"). There is no trace of discussion as to why this cleanup wasn't done earlier. Co-developed-by: Gorbunov Ivan Signed-off-by: Gorbunov Ivan Signed-off-by: Gladyshev Ilya Acked-by: David Hildenbrand (Arm) Acked-by: Kiryl Shutsemau Acked-by: Zi Yan --- include/linux/mm.h | 2 +- include/linux/page-flags.h | 6 +++--- include/linux/page_ref.h | 14 +++++++------- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/include/linux/mm.h b/include/linux/mm.h index f0d5be9dc736..0b30305512dc 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -1306,7 +1306,7 @@ static inline int folio_put_testzero(struct folio *fo= lio) */ static inline bool get_page_unless_zero(struct page *page) { - return page_ref_add_unless(page, 1, 0); + return page_ref_add_unless_zero(page, 1); } =20 static inline struct folio *folio_get_nontail_page(struct page *page) diff --git a/include/linux/page-flags.h b/include/linux/page-flags.h index f7a0e4af0c73..fb6a83fe88b0 100644 --- a/include/linux/page-flags.h +++ b/include/linux/page-flags.h @@ -231,7 +231,7 @@ static __always_inline const struct page *page_fixed_fa= ke_head(const struct page return page; } =20 -static __always_inline bool page_count_writable(const struct page *page, i= nt u) +static __always_inline bool page_count_writable(const struct page *page) { if (!static_branch_unlikely(&hugetlb_optimize_vmemmap_key)) return true; @@ -257,7 +257,7 @@ static __always_inline bool page_count_writable(const s= truct page *page, int u) * The refcount check also prevents modification attempts to other (r/o) * tail pages that are not fake heads. */ - if (atomic_read_acquire(&page->_refcount) =3D=3D u) + if (!atomic_read_acquire(&page->_refcount)) return false; =20 return page_fixed_fake_head(page) =3D=3D page; @@ -268,7 +268,7 @@ static inline const struct page *page_fixed_fake_head(c= onst struct page *page) return page; } =20 -static inline bool page_count_writable(const struct page *page, int u) +static inline bool page_count_writable(const struct page *page) { return true; } diff --git a/include/linux/page_ref.h b/include/linux/page_ref.h index 544150d1d5fd..b0e3f4a4b4b8 100644 --- a/include/linux/page_ref.h +++ b/include/linux/page_ref.h @@ -228,14 +228,14 @@ static inline int folio_ref_dec_return(struct folio *= folio) return page_ref_dec_return(&folio->page); } =20 -static inline bool page_ref_add_unless(struct page *page, int nr, int u) +static inline bool page_ref_add_unless_zero(struct page *page, int nr) { bool ret =3D false; =20 rcu_read_lock(); /* avoid writing to the vmemmap area being remapped */ - if (page_count_writable(page, u)) - ret =3D atomic_add_unless(&page->_refcount, nr, u); + if (page_count_writable(page)) + ret =3D atomic_add_unless(&page->_refcount, nr, 0); rcu_read_unlock(); =20 if (page_ref_tracepoint_active(page_ref_mod_unless)) @@ -243,9 +243,9 @@ static inline bool page_ref_add_unless(struct page *pag= e, int nr, int u) return ret; } =20 -static inline bool folio_ref_add_unless(struct folio *folio, int nr, int u) +static inline bool folio_ref_add_unless_zero(struct folio *folio, int nr) { - return page_ref_add_unless(&folio->page, nr, u); + return page_ref_add_unless_zero(&folio->page, nr); } =20 /** @@ -261,12 +261,12 @@ static inline bool folio_ref_add_unless(struct folio = *folio, int nr, int u) */ static inline bool folio_try_get(struct folio *folio) { - return folio_ref_add_unless(folio, 1, 0); + return folio_ref_add_unless_zero(folio, 1); } =20 static inline bool folio_ref_try_add(struct folio *folio, int count) { - return folio_ref_add_unless(folio, count, 0); + return folio_ref_add_unless_zero(folio, count); } =20 static inline int page_ref_freeze(struct page *page, int count) --=20 2.43.0