[PATCH v13 17/85] KVM: x86/mmu: Drop kvm_page_fault.hva, i.e. don't track... | linux-arm-kernel

[PATCH v13 00/85] KVM: Stop grabbing references to PFNMAP'd pages · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 01/85] KVM: Drop KVM_ERR_PTR_BAD_PAGE and instead return NULL to indicate an error · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 02/85] KVM: Allow calling kvm_release_page_{clean,dirty}() on a NULL page pointer · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 03/85] KVM: Add kvm_release_page_unused() API to put pages that KVM never consumes · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 04/85] KVM: x86/mmu: Skip the "try unsync" path iff the old SPTE was a leaf SPTE · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 05/85] KVM: x86/mmu: Don't overwrite shadow-present MMU SPTEs when prefaulting · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 06/85] KVM: x86/mmu: Invert @can_unsync and renamed to @synchronizing · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 07/85] KVM: x86/mmu: Mark new SPTE as Accessed when synchronizing existing SPTE · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 08/85] KVM: x86/mmu: Mark folio dirty when creating SPTE, not when zapping/modifying · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 09/85] KVM: x86/mmu: Mark page/folio accessed only when zapping leaf SPTEs · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 10/85] KVM: x86/mmu: Use gfn_to_page_many_atomic() when prefetching indirect PTEs · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 11/85] KVM: Rename gfn_to_page_many_atomic() to kvm_prefetch_pages() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 12/85] KVM: Drop @atomic param from gfn=>pfn and hva=>pfn APIs · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 13/85] KVM: Annotate that all paths in hva_to_pfn() might sleep · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 14/85] KVM: Return ERR_SIGPENDING from hva_to_pfn() if GUP returns -EGAIN · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 15/85] KVM: Drop extra GUP (via check_user_page_hwpoison()) to detect poisoned page · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 16/85] KVM: Replace "async" pointer in gfn=>pfn with "no_wait" and error code · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 17/85] KVM: x86/mmu: Drop kvm_page_fault.hva, i.e. don't track intermediate hva · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 18/85] KVM: Drop unused "hva" pointer from __gfn_to_pfn_memslot() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 19/85] KVM: Introduce kvm_follow_pfn() to eventually replace "gfn_to_pfn" APIs · Sean Christopherson <seanjc@google.com> · 2024-10-10
Re: [PATCH v13 19/85] KVM: Introduce kvm_follow_pfn() to eventually replace "gfn_to_pfn" APIs · Yan Zhao <hidden> · 2024-10-21
Re: [PATCH v13 19/85] KVM: Introduce kvm_follow_pfn() to eventually replace "gfn_to_pfn" APIs · Sean Christopherson <seanjc@google.com> · 2024-10-21
Re: [PATCH v13 19/85] KVM: Introduce kvm_follow_pfn() to eventually replace "gfn_to_pfn" APIs · Yan Zhao <hidden> · 2024-10-22
[PATCH v13 20/85] KVM: Remove pointless sanity check on @map param to kvm_vcpu_(un)map() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 21/85] KVM: Explicitly initialize all fields at the start of kvm_vcpu_map() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 22/85] KVM: Use NULL for struct page pointer to indicate mremapped memory · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 23/85] KVM: nVMX: Rely on kvm_vcpu_unmap() to track validity of eVMCS mapping · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 24/85] KVM: nVMX: Drop pointless msr_bitmap_map field from struct nested_vmx · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 25/85] KVM: nVMX: Add helper to put (unmap) vmcs12 pages · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 26/85] KVM: Use plain "struct page" pointer instead of single-entry array · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 27/85] KVM: Provide refcounted page as output field in struct kvm_follow_pfn · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 28/85] KVM: Move kvm_{set,release}_page_{clean,dirty}() helpers up in kvm_main.c · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 29/85] KVM: pfncache: Precisely track refcounted pages · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 30/85] KVM: Migrate kvm_vcpu_map() to kvm_follow_pfn() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 31/85] KVM: Pin (as in FOLL_PIN) pages during kvm_vcpu_map() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 32/85] KVM: nVMX: Mark vmcs12's APIC access page dirty when unmapping · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 33/85] KVM: Pass in write/dirty to kvm_vcpu_map(), not kvm_vcpu_unmap() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 34/85] KVM: Get writable mapping for __kvm_vcpu_map() only when necessary · Sean Christopherson <seanjc@google.com> · 2024-10-10
Re: [PATCH v13 34/85] KVM: Get writable mapping for __kvm_vcpu_map() only when necessary · Yan Zhao <hidden> · 2024-10-21
Re: [PATCH v13 34/85] KVM: Get writable mapping for __kvm_vcpu_map() only when necessary · Sean Christopherson <seanjc@google.com> · 2024-10-21
Re: [PATCH v13 34/85] KVM: Get writable mapping for __kvm_vcpu_map() only when necessary · Yan Zhao <hidden> · 2024-10-22
[PATCH v13 35/85] KVM: Disallow direct access (w/o mmu_notifier) to unpinned pfn by default · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 36/85] KVM: x86: Don't fault-in APIC access page during initial allocation · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 37/85] KVM: x86/mmu: Add "mmu" prefix fault-in helpers to free up generic names · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 38/85] KVM: x86/mmu: Put direct prefetched pages via kvm_release_page_clean() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 39/85] KVM: x86/mmu: Add common helper to handle prefetching SPTEs · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 40/85] KVM: x86/mmu: Add helper to "finish" handling a guest page fault · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 41/85] KVM: x86/mmu: Mark pages/folios dirty at the origin of make_spte() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 42/85] KVM: Move declarations of memslot accessors up in kvm_host.h · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 43/85] KVM: Add kvm_faultin_pfn() to specifically service guest page faults · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 44/85] KVM: x86/mmu: Convert page fault paths to kvm_faultin_pfn() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 45/85] KVM: guest_memfd: Pass index, not gfn, to __kvm_gmem_get_pfn() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 46/85] KVM: guest_memfd: Provide "struct page" as output from kvm_gmem_get_pfn() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 47/85] KVM: x86/mmu: Put refcounted pages instead of blindly releasing pfns · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 48/85] KVM: x86/mmu: Don't mark unused faultin pages as accessed · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 49/85] KVM: Move x86's API to release a faultin page to common KVM · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 50/85] KVM: VMX: Hold mmu_lock until page is released when updating APIC access page · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 51/85] KVM: VMX: Use __kvm_faultin_page() to get APIC access page/pfn · Sean Christopherson <seanjc@google.com> · 2024-10-10
Re: [PATCH v13 51/85] KVM: VMX: Use __kvm_faultin_page() to get APIC access page/pfn · Yan Zhao <hidden> · 2024-10-21
Re: [PATCH v13 51/85] KVM: VMX: Use __kvm_faultin_page() to get APIC access page/pfn · Sean Christopherson <seanjc@google.com> · 2024-10-21
Re: [PATCH v13 51/85] KVM: VMX: Use __kvm_faultin_page() to get APIC access page/pfn · Yan Zhao <hidden> · 2024-10-22
[PATCH v13 52/85] KVM: PPC: e500: Mark "struct page" dirty in kvmppc_e500_shadow_map() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 53/85] KVM: PPC: e500: Mark "struct page" pfn accessed before dropping mmu_lock · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 54/85] KVM: PPC: e500: Use __kvm_faultin_pfn() to handle page faults · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 55/85] KVM: arm64: Mark "struct page" pfns accessed/dirty before dropping mmu_lock · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 56/85] KVM: arm64: Use __kvm_faultin_pfn() to handle memory aborts · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 57/85] KVM: RISC-V: Mark "struct page" pfns dirty iff a stage-2 PTE is installed · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 58/85] KVM: RISC-V: Mark "struct page" pfns accessed before dropping mmu_lock · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 59/85] KVM: RISC-V: Use kvm_faultin_pfn() when mapping pfns into the guest · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 60/85] KVM: PPC: Use __kvm_faultin_pfn() to handle page faults on Book3s HV · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 61/85] KVM: PPC: Use __kvm_faultin_pfn() to handle page faults on Book3s Radix · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 62/85] KVM: PPC: Drop unused @kvm_ro param from kvmppc_book3s_instantiate_page() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 63/85] KVM: PPC: Book3S: Mark "struct page" pfns dirty/accessed after installing PTE · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 64/85] KVM: PPC: Use kvm_faultin_pfn() to handle page faults on Book3s PR · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 65/85] KVM: LoongArch: Mark "struct page" pfns dirty only in "slow" page fault path · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 66/85] KVM: LoongArch: Mark "struct page" pfns accessed only in "slow" page fault path · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 67/85] KVM: LoongArch: Mark "struct page" pfn accessed before dropping mmu_lock · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 68/85] KVM: LoongArch: Use kvm_faultin_pfn() to map pfns into the guest · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 69/85] KVM: MIPS: Mark "struct page" pfns dirty only in "slow" page fault path · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 70/85] KVM: MIPS: Mark "struct page" pfns accessed only in "slow" page fault path · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 71/85] KVM: MIPS: Mark "struct page" pfns accessed prior to dropping mmu_lock · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 72/85] KVM: MIPS: Use kvm_faultin_pfn() to map pfns into the guest · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 73/85] KVM: PPC: Remove extra get_page() to fix page refcount leak · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 74/85] KVM: PPC: Use kvm_vcpu_map() to map guest memory to patch dcbz instructions · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 75/85] KVM: Convert gfn_to_page() to use kvm_follow_pfn() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 76/85] KVM: Add support for read-only usage of gfn_to_page() · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 77/85] KVM: arm64: Use __gfn_to_page() when copying MTE tags to/from userspace · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 78/85] KVM: PPC: Explicitly require struct page memory for Ultravisor sharing · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 79/85] KVM: Drop gfn_to_pfn() APIs now that all users are gone · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 80/85] KVM: s390: Use kvm_release_page_dirty() to unpin "struct page" memory · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 81/85] KVM: Make kvm_follow_pfn.refcounted_page a required field · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 82/85] KVM: x86/mmu: Don't mark "struct page" accessed when zapping SPTEs · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 83/85] KVM: arm64: Don't mark "struct page" accessed when making SPTE young · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 84/85] KVM: Drop APIs that manipulate "struct page" via pfns · Sean Christopherson <seanjc@google.com> · 2024-10-10
[PATCH v13 85/85] KVM: Don't grab reference on VM_MIXEDMAP pfns that have a "struct page" · Sean Christopherson <seanjc@google.com> · 2024-10-10
Re: [PATCH v13 00/85] KVM: Stop grabbing references to PFNMAP'd pages · Paolo Bonzini <pbonzini@redhat.com> · 2024-10-17
Re: [PATCH v13 00/85] KVM: Stop grabbing references to PFNMAP'd pages · Sean Christopherson <seanjc@google.com> · 2024-10-22
Re: [PATCH v13 00/85] KVM: Stop grabbing references to PFNMAP'd pages · Paolo Bonzini <pbonzini@redhat.com> · 2024-10-25
Re: [PATCH v13 00/85] KVM: Stop grabbing references to PFNMAP'd pages · Dmitry Osipenko <dmitry.osipenko@collabora.com> · 2024-10-24

