lkml.org 
[lkml]   [2013]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/3] KVM: Make kvm_lock non-raw
On Mon, Sep 16, 2013 at 04:06:10PM +0200, Paolo Bonzini wrote:
> Paul Gortmaker reported a BUG on preempt-rt kernels, due to taking the
> mmu_lock within the raw kvm_lock in mmu_shrink_scan. He provided a
> patch that shrunk the kvm_lock critical section so that the mmu_lock
> critical section does not nest with it, but in the end there is no reason
> for the vm_list to be protected by a raw spinlock. Only manipulations
> of kvm_usage_count and the consequent hardware_enable/disable operations
> are not preemptable.
>
> This small series thus splits the kvm_lock in the "raw" part and the
> "non-raw" part.
>
> Paul, could you please provide your Tested-by?
>
Reviewed-by: Gleb Natapov <gleb@redhat.com>

But why should it go to stable?

> Thanks,
>
> Paolo
>
> Paolo Bonzini (3):
> KVM: cleanup (physical) CPU hotplug
> KVM: protect kvm_usage_count with its own spinlock
> KVM: Convert kvm_lock back to non-raw spinlock
>
> Documentation/virtual/kvm/locking.txt | 8 ++++--
> arch/x86/kvm/mmu.c | 4 +--
> arch/x86/kvm/x86.c | 8 +++---
> include/linux/kvm_host.h | 2 +-
> virt/kvm/kvm_main.c | 51 ++++++++++++++++++-----------------
> 5 files changed, 40 insertions(+), 33 deletions(-)
>
> --
> 1.8.3.1

--
Gleb.


\
 
 \ /
  Last update: 2013-09-22 10:01    [W:0.146 / U:0.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site