diff options
author | Oleg Nesterov <oleg@redhat.com> | 2014-04-28 17:03:00 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2014-04-28 17:24:55 +0200 |
commit | e9545b9f8aeb63e05818e4b3250057260bc072aa (patch) | |
tree | f89d5a69691b0f61871604aa2b09ce140d236cc8 | |
parent | KVM: async_pf: kill the unnecessary use_mm/unuse_mm async_pf_execute() (diff) | |
download | linux-e9545b9f8aeb63e05818e4b3250057260bc072aa.tar.xz linux-e9545b9f8aeb63e05818e4b3250057260bc072aa.zip |
KVM: async_pf: change async_pf_execute() to use get_user_pages(tsk => NULL)
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>
-rw-r--r-- | virt/kvm/async_pf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/virt/kvm/async_pf.c b/virt/kvm/async_pf.c index 0ced4f31bc65..62f4223f4c9e 100644 --- a/virt/kvm/async_pf.c +++ b/virt/kvm/async_pf.c @@ -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); |