From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 82B61606A326 for ; Fri, 6 Nov 2026 10:46:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=dW6eV4BW; dkim=pass (1024-bit key, unprotected) header.d=arm.com header.i=@arm.com header.a=rsa-sha256 header.s=foss header.b=DSxvniHD DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Unsubscribe-Post:List-Owner; bh=EBOzhR38zAbfIVOw2so9GKfjXciI6NC0gGb/q053Uvw=; b=dW6eV4BW1PBuHA86tlxEmVJRc0 bSQHfjpynyALCMJQEg586Ewaqd9bEXOc31ZBAq0gMPPfpvSSXgOSFEUu6HCTypmudWjcG6HcnvxVv RFyG2SzfGudckSqcdWa+r+hWhJSWR/RRe6jLXlgCRYDt0seAGNtkp36m53w8hvYbm1jptXa7mwn1j pR3fWevtGv4vDvhHmgx2yHk73UHkDSjwtKGUlkofJRkiXHbRpqRcTNxgqcb8xV73BAtdgEbMKD6X0 iHpqyuD+a4zHxWLMon1w16BfiQaIh8tuMkMgxo7e6K/j9l/f9vRLPU3ssQ9+PPfhqL/03oM8rkOfs NAFmkcqQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPHSR-0000000BzcY-17iS; Fri, 06 Nov 2026 10:46:07 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xPHSO-0000000Bzap-25DU for linux-arm-kernel@lists.infradead.org; Fri, 06 Nov 2026 10:46:05 +0000 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 1A57415A1; Fri, 6 Nov 2026 02:45:58 -0800 (PST) Received: from raptor (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 74B503F673; Fri, 6 Nov 2026 02:45:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1793961961; bh=Kz0mSCmJER1t1vnDARZyfMev49ZrLe9m/+tlQWtAGgU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=DSxvniHDbqcztfN5wXQ+N71HLuRwr1uD8C0kMxaoe5FTSBtVaGGF4RjPVSfcQL6p3 jCXxICd99aAnXYQI66kJlgyBYM5pciaOPd9d5JvxC6SNe5VPJ6jxZ2U60EU2qW5eI5 COBjnYgGy5DFy/yemQkddtpjAdQ1ebJFXF5tu96I= Date: Fri, 6 Nov 2026 10:45:56 +0000 From: Alexandru Elisei To: Sean Christopherson Cc: "Lorenzo Stoakes (ARM)" , pbonzini@redhat.com, kvm@vger.kernel.org, linux-kernel@vger.kernel.org, maz@kernel.org, oupton@kernel.org, suzuki.poulose@arm.com, kvmarm@lists.linux.dev, linux-arm-kernel@lists.infradead.org, david@kernel.org, mark.rutland@arm.com, ackerleytng@google.com Subject: Re: [PATCH v2] KVM: Ignore MMU notifiers for guest_memfd-only memslots Message-ID: References: <20260714160411.302386-1-alexandru.elisei@arm.com> X-Archived-By: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20261106_024604_622988_18CC2363 X-CRM114-Status: GOOD ( 40.59 ) X-Spam-Score: -4.4 (----) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" X-Spam-Status: No, score=-15.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 Hi Sean, On Thu, Nov 05, 2026 at 11:02:28AM -0800, Sean Christopherson wrote: > On Wed, Nov 04, 2026, Lorenzo Stoakes (ARM) wrote: > > [updated David's email address to @kernel.org] > > > > On Wed, Aug 12, 2026 at 10:42:50AM +0100, Alexandru Elisei wrote: > > [ ... ] > > > On Fri, Jul 24, 2026 at 12:52:20PM -0700, Sean Christopherson wrote: > > > > It'll probably take me a few weeks to circle back to this, as I finally crawled > > > > out from underneath a pile of urgent and am just now getting back into review > > > > mode, but this is fairly high on my todo list, so with luck we might still squeeze > > > > this into 7.3. > > > > Hey Sean, > > > > I wondered whether you'd had a chance to look at this? > > > > After discussing this with Marc and Alex it is clear that this is really a > > blocker for Alex's SPE series, as we need a sensible means of ensuring > > that stage-2 mappings stay in place. > > > > So ideally would prefer to get something like this in in the next cycle, > > if you don't have any objections to it? :) > > > > If it'd be easier, I think a smaller version of the same patch would also > > give us what we need? I.e. your snippet from June with the page ageing part > > dropped, as below. > > The GPC invalidation fix landed, so we can go straight to the super simple v1. > Compile tested only at this point, I'll formally post this (hopefully today), > assuming nothing explodes in testing. > > -- > From: Alexandru Elisei > Date: Mon, 15 Jun 2026 16:52:44 +0100 > Subject: [PATCH] KVM: Skip gfn-based MMU notifier invalidations for > guest_memfd-only memslots > > Don't invalidate gmem-only memslots when responding to MMU notifier > invalidations, as the entire point of gmem-only memslots is so that KVM > doesn't need a VMA to install mappings into KVM's MMU. Specificially, > page faults are resolved by going straight to guest_memfd, using the gfn > and the associated memslot's gmem.file. I.e. whether or not userspace > happens to have valid mappings to the same gmem instance is irrelevant, > as are any invalidations for those mappings. > > Note! The exception is KVM's gfn_to_pfn_cache implementation which isn't > yet wired up to pull pfns directly from guest_memfd, i.e. still relies on > userspace mappings/VMAs and thus need to be invalidated, e.g. to ensure KVM > doesn't retain a kernel mapping+pointer to PRIVATE memory. However, as of > commit da260bcb56fe ("KVM: pfncache: use a dedicated invalidation sequence > for cache refresh"), GPC invalidation no longer relies on memslot lookups, > because GPCs also support HVA-based mappings. I.e. KVM *must* be able to > detect and handle relevant MMU notifier invalidations even if there is *no* > memslot whatsoever. > > In addition to the obvious performance benefits, avoiding unnecessary and > spurious SPTE zaps and TLB flushes will allow KVM to support features like > ARM's Statistical Profiling Extension (SPE)[1] and x86's Secure AVIC[2], > both of which rely on the pages *never* being unmapped from the stage-2 > page tables. Looks great, and thank you for updating the commit message! Alex > > Link: https://lore.kernel.org/all/20260903160623.315525-1-alexandru.elisei@arm.com [1] > Link: https://lore.kernel.org/all/cover.1783490022.git.naveen@kernel.org [2] > Cc: Lorenzo Stoakes (ARM) > Cc: Naveen N Rao (AMD) > Signed-off-by: Alexandru Elisei > [sean: rewrite changelog to capture the new gpc behavior] > Signed-off-by: Sean Christopherson > --- > virt/kvm/kvm_main.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index cf4cb586f140..599dea22afb8 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -601,6 +601,10 @@ static __always_inline kvm_mn_ret_t kvm_handle_hva_range(struct kvm *kvm, > unsigned long hva_start, hva_end; > > slot = container_of(node, struct kvm_memory_slot, hva_node[slots->node_idx]); > + > + if (kvm_slot_has_gmem(slot) && kvm_memslot_is_gmem_only(slot)) > + continue; > + > hva_start = max_t(unsigned long, range->start, slot->userspace_addr); > hva_end = min_t(unsigned long, range->end, > slot->userspace_addr + (slot->npages << PAGE_SHIFT)); > > base-commit: d21eef2084a9af87ac2a783bc7c2b175ba114e4e > --