Re: [PULL 00/12] ppc patch queue 2015-08-22

2015-08-24 Thread Paolo Bonzini
On 24/08/2015 06:49, Alexander Graf wrote: > > > Hi Paolo, > > > > > > This is my current patch queue for ppc. Please pull. > > > > Done, but this queue has not been in linux-next. Please push to > > kvm-ppc-next on your github Linux tree as well; please keep an eye on > > Ah, sorry. I pushed

[PATCH 0/3] KVM: Dynamic halt_poll_ns

2015-08-24 Thread Wanpeng Li
There is a downside of halt_poll_ns since poll is still happen for idle VCPU which can waste cpu usage. This patchset add the ability to adjust halt_poll_ns dynamically, grows halt_poll_ns if an interrupt arrives and shrinks halt_poll_ns when idle VCPU is detected. There are two new kernel para

[PATCH 3/3] KVM: trace kvm_halt_poll_ns grow/shrink

2015-08-24 Thread Wanpeng Li
Tracepoint for dynamic halt_pool_ns, fired on every potential change. Signed-off-by: Wanpeng Li --- include/trace/events/kvm.h | 30 ++ virt/kvm/kvm_main.c| 8 2 files changed, 38 insertions(+) diff --git a/include/trace/events/kvm.h b/include/trace

[PATCH 1/3] KVM: make halt_poll_ns per-VCPU

2015-08-24 Thread Wanpeng Li
Change halt_poll_ns into per-VCPU variable, seeded from module parameter, to allow greater flexibility. Signed-off-by: Wanpeng Li --- include/linux/kvm_host.h | 1 + virt/kvm/kvm_main.c | 1 + 2 files changed, 2 insertions(+) diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h

[PATCH 2/3] KVM: dynamise halt_poll_ns adjustment

2015-08-24 Thread Wanpeng Li
There are two new kernel parameters for changing the halt_poll_ns: halt_poll_ns_grow and halt_poll_ns_shrink. halt_poll_ns_grow affects halt_poll_ns when an interrupt arrives and halt_poll_ns_shrink does it when idle VCPU is detected. halt_poll_ns_shrink/ | halt_poll_ns_grow| interrupt a

Re: [PATCH 2/3] kvm: don't register wildcard MMIO EVENTFD on two buses

2015-08-24 Thread Cornelia Huck
On Mon, 24 Aug 2015 11:29:29 +0800 Jason Wang wrote: > On 08/21/2015 05:29 PM, Cornelia Huck wrote: > > On Fri, 21 Aug 2015 16:03:52 +0800 > > Jason Wang wrote: > >> @@ -850,9 +845,15 @@ kvm_assign_ioeventfd(struct kvm *kvm, struct > >> kvm_ioeventfd *args) > > Unfortunately snipped by diff, b

Re: [PATCH v2 14/15] KVM: arm64: implement MSI injection in ITS emulation

