Re: [PATCH] kvmclock: Ensure time in migration never goes backward

2014-05-14 Thread Marcelo Tosatti
On Fri, May 09, 2014 at 01:53:32PM +0200, Paolo Bonzini wrote: Il 09/05/2014 04:28, Marcelo Tosatti ha scritto: Alex, Unability to upgrade systems is not an excuse to fix the bug in the wrong place. It may be an excuse to fix the bug in both places though. Paolo Actually, its messy to

Re: [PATCH] kvmclock: Ensure time in migration never goes backward

2014-05-14 Thread Marcelo Tosatti
On Mon, May 12, 2014 at 10:26:37PM +0200, Alexander Graf wrote: On 09.05.14 13:53, Paolo Bonzini wrote: Il 09/05/2014 04:28, Marcelo Tosatti ha scritto: Alex, Unability to upgrade systems is not an excuse to fix the bug in the wrong place. It may be an excuse to fix the bug in both

KVM: x86: disable master clock if TSC is reset during suspend

2014-05-14 Thread Marcelo Tosatti
Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume path. Signed-off-by: Marcelo Tosatti

Re: [PATCH 0/5] KVM: x86: support setting the CPL independent of CS

2014-05-14 Thread Paolo Bonzini
Il 13/05/2014 16:55, Paolo Bonzini ha scritto: Until now, KVM used to assume that CS.RPL could always be used as the CPL value when KVM_SET_SREGS is called. Unfortunately this is not the case. If userspace decides to call KVM_GET_SREGS/KVM_SET_SREGS exactly after CR0.PE has been set to 1, but

[PATCH] KVM: x86: get CPL from SS.DPL

2014-05-14 Thread Paolo Bonzini
CS.RPL is not equal to the CPL in the few instructions between setting CR0.PE and reloading CS. And CS.DPL is also not equal to the CPL for conforming code segments. However, SS.DPL *is* always equal to the CPL except for the weird case of SYSRET on AMD processors, which sets SS.DPL=SS.RPL from

Re: [PATCH v1] vhost: avoid large order allocations

2014-05-14 Thread Michael Mueller
On Tue, 13 May 2014 18:15:27 +0300 Michael S. Tsirkin m...@redhat.com wrote: On Tue, May 13, 2014 at 04:29:58PM +0200, Romain Francoise wrote: Michael S. Tsirkin m...@redhat.com writes: Please dont' do this, extra indirection hurts performance. Instead, please change vhost_net_open

Re: [PATCH] KVM: x86: get/set CPL separately from CS

2014-05-14 Thread Paolo Bonzini
Il 13/05/2014 17:04, Paolo Bonzini ha scritto: KVM used to assume that CS.RPL could always be used as the CPL value when KVM_SET_SREGS is called. QEMU could call KVM_GET_SREGS and the KVM_SET_SREGS exactly after CR0.PE has been set to 1, but before the long jump that reloads CS. Then, KVM

[Bug 76141] New: kvm intel nested virtualization get PANIC: double fault, error_code: 0x0

2014-05-14 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=76141 Bug ID: 76141 Summary: kvm intel nested virtualization get PANIC: double fault, error_code: 0x0 Product: Virtualization Version: unspecified Kernel Version: 3.12.18

Re: [Qemu-devel] [PATCH v1 RFC 09/10] QEMU: s390: cpu model QMP query-cpu-model

2014-05-14 Thread Michael Mueller
On Tue, 13 May 2014 09:29:37 -0600 Eric Blake ebl...@redhat.com wrote: Hi Eric, On 05/13/2014 09:00 AM, Michael Mueller wrote: This patch implements a new QMP request named query-cpu-model. It returns the cpu model of cpu 0. eg: request: '{execute : query-cpu-model } answer:

Re: KVM: x86: disable master clock if TSC is reset during suspend

2014-05-14 Thread Paolo Bonzini
Il 14/05/2014 09:26, Marcelo Tosatti ha scritto: Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume

KVM: x86: disable master clock if TSC is reset during suspend (v2)

2014-05-14 Thread Marcelo Tosatti
Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume path. Signed-off-by: Marcelo Tosatti

Re: KVM: x86: disable master clock if TSC is reset during suspend

2014-05-14 Thread Marcelo Tosatti
On Wed, May 14, 2014 at 11:09:36AM +0200, Paolo Bonzini wrote: Il 14/05/2014 09:26, Marcelo Tosatti ha scritto: Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency.

Re: KVM: x86: disable master clock if TSC is reset during suspend (v2)

2014-05-14 Thread Paolo Bonzini
Il 14/05/2014 12:06, Marcelo Tosatti ha scritto: Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume

[Bug 76141] kvm intel nested virtualization get PANIC: double fault, error_code: 0x0

2014-05-14 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=76141 Paolo Bonzini bonz...@gnu.org changed: What|Removed |Added CC||bonz...@gnu.org ---

Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread Zoltan Kiss
Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list were modified. I came across this function skb_shift(), which moves frags between skbs. And there are a lot more of such kind, skb_split or skb_try_coalesce, for example. It could be a dangerous thing if

[Bug 76191] New: vt-d intel IOMMU - jump label: negative count!

2014-05-14 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=76191 Bug ID: 76191 Summary: vt-d intel IOMMU - jump label: negative count! Product: Virtualization Version: unspecified Kernel Version: Linux venus 3.12-1-amd64 #1 SMP Debian 3.12.9-1

Re: Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread Eric Dumazet
On Wed, 2014-05-14 at 14:40 +0100, Zoltan Kiss wrote: Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list were modified. I came across this function skb_shift(), which moves frags between skbs. And there are a lot more of such kind, skb_split or

Poor performance with virtio network driver on windows 2008 R2 on KVM

