diff options
author | David Woodhouse <dwmw@amazon.co.uk> | 2021-02-09 00:23:26 +0100 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2021-02-09 13:42:03 +0100 |
commit | 99df541dcc97ce4436528a4eeb23165d8c672a64 (patch) | |
tree | 507827f4d305f45c903aa1726e76185b12c32aeb | |
parent | KVM: Use kvm_pfn_t for local PFN variable in hva_to_pfn_remapped() (diff) | |
download | linux-99df541dcc97ce4436528a4eeb23165d8c672a64.tar.xz linux-99df541dcc97ce4436528a4eeb23165d8c672a64.zip |
KVM: x86/xen: Remove extra unlock in kvm_xen_hvm_set_attr()
This accidentally ended up locking and then immediately unlocking kvm->lock
at the beginning of the function. Fix it.
Fixes: a76b9641ad1c ("KVM: x86/xen: add KVM_XEN_HVM_SET_ATTR/KVM_XEN_HVM_GET_ATTR")
Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
Message-Id: <20210208232326.1830370-2-dwmw2@infradead.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-rw-r--r-- | arch/x86/kvm/xen.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/x86/kvm/xen.c b/arch/x86/kvm/xen.c index 39a7ffcdcf22..54a4895d4440 100644 --- a/arch/x86/kvm/xen.c +++ b/arch/x86/kvm/xen.c @@ -105,8 +105,6 @@ int kvm_xen_hvm_set_attr(struct kvm *kvm, struct kvm_xen_hvm_attr *data) mutex_lock(&kvm->lock); - mutex_unlock(&kvm->lock); - switch (data->type) { case KVM_XEN_ATTR_TYPE_LONG_MODE: if (!IS_ENABLED(CONFIG_64BIT) && data->u.long_mode) { |