2015-08-24 Thread Andre Przywara
Hi, On 03/08/15 18:06, Marc Zyngier wrote: > On 03/08/15 16:37, Eric Auger wrote: >> Andre, Pavel, >> On 08/03/2015 11:16 AM, Pavel Fedin wrote: >>> Hello! >>> Again the case that leaves me uncomfortable is the one where the userspace does not provide the devid whereas it must (GICv3 IT

Re: KVM slow LAMP guest

2015-08-24 Thread Hansa
On 24-8-2015 1:26, Wanpeng Li wrote: On 8/24/15 3:18 AM, Hansa wrote: On 16-7-2015 13:27, Paolo Bonzini wrote: On 15/07/2015 22:02, "C. Bröcker" wrote: What OS is this? Is it RHEL/CentOS? If so, halt_poll_ns will be in 6.7 which will be out in a few days/weeks. Paolo OK. As said CentOS 6.6.

[Bug 103141] Host-triggerable NULL pointer oops

2015-08-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=103141 --- Comment #2 from felix --- Created attachment 185681 --> https://bugzilla.kernel.org/attachment.cgi?id=185681&action=edit Test program 2 (C99) You mean "can" as in "I think it does" or "it did for me"? And anyway, it seems to only fix the

[Bug 103141] Host-triggerable NULL pointer oops

2015-08-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=103141 felix changed: What|Removed |Added Attachment #185681|0 |1 is obsolete|

linux-kvm.org wiki

2015-08-24 Thread Jonathon Reinhart
To whom it may concern: I'm trying to make an edit to this page: http://www.linux-kvm.org/index.php?title=Windows7Install I have successfully created an account. When I "Save" the edit to the page, it looks like everything worked okay. However, when I refresh the page, my edit is gone, and no hi

Re: [PATCH v2 01/15] KVM: arm/arm64: VGIC: don't track used LRs in the distributor

2015-08-24 Thread Andre Przywara
Hi Eric, On 12/08/15 10:01, Eric Auger wrote: >> diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c >> index bc40137..394622c 100644 >> --- a/virt/kvm/arm/vgic.c >> +++ b/virt/kvm/arm/vgic.c >> @@ -79,7 +79,6 @@ >> #include "vgic.h" >> >> static void vgic_retire_disabled_irqs(struct k

Re: [PATCH 1/3] KVM: make halt_poll_ns per-VCPU

2015-08-24 Thread David Matlack
On Mon, Aug 24, 2015 at 5:53 AM, Wanpeng Li wrote: > Change halt_poll_ns into per-VCPU variable, seeded from module parameter, > to allow greater flexibility. You should also change kvm_vcpu_block to read halt_poll_ns from the vcpu instead of the module parameter. > > Signed-off-by: Wanpeng Li

Re: [PATCH 2/3] KVM: dynamise halt_poll_ns adjustment

2015-08-24 Thread David Matlack
On Mon, Aug 24, 2015 at 5:53 AM, Wanpeng Li wrote: > There are two new kernel parameters for changing the halt_poll_ns: > halt_poll_ns_grow and halt_poll_ns_shrink. halt_poll_ns_grow affects > halt_poll_ns when an interrupt arrives and halt_poll_ns_shrink > does it when idle VCPU is detected. > >

Re: [PATCH v2 05/15] KVM: arm/arm64: make GIC frame address initialization model specific

2015-08-24 Thread Andre Przywara
Hi, On 12/08/15 14:02, Eric Auger wrote: > On 07/10/2015 04:21 PM, Andre Przywara wrote: >> Currently we initialize all the possible GIC frame addresses in one >> function, without looking at the specific GIC model we instantiate >> for the guest. >> As this gets confusing when adding another VGIC

Re: [PATCH v2 07/15] KVM: arm64: handle ITS related GICv3 redistributor registers

2015-08-24 Thread Andre Przywara
Hi Eric, On 13/08/15 13:17, Eric Auger wrote: > On 07/10/2015 04:21 PM, Andre Przywara wrote: >> In the GICv3 redistributor there are the PENDBASER and PROPBASER >> registers which we did not emulate so far, as they only make sense >> when having an ITS. In preparation for that emulate those MMIO

Re: [PATCH 2/3] KVM: dynamise halt_poll_ns adjustment

2015-08-24 Thread Wanpeng Li
Hi David, On 8/25/15 1:00 AM, David Matlack wrote: On Mon, Aug 24, 2015 at 5:53 AM, Wanpeng Li wrote: There are two new kernel parameters for changing the halt_poll_ns: halt_poll_ns_grow and halt_poll_ns_shrink. halt_poll_ns_grow affects halt_poll_ns when an interrupt arrives and halt_poll_ns_s

Re: [PATCH 1/3] KVM: make halt_poll_ns per-VCPU

2015-08-24 Thread Wanpeng Li
On 8/25/15 12:59 AM, David Matlack wrote: On Mon, Aug 24, 2015 at 5:53 AM, Wanpeng Li wrote: Change halt_poll_ns into per-VCPU variable, seeded from module parameter, to allow greater flexibility. You should also change kvm_vcpu_block to read halt_poll_ns from the vcpu instead of the module pa

Re: [Bug 103321] New: NPT page attribute support causes extreme slowdown

2015-08-24 Thread Sebastian Schütte
> Please try this: Still no difference I guess the trace_kvm_cr_write() call in that patch was supposed to trigger kvm_cr entries while tracing? I couldn't find any, though, the only entries containing "cr" within the output of "trace-cmd report" were kvm_exit ones that looked quite similar to th

Re: [RFC PATCH 4/5] KVM: x86: enable unhandled MSR exits for vmx

2015-08-24 Thread Bandan Das
Peter Hornyack writes: > Set the vm's unhandled_msr_exits flag when user space calls the > KVM_ENABLE_CAP ioctl with KVM_CAP_UNHANDLED_MSR_EXITS. After kvm fails > to handle a guest rdmsr or wrmsr, check this flag and exit to user space > with KVM_EXIT_MSR rather than immediately injecting a GP f

Re: [RFC PATCH 3/5] KVM: x86: add msr_exits_supported to kvm_x86_ops

2015-08-24 Thread Bandan Das
Peter Hornyack writes: > msr_exits_supported will be checked when user space attempts to enable > the KVM_CAP_UNHANDLED_MSR_EXITS capability for the vm. This is needed > because MSR exit support will be implemented for vmx but not svm later > in this patchset. Is svm future work ? :) Are there a

