RE: [PATCH 2/2] vfio: hugepage support for vfio_iommu_type1

2013-05-27 Thread Sethi Varun-B16395
-Original Message- From: iommu-boun...@lists.linux-foundation.org [mailto:iommu- boun...@lists.linux-foundation.org] On Behalf Of Alex Williamson Sent: Friday, May 24, 2013 10:55 PM To: alex.william...@redhat.com Cc: io...@lists.linux-foundation.org; chegu_vi...@hp.com; qemu-

Re: [PATCH 1/4] KVM: PPC: Add support for multiple-TCE hcalls

2013-05-27 Thread Paolo Bonzini
Il 21/05/2013 05:06, Alexey Kardashevskiy ha scritto: This adds real mode handlers for the H_PUT_TCE_INDIRECT and H_STUFF_TCE hypercalls for QEMU emulated devices such as virtio devices or emulated PCI. Do you mean vio? virtio (without getting into whether that's good or bad) will bypass the

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-05-27 Thread Paolo Bonzini
Il 25/05/2013 04:45, David Gibson ha scritto: + case KVM_CREATE_SPAPR_TCE_IOMMU: { + struct kvm_create_spapr_tce_iommu create_tce_iommu; + struct kvm *kvm = filp-private_data; + + r = -EFAULT; + if (copy_from_user(create_tce_iommu, argp, +

Re: [PATCH] kvm: exclude ioeventfd from counting kvm_io_range limit

2013-05-27 Thread Stefan Hajnoczi
On Sat, May 25, 2013 at 06:44:15AM +0800, Amos Kong wrote: We can easily reach the 1000 limit by start VM with a couple hundred I/O devices (multifunction=on). The hardcode limit already been adjusted 3 times (6 ~ 200 ~ 300 ~ 1000). In userspace, we already have maximum file descriptor to

Re: [RFC PATCH v2 2/2] add support for Hyper-V invariant TSC

2013-05-27 Thread Vadim Rozenfeld
On Fri, 2013-05-24 at 16:41 -0300, Marcelo Tosatti wrote: On Fri, May 24, 2013 at 06:11:16AM -0400, Vadim Rozenfeld wrote: Is there a better option? If setting TscSequence to zero makes Windows fall back to the MSR this is a better option. +1 This is why MS has two different

Re: [PATCH 00/18] KVM/MIPS32: Support for the new Virtualization ASE (VZ-ASE)

2013-05-27 Thread Maciej W. Rozycki
On Mon, 20 May 2013, David Daney wrote: That's rather risky as the implementation of this register (and its presence in the first place) is processor-specific. Do you maintain a list of PRId values the use of this register is safe with? FWIW: The MIPS-VZ architecture module

Re: [PATCH 2/2] vfio: hugepage support for vfio_iommu_type1

2013-05-27 Thread Alex Williamson
On Mon, 2013-05-27 at 08:41 +, Sethi Varun-B16395 wrote: -Original Message- From: iommu-boun...@lists.linux-foundation.org [mailto:iommu- boun...@lists.linux-foundation.org] On Behalf Of Alex Williamson Sent: Friday, May 24, 2013 10:55 PM To: alex.william...@redhat.com

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-05-27 Thread Alexey Kardashevskiy
On 05/27/2013 08:23 PM, Paolo Bonzini wrote: Il 25/05/2013 04:45, David Gibson ha scritto: + case KVM_CREATE_SPAPR_TCE_IOMMU: { + struct kvm_create_spapr_tce_iommu create_tce_iommu; + struct kvm *kvm = filp-private_data; + + r = -EFAULT; + if

Re: [PATCH 1/4] KVM: PPC: Add support for multiple-TCE hcalls

2013-05-27 Thread Alexey Kardashevskiy
On 05/27/2013 08:08 PM, Paolo Bonzini wrote: Il 21/05/2013 05:06, Alexey Kardashevskiy ha scritto: This adds real mode handlers for the H_PUT_TCE_INDIRECT and H_STUFF_TCE hypercalls for QEMU emulated devices such as virtio devices or emulated PCI. Do you mean vio? virtio (without getting

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 16:26, Alexey Kardashevskiy ha scritto: On 05/27/2013 08:23 PM, Paolo Bonzini wrote: Il 25/05/2013 04:45, David Gibson ha scritto: + case KVM_CREATE_SPAPR_TCE_IOMMU: { + struct kvm_create_spapr_tce_iommu create_tce_iommu; + struct kvm *kvm = filp-private_data; +

Re: [PATCH] virtio-balloon spec: rework VIRTIO_BALLOON_F_MUST_TELL_HOST feature, support silent deflation

2013-05-27 Thread Paolo Bonzini
Il 08/05/2013 12:10, Paolo Bonzini ha scritto: The idea of the VIRTIO_BALLOON_F_MUST_TELL_HOST feature is to let drivers skip usage of the deflate queue when leaking the balloon (silent deflation). Guests may benefit from silent deflate by aggressively inflating the balloon; they know that

[PATCH rusty/virtio-pci-new-layout] virtio: new layout minor header fixups

2013-05-27 Thread Michael S. Tsirkin
Fix issues observed with the new layout code, seen when implementing device in qemu: - use of uXX in uapi header - incorrect readonly tag on one field - unconditional warning breaks builds with -Werr Signed-off-by: Michael S. Tsirkin m...@redhat.com --- This patch is on

Re: [PATCH] virtio-balloon spec: rework VIRTIO_BALLOON_F_MUST_TELL_HOST feature, support silent deflation

2013-05-27 Thread Michael S. Tsirkin
On Mon, May 27, 2013 at 05:55:05PM +0200, Paolo Bonzini wrote: Il 08/05/2013 12:10, Paolo Bonzini ha scritto: The idea of the VIRTIO_BALLOON_F_MUST_TELL_HOST feature is to let drivers skip usage of the deflate queue when leaking the balloon (silent deflation). Guests may benefit from

Re: [PATCH] virtio-balloon spec: rework VIRTIO_BALLOON_F_MUST_TELL_HOST feature, support silent deflation

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 18:04, Michael S. Tsirkin ha scritto: I don't think we need a new feature. Hosts do not in practice treat the feature as negative (that is, required), whatever the spec says. Further, windows guests don't treat it is such either. Windows guests not treating as such is what makes

Re: [PATCH v3-resend 00/11] uaccess: better might_sleep/might_fault behavior

2013-05-27 Thread Peter Zijlstra
On Sun, May 26, 2013 at 05:21:30PM +0300, Michael S. Tsirkin wrote: If the changes look good, would sched maintainers please consider merging them through sched/core because of the interaction with the scheduler? Please review, and consider for 3.11. I'll stick them in my queue, we'll see

[PATCH] KVM: s390: Add devname:kvm alias.

2013-05-27 Thread Cornelia Huck
Providing a devname:kvm module alias enables automatic loading of the kvm module when /dev/kvm is opened. Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- arch/s390/kvm/kvm-s390.c | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/s390/kvm/kvm-s390.c

[PATCH] KVM: s390: Module autoloading.

2013-05-27 Thread Cornelia Huck
This patch enables automatically loading the kvm module when /dev/kvm is opened, allowing distros to stop loading the module statically. I think it is trivial enough for 3.10; otherwise, please queue this for 3.11. Cornelia Huck (1): KVM: s390: Add devname:kvm alias. arch/s390/kvm/kvm-s390.c

Re: [PATCH] virtio-balloon spec: rework VIRTIO_BALLOON_F_MUST_TELL_HOST feature, support silent deflation

2013-05-27 Thread Michael S. Tsirkin
On Mon, May 27, 2013 at 06:09:54PM +0200, Paolo Bonzini wrote: Il 27/05/2013 18:04, Michael S. Tsirkin ha scritto: I don't think we need a new feature. Hosts do not in practice treat the feature as negative (that is, required), whatever the spec says. Further, windows guests don't treat it

[Bug 58771] VM performance degradation after KVM QEMU migration or save/restore with Intel EPT enabled

2013-05-27 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=58771 --- Comment #1 from Chris ccorm...@gmail.com 2013-05-27 18:37:46 --- Apologies, mentions of Kernel 2.8.x above should be 2.6. -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail

Re: [PATCH v3 7/7] ARM: KVM: drop use of PAGE_S2_DEVICE

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 4:11 AM, Marc Zyngier marc.zyng...@arm.com wrote: At the moment, when mapping a device into Stage-2 for a guest, we override whatever the guest uses by forcing a device memory type in Stage-2. While this is not exactly wrong, this isn't really the spirit of the

Unable to boot from SCSI disk

2013-05-27 Thread Daniel Guillermo Bareiro
Hi all! I'm trying libvirt and virt-manager and I found that if I install a virtual machine with SCSI disks, the installation is done without problems but when it boots, it is unable to boot from these disks. If I use bus=sata or bus=virtio, this problem does not happen. I also tried installing

Re: Unable to boot from SCSI disk

2013-05-27 Thread Paolo Bonzini
Il 27/05/2013 22:53, Daniel Guillermo Bareiro ha scritto: Hi all! I'm trying libvirt and virt-manager and I found that if I install a virtual machine with SCSI disks, the installation is done without problems but when it boots, it is unable to boot from these disks. If I use bus=sata or

Re: Unable to boot from SCSI disk

2013-05-27 Thread Michael Tokarev
28.05.2013 00:59, Paolo Bonzini wrote: Thank you Paolo for forwarding this email to me. Il 27/05/2013 22:53, Daniel Guillermo Bareiro ha scritto: Hi all! I'm trying libvirt and virt-manager and I found that if I install a virtual machine with SCSI disks, the installation is done without

Re: [PATCH v7 03/11] KVM: MMU: fast invalidate all pages

2013-05-27 Thread Marcelo Tosatti
On Sun, May 26, 2013 at 11:26:49AM +0300, Gleb Natapov wrote: On Fri, May 24, 2013 at 05:23:07PM -0300, Marcelo Tosatti wrote: Hi Xiao, On Thu, May 23, 2013 at 03:55:52AM +0800, Xiao Guangrong wrote: The current kvm_mmu_zap_all is really slow - it is holding mmu-lock to walk and zap

Re: [PATCH v7 03/11] KVM: MMU: fast invalidate all pages

2013-05-27 Thread Xiao Guangrong
On 05/27/2013 04:37 AM, Marcelo Tosatti wrote: On Sun, May 26, 2013 at 11:26:49AM +0300, Gleb Natapov wrote: On Fri, May 24, 2013 at 05:23:07PM -0300, Marcelo Tosatti wrote: Hi Xiao, On Thu, May 23, 2013 at 03:55:52AM +0800, Xiao Guangrong wrote: The current kvm_mmu_zap_all is really slow -

Re: [Qemu-devel] VFIO VGA test branches

2013-05-27 Thread Maik Broemme
Hi Alex, Maik Broemme mbroe...@parallels.com wrote: Hi Alex, Alex Williamson alex.william...@redhat.com wrote: Good to hear. It looks like you have the same motherboard as my AMD test system. An HD7850 in that system runs quite reliably with the branches above although I do

Re: [PATCH v3 1/7] ARM: KVM: be more thorough when invalidating TLBs

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:34PM +0100, Marc Zyngier wrote: The KVM/ARM MMU code doesn't take care of invalidating TLBs before freeing a {pte,pmd} table. This could cause problems if the page is reallocated and then speculated into by another CPU. Reported-by: Catalin Marinas

Re: [PATCH v3 2/7] ARM: KVM: remove dead prototype for __kvm_tlb_flush_vmid

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:35PM +0100, Marc Zyngier wrote: __kvm_tlb_flush_vmid has been renamed to __kvm_tlb_flush_vmid_ipa, and the old prototype should have been removed when the code was modified. Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- arch/arm/include/asm/kvm_asm.h |

Re: [PATCH v3 3/7] ARM: KVM: relax cache maintainance when building page tables

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:36PM +0100, Marc Zyngier wrote: Patch 5a677ce044f1 (ARM: KVM: switch to a dual-step HYP init code) introduced code that flushes page tables to the point of coherency. This is overkill (point of unification is enough and already done), and actually not required if

Re: [PATCH v3 4/7] ARM: KVM: use phys_addr_t instead of unsigned long long for HYP PGDs

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:37PM +0100, Marc Zyngier wrote: HYP PGDs are passed around as phys_addr_t, except just before calling into the hypervisor init code, where they are cast to a rather weird unsigned long long. Just keep them around as phys_addr_t, which is what makes the most

Re: [PATCH v3 5/7] ARM: KVM: don't special case PC when doing an MMIO

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:38PM +0100, Marc Zyngier wrote: Admitedly, reading a MMIO register to load PC is very weird. Writing PC to a MMIO register is probably even worse. But the architecture doesn't forbid any of these, and injecting a Prefetch Abort is the wrong thing to do anyway.

Re: [PATCH v3 6/7] ARM: KVM: get rid of S2_PGD_SIZE

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:39PM +0100, Marc Zyngier wrote: S2_PGD_SIZE defines the number of pages used by a stage-2 PGD and is unused, except for a VM_BUG_ON check that missuses the define. As the check is very unlikely to ever triggered except in circumstances where KVM is the least of

Re: [PATCH v3 6/7] ARM: KVM: get rid of S2_PGD_SIZE

2013-05-27 Thread Christoffer Dall
On Tue, May 14, 2013 at 12:11:39PM +0100, Marc Zyngier wrote: S2_PGD_SIZE defines the number of pages used by a stage-2 PGD and is unused, except for a VM_BUG_ON check that missuses the define. As the check is very unlikely to ever triggered except in circumstances where KVM is the least of

Re: [Qemu-devel] VFIO VGA test branches

2013-05-27 Thread Alex Williamson
On Tue, 2013-05-28 at 03:40 +0200, Maik Broemme wrote: Hi Alex, Maik Broemme mbroe...@parallels.com wrote: Hi Alex, Alex Williamson alex.william...@redhat.com wrote: Good to hear. It looks like you have the same motherboard as my AMD test system. An HD7850 in that system

Re: [PATCH rusty/virtio-pci-new-layout] virtio: new layout minor header fixups

2013-05-27 Thread Rusty Russell
Michael S. Tsirkin m...@redhat.com writes: Fix issues observed with the new layout code, seen when implementing device in qemu: - use of uXX in uapi header - incorrect readonly tag on one field - unconditional warning breaks builds with -Werr Signed-off-by: Michael S.

Re: [Qemu-devel] VFIO VGA test branches

2013-05-27 Thread Knut Omang
On Mon, 2013-05-20 at 23:11 +0200, Knut Omang wrote: On Sun, 2013-05-19 at 22:15 -0600, Alex Williamson wrote: On Sun, 2013-05-19 at 17:35 +0200, Knut Omang wrote: On Mon, 2013-05-13 at 16:23 -0600, Alex Williamson wrote: On Mon, 2013-05-13 at 22:55 +0200, Knut Omang wrote: Hi all,

Re: [PATCH 1/4] KVM: PPC: Add support for multiple-TCE hcalls

2013-05-27 Thread Paolo Bonzini
Il 21/05/2013 05:06, Alexey Kardashevskiy ha scritto: This adds real mode handlers for the H_PUT_TCE_INDIRECT and H_STUFF_TCE hypercalls for QEMU emulated devices such as virtio devices or emulated PCI. Do you mean vio? virtio (without getting into whether that's good or bad) will bypass the

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-05-27 Thread Paolo Bonzini
Il 25/05/2013 04:45, David Gibson ha scritto: + case KVM_CREATE_SPAPR_TCE_IOMMU: { + struct kvm_create_spapr_tce_iommu create_tce_iommu; + struct kvm *kvm = filp-private_data; + + r = -EFAULT; + if (copy_from_user(create_tce_iommu, argp, +

Re: [PATCH 3/4] KVM: PPC: Add support for IOMMU in-kernel handling

2013-05-27 Thread Alexey Kardashevskiy
On 05/27/2013 08:23 PM, Paolo Bonzini wrote: Il 25/05/2013 04:45, David Gibson ha scritto: + case KVM_CREATE_SPAPR_TCE_IOMMU: { + struct kvm_create_spapr_tce_iommu create_tce_iommu; + struct kvm *kvm = filp-private_data; + + r = -EFAULT; + if