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]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B9803C79FB6 for ; Wed, 9 Sep 2026 11:22:46 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C4ABF6B0093; Wed, 9 Sep 2026 07:22:45 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C22066B0095; Wed, 9 Sep 2026 07:22:45 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id AECDC6B0096; Wed, 9 Sep 2026 07:22:45 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 886106B0093 for ; Wed, 9 Sep 2026 07:22:45 -0400 (EDT) Received: from smtpin29.hostedemail.com (lb01a-stub [10.200.18.249]) by unirelay08.hostedemail.com (Postfix) with ESMTP id BA8F214019A for ; Wed, 9 Sep 2026 11:22:44 +0000 (UTC) X-FDA: 85193986248.29.FCC62B9 Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by imf12.hostedemail.com (Postfix) with ESMTP id 4EAEF40008 for ; Wed, 9 Sep 2026 11:22:42 +0000 (UTC) Authentication-Results: imf12.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=qhguwTuN; spf=pass (imf12.hostedemail.com: domain of imbrenda@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=imbrenda@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Seal: i=1; a=rsa-sha256; d=hostedemail.com; s=arc-20220608; cv=none; t=1788952962; b=YfytLFcNwH2MJGLtYcbsGGBkANq/Y6ai7lDMeHAoqMJNoYcZPct5/9fEk6Dk9DVDiHTEsP MMNARsNUZIbqmC085CZWO5fG/phreGfUFc/navWgh+wHiOckfuVnMKNqMWpfGVtRMx/me+ VLy04CpPpi50dsISS517VACdxcjOPvU= ARC-Authentication-Results: i=1; imf12.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=qhguwTuN; spf=pass (imf12.hostedemail.com: domain of imbrenda@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=imbrenda@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1788952962; 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=/8IaYvCFo6fz+OwPEEmckG3lCFihAL43nD3n3f2T7Gc=; b=To3MAvWIHr3Sl8V2aJJDQM0sOzye9kFe/jHfwQ1iXPpbTCyOJ2URWmLvNbu1eBSa0MJE9N zwDLhuTCqen2ILRdOHXAJQzDEHkzP0cGyOMw/UNyHsFrYEsoZ6CdZhMH2bPBoV7bK2y2hv SxyeHgb6LJUJ/91XXGCubtrEuBZL6bo= Received: from pps.filterd (m0353725.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 689B1duw2666182; Wed, 9 Sep 2026 11:22:16 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=pp1; bh=/8IaYv CFo6fz+OwPEEmckG3lCFihAL43nD3n3f2T7Gc=; b=qhguwTuN778X3cOaWWW1OU aeZ7AJr5+u/cvNAs7RIOnmqYZbMjHAKojx4nXE/wh5sMGNOH6xbbXBYRV6W5CP5m 1LfKeLsLqhxoM/rtvfSFecMgjFvQaVkTmtglAbnTNxS9dCCPH/KKlDVi1+BNFW8b A4InRafmKCz6Cv0j8uRLUCdygzECd2S/44yMh+FBsFGBpXzX182KSY9KbjvUJx3R Y9u7hnBxycqsfJ6mB5bAL8bE1MRjX8qA9TLrZaAXWBKpiTNyT5Ad1gPC1O2J0Q1o sK1AZgxMslVpj/H1udTip6Q7YBFvLBe6aJ93X/kYeUNT6ud7ShMMaO3pvDg3tNaA == Received: from ppma21.wdc07v.mail.ibm.com (5b.69.3da9.ip4.static.sl-reverse.com [169.61.105.91]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 4ggbjrw4cr-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 09 Sep 2026 11:22:15 +0000 (GMT) Received: from pps.filterd (ppma21.wdc07v.mail.ibm.com [127.0.0.1]) by ppma21.wdc07v.mail.ibm.com (8.18.1.7/8.18.1.7) with ESMTP id 689AfLZI023073; Wed, 9 Sep 2026 11:22:14 GMT Received: from smtprelay03.fra02v.mail.ibm.com ([9.218.2.224]) by ppma21.wdc07v.mail.ibm.com (PPS) with ESMTPS id 4ggxdk1xnq-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 09 Sep 2026 11:22:14 +0000 (GMT) Received: from smtpav07.fra02v.mail.ibm.com (smtpav07.fra02v.mail.ibm.com [10.20.54.106]) by smtprelay03.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 689BM9Ka46072296 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 9 Sep 2026 11:22:10 GMT Received: from smtpav07.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id D56142004B; Wed, 9 Sep 2026 11:22:09 +0000 (GMT) Received: from smtpav07.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 84D262004D; Wed, 9 Sep 2026 11:22:09 +0000 (GMT) Received: from p-imbrenda (unknown [9.224.75.30]) by smtpav07.fra02v.mail.ibm.com (Postfix) with SMTP; Wed, 9 Sep 2026 11:22:09 +0000 (GMT) Date: Wed, 9 Sep 2026 13:22:07 +0200 From: Claudio Imbrenda To: "Lorenzo Stoakes (ARM)" Cc: Andrew Morton , David Hildenbrand , Mike Rapoport , Jonathan Corbet , Greg Kroah-Hartman , Jason Gunthorpe , John Hubbard , Peter Zijlstra , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Mark Rutland , Nick Piggin , Arnd Bergmann , "Matthew Wilcox (Oracle)" , Marc Zyngier , Christian Borntraeger , Janosch Frank , Alexander Gordeev , Gerald Schaefer , Heiko Carstens , Vasily Gorbik , "David S. Miller" , Alexander Viro , linux-mm@kvack.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-s390@vger.kernel.org Subject: Re: [PATCH 27/39] mm: rename is_vm_hugetlb_page() to vma_is_hugetlb() Message-ID: <20260909130141.2c5d250d@p-imbrenda> In-Reply-To: <20260908-b4-mmap-prepare-vma-flag-sanify-v1-27-dacf19cce22b@kernel.org> References: <20260908-b4-mmap-prepare-vma-flag-sanify-v1-0-dacf19cce22b@kernel.org> <20260908-b4-mmap-prepare-vma-flag-sanify-v1-27-dacf19cce22b@kernel.org> Organization: IBM X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 X-Proofpoint-Reinject: loops=2 maxloops=12 X-Authority-Analysis: v=2.4 cv=E7T9Y6dl c=1 sm=1 tr=0 ts=6aa14167 cx=c_pps a=GFwsV6G8L6GxiO2Y/PsHdQ==:117 a=GFwsV6G8L6GxiO2Y/PsHdQ==:17 a=kj9zAlcOel0A:10 a=VdqzKS8jKosA:10 a=VkNPw1HP01LnGYTKEx00:22 a=RnoormkPH1_aCDwRdu11:22 a=V8glGbnc2Ofi9Qvn3v5h:22 a=VwQbUJbxAAAA:8 a=VnNF1IyMAAAA:8 a=OpMjZilIJQQ69o6v8fAA:9 a=CjuIK1q_8ugA:10 X-Proofpoint-Spam-Info: AW1haW4tMjYwOTA5MDEyMiBTYWx0ZWRfXySxVHdsN1ufG wxjpHwj2VKPdL+MdVQie3JyfabMJU/26wLQwlGWO0QZf5MNW6jvmMiP9yN+m5dAuCWYjRliYhq4 TcA4SKzj97ajqgowLAlPjKHLqae4ogE= X-Proofpoint-ORIG-GUID: 9hCeolFJXa7mLi7YIHynK8hgkmct4qqF X-Proofpoint-GUID: H6kaRUBVX1M87FU8LUMUUE3VPbQB106q X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwOTA5MDEyMiBTYWx0ZWRfXwfSodi+GpBAe EWgvlM2SUJLuFEu9GHXrLlP2p81ohn4dYO/6vEBPnJTS7AYCF12oOwWJKBdy11FngCnfzjafrml x7eCREr73cwTqFnu6IA00UumMxG+FNDgYEdClHlPwbc1pNySdmscuIEFgiyEscAMb5CvqHCngH4 9etlThpWCx68gjVkvvt+o9hbhSMj/PU4F8xWzqFViAnbRaCUuf1D7Td9B3JQ4Vtu0fpb5dCBdIH aSEttzOvUQ4yhnFMkS4XYCLLFCOA4y1iHeW0G8CMS66W75wzs1pMgXqxvCQixpiQnCtN4jJpwxz MYPEBvggS/svZxLrh5QF1aQ4q+brQniNhqPP/TzsUGdpI+461ReTNyBLe9RDMDx4OF5Tcq5JaVY mAc/mvm2bod5UBNrSuH+QS/D5oV2zUf4Ngzn1gtQhCitsLJX3IgoC1sGFmMRvwFXIVDA1tW6YwN 6ze7nzFRPTPgnrVQ3SQ== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1176,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-09-08_03,2026-09-09_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1011 phishscore=0 spamscore=0 suspectscore=0 priorityscore=1501 bulkscore=0 impostorscore=0 malwarescore=0 lowpriorityscore=0 adultscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2609090122 X-Rspamd-Server: rspam10 X-Rspamd-Queue-Id: 4EAEF40008 X-Stat-Signature: kq6p5db9kbx6f5z4qod5eqspm7pu4ncy X-Rspam-User: X-HE-Tag: 1788952962-580443 X-HE-Meta: U2FsdGVkX18Ps21gbFWrhPqVO5gvrsM4b7CRaxqQf4cnT75bY60NxDxj+zpZ6p5wnH7urzU0rS9M5w8JH4cAbkD6QW6QXa5IiVyffhGnzsruW23e7yYD6fW+NMKsOnGST+Cgi1QchcKbicXpLgoO3c4SClumvGFaWmtLnis6WZngJaibH1Q8Nf9rCvgpINUrWwhP3Eb10u6vd9NZLziSTihRbyzzAKy38DerVlCD9rOSJv82z6s7Li3MqgsX6MbxBD5eS4SzbgIU+ZNbkFz/yBZFEC55Lnq6dtu+FAsUPTfuWyX9nX4We9crOY7lL0enFK7BfoWB8N/fhJRYca50zw0k7VKTt8I2ytbNPG6nMyEAu8eatIrEv10+3B2/DdGsNqS7eA9Euw7DIyuBAZCYR6ufz3v+ShqENbHdk5+LYU1a2CIisA3innQyCxeusHYihDRpCQafl9GHkXXWnm/va3hdK44e3F/6BEz1bjTra1Uyfza3hUIOXkXZ4ipe8X3R0jThDY7tkwm0BsLnbRLSAVSYfds14tqga1Gb/Za2QdzlgqrxopDUV5sxTIBunMCoYd6R/WYR00TtyDeFQ8+59O43Ec6IRPIWh5SYYRZorfZ3X3hp300p2gLH9ONSDNRbpUcCrIXV3RwGCcSyQMPAUpNUyggpDVsV7MrDIBZTqsnVV4I70ONWPtqmlzsR3aZvRcBzCSt08nH29UgSc+zCLo5akmZ/PSJi6odjg5ANtRIJ1vKSNDdoFBn2G6xaItOM6sONGWFRIykPz95pi+jU9teMxnv6Ym/CFMTNqtgcy675Q933ZB97AwXtWu4dJ+VexrFAnFIK905QSChk4FHlLojWtoN6F+KT4cvlR1R4KQwh031TxyIgr4k8n5HPwEbkCDI40cG6DJ4oAEje3azHkozIRMJWcfDIa4M2wqgBIyWH5XYFITknPG7XUx14sZGkcOFWvpTSjgy1s1bYAhz 0d4TRyd4 6vEKpV98QFXhAQ7jtcjwbUES/Av2H308EHtpK8hFmMUjPP4nuW2X67Of6rNka9S99TqRsBj+uSsubV55ff9vjQnnlzGrEsXW3mrzYraxtidnr1A0K3KrqeNiwdSW5O6xbZxLwMLA04lJgnUbeYMRwXKRb8RPrReh0nIxQIP+rAlVQ9LBw/9tF6XV1tpUCvJBMlZaacX+7NbLaqmZbZhn2U+Q6AAC8H9AceCNqr38IWSsYm8NkDpPJQMXG694Ru7aBGZQXbAnTyRg4ixAedklkGIYoPthzvIZVVKdvsUBY/v6S55CDFvZIvy8xcMSwKK524+Z2 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Tue, 08 Sep 2026 21:01:31 +0100 "Lorenzo Stoakes (ARM)" wrote: [please note that I had to prune the CC list quite aggressively because my SMTP server was not happy] > The is_vm_hugetlb_page() predicate is badly named - the mapping can span > more than a page and it is inconsistent with other VMA predicates that > typically are prefixed by vma_. > > Rename to vma_is_hugetlb() for consistency, and while we're here update > some VM_BUG_ON_VMA() to VM_WARN_ON_ONCE_VMA() as to avoid unnecessary > oopses. > > No functional change intended. > > Signed-off-by: Lorenzo Stoakes (ARM) for s390: Acked-by: Claudio Imbrenda but please see the comment below > --- > arch/arm64/kvm/mmu.c | 4 ++-- > arch/powerpc/mm/book3s64/radix_tlb.c | 6 +++--- > arch/powerpc/mm/nohash/e500_hugetlbpage.c | 2 +- > arch/powerpc/mm/nohash/tlb.c | 2 +- > arch/riscv/kvm/mmu.c | 2 +- > arch/riscv/mm/tlbflush.c | 2 +- > arch/s390/mm/gmap_helpers.c | 6 +++--- > arch/sparc/mm/init_64.c | 2 +- > drivers/gpu/drm/drm_gpusvm.c | 2 +- > fs/coredump.c | 2 +- > fs/hugetlbfs/inode.c | 2 +- > fs/proc/task_mmu.c | 8 ++++---- > include/asm-generic/tlb.h | 2 +- > include/linux/hugetlb.h | 4 ++-- > include/linux/mm.h | 19 ++++++++++++++++--- > include/linux/rmap.h | 2 +- > kernel/events/core.c | 2 +- > kernel/sched/fair.c | 2 +- > mm/gup.c | 4 ++-- > mm/huge_memory.c | 2 +- > mm/hugetlb.c | 14 +++++++------- > mm/internal.h | 2 +- > mm/madvise.c | 4 ++-- > mm/memory.c | 12 ++++++------ > mm/mempolicy.c | 2 +- > mm/migrate_device.c | 2 +- > mm/mmap.c | 2 +- > mm/mmu_gather.c | 2 +- > mm/mprotect.c | 2 +- > mm/mremap.c | 6 +++--- > mm/page_vma_mapped.c | 4 ++-- > mm/pagewalk.c | 2 +- > mm/swapfile.c | 2 +- > mm/userfaultfd.c | 26 +++++++++++++------------- > mm/vma.c | 8 ++++---- > mm/vmscan.c | 2 +- > tools/testing/vma/include/stubs.h | 2 +- > 37 files changed, 92 insertions(+), 79 deletions(-) [...] > diff --git a/arch/s390/mm/gmap_helpers.c b/arch/s390/mm/gmap_helpers.c > index ff63ffb1dbd2..3f6783b93e67 100644 > --- a/arch/s390/mm/gmap_helpers.c > +++ b/arch/s390/mm/gmap_helpers.c > @@ -102,7 +102,7 @@ __context_unsafe(/* pte_unmap_unlock() not instrumented */) > > /* Find the vm address for the guest address */ > vma = vma_lookup(mm, vmaddr); > - if (!vma || is_vm_hugetlb_page(vma)) > + if (!vma || vma_is_hugetlb(vma)) > return; > > /* Get pointer to the page table entry */ > @@ -139,7 +139,7 @@ void gmap_helper_discard(struct mm_struct *mm, unsigned long vmaddr, unsigned lo > vma = find_vma_intersection(mm, vmaddr, end); > if (!vma) > return; > - if (!is_vm_hugetlb_page(vma)) > + if (!vma_is_hugetlb(vma)) > zap_vma_range(vma, vmaddr, min(end, vma->vm_end) - vmaddr); > vmaddr = vma->vm_end; > } > @@ -247,7 +247,7 @@ static int __gmap_helper_unshare_zeropages(struct mm_struct *mm) > * proof to catch unexpected zeropages in other mappings and > * fail. > */ > - if ((vma->vm_flags & VM_PFNMAP) || is_vm_hugetlb_page(vma)) > + if ((vma->vm_flags & VM_PFNMAP) || vma_is_hugetlb(vma)) Would it be possible to replace this ^ with !vma_can_gup() ? (obviously not in this patch, since vma_can_gup() is only introduced at the end of the series) > continue; > addr = vma->vm_start; > [...]