[PATCH 3/4] kvm: Replace vcpu->swait with rcuwait

2020-03-24 Thread Davidlohr Bueso
.columbia.edu Cc: linux-m...@vger.kernel.org Signed-off-by: Davidlohr Bueso --- arch/mips/kvm/mips.c | 6 ++ arch/powerpc/include/asm/kvm_book3s.h | 2 +- arch/powerpc/include/asm/kvm_host.h | 2 +- arch/powerpc/kvm/book3s_hv.c | 22 --

Re: [PATCH 3/4] kvm: Replace vcpu->swait with rcuwait

2020-03-24 Thread Davidlohr Bueso
On Wed, 25 Mar 2020, kbuild test robot wrote: arch/x86/kvm/../../../virt/kvm/kvm_main.c: In function 'kvm_vcpu_block': arch/x86/kvm/../../../virt/kvm/kvm_main.c:2698:16: error: macro "rcuwait_wait_event" passed 3 arguments, but takes just 2 TASK_IDLE); This is because this series

Re: [PATCH 3/4] kvm: Replace vcpu->swait with rcuwait

2020-04-15 Thread Davidlohr Bueso
On Wed, 25 Mar 2020, Paolo Bonzini wrote: On 24/03/20 05:44, Davidlohr Bueso wrote: diff --git a/arch/mips/kvm/mips.c b/arch/mips/kvm/mips.c index 71244bf87c3a..e049fcb3dffb 100644 --- a/arch/mips/kvm/mips.c +++ b/arch/mips/kvm/mips.c @@ -290,8 +290,7 @@ static enum hrtimer_restart

[PATCH v2] kvm: Replace vcpu->swait with rcuwait

2020-04-20 Thread Davidlohr Bueso
.columbia.edu Cc: linux-m...@vger.kernel.org Signed-off-by: Davidlohr Bueso --- v2: Added missing semicolon in mips change. The rest of the patches in this series continues to apply on tip, as such I am only sending a v2 for this particular patch. arch/mips/kvm/mips.c | 6 ++ arch

Re: [PATCH v2] kvm: Replace vcpu->swait with rcuwait

2020-04-21 Thread Davidlohr Bueso
On Mon, 20 Apr 2020, Paolo Bonzini wrote: On 20/04/20 22:56, Davidlohr Bueso wrote: On Mon, 20 Apr 2020, Marc Zyngier wrote: This looks like a change in the semantics of the tracepoint. Before this change, 'waited' would have been true if the vcpu waited at all. Here, you'd h

Re: [PATCH v2] kvm: Replace vcpu->swait with rcuwait

2020-04-21 Thread Davidlohr Bueso
On Mon, 20 Apr 2020, Marc Zyngier wrote: This looks like a change in the semantics of the tracepoint. Before this change, 'waited' would have been true if the vcpu waited at all. Here, you'd have false if it has been interrupted by a signal, even if the vcpu has waited for a period of time.

Re: [PATCH v2] kvm: Replace vcpu->swait with rcuwait

2020-04-21 Thread Davidlohr Bueso
On Tue, 21 Apr 2020, Paolo Bonzini wrote: On 20/04/20 23:50, Davidlohr Bueso wrote: On Mon, 20 Apr 2020, Paolo Bonzini wrote: On 20/04/20 22:56, Davidlohr Bueso wrote: On Mon, 20 Apr 2020, Marc Zyngier wrote: This looks like a change in the semantics of the tracepoint. Before this change

[PATCH 4/5] kvm: Replace vcpu->swait with rcuwait

2020-04-22 Thread Davidlohr Bueso
uot;) shows that, on avg, latency is reduced by around 15-20% with this change. Cc: Paul Mackerras Cc: kvmarm@lists.cs.columbia.edu Cc: linux-m...@vger.kernel.org Signed-off-by: Davidlohr Bueso --- arch/mips/kvm/mips.c | 6 ++ arch/powerpc/include/asm/kvm_book3s.h | 2

[PATCH 5/5] kvm: Replace vcpu->swait with rcuwait

2020-04-24 Thread Davidlohr Bueso
uot;) shows that, on avg, latency is reduced by around 15-20% with this change. Cc: Paul Mackerras Cc: kvmarm@lists.cs.columbia.edu Cc: linux-m...@vger.kernel.org Reviewed-by: Marc Zyngier Signed-off-by: Davidlohr Bueso --- arch/mips/kvm/mips.c | 6 ++ arch/powerpc/include/

Re: [PATCH 5/5] kvm: Replace vcpu->swait with rcuwait

2020-04-26 Thread Davidlohr Bueso
On Sat, 25 Apr 2020, Paolo Bonzini wrote: I'm squashing this in to keep the changes compared to the current code minimal, and queuing the series. Thanks! ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listi