Re: [PULL 0/8] ppc patch queue 2013-01-10

2013-01-14 Thread Gleb Natapov
On Thu, Jan 10, 2013 at 01:45:04PM +0100, Alexander Graf wrote: Hi Marcelo / Gleb, This is my current patch queue for ppc. Please pull. Highlights this time: - Book3S: enable potential sPAPR guest emulation on PR KVM on pHyp - BookE: EPR (External Proxy Register) support Alex

Re: [PATCH V3 2/2] vhost: handle polling errors

2013-01-14 Thread Michael S. Tsirkin
On Mon, Jan 14, 2013 at 03:37:58PM +0800, Jason Wang wrote: On 01/14/2013 02:57 PM, Michael S. Tsirkin wrote: On Mon, Jan 14, 2013 at 10:59:02AM +0800, Jason Wang wrote: On 01/13/2013 07:10 PM, Michael S. Tsirkin wrote: On Mon, Jan 07, 2013 at 11:04:32PM +0800, Jason Wang wrote: On

Re: [PATCH 0/7 -v2] KVM: Alleviate mmu_lock hold time when we start dirty logging

2013-01-14 Thread Gleb Natapov
On Tue, Jan 08, 2013 at 07:42:38PM +0900, Takuya Yoshikawa wrote: Changelog v1-v2: The condition in patch 1 was changed like this: npages (mem-flags KVM_MEM_LOG_DIRTY_PAGES) This patch set makes kvm_mmu_slot_remove_write_access() rmap based and adds conditional rescheduling to it.

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Gleb Natapov
On Tue, Jan 08, 2013 at 01:38:48PM -0500, Christoffer Dall wrote: Add a method (hyp_idmap_setup) to populate a hyp pgd with an identity mapping of the code contained in the .hyp.idmap.text section. Offer a method to drop this identity mapping through hyp_idmap_teardown. Make all the

Re: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote: From: Yang Zhang yang.z.zh...@intel.com basically to benefit from apicv, we need to enable virtualized x2apic mode. Currently, we only enable it when guest is really using x2apic. Also, clear MSR bitmap for corresponding x2apic

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 10:27:21AM +, Gleb Natapov wrote: On Tue, Jan 08, 2013 at 01:38:48PM -0500, Christoffer Dall wrote: Add a method (hyp_idmap_setup) to populate a hyp pgd with an identity mapping of the code contained in the .hyp.idmap.text section. Offer a method to drop

RE: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Zhang, Yang Z
Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote: From: Yang Zhang yang.z.zh...@intel.com basically to benefit from apicv, we need to enable virtualized x2apic mode. Currently, we only enable it when guest is really using x2apic. Also, clear MSR

Re: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 11:01:02AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote: From: Yang Zhang yang.z.zh...@intel.com basically to benefit from apicv, we need to enable virtualized x2apic mode. Currently, we

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 10:49:53AM +, Will Deacon wrote: On Mon, Jan 14, 2013 at 10:27:21AM +, Gleb Natapov wrote: On Tue, Jan 08, 2013 at 01:38:48PM -0500, Christoffer Dall wrote: Add a method (hyp_idmap_setup) to populate a hyp pgd with an identity mapping of the code contained

RE: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Zhang, Yang Z
Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:01:02AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote: From: Yang Zhang yang.z.zh...@intel.com basically to benefit from apicv, we need to enable

Re: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 11:10:26AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:01:02AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote: From: Yang Zhang

RE: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Zhang, Yang Z
Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:10:26AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:01:02AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 03:13:34PM +0800, Yang Zhang wrote:

Re: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 11:25:45AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:10:26AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:01:02AM +, Zhang, Yang Z wrote: Gleb Natapov wrote on

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Russell King - ARM Linux
On Mon, Jan 14, 2013 at 01:07:56PM +0200, Gleb Natapov wrote: Ah, of course. This is ident map so by definition it cannot map phys addresses above 4G. And since __virt_to_phys() suppose to work only on ident map it's OK to returns unsigned long. Let's get this right... the lack of correct

KVM call for 2013-01-15

2013-01-14 Thread Juan Quintela
Hi Please send in any agenda topics that you have. Thanks, Juan. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: qemu-kvm hangs at start up under 3.8.0-rc3-00074-gb719f43 (works with CONFIG_LOCKDEP)

