kvm: Faults which trigger IO release the mmap_sem
authorAndres Lagar-Cavilla <andreslc@google.com>
Wed, 17 Sep 2014 17:51:48 +0000 (10:51 -0700)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 24 Sep 2014 12:07:54 +0000 (14:07 +0200)
commit234b239bea395316d7f78018c672f4a88b3cdf0d
tree3defdf8536d0b73b1130d23fc6c79343a9831d59
parentb4619660635732bd2da376bb8f31f94d0f15fc98
kvm: Faults which trigger IO release the mmap_sem

When KVM handles a tdp fault it uses FOLL_NOWAIT. If the guest memory
has been swapped out or is behind a filemap, this will trigger async
readahead and return immediately. The rationale is that KVM will kick
back the guest with an "async page fault" and allow for some other
guest process to take over.

If async PFs are enabled the fault is retried asap from an async
workqueue. If not, it's retried immediately in the same code path. In
either case the retry will not relinquish the mmap semaphore and will
block on the IO. This is a bad thing, as other mmap semaphore users
now stall as a function of swap or filemap latency.

This patch ensures both the regular and async PF path re-enter the
fault allowing for the mmap semaphore to be relinquished in the case
of IO wait.

Reviewed-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Andres Lagar-Cavilla <andreslc@google.com>
Acked-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
include/linux/kvm_host.h
include/linux/mm.h
mm/gup.c
virt/kvm/async_pf.c
virt/kvm/kvm_main.c