KVM: async_pf: change async_pf_execute() to use get_user_pages(tsk => NULL)
authorOleg Nesterov <oleg@redhat.com>
Mon, 28 Apr 2014 15:03:00 +0000 (17:03 +0200)
committerChristoffer Dall <christoffer.dall@linaro.org>
Thu, 2 Oct 2014 15:18:41 +0000 (17:18 +0200)
async_pf_execute() passes tsk == current to gup(), this is doesn't
hurt but unnecessary and misleading. "tsk" is only used to account
the number of faults and current is the random workqueue thread.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Suggested-by: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
(cherry picked from commit e9545b9f8aeb63e05818e4b3250057260bc072aa)
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
virt/kvm/async_pf.c

index cda703e512d3dd46a4be8db299293929428cb6b7..d6a3d0993d8812c8527274d01e8c08ce942746a4 100644 (file)
@@ -81,7 +81,7 @@ static void async_pf_execute(struct work_struct *work)
        might_sleep();
 
        down_read(&mm->mmap_sem);
-       get_user_pages(current, mm, addr, 1, 1, 0, NULL, NULL);
+       get_user_pages(NULL, mm, addr, 1, 1, 0, NULL, NULL);
        up_read(&mm->mmap_sem);
        kvm_async_page_present_sync(vcpu, apf);