2013-01-14 Thread Gleb Natapov
On Sun, Jan 13, 2013 at 10:29:58PM +, Andrew Clayton wrote: When running qemu-kvm under 64but Fedora 16 under current 3.8, it just hangs at start up. Dong a ps -ef hangs the process at the point where it would display the qemu process (trying to list the qemu-kvm /proc pid directory

Shared memory version of tap+vhost_net to support a userspace switch?

2013-01-14 Thread Luke Gorrie
Hello, I am writing on behalf of a newly developing open source project called Snabb Switch (http://snabb.co/snabbswitch/). This is a Lua-scriptable userspace vswitch that's designed for performance, as funny as that may sound at first :-). The goal is to have an über-extensible ethernet fabric

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:38:55PM +, Christoffer Dall wrote: Targets KVM support for Cortex A-15 processors. Contains all the framework components, make files, header files, some tracing functionality, and basic user space API. Only supported core is Cortex-A15 for now. Most

Re: [PATCH v5 04/14] KVM: ARM: Hypervisor initialization

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:39:03PM +, Christoffer Dall wrote: Sets up KVM code to handle all exceptions taken to Hyp mode. When the kernel is booted in Hyp mode, calling an hvc instruction with r0 pointing to the new vectors, the HVBAR is changed to the the vector pointers. This allows

Re: [PATCH v5 2/4] ARM: KVM: arch_timers: Add guest timer core support

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:43:20PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add some the architected timer related infrastructure, and support timer interrupt injection, which can happen as a resultof three possible events: - The virtual timer interrupt has

Re: [PATCH v5 3/4] ARM: KVM: arch_timers: Add timer world switch

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:43:27PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Do the necessary save/restore dance for the timers in the world switch code. In the process, allow the guest to read the physical counter, which is useful for its own clock_event_device.

Re: [PATCH v5 04/12] ARM: KVM: Initial VGIC infrastructure code

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:41:51PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Wire the basic framework code for VGIC support and the initial in-kernel MMIO support code for the VGIC, used for the distributor emulation. [...] +/** + * vgic_reg_access - access

Re: [PATCH v5 06/12] ARM: KVM: VGIC distributor handling

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:42:04PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add the GIC distributor emulation code. A number of the GIC features are simply ignored as they are not required to boot a Linux guest. Signed-off-by: Marc Zyngier marc.zyng...@arm.com

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:09 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:38:55PM +, Christoffer Dall wrote: Targets KVM support for Cortex A-15 processors. Contains all the framework components, make files, header files, some tracing functionality, and basic

Re: [PATCH v5 07/12] ARM: KVM: VGIC virtual CPU interface management

2013-01-14 Thread Will Deacon
On Tue, Jan 08, 2013 at 06:42:11PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add VGIC virtual CPU interface code, picking pending interrupts from the distributor and stashing them in the VGIC control interface list registers. Signed-off-by: Marc Zyngier

Re: [PATCH v5 00/14] KVM/ARM Implementation

2013-01-14 Thread Will Deacon
Hi Christoffer, On Tue, Jan 08, 2013 at 06:38:34PM +, Christoffer Dall wrote: The following series implements KVM support for ARM processors, specifically on the Cortex-A15 platform. [...] This is looking pretty good to me now and I feel that the longer it stays out-of-tree, the more

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Russell King - ARM Linux
On Tue, Jan 08, 2013 at 01:38:48PM -0500, Christoffer Dall wrote: + pr_info(Setting up static %sidentity map for 0x%llx - 0x%llx\n, + prot ? HYP : , + (long long)addr, (long long)end); There's no point using 0x%llx and casting to 64-bit longs if the arguments are

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Russell King - ARM Linux
On Tue, Jan 08, 2013 at 01:38:55PM -0500, Christoffer Dall wrote: + /* -ENOENT for unknown features, -EINVAL for invalid combinations. */ + for (i = 0; i sizeof(init-features)*8; i++) { + if (init-features[i / 32] (1 (i % 32))) { Isn't this an open-coded version of

Re: [PATCH v5 08/14] KVM: ARM: Emulation framework and CP15 emulation

2013-01-14 Thread Russell King - ARM Linux
On Tue, Jan 08, 2013 at 01:39:31PM -0500, Christoffer Dall wrote: + /* + * Check whether this vcpu requires the cache to be flushed on + * this physical CPU. This is a consequence of doing dcache + * operations by set/way on this vcpu. We do it here to be in + * a

Re: [PATCH v5 04/14] KVM: ARM: Hypervisor initialization

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:11 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:39:03PM +, Christoffer Dall wrote: Sets up KVM code to handle all exceptions taken to Hyp mode. When the kernel is booted in Hyp mode, calling an hvc instruction with r0 pointing to the new

Re: [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Russell King - ARM Linux
On Tue, Jan 08, 2013 at 01:40:05PM -0500, Christoffer Dall wrote: diff --git a/arch/arm/kvm/decode.c b/arch/arm/kvm/decode.c new file mode 100644 index 000..469cf14 --- /dev/null +++ b/arch/arm/kvm/decode.c @@ -0,0 +1,462 @@ +/* + * Copyright (C) 2012 - Virtual Open Systems and

Re: [PATCH v5 02/14] ARM: Section based HYP idmap

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 11:13 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Jan 08, 2013 at 01:38:48PM -0500, Christoffer Dall wrote: + pr_info(Setting up static %sidentity map for 0x%llx - 0x%llx\n, + prot ? HYP : , + (long long)addr, (long

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 11:24 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Jan 08, 2013 at 01:38:55PM -0500, Christoffer Dall wrote: + /* -ENOENT for unknown features, -EINVAL for invalid combinations. */ + for (i = 0; i sizeof(init-features)*8; i++) { +

Re: [PATCH v5 08/14] KVM: ARM: Emulation framework and CP15 emulation

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 11:36 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Jan 08, 2013 at 01:39:31PM -0500, Christoffer Dall wrote: + /* + * Check whether this vcpu requires the cache to be flushed on + * this physical CPU. This is a consequence of doing

Re: [PATCH v5 3/4] ARM: KVM: arch_timers: Add timer world switch

2013-01-14 Thread Marc Zyngier
On 14/01/13 15:21, Will Deacon wrote: On Tue, Jan 08, 2013 at 06:43:27PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Do the necessary save/restore dance for the timers in the world switch code. In the process, allow the guest to read the physical counter, which is

BSoD, HAL.DLL

2013-01-14 Thread Sean Kennedy
I continue to get bluescreens on an XP virtual machine running on CentOS 6.3 x86_64 using VirtIO drivers. QEMU: qemu-kvm-0.12.1.2-2.295.el6.x86_64 KERNEL: Linux vmhost1 2.6.32-279.1.1.el6.x86_64 #1 SMP Tue Jul 10 13:47:21 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux VIRSH DUMPXML APOLLO: domain

Re: qemu-kvm hangs at start up under 3.8.0-rc3-00074-gb719f43 (works with CONFIG_LOCKDEP)

2013-01-14 Thread Andrew Clayton
On Mon, 14 Jan 2013 15:27:36 +0200, Gleb Natapov wrote: On Sun, Jan 13, 2013 at 10:29:58PM +, Andrew Clayton wrote: When running qemu-kvm under 64but Fedora 16 under current 3.8, it just hangs at start up. Dong a ps -ef hangs the process at the point where it would display the qemu

Re: [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 11:43 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Jan 08, 2013 at 01:40:05PM -0500, Christoffer Dall wrote: diff --git a/arch/arm/kvm/decode.c b/arch/arm/kvm/decode.c new file mode 100644 index 000..469cf14 --- /dev/null +++

Re: [PATCH v5 08/14] KVM: ARM: Emulation framework and CP15 emulation

2013-01-14 Thread Russell King - ARM Linux
On Mon, Jan 14, 2013 at 12:38:17PM -0500, Christoffer Dall wrote: On Mon, Jan 14, 2013 at 11:36 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Jan 08, 2013 at 01:39:31PM -0500, Christoffer Dall wrote: + /* + * Check whether this vcpu requires the cache to be

Re: [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Russell King - ARM Linux
On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction decoding within arch/arm is quite the heavy task, and requires agreeing on some canonical API that people can live with and it will likely take a long time. I seem to recall there were also

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Gleb Natapov
A couple of general question about ABI. If they were already answered just refer me to the previous discussion. On Tue, Jan 08, 2013 at 01:38:55PM -0500, Christoffer Dall wrote: diff --git a/Documentation/virtual/kvm/api.txt b/Documentation/virtual/kvm/api.txt index a4df553..4237c27 100644

Re: [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 06:43:19PM +, Russell King - ARM Linux wrote: On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction decoding within arch/arm is quite the heavy task, and requires agreeing on some canonical API that people can live

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Alexander Graf
On 01/14/2013 07:50 PM, Will Deacon wrote: On Mon, Jan 14, 2013 at 06:43:19PM +, Russell King - ARM Linux wrote: On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction decoding within arch/arm is quite the heavy task, and requires agreeing on

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 1:53 PM, Alexander Graf ag...@suse.de wrote: On 01/14/2013 07:50 PM, Will Deacon wrote: On Mon, Jan 14, 2013 at 06:43:19PM +, Russell King - ARM Linux wrote: On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction

Re: qemu-kvm hangs at start up under 3.8.0-rc3-00074-gb719f43 (works with CONFIG_LOCKDEP)

2013-01-14 Thread Gleb Natapov
Copying linux-mm. On Mon, Jan 14, 2013 at 06:24:49PM +, Andrew Clayton wrote: On Mon, 14 Jan 2013 15:27:36 +0200, Gleb Natapov wrote: On Sun, Jan 13, 2013 at 10:29:58PM +, Andrew Clayton wrote: When running qemu-kvm under 64but Fedora 16 under current 3.8, it just hangs at

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: On 01/14/2013 07:50 PM, Will Deacon wrote: FWIW, KVM only needs this code for handling complex MMIO instructions, which aren't even generated by recent guest kernels. I'm inclined to suggest removing this emulation code from

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon will.dea...@arm.com wrote: On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: On 01/14/2013 07:50 PM, Will Deacon wrote: FWIW, KVM only needs this code for handling complex MMIO instructions, which aren't even generated by recent

Re: [PATCH v5 2/4] ARM: KVM: arch_timers: Add guest timer core support

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:18 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:43:20PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add some the architected timer related infrastructure, and support timer interrupt injection, which can happen as

Re: [PATCH 00/12] Multiqueue virtio-net

2013-01-14 Thread Anthony Liguori
Jason Wang jasow...@redhat.com writes: Hello all: This seires is an update of last version of multiqueue virtio-net support. Recently, linux tap gets multiqueue support. This series implements basic support for multiqueue tap, nic and vhost. Then use it as an infrastructure to enable the

Re: [PATCH v5 04/12] ARM: KVM: Initial VGIC infrastructure code

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:31 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:41:51PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Wire the basic framework code for VGIC support and the initial in-kernel MMIO support code for the VGIC, used for

Re: [kvmarm] [PATCH v5 04/12] ARM: KVM: Initial VGIC infrastructure code

2013-01-14 Thread Alexander Graf
Am 14.01.2013 um 22:08 schrieb Christoffer Dall c.d...@virtualopensystems.com: On Mon, Jan 14, 2013 at 10:31 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:41:51PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Wire the basic framework code

[PATCH] s390: kvm/sigp.c: fix memory leakage

2013-01-14 Thread Cong Ding
the variable inti should be freed in the branch CPUSTAT_STOPPED. Signed-off-by: Cong Ding ding...@gmail.com --- arch/s390/kvm/sigp.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/s390/kvm/sigp.c b/arch/s390/kvm/sigp.c index 461e841..1c48ab2 100644 ---

Re: [PATCH v5 06/12] ARM: KVM: VGIC distributor handling

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:39 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:42:04PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add the GIC distributor emulation code. A number of the GIC features are simply ignored as they are not required

Re: [PATCH v5 07/12] ARM: KVM: VGIC virtual CPU interface management

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 10:42 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:42:11PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Add VGIC virtual CPU interface code, picking pending interrupts from the distributor and stashing them in the VGIC

Re: [PATCH v5 3/4] ARM: KVM: arch_timers: Add timer world switch

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 12:51 PM, Marc Zyngier marc.zyng...@arm.com wrote: On 14/01/13 15:21, Will Deacon wrote: On Tue, Jan 08, 2013 at 06:43:27PM +, Christoffer Dall wrote: From: Marc Zyngier marc.zyng...@arm.com Do the necessary save/restore dance for the timers in the world switch

Re: [PATCH v5 03/14] KVM: ARM: Initial skeleton to compile KVM support

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 1:49 PM, Gleb Natapov g...@redhat.com wrote: A couple of general question about ABI. If they were already answered just refer me to the previous discussion. On Tue, Jan 08, 2013 at 01:38:55PM -0500, Christoffer Dall wrote: diff --git a/Documentation/virtual/kvm/api.txt

Re: [PATCH v5 3/4] ARM: KVM: arch_timers: Add timer world switch

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 10:08:39PM +, Christoffer Dall wrote: can't we also get rid of the isb on the return path then? do you agree with this patch: diff --git a/arch/arm/kvm/interrupts_head.S b/arch/arm/kvm/interrupts_head.S index 57cfa84..7e6eedf 100644 ---

Re: [PATCH v5 00/14] KVM/ARM Implementation

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 11:00 AM, Will Deacon will.dea...@arm.com wrote: Hi Christoffer, On Tue, Jan 08, 2013 at 06:38:34PM +, Christoffer Dall wrote: The following series implements KVM support for ARM processors, specifically on the Cortex-A15 platform. [...] This is looking pretty

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon will.dea...@arm.com wrote: On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: On 01/14/2013 07:50 PM, Will Deacon wrote: FWIW, KVM only needs this code for handling

Re: [PATCH v5 04/12] ARM: KVM: Initial VGIC infrastructure code

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 09:08:54PM +, Christoffer Dall wrote: On Mon, Jan 14, 2013 at 10:31 AM, Will Deacon will.dea...@arm.com wrote: On Tue, Jan 08, 2013 at 06:41:51PM +, Christoffer Dall wrote: + case ACCESS_READ_VALUE: + *((u32 *)mmio-data) =

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 5:36 PM, Will Deacon will.dea...@arm.com wrote: On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon will.dea...@arm.com wrote: On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: On 01/14/2013

RE: [PATCH v10 2/3] x86, apicv: add virtual x2apic support

2013-01-14 Thread Zhang, Yang Z
Attached patch is modified according your comments to use 4 msr bitmaps. Please help to review it. Also, is there any comments for the other part of this series? If yes, welcome. :) Gleb Natapov wrote on 2013-01-14: On Mon, Jan 14, 2013 at 11:25:45AM +, Zhang, Yang Z wrote: Gleb Natapov

[PATCH] vfio-pci: Fix buffer overfill

2013-01-14 Thread Alex Williamson
A read from a range hidden from the user (ex. MSI-X vector table) attempts to fill the user buffer up to the end of the excluded range instead of up to the requested count. Fix it. Signed-off-by: Alex Williamson alex.william...@redhat.com Cc: sta...@vger.kernel.org ---

[PATCH 0/3] vfio-pci: VGA and legacy MMIO I/O port access

2013-01-14 Thread Alex Williamson
This series includes a couple cleanups that I'll target for 3.9 plus RFC patch 3/3 that adds device regions for legacy I/O and makes use of the VGA arbiter for enabling VGA assignment. I know the POWER guys don't likely care about VGA support, but Alexey, Ben, and David, I'd appreciate any

[PATCH 1/3] vfio-pci: Cleanup read/write functions

2013-01-14 Thread Alex Williamson
The read and write functions are nearly identical, combine them and convert to a switch statement. This also makes it easy to narrow the scope of when we use the io/mem accessors in case new regions are added. Signed-off-by: Alex Williamson alex.william...@redhat.com ---

[PATCH 2/3] vfio-pci: Cleanup BAR access

2013-01-14 Thread Alex Williamson
We can actually handle MMIO and I/O port from the same access function since PCI already does abstraction of this. The ROM BAR only requires a minor difference, so it gets included too. vfio_pci_config_readwrite gets renamed for consistency. Signed-off-by: Alex Williamson

[RFC PATCH 0/2] qemu vfo-pci: VGA support + initial round of device quirks

2013-01-14 Thread Alex Williamson
This series make use of the recently posted kernel-side vfio-pci support for accessing VGA ranges. This enables a handful of devices to be used as the primary (only) VM console. See patch 2 for a list of devices. These show seabios text mode output and work with standard VGA drivers.

[RFC PATCH 1/2] qemu vfio-pci: Add support for VGA MMIO and I/O port access

2013-01-14 Thread Alex Williamson
Most VGA cards need some kind of quirk to fully operate since they hide backdoors to get to other registers outside of PCI config space within the registers, but this provides the base infrastructure. If we could identity map PCI resources for assigned devices we would need a lot fewer quirks.

[RFC PATCH 2/2] qemu vfio-pci: Graphics device quirks

2013-01-14 Thread Alex Williamson
Apparently graphics vendors need to come up with new ways to retrieve PCI BAR addresses on every revision of their chip. These are the ones that I've found on the following assortment of cards: Advanced Micro Devices [AMD] nee ATI Cedar PRO [Radeon HD 5450/6350] Advanced Micro Devices [AMD] nee

[RFC PATCH 3/3] vfio-pci: Add support for legacy MMIO I/O port towards VGA support

2013-01-14 Thread Alex Williamson
Create two new legacy regions, one for MMIO space below 1MB and another for 64k of I/O port space. For devices of PCI class VGA these ranges will be exposed and allow direct access to the device at the PCI defined VGA addresses, 0xa, 0x3b0, 0x3c0. VFIO makes use of the host VGA arbiter to

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 10:36:38PM +, Will Deacon wrote: On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon will.dea...@arm.com wrote: On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: On 01/14/2013 07:50 PM,

Re: [PATCH] access: add test for dirty bit tracking if CR0.WP = 0

2013-01-14 Thread Gleb Natapov
On Sat, Dec 15, 2012 at 03:03:32PM +0800, Xiao Guangrong wrote: If the write-fault access is from supervisor and CR0.WP is not set on the vcpu, kvm will fix it by adjusting pte access - it sets the W bit on pte and clears U bit. This is the chance that kvm can change pte access from readonly