2024-03-05 00:37:59

by Sean Christopherson

[permalink] [raw]
Subject: [PATCH] KVM: Drop unused @may_block param from gfn_to_pfn_cache_invalidate_start()

Remove gfn_to_pfn_cache_invalidate_start()'s unused @may_block parameter,
which was leftover from KVM's abandoned (for now) attempt to support guest
usage of gfn_to_pfn caches.

Fixes: a4bff3df5147 ("KVM: pfncache: remove KVM_GUEST_USES_PFN usage")
Reported-by: Like Xu <[email protected]>
Cc: Paul Durrant <[email protected]>
Cc: David Woodhouse <[email protected]>
Signed-off-by: Sean Christopherson <[email protected]>
---
virt/kvm/kvm_main.c | 3 +--
virt/kvm/kvm_mm.h | 6 ++----
virt/kvm/pfncache.c | 2 +-
3 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
index ff588677beb7..7106b57b8468 100644
--- a/virt/kvm/kvm_main.c
+++ b/virt/kvm/kvm_main.c
@@ -832,8 +832,7 @@ static int kvm_mmu_notifier_invalidate_range_start(struct mmu_notifier *mn,
* mn_active_invalidate_count (see above) instead of
* mmu_invalidate_in_progress.
*/
- gfn_to_pfn_cache_invalidate_start(kvm, range->start, range->end,
- hva_range.may_block);
+ gfn_to_pfn_cache_invalidate_start(kvm, range->start, range->end);

/*
* If one or more memslots were found and thus zapped, notify arch code
diff --git a/virt/kvm/kvm_mm.h b/virt/kvm/kvm_mm.h
index ecefc7ec51af..715f19669d01 100644
--- a/virt/kvm/kvm_mm.h
+++ b/virt/kvm/kvm_mm.h
@@ -26,13 +26,11 @@ kvm_pfn_t hva_to_pfn(unsigned long addr, bool atomic, bool interruptible,
#ifdef CONFIG_HAVE_KVM_PFNCACHE
void gfn_to_pfn_cache_invalidate_start(struct kvm *kvm,
unsigned long start,
- unsigned long end,
- bool may_block);
+ unsigned long end);
#else
static inline void gfn_to_pfn_cache_invalidate_start(struct kvm *kvm,
unsigned long start,
- unsigned long end,
- bool may_block)
+ unsigned long end)
{
}
#endif /* HAVE_KVM_PFNCACHE */
diff --git a/virt/kvm/pfncache.c b/virt/kvm/pfncache.c
index 9ac8c9da4eda..bff9875cdcd2 100644
--- a/virt/kvm/pfncache.c
+++ b/virt/kvm/pfncache.c
@@ -23,7 +23,7 @@
* MMU notifier 'invalidate_range_start' hook.
*/
void gfn_to_pfn_cache_invalidate_start(struct kvm *kvm, unsigned long start,
- unsigned long end, bool may_block)
+ unsigned long end)
{
struct gfn_to_pfn_cache *gpc;


base-commit: 003d914220c97ef93cabfe3ec4e245e2383e19e9
--
2.44.0.278.ge034bb2e1d-goog



2024-03-05 09:32:28

by Paul Durrant

[permalink] [raw]
Subject: Re: [PATCH] KVM: Drop unused @may_block param from gfn_to_pfn_cache_invalidate_start()

On 05/03/2024 00:37, Sean Christopherson wrote:
> Remove gfn_to_pfn_cache_invalidate_start()'s unused @may_block parameter,
> which was leftover from KVM's abandoned (for now) attempt to support guest
> usage of gfn_to_pfn caches.
>
> Fixes: a4bff3df5147 ("KVM: pfncache: remove KVM_GUEST_USES_PFN usage")
> Reported-by: Like Xu <[email protected]>
> Cc: Paul Durrant <[email protected]>
> Cc: David Woodhouse <[email protected]>
> Signed-off-by: Sean Christopherson <[email protected]>
> ---
> virt/kvm/kvm_main.c | 3 +--
> virt/kvm/kvm_mm.h | 6 ++----
> virt/kvm/pfncache.c | 2 +-
> 3 files changed, 4 insertions(+), 7 deletions(-)
>

Reviewed-by: Paul Durrant <[email protected]>


2024-03-06 13:00:23

by David Woodhouse

[permalink] [raw]
Subject: Re: [PATCH] KVM: Drop unused @may_block param from gfn_to_pfn_cache_invalidate_start()

On Mon, 2024-03-04 at 16:37 -0800, Sean Christopherson wrote:
> Remove gfn_to_pfn_cache_invalidate_start()'s unused @may_block parameter,
> which was leftover from KVM's abandoned (for now) attempt to support guest
> usage of gfn_to_pfn caches.
>
> Fixes: a4bff3df5147 ("KVM: pfncache: remove KVM_GUEST_USES_PFN usage")
> Reported-by: Like Xu <[email protected]>
> Cc: Paul Durrant <[email protected]>
> Cc: David Woodhouse <[email protected]>
> Signed-off-by: Sean Christopherson <[email protected]>

Reviewed-by: David Woodhouse <[email protected]>


Attachments:
smime.p7s (5.83 kB)

2024-04-10 00:20:47

by Sean Christopherson

[permalink] [raw]
Subject: Re: [PATCH] KVM: Drop unused @may_block param from gfn_to_pfn_cache_invalidate_start()

On Mon, 04 Mar 2024 16:37:42 -0800, Sean Christopherson wrote:
> Remove gfn_to_pfn_cache_invalidate_start()'s unused @may_block parameter,
> which was leftover from KVM's abandoned (for now) attempt to support guest
> usage of gfn_to_pfn caches.
>
>

Applied to kvm-x86 fixes, thanks!

[1/1] KVM: Drop unused @may_block param from gfn_to_pfn_cache_invalidate_start()
https://github.com/kvm-x86/linux/commit/b326ec064c9a

--
https://github.com/kvm-x86/linux/tree/next