Re: [RFC PATCH 0/5] KVM: x86: exit to user space on unhandled MSR accesses

2015-08-24 Thread Bandan Das
Peter Hornyack writes: > On Wed, Aug 19, 2015 at 2:43 PM, Bandan Das wrote: >> Peter Hornyack writes: >> >>> There are numerous MSRs that kvm does not currently handle. On Intel >>> platforms we have observed guest VMs accessing some of these MSRs (for >>> example, MSR_PLATFORM_INFO) and behavi

Re: [PATCH 2/3] kvm: don't register wildcard MMIO EVENTFD on two buses

2015-08-24 Thread Jason Wang
On 08/24/2015 10:05 PM, Cornelia Huck wrote: > On Mon, 24 Aug 2015 11:29:29 +0800 > Jason Wang wrote: > >> On 08/21/2015 05:29 PM, Cornelia Huck wrote: >>> On Fri, 21 Aug 2015 16:03:52 +0800 >>> Jason Wang wrote: @@ -850,9 +845,15 @@ kvm_assign_ioeventfd(struct kvm *kvm, struct kvm_i

[PATCH v2 1/3] KVM: make halt_poll_ns per-VCPU

2015-08-24 Thread Wanpeng Li
Change halt_poll_ns into per-VCPU variable, seeded from module parameter, to allow greater flexibility. Signed-off-by: Wanpeng Li --- include/linux/kvm_host.h | 1 + virt/kvm/kvm_main.c | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/include/linux/kvm_host.h b/incl

[PATCH v2 0/3] KVM: Dynamic halt_poll_ns

2015-08-24 Thread Wanpeng Li
v1 -> v2: * change kvm_vcpu_block to read halt_poll_ns from the vcpu instead of the module parameter * use the shrink/grow matrix which is suggested by David * set halt_poll_ns_max to 2ms There is a downside of halt_poll_ns since poll is still happen for idle VCPU which can waste cpu usage

[PATCH v2 3/3] KVM: trace kvm_halt_poll_ns grow/shrink

2015-08-24 Thread Wanpeng Li
Tracepoint for dynamic halt_pool_ns, fired on every potential change. Signed-off-by: Wanpeng Li --- include/trace/events/kvm.h | 30 ++ virt/kvm/kvm_main.c| 8 2 files changed, 38 insertions(+) diff --git a/include/trace/events/kvm.h b/include/trace

[PATCH v2 2/3] KVM: dynamic halt_poll_ns adjustment

2015-08-24 Thread Wanpeng Li
There is a downside of halt_poll_ns since poll is still happen for idle VCPU which can waste cpu usage. This patch adds the ability to adjust halt_poll_ns dynamically. There are two new kernel parameters for changing the halt_poll_ns: halt_poll_ns_grow and halt_poll_ns_shrink. A third new param