KVM: SVM: remove needless mmap_sem acquision from nested_svm_map
authorMarcelo Tosatti <mtosatti@redhat.com>
Tue, 8 Sep 2009 17:47:38 +0000 (14:47 -0300)
committerAvi Kivity <avi@redhat.com>
Thu, 3 Dec 2009 07:32:10 +0000 (09:32 +0200)
nested_svm_map unnecessarily takes mmap_sem around gfn_to_page, since
gfn_to_page / get_user_pages are responsible for it.

Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Acked-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/svm.c

index 92048a626d4ebd563c54e146ebe28d3d911affc1..f54c4f9d28657ba80e53edb4d5a75e01508c1872 100644 (file)
@@ -1396,10 +1396,7 @@ static void *nested_svm_map(struct vcpu_svm *svm, u64 gpa, enum km_type idx)
 {
        struct page *page;
 
-       down_read(&current->mm->mmap_sem);
        page = gfn_to_page(svm->vcpu.kvm, gpa >> PAGE_SHIFT);
-       up_read(&current->mm->mmap_sem);
-
        if (is_error_page(page))
                goto error;