STALE582d REVIEWED: 1 (0M)

[PATCH v13 17/85] KVM: x86/mmu: Drop kvm_page_fault.hva, i.e. don't track intermediate hva

From: Sean Christopherson <seanjc@google.com>
Date: 2024-10-10 18:25:28
Also in: kvm, kvm-riscv, kvmarm, linux-mips, linux-riscv, linuxppc-dev, lkml, loongarch
Subsystem: kernel virtual machine for x86 (kvm/x86), the rest, x86 architecture (32-bit and 64-bit) · Maintainers: Sean Christopherson, Paolo Bonzini, Linus Torvalds, Thomas Gleixner, Ingo Molnar, Borislav Petkov, Dave Hansen

Remove kvm_page_fault.hva as it is never read, only written.  This will
allow removing the @hva param from __gfn_to_pfn_memslot().

Tested-by: Alex Bennée <redacted>
Signed-off-by: Sean Christopherson <seanjc@google.com>
---
 arch/x86/kvm/mmu/mmu.c          | 5 ++---
 arch/x86/kvm/mmu/mmu_internal.h | 2 --
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c
index fa8f3fb7c14b..c67228b46bd5 100644
--- a/arch/x86/kvm/mmu/mmu.c
+++ b/arch/x86/kvm/mmu/mmu.c

