KVM: x86: Simplify msr_filter update

Replace srcu_dereference()+rcu_assign_pointer() sequence with
a single rcu_replace_pointer().

Suggested-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Michal Luczaj <mhal@rbox.co>
Link: https://lore.kernel.org/r/20230107001256.2365304-4-mhal@rbox.co
Signed-off-by: Sean Christopherson <seanjc@google.com>

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index 4f8e78d495852dcfe4dd947ab0e61001171a9972..9c66cb3657b45c57d4c2ea31d3ac2a2ca29029f4 100644 (file)
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -6492,11 +6492,8 @@ static int kvm_vm_ioctl_set_msr_filter(struct kvm *kvm,
        }
 
        mutex_lock(&kvm->lock);
-
        /* The per-VM filter is protected by kvm->lock... */
-       old_filter = srcu_dereference_check(kvm->arch.msr_filter, &kvm->srcu, 1);
-
-       rcu_assign_pointer(kvm->arch.msr_filter, new_filter);
+       old_filter = rcu_replace_pointer(kvm->arch.msr_filter, new_filter, 1);
        mutex_unlock(&kvm->lock);
        synchronize_srcu(&kvm->srcu);