Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-23 Thread Jan Kiszka
On 2013-09-22 11:53, Gleb Natapov wrote: On Sun, Sep 22, 2013 at 10:53:14AM +0200, Paolo Bonzini wrote: Il 22/09/2013 09:42, Gleb Natapov ha scritto: 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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-23 Thread Paul Gortmaker
On 13-09-22 05:53 AM, Gleb Natapov wrote: On Sun, Sep 22, 2013 at 10:53:14AM +0200, Paolo Bonzini wrote: Il 22/09/2013 09:42, Gleb Natapov ha scritto: 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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-23 Thread Paolo Bonzini
Il 23/09/2013 15:36, Paul Gortmaker ha scritto: The change is not completely trivial, it splits lock. There is no obvious problem of course, otherwise you wouldn't send it and I would ack it :), but it does not mean that the chance for problem is zero, so why risk stability of stable even

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-23 Thread Gleb Natapov
On Mon, Sep 23, 2013 at 03:44:21PM +0200, Paolo Bonzini wrote: Il 23/09/2013 15:36, Paul Gortmaker ha scritto: The change is not completely trivial, it splits lock. There is no obvious problem of course, otherwise you wouldn't send it and I would ack it :), but it does not mean that the

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-23 Thread Paolo Bonzini
Il 23/09/2013 16:59, Gleb Natapov ha scritto: Perfect, I'll queue [v2 of] these patches for 3.12 then. Why 3.12 if it is not going to stable? Off-by-one. :) Paolo -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-22 Thread Gleb Natapov
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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-22 Thread Paolo Bonzini
Il 22/09/2013 09:42, Gleb Natapov ha scritto: 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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-22 Thread Gleb Natapov
On Sun, Sep 22, 2013 at 10:53:14AM +0200, Paolo Bonzini wrote: Il 22/09/2013 09:42, Gleb Natapov ha scritto: 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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-21 Thread Michael S. Tsirkin
On Fri, Sep 20, 2013 at 08:04:19PM +0200, Jan Kiszka wrote: On 2013-09-20 19:51, Paul Gortmaker wrote: [Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul Gortmaker wrote: On 13-09-16 10:06 AM, Paolo Bonzini wrote: Paul Gortmaker reported a BUG on preempt-rt

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-20 Thread Paul Gortmaker
[Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul Gortmaker wrote: On 13-09-16 10:06 AM, 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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-20 Thread Paul Gortmaker
On 13-09-20 02:04 PM, Jan Kiszka wrote: On 2013-09-20 19:51, Paul Gortmaker wrote: [Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul Gortmaker wrote: On 13-09-16 10:06 AM, Paolo Bonzini wrote: Paul Gortmaker reported a BUG on preempt-rt kernels, due to taking

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-20 Thread Jan Kiszka
On 2013-09-20 20:18, Paul Gortmaker wrote: On 13-09-20 02:04 PM, Jan Kiszka wrote: On 2013-09-20 19:51, Paul Gortmaker wrote: [Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul Gortmaker wrote: On 13-09-16 10:06 AM, Paolo Bonzini wrote: Paul Gortmaker reported

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-20 Thread Jan Kiszka
On 2013-09-20 19:51, Paul Gortmaker wrote: [Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul Gortmaker wrote: On 13-09-16 10:06 AM, Paolo Bonzini wrote: Paul Gortmaker reported a BUG on preempt-rt kernels, due to taking the mmu_lock within the raw kvm_lock

[PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-16 Thread Paolo Bonzini
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

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-16 Thread Paul Gortmaker
On 13-09-16 10:06 AM, 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