2014-05-14 Thread JR
Greetings all, Though I don't believe that this is KVM problem per se, I'm seeing very poor network performance when using windows 2008R2 KVM instances with the virtio driver when on a 10G network. iperf results show linux VMs getting close to wire speeds, while windows only does about

KVM: x86: disable master clock if TSC is reset during suspend (v3)

2014-05-14 Thread Marcelo Tosatti
Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume path. Signed-off-by: Marcelo Tosatti

Re: KVM: x86: disable master clock if TSC is reset during suspend (v3)

2014-05-14 Thread Paolo Bonzini
Il 14/05/2014 17:43, Marcelo Tosatti ha scritto: Updating system_time from the kernel clock once master clock has been enabled can result in time backwards event, in case kernel clock frequency is lower than TSC frequency. Disable master clock in case its necessary to update it from the resume

Re: [PATCH v5 1/4] add ARMv7 HYP API to flush VM TLBs without address param

2014-05-14 Thread Christoffer Dall
On Wed, May 07, 2014 at 05:40:13PM -0700, Mario Smarduch wrote: Patch adds HYP interface for global VM TLB invalidation without address parameter. - Added ARM version of kvm_flush_remote_tlbs() Signed-off-by: Mario Smarduch m.smard...@samsung.com --- arch/arm/include/asm/kvm_asm.h |

Re: [PATCH v10 03/19] qspinlock: Add pending bit

2014-05-14 Thread Radim Krčmář
2014-05-13 15:47-0400, Waiman Long: On 05/12/2014 11:22 AM, Radim Krčmář wrote: I think there is an unwanted scenario on virtual machines: 1) VCPU sets the pending bit and start spinning. 2) Pending VCPU gets descheduled. - we have PLE and lock holder isn't running [1] - the

Re: [PATCH v10 03/19] qspinlock: Add pending bit

2014-05-14 Thread Peter Zijlstra
On Wed, May 14, 2014 at 06:51:24PM +0200, Radim Krčmář wrote: Ok. I've seen merit in pvqspinlock even with slightly slower first-waiter, so I would have happily sacrificed those horrible branches. (I prefer elegant to optimized code, but I can see why we want to be strictly better than

Re: Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread David Miller
From: Eric Dumazet eric.duma...@gmail.com Date: Wed, 14 May 2014 07:23:52 -0700 On Wed, 2014-05-14 at 14:40 +0100, Zoltan Kiss wrote: Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list were modified. I came across this function skb_shift(), which

Re: Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread Eric Dumazet
On Wed, 2014-05-14 at 13:42 -0400, David Miller wrote: From: Eric Dumazet eric.duma...@gmail.com Date: Wed, 14 May 2014 07:23:52 -0700 On Wed, 2014-05-14 at 14:40 +0100, Zoltan Kiss wrote: Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list

Re: [PATCH v10 03/19] qspinlock: Add pending bit

2014-05-14 Thread Radim Krčmář
2014-05-14 19:00+0200, Peter Zijlstra: On Wed, May 14, 2014 at 06:51:24PM +0200, Radim Krčmář wrote: Ok. I've seen merit in pvqspinlock even with slightly slower first-waiter, so I would have happily sacrificed those horrible branches. (I prefer elegant to optimized code, but I can see

Re: Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread Zoltan Kiss
On 14/05/14 15:23, Eric Dumazet wrote: On Wed, 2014-05-14 at 14:40 +0100, Zoltan Kiss wrote: Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list were modified. I came across this function skb_shift(), which moves frags between skbs. And there are a lot

Re: Moving frags and SKBTX_DEV_ZEROCOPY skbs

2014-05-14 Thread Eric Dumazet
On Wed, 2014-05-14 at 20:41 +0100, Zoltan Kiss wrote: On 14/05/14 15:23, Eric Dumazet wrote: On Wed, 2014-05-14 at 14:40 +0100, Zoltan Kiss wrote: Hi, Recently I've investigated issues around SKBTX_DEV_ZEROCOPY skbs where the frags list were modified. I came across this function

Re: [PATCH v5 1/4] add ARMv7 HYP API to flush VM TLBs without address param

2014-05-14 Thread Mario Smarduch
On 05/14/2014 09:47 AM, Christoffer Dall wrote: On Wed, May 07, 2014 at 05:40:13PM -0700, Mario Smarduch wrote: Patch adds HYP interface for global VM TLB invalidation without address parameter. - Added ARM version of kvm_flush_remote_tlbs() Signed-off-by: Mario Smarduch

RE: Poor performance with virtio network driver on windows 2008 R2 on KVM

2014-05-14 Thread Venkateswara Rao Nandigam
Try TSO offload on Guest Virtio interface And try passthrough interface mode at Host Level for this Guest Virtio Interface Thanks, Venkatesh -Original Message- From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] On Behalf Of JR Sent: Wednesday, May 14, 2014 9:07 PM To:

Re: [PATCH] ARM: KVM: add irqfd and irq routing support

2014-05-14 Thread Anup Patel
On Fri, May 9, 2014 at 6:15 PM, Eric Auger eric.au...@linaro.org wrote: This patch enables irqfd and irq routing on ARM. It turns on CONFIG_HAVE_KVM_EVENTFD and CONFIG_HAVE_KVM_IRQ_ROUTING irqfd framework enables to assign physical IRQs to guests. 1) user-side uses KVM_IRQFD VM ioctl to

How to disable IDE DMA in KVM or in guest OS

2014-05-14 Thread Arthur Chunqi Li
Hi Jan and there, I want to disable IDE BMDMA in Qemu/KVM and let guest OS uses only PIO mode. Are there any configurations in Qemu or KVM to disable the hardware support of DMA? I have tried to disable IDE DMA in guest OS booting params as follows: ide-core.nodma=0.0 libata.dma=0 ide=nodma