From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f74.google.com (mail-wm1-f74.google.com [209.85.128.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2BFE0392811 for ; Sat, 28 Feb 2026 12:44:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.74 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772282672; cv=none; b=NYJ2kPOatm4PJBBu2bhxjUoTNN0s++VIJuNf9TL2ZhIgf79Zyx2NV0Sax3zgfCwvuY5w713LQEIXSDj1HAiGXR2JR+6HCebnThakDyObQpM1C0mLsQ8fJC+WnNpUjeYKoYxlM6AWLoy23KvbTX4SBQHrgOqSXw1YHU5KL1+t/6E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772282672; c=relaxed/simple; bh=tFmr9Byzvb8HJGLej5py8cdOGPaTsp9oa5qHurqtjoY=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=tt4dDRRY0xH0K+EGTeK29bj/yVtZLz/MCUaJEU5LkPr0L0Tt2xq/gOWyhkwkB6Tv4SpaXY2kuUDeT7F63mHkB9JR76OUtiqn5mYtd9z8eED9Nt5jPhMuC32jlk8M/izTSZjsE7n+MUiIXSw3JDKDpGjg1C6B+oVLoUZkpbPwI9M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=kr5HmvWO; arc=none smtp.client-ip=209.85.128.74 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="kr5HmvWO" Received: by mail-wm1-f74.google.com with SMTP id 5b1f17b1804b1-483a2db68caso23191695e9.0 for ; Sat, 28 Feb 2026 04:44:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1772282668; x=1772887468; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=RH0L9WD4/u9mQiuU+Jc2bLYt2zoxsRLi5sNyM2YzgMo=; b=kr5HmvWO4Za7oHkdy3Q6/y7Snpak/F3hJxMr1t8WFZoaa1XfBQrT0gu3Rp1Nnji4gy +Y5N9poFyC/7fvBuXe8fDWbR7D/StCKmDBtQI+2+2avUq1SDHR4rNE4fBFL91o9ZU+zw bLnHWbtznlJ5LNsU3fhYfrmdmAdFj+FlrYeK9kNA7g+e9AwVwsGvfYFhCbatmldQ4bAy o+2SmFgJDrkCBGQKNZ9TZKOK91liCi2uTETOrUZisgPC2WT2dvFLSS2vKE6n5PUf8/xa eEJTHgr+TLr6yXReJhINV89fqZrWBvdr+ZKu65KyNbQPZeUuGD+zBTe+MbvkmVJ2+jnj yklw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772282668; x=1772887468; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=RH0L9WD4/u9mQiuU+Jc2bLYt2zoxsRLi5sNyM2YzgMo=; b=aJEEkiC+M1h8Dn2Vdo5nhE9ZoEhqtdXGTtOAUax1pExdeDH/jjkqmbh5pCrNlTHVja Zcxre5NWzRE7f7hgOAoiaPs0+K2eqiZAZ7lG+rGtocTvavoTKyCOZdy6y3aM7qTIpUUm vL8g2pDnwd+ohvolNkQgDcU/pxSOX9zzsUIqxtVmOQp/CrUE0xPEL8iw/B3vktmvtA+4 kXEQqsOn9h61XOXAohS1zhudJmeiRRJC1BLjG8tC31bH0N6pFyL0g17CyZwDZBqBjNFf tL7rdqFoaQW/x9AbUUmQAyHUQrpcEI8NxiQkef6W9LeK9/GidYmZw6XC75/QOXwo+oBO /zEQ== X-Gm-Message-State: AOJu0YzHPvMfjUVKUVBycudlvzQXiMVZh43uMxVaTZa8PGIA7ZhMkIgv iAqi+hzX1zn72+ENwVOjh1WY5uv2xpSvD1OJo8YGfAhXd/YfXzJMaqs1L/WBfMifHUvyAGNawXS MOFm4N+xXhmk4Tx9Dbw== X-Received: from wmqi19.prod.google.com ([2002:a05:600c:3553:b0:483:6fe1:c054]) (user=aliceryhl job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:4fc8:b0:47b:e2a9:2bd7 with SMTP id 5b1f17b1804b1-483c9beaca0mr120569145e9.19.1772282668287; Sat, 28 Feb 2026 04:44:28 -0800 (PST) Date: Sat, 28 Feb 2026 12:44:27 +0000 In-Reply-To: <20260227200848.114019-15-david@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260227200848.114019-1-david@kernel.org> <20260227200848.114019-15-david@kernel.org> Message-ID: Subject: Re: [PATCH v1 14/16] mm: rename zap_page_range_single() to zap_vma_range() From: Alice Ryhl To: "David Hildenbrand (Arm)" Cc: linux-kernel@vger.kernel.org, "linux-mm @ kvack . org" , Andrew Morton , Lorenzo Stoakes , "Liam R. Howlett" , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Jann Horn , Pedro Falcato , David Rientjes , Shakeel Butt , "Matthew Wilcox (Oracle)" , Madhavan Srinivasan , Michael Ellerman , Christian Borntraeger , Janosch Frank , Claudio Imbrenda , Alexander Gordeev , Gerald Schaefer , Heiko Carstens , Vasily Gorbik , Jarkko Sakkinen , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Greg Kroah-Hartman , "Arve =?utf-8?B?SGrDuG5uZXbDpWc=?=" , Todd Kjos , Christian Brauner , Carlos Llamas , Ian Abbott , H Hartley Sweeten , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , Tvrtko Ursulin , David Airlie , Simona Vetter , Jason Gunthorpe , Leon Romanovsky , Dimitri Sivanich , Arnd Bergmann , Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Peter Zijlstra , Arnaldo Carvalho de Melo , Namhyung Kim , Andy Lutomirski , Vincenzo Frascino , Eric Dumazet , Neal Cardwell , "David S. Miller" , David Ahern , Jakub Kicinski , Paolo Abeni , Miguel Ojeda , linuxppc-dev@lists.ozlabs.org, kvm@vger.kernel.org, linux-s390@vger.kernel.org, linux-sgx@vger.kernel.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-rdma@vger.kernel.org, bpf@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-fsdevel@vger.kernel.org, netdev@vger.kernel.org, rust-for-linux@vger.kernel.org, x86@kernel.org Content-Type: text/plain; charset="utf-8" On Fri, Feb 27, 2026 at 09:08:45PM +0100, David Hildenbrand (Arm) wrote: > diff --git a/drivers/android/binder/page_range.rs b/drivers/android/binder/page_range.rs > index fdd97112ef5c..2fddd4ed8d4c 100644 > --- a/drivers/android/binder/page_range.rs > +++ b/drivers/android/binder/page_range.rs > @@ -130,7 +130,7 @@ pub(crate) struct ShrinkablePageRange { > pid: Pid, > /// The mm for the relevant process. > mm: ARef, > - /// Used to synchronize calls to `vm_insert_page` and `zap_page_range_single`. > + /// Used to synchronize calls to `vm_insert_page` and `zap_vma_range`. > #[pin] > mm_lock: Mutex<()>, > /// Spinlock protecting changes to pages. > @@ -719,7 +719,7 @@ fn drop(self: Pin<&mut Self>) { > > if let Some(vma) = mmap_read.vma_lookup(vma_addr) { > let user_page_addr = vma_addr + (page_index << PAGE_SHIFT); > - vma.zap_page_range_single(user_page_addr, PAGE_SIZE); > + vma.zap_vma_range(user_page_addr, PAGE_SIZE); > } LGTM. Be aware that this will have a merge conflict with patches currently in char-misc-linus that are scheduled to land in an -rc. > diff --git a/drivers/android/binder_alloc.c b/drivers/android/binder_alloc.c > index dd2046bd5cde..e4488ad86a65 100644 > --- a/drivers/android/binder_alloc.c > +++ b/drivers/android/binder_alloc.c > @@ -1185,7 +1185,7 @@ enum lru_status binder_alloc_free_page(struct list_head *item, > if (vma) { > trace_binder_unmap_user_start(alloc, index); > > - zap_page_range_single(vma, page_addr, PAGE_SIZE); > + zap_vma_range(vma, page_addr, PAGE_SIZE); > > trace_binder_unmap_user_end(alloc, index); LGTM. > diff --git a/rust/kernel/mm/virt.rs b/rust/kernel/mm/virt.rs > index b8e59e4420f3..04b3cc925d67 100644 > --- a/rust/kernel/mm/virt.rs > +++ b/rust/kernel/mm/virt.rs > @@ -113,7 +113,7 @@ pub fn end(&self) -> usize { > /// kernel goes further in freeing unused page tables, but for the purposes of this operation > /// we must only assume that the leaf level is cleared. > #[inline] > - pub fn zap_page_range_single(&self, address: usize, size: usize) { > + pub fn zap_vma_range(&self, address: usize, size: usize) { > let (end, did_overflow) = address.overflowing_add(size); > if did_overflow || address < self.start() || self.end() < end { > // TODO: call WARN_ONCE once Rust version of it is added > @@ -124,7 +124,7 @@ pub fn zap_page_range_single(&self, address: usize, size: usize) { > // sufficient for this method call. This method has no requirements on the vma flags. The > // address range is checked to be within the vma. > unsafe { > - bindings::zap_page_range_single(self.as_ptr(), address, size) > + bindings::zap_vma_range(self.as_ptr(), address, size) > }; > } Same as previous patch: please run rustfmt. It will format on a single line, like this: unsafe { bindings::zap_vma_range(self.as_ptr(), address, size) }; with the above change applied: Acked-by: Alice Ryhl # Rust and Binder Alice