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 92D62C4167B for ; Mon, 11 Dec 2023 16:14:36 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1CA8D6B0149; Mon, 11 Dec 2023 11:14:36 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 155DB6B014A; Mon, 11 Dec 2023 11:14:36 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F36306B014B; Mon, 11 Dec 2023 11:14:35 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id DD8216B0149 for ; Mon, 11 Dec 2023 11:14:35 -0500 (EST) Received: from smtpin18.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id D0135A065E for ; Mon, 11 Dec 2023 16:14:28 +0000 (UTC) X-FDA: 81555035016.18.05A8B88 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf09.hostedemail.com (Postfix) with ESMTP id 8890D14000E for ; Mon, 11 Dec 2023 16:14:26 +0000 (UTC) Authentication-Results: imf09.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf09.hostedemail.com: domain of ryan.roberts@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=ryan.roberts@arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1702311267; 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; bh=IZp1xAsGlrX2tBwEXSq4B+lNS7wG1T2sAYfQwYpIFo8=; b=qKQMe5FT4IxbkUz4hsoNyrdfrDPXbVpikVFXb4QYr8oagVHUNsYmxl/tE8ahGC/3O9ERaH 4b2cj8LHXaPV8fQMZpr/KC/IaH+3Z3D6nEXnhZYGChrIjT+w5sUwOEO9QGyLiTl7s9UJQ8 ivTj7kkjiH4Gc0GwNdwuib65Ncyiaq8= ARC-Authentication-Results: i=1; imf09.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf09.hostedemail.com: domain of ryan.roberts@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=ryan.roberts@arm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1702311267; a=rsa-sha256; cv=none; b=hO6iWxwWVP1V4qSwg+IbueKbVB4nrK2iVPSJ8jYob0OExweLlyVawebiAhYqOPb4vNu+jd N5EqK6DzcGyvpDY4M0oYqFkMBE9sxb61gnyIhKrMSSm2uCgdtw2crjWzHRI1W7lVQq1f9c R3m3U1bVyqDbh2uruAkLCEcXLc0VkuU= Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CAF0516F2; Mon, 11 Dec 2023 08:15:11 -0800 (PST) Received: from [10.57.73.30] (unknown [10.57.73.30]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id BBE4A3F738; Mon, 11 Dec 2023 08:14:23 -0800 (PST) Message-ID: Date: Mon, 11 Dec 2023 16:14:22 +0000 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 01/39] mm/rmap: rename hugepage_add* to hugetlb_add* Content-Language: en-GB To: David Hildenbrand , linux-kernel@vger.kernel.org Cc: linux-mm@kvack.org, Andrew Morton , "Matthew Wilcox (Oracle)" , Hugh Dickins , Yin Fengwei , Mike Kravetz , Muchun Song , Peter Xu , Muchun Song References: <20231211155652.131054-1-david@redhat.com> <20231211155652.131054-2-david@redhat.com> From: Ryan Roberts In-Reply-To: <20231211155652.131054-2-david@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 8890D14000E X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: bq6xsj5zf5kn7gkwht9pw1dnoc9e6bf4 X-HE-Tag: 1702311266-666831 X-HE-Meta: U2FsdGVkX18GNvyfWODxR3yUqXoHeXL1IhbRobqcmTZkxJvbliNXUoefug+tnDXKbIQfU3UnfqMyqYA2rjTpcBM2HDXrewCrN9VzPl1ASSuKhO2I9nPS4kOA5ezDoJM09tRYoLjvD7YvN1JfALA8nZzVI47eUs99ehoCPuflqTBlCGdrs6cPCFUZkfdujVTJVIGLleLqurC+XGDj1FUkuAgrudhFyrQXrTHV/iXiV6eoK8ufQ5aLLDdiZiqZ3pZSauKGJOgLh2iD9F/bwIexJzqjRD9J1D32i3JTPv5PmRFFqoUZqlbgw+W0fxgt+fVcvLQYWGhZndoncc0zXKIUSLx4ZLFnKYhvmoMYAAxApN0q4SuAw6Ul2HwwEWY8QM+01MaTr9Cjb8duPluAsc9k9G8aAKZgzjJrt7yecMzbMmREN6R+sZWD1db8uI/8L5B32EEoA1nh/PKGFeFXU4f3bQv2FZFhvvhm8BbwNlaU1MUyO/IH1IH++27D4AOrjIkCeYppyVCPQJR+uvGAjOfowEXxpMJc+K3FLnkhwijRa3Fds0YLPsrdOpsHni0ysTX/AFaYT+3+cdXjg4nnmyqbwXcAUDuOJBLAGUPi6+DWqUFe9HpLhLuh5qO4HsT3gUlmO0Lctqh9W12xd1khzGsMgc/2yRC3RtkAxeNdoSg/1xxKJarZ4NYgaaMcuAsTjSakigbcyJNrVoY+rwMCATSjQVs86otMDXIlw/5t0yycrQazTC1IsthTGHJvOxPfS08gi3S5ZKnG3Xaeow5AyyVCldNHjw+KW3ZkAO+x5QCBmubVwAmaPzlDJ6MONNbjOQkNdbPOnzAJG0A+9X/ixw6gUEIlkgnT1tv/TLxt85Q/qQg6RAsaT9JisiopFzCpS2Gh36wCHilK9cFLc5uYAQaOjRkqG0lwNLeJUaQ3MoKrxLfwhsCr4/VL4Hncs9Z3eZZb0egzBWaEC+vE4H/PgP/ 3oDuEjta kl5lXHv5XiKzgbrIIDp04N0ypOcTLDWsW2NuAspZ0MoZlDyTCsythqTq+hiqV79XdqNiPMBDrOZ8xjXUJIXlppm0uyyHRF2s9B1A4Y7OAxPOTQKlBdTXhomgC0WTyNYRxevV2GzAumdyYF17DuIooiQCwBPP4ADyOpoZAKXIB7PEJmEbpeNXo0n9mw7KyL2rH7Maj3ql02VopLP4LWNXRWI5VqUMIgGTVzaxqvAqg7od1YmGX9A6SEACyNFfRGJzqbT6SPeDuQSCaFhJ1JQI1F/wT85GznbCBIfNm/ac1U6/obJR/p4Tzkej2mlwzgu+XxPwEF+y+9rmQyLT4voFnO7x63g== 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: On 11/12/2023 15:56, David Hildenbrand wrote: > Let's just call it "hugetlb_". > > Yes, it's all already inconsistent and confusing because we have a lot > of "hugepage_" functions for legacy reasons. But "hugetlb" cannot possibly > be confused with transparent huge pages, and it matches "hugetlb.c" and > "folio_test_hugetlb()". So let's minimize confusion in rmap code. > > Reviewed-by: Muchun Song > Signed-off-by: David Hildenbrand Reviewed-by: Ryan Roberts > --- > include/linux/rmap.h | 4 ++-- > mm/hugetlb.c | 8 ++++---- > mm/migrate.c | 4 ++-- > mm/rmap.c | 8 ++++---- > 4 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/include/linux/rmap.h b/include/linux/rmap.h > index af6a32b6f3e7..0bfea866f39b 100644 > --- a/include/linux/rmap.h > +++ b/include/linux/rmap.h > @@ -208,9 +208,9 @@ void folio_add_file_rmap_range(struct folio *, struct page *, unsigned int nr, > void page_remove_rmap(struct page *, struct vm_area_struct *, > bool compound); > > -void hugepage_add_anon_rmap(struct folio *, struct vm_area_struct *, > +void hugetlb_add_anon_rmap(struct folio *, struct vm_area_struct *, > unsigned long address, rmap_t flags); > -void hugepage_add_new_anon_rmap(struct folio *, struct vm_area_struct *, > +void hugetlb_add_new_anon_rmap(struct folio *, struct vm_area_struct *, > unsigned long address); > > static inline void __page_dup_rmap(struct page *page, bool compound) > diff --git a/mm/hugetlb.c b/mm/hugetlb.c > index 6feb3e0630d1..305f3ca1dee6 100644 > --- a/mm/hugetlb.c > +++ b/mm/hugetlb.c > @@ -5285,7 +5285,7 @@ hugetlb_install_folio(struct vm_area_struct *vma, pte_t *ptep, unsigned long add > pte_t newpte = make_huge_pte(vma, &new_folio->page, 1); > > __folio_mark_uptodate(new_folio); > - hugepage_add_new_anon_rmap(new_folio, vma, addr); > + hugetlb_add_new_anon_rmap(new_folio, vma, addr); > if (userfaultfd_wp(vma) && huge_pte_uffd_wp(old)) > newpte = huge_pte_mkuffd_wp(newpte); > set_huge_pte_at(vma->vm_mm, addr, ptep, newpte, sz); > @@ -5988,7 +5988,7 @@ static vm_fault_t hugetlb_wp(struct mm_struct *mm, struct vm_area_struct *vma, > /* Break COW or unshare */ > huge_ptep_clear_flush(vma, haddr, ptep); > page_remove_rmap(&old_folio->page, vma, true); > - hugepage_add_new_anon_rmap(new_folio, vma, haddr); > + hugetlb_add_new_anon_rmap(new_folio, vma, haddr); > if (huge_pte_uffd_wp(pte)) > newpte = huge_pte_mkuffd_wp(newpte); > set_huge_pte_at(mm, haddr, ptep, newpte, huge_page_size(h)); > @@ -6277,7 +6277,7 @@ static vm_fault_t hugetlb_no_page(struct mm_struct *mm, > goto backout; > > if (anon_rmap) > - hugepage_add_new_anon_rmap(folio, vma, haddr); > + hugetlb_add_new_anon_rmap(folio, vma, haddr); > else > page_dup_file_rmap(&folio->page, true); > new_pte = make_huge_pte(vma, &folio->page, ((vma->vm_flags & VM_WRITE) > @@ -6732,7 +6732,7 @@ int hugetlb_mfill_atomic_pte(pte_t *dst_pte, > if (folio_in_pagecache) > page_dup_file_rmap(&folio->page, true); > else > - hugepage_add_new_anon_rmap(folio, dst_vma, dst_addr); > + hugetlb_add_new_anon_rmap(folio, dst_vma, dst_addr); > > /* > * For either: (1) CONTINUE on a non-shared VMA, or (2) UFFDIO_COPY > diff --git a/mm/migrate.c b/mm/migrate.c > index 35a88334bb3c..4cb849fa0dd2 100644 > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -249,8 +249,8 @@ static bool remove_migration_pte(struct folio *folio, > > pte = arch_make_huge_pte(pte, shift, vma->vm_flags); > if (folio_test_anon(folio)) > - hugepage_add_anon_rmap(folio, vma, pvmw.address, > - rmap_flags); > + hugetlb_add_anon_rmap(folio, vma, pvmw.address, > + rmap_flags); > else > page_dup_file_rmap(new, true); > set_huge_pte_at(vma->vm_mm, pvmw.address, pvmw.pte, pte, > diff --git a/mm/rmap.c b/mm/rmap.c > index 846fc79f3ca9..80d42c31281a 100644 > --- a/mm/rmap.c > +++ b/mm/rmap.c > @@ -2625,8 +2625,8 @@ void rmap_walk_locked(struct folio *folio, struct rmap_walk_control *rwc) > * > * RMAP_COMPOUND is ignored. > */ > -void hugepage_add_anon_rmap(struct folio *folio, struct vm_area_struct *vma, > - unsigned long address, rmap_t flags) > +void hugetlb_add_anon_rmap(struct folio *folio, struct vm_area_struct *vma, > + unsigned long address, rmap_t flags) > { > VM_WARN_ON_FOLIO(!folio_test_anon(folio), folio); > > @@ -2637,8 +2637,8 @@ void hugepage_add_anon_rmap(struct folio *folio, struct vm_area_struct *vma, > PageAnonExclusive(&folio->page), folio); > } > > -void hugepage_add_new_anon_rmap(struct folio *folio, > - struct vm_area_struct *vma, unsigned long address) > +void hugetlb_add_new_anon_rmap(struct folio *folio, > + struct vm_area_struct *vma, unsigned long address) > { > BUG_ON(address < vma->vm_start || address >= vma->vm_end); > /* increment count (starts at -1) */