From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (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 BD46333F5BA for ; Tue, 21 Apr 2026 13:19:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776777601; cv=none; b=q/du2XEazYgrphUVEQxant7zNgYysLXFzkURhzWQKLzUxSfOJlnbYHMuDu6iVSeZkkf+a5nQc0ckXCHmNw+r2B5bCn5GcSJr6FR5Obm2YRqobk3/1Z7F1R0M7LPoq688XoUAMTv74CR1MTSVllt8tsH2waj6LfjLvmXLzzdSz5k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776777601; c=relaxed/simple; bh=l5nh2tX+tG4PyQxEGrsy245Ln5RZlFhnAyXMUDP8PX4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=pmrG7OZrlrXq3mOaF1rdgNGVNokj2x2jcECWrJi5FeC/mqJLGR7CDHbZd4tpx2fYEodSIsLSVsSxpIGZesg7mlTXwo3nzCHN4BypyvTjEA+sAbe+lWGRRLBq6Cdli3E6ilG4NfvSW2iwK8WV2RuGyqb76hsg5e5ZAeplRLiXQhI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=XpZsz+2k; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=XbIMbwQ2; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="XpZsz+2k"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="XbIMbwQ2" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1776777599; 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: in-reply-to:in-reply-to:references:references; bh=xexzte0yTqtWkjudFGvCNSwnTYifigEpG3qwMv3/9fw=; b=XpZsz+2kwAm3I+f7UShielI4NnOb2k0Ah0tJi2o9/QekciBu4+9AtWCojJ8HLT2rJ5+iyw eS2iUSs5T1lS16cwSh/+3ut6v5sdv6sgxjhu2+zdmOlbZA6d4xWjElpOUis2+8SfJ+/2RU WTt8iwksgNfYeAl/+OoOK9GnVaBC8UM= Received: from mail-ed1-f72.google.com (mail-ed1-f72.google.com [209.85.208.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-201-9MBLgMggMfCkR3VC74JKfQ-1; Tue, 21 Apr 2026 09:19:56 -0400 X-MC-Unique: 9MBLgMggMfCkR3VC74JKfQ-1 X-Mimecast-MFC-AGG-ID: 9MBLgMggMfCkR3VC74JKfQ_1776777595 Received: by mail-ed1-f72.google.com with SMTP id 4fb4d7f45d1cf-66b0d92bffaso3374733a12.1 for ; Tue, 21 Apr 2026 06:19:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1776777595; x=1777382395; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=xexzte0yTqtWkjudFGvCNSwnTYifigEpG3qwMv3/9fw=; b=XbIMbwQ276/Bz4ZcH8AMyBgNzFunskbDbZiLFQ7TU1GZYsw1Fyp1G2vkh4N9cduzRs Eoo8c+LYdnPrmndL3+VJSeY9ONeI8YtkfCwhDNSY6xcpzfqVH37O8itUK1OJzZFM5JUW NjHfeJiRsc0wFZ556mwGlrvCTrQbE6YWLGymOiSEYw+O2BlXgt9gJY9KkhSR2XJrF9LS 2Ket7CBWbeMl6g+4v4u6ZSpeBhHmm4L++vzRDX1Rqcv8wv/xzoojfvtsLG7fksc7ws/W XQuUaNSNZztyw55eH7Jmr4+MTgp37ldRtatIOXCzxfFpHxozHctLeHZkG8T+QwboEDDi 7utQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776777595; x=1777382395; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=xexzte0yTqtWkjudFGvCNSwnTYifigEpG3qwMv3/9fw=; b=gk5z917U7lcYN5IX0M/b7BM828pXvrFDJ8Qv4XhfVe3KVrrhY+zZrkdwpNk3nqI6SI fOdV89tI1wgxq7qRoSdPqTlIyEmBokxQ8JZarHY61RKdFKYsjmdFx5iZHIuqL6zqLk6X VLJzwfnO1aAaV0snBeOUGVL/juCabs8Cjqd1i9ru3rMP/PU0ZNEnrQXqVEjnFKBM4vcE v9/9y2WtQg0KvES7rHpxtWGQrLNUuE5ZAhHXYx39d6e2r97L/WGJNnyoRY+Y9SQfJFRW 5M3lYzcu/VNyhyACytCU65Ex3Je4u1nuqtPXOfbLXhDyJ3zoeCL8ypP1i8kfVTPumTBD rJNw== X-Gm-Message-State: AOJu0YwNWzi7kwJtJEtF7HuDYWjw32FdUFJgkBlHPUYAXhgxbS9nVf4Q 1VB6ccIFxdfpkI5RNxfkgEDp9a2u89jDUK44+neLuBvDkZzNVgxW3uGwV5B3vA74AzyZn2WZAru QyX6JJflSn0LDooZmbzJr5NIp/JQ+8jzr5drwtmQICAbjzGO18B6vpwSID6oJAbEvxgIv9bHRwg == X-Gm-Gg: AeBDieujE51gm1NujfOnBFDvIGxswWlxOkKnXiTRGPcnYcpR6mQkrr3ZeMZrWufIIO5 lgJoH0oQtBlrt5NG4vEQ/VAlERiH+e2eotcRFB7sqYG7TxPBxMCOW+OdP5s81IIqVlTBEqaSY3u o9pDLBjpFU5OMv+NZPB/DdFcK92QiXJ+4xDgsI3Raa+lPrT0f/eP5bysi/armsoobAjdu30bMPv z0Dhp73C6QMPrBifmB33Sz2JzsGiSr8T7Y4NuSKLpgjc2llLo6LlOy9nWkf+V5HcdwSaNCExIsZ 3nSYmzbEBHFgQIojFuZ8DsFWtFHQR8Tc9DtvAKqLAFYuIY0O3lCUKXAJ8TtnRfTpknCSuji5I6i R3zTDIrwxBezgRvad2CLRe7/lAvEEN05Kk/Y48LXpN4MUUj+xebM8zA== X-Received: by 2002:a05:6402:2b9b:b0:670:8b30:a8a7 with SMTP id 4fb4d7f45d1cf-672bfc12ae6mr8152682a12.0.1776777594813; Tue, 21 Apr 2026 06:19:54 -0700 (PDT) X-Received: by 2002:a05:6402:2b9b:b0:670:8b30:a8a7 with SMTP id 4fb4d7f45d1cf-672bfc12ae6mr8152659a12.0.1776777594246; Tue, 21 Apr 2026 06:19:54 -0700 (PDT) Received: from redhat.com (IGLD-80-230-25-21.inter.net.il. [80.230.25.21]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6744dcdf29fsm2080687a12.30.2026.04.21.06.19.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Apr 2026 06:19:53 -0700 (PDT) Date: Tue, 21 Apr 2026 09:19:49 -0400 From: "Michael S. Tsirkin" To: "David Hildenbrand (Arm)" Cc: linux-kernel@vger.kernel.org, Andrew Morton , Vlastimil Babka , Brendan Jackman , Michal Hocko , Suren Baghdasaryan , Jason Wang , Andrea Arcangeli , linux-mm@kvack.org, virtualization@lists.linux.dev, Lorenzo Stoakes , "Liam R. Howlett" , Mike Rapoport Subject: Re: [PATCH RFC v2 11/18] mm: skip zeroing in vma_alloc_zeroed_movable_folio for pre-zeroed pages Message-ID: <20260421091759-mutt-send-email-mst@kernel.org> References: <39c3c8e647fc4c8e3dcb5c9d12f2e29b5404e8f2.1776689093.git.mst@redhat.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Tue, Apr 21, 2026 at 12:58:11PM +0200, David Hildenbrand (Arm) wrote: > On 4/20/26 14:50, Michael S. Tsirkin wrote: > > Use vma_alloc_folio_hints() and check PGHINT_ZEROED to skip > > clear_user_highpage() when the page is already zeroed. > > > > On x86, vma_alloc_zeroed_movable_folio is overridden by a macro > > that uses __GFP_ZERO directly, so this change has no effect there. > > > > Signed-off-by: Michael S. Tsirkin > > Assisted-by: Claude:claude-opus-4-6 > > Assisted-by: cursor-agent:GPT-5.4-xhigh > > --- > > include/linux/highmem.h | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/include/linux/highmem.h b/include/linux/highmem.h > > index af03db851a1d..8bb67772c1cb 100644 > > --- a/include/linux/highmem.h > > +++ b/include/linux/highmem.h > > @@ -321,9 +321,11 @@ struct folio *vma_alloc_zeroed_movable_folio(struct vm_area_struct *vma, > > unsigned long vaddr) > > { > > struct folio *folio; > > + pghint_t hints; > > > > - folio = vma_alloc_folio(GFP_HIGHUSER_MOVABLE, 0, vma, vaddr); > > - if (folio && user_alloc_needs_zeroing()) > > + folio = vma_alloc_folio_hints(GFP_HIGHUSER_MOVABLE, 0, vma, vaddr, > > + &hints); > > + if (folio && user_alloc_needs_zeroing() && !(hints & PGHINT_ZEROED)) > > clear_user_highpage(&folio->page, vaddr); > > > > return folio; > > > For others reading along, the variant on your other branch: > > diff --git a/include/linux/highmem.h b/include/linux/highmem.h > index af03db851a1d9..ffa683f64f1d1 100644 > --- a/include/linux/highmem.h > +++ b/include/linux/highmem.h > @@ -320,13 +320,8 @@ static inline > struct folio *vma_alloc_zeroed_movable_folio(struct vm_area_struct *vma, > unsigned long vaddr) > { > - struct folio *folio; > - > - folio = vma_alloc_folio(GFP_HIGHUSER_MOVABLE, 0, vma, vaddr); > - if (folio && user_alloc_needs_zeroing()) > - clear_user_highpage(&folio->page, vaddr); > - > - return folio; > + return vma_alloc_folio(GFP_HIGHUSER_MOVABLE | __GFP_ZERO, > + 0, vma, vaddr); > } > #endif > > Looks like an extremely clean interface. OK, sure. I will post that one as v3. Let me just add a changelog in the cover letter, and see what to do about balloon. > -- > Cheers, > > David