@@ -3294,7 +3294,6 @@ static int kvm_handle_noslot_fault(struct kvm_vcpu *vcpu,
 	fault->slot = NULL;
 	fault->pfn = KVM_PFN_NOSLOT;
 	fault->map_writable = false;
-	fault->hva = KVM_HVA_ERR_BAD;
 
 	/*
 	 * If MMIO caching is disabled, emulate immediately without

@@ -4379,7 +4378,7 @@ static int __kvm_faultin_pfn(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault
 
 	fault->pfn = __gfn_to_pfn_memslot(fault->slot, fault->gfn, false, true,
 					  fault->write, &fault->map_writable,
-					  &fault->hva);
+					  NULL);
 
 	/*
 	 * If resolving the page failed because I/O is needed to fault-in the

@@ -4408,7 +4407,7 @@ static int __kvm_faultin_pfn(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault
 	 */
 	fault->pfn = __gfn_to_pfn_memslot(fault->slot, fault->gfn, true, true,
 					  fault->write, &fault->map_writable,
-					  &fault->hva);
+					  NULL);
 	return RET_PF_CONTINUE;
 }

diff --git a/arch/x86/kvm/mmu/mmu_internal.h b/arch/x86/kvm/mmu/mmu_internal.h
index 4da83544c4e1..633aedec3c2e 100644
--- a/arch/x86/kvm/mmu/mmu_internal.h
+++ b/arch/x86/kvm/mmu/mmu_internal.h

@@ -238,7 +238,6 @@ struct kvm_page_fault {
 	/* Outputs of kvm_faultin_pfn.  */
 	unsigned long mmu_seq;
 	kvm_pfn_t pfn;
-	hva_t hva;
 	bool map_writable;
 
 	/*

@@ -313,7 +312,6 @@ static inline int kvm_mmu_do_page_fault(struct kvm_vcpu *vcpu, gpa_t cr2_or_gpa,
 		.is_private = err & PFERR_PRIVATE_ACCESS,
 
 		.pfn = KVM_PFN_ERR_FAULT,
-		.hva = KVM_HVA_ERR_BAD,
 	};
 	int r;

-- 
2.47.0.rc1.288.g06298d1525-goog

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help