Re: KVM guest 100% cpu lock up

2013-03-27 Thread Phil Daws
This problem still persists :( I have read a few articles about setting the clocksource to acpi_pm and the guest was stable for a couple of days, then low and behold the CPU on the guest started to climb too 100% and completely froze. Only way to recover it was hard reset on the guest machine

Re: [PATCH V2 1/3] tcm_vhost: Wait for pending requests in vhost_scsi_flush()

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 11:01:34AM +0800, Asias He wrote: > On Tue, Mar 26, 2013 at 10:51:54PM +0200, Michael S. Tsirkin wrote: > > On Tue, Mar 26, 2013 at 10:38:27AM +0800, Asias He wrote: > > > On Mon, Mar 25, 2013 at 01:13:39PM +0200, Michael S. Tsirkin wrote: > > > > On Mon, Mar 25, 2013 at 03:

Re: [PATCH v2 01/30] arm64: add explicit symbols to ESR_EL1 decoding

2013-03-27 Thread Catalin Marinas
On Tue, Mar 26, 2013 at 05:00:56PM +, Marc Zyngier wrote: > --- a/arch/arm64/kernel/entry.S > +++ b/arch/arm64/kernel/entry.S > @@ -24,6 +24,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -239,18 +240,18 @@ ENDPROC(el1_error_invalid) > el1_

Re: [PATCH v2 01/30] arm64: add explicit symbols to ESR_EL1 decoding

2013-03-27 Thread Marc Zyngier
On 27/03/13 13:51, Catalin Marinas wrote: > On Tue, Mar 26, 2013 at 05:00:56PM +, Marc Zyngier wrote: >> --- a/arch/arm64/kernel/entry.S >> +++ b/arch/arm64/kernel/entry.S >> @@ -24,6 +24,7 @@ >> #include >> #include >> #include >> +#include >> #include >> #include >> #include >> @

Re: [PATCH v2 02/30] arm64: KVM: define HYP and Stage-2 translation page flags

2013-03-27 Thread Catalin Marinas
On Tue, Mar 26, 2013 at 05:00:57PM +, Marc Zyngier wrote: > --- a/arch/arm64/mm/mmu.c > +++ b/arch/arm64/mm/mmu.c > @@ -44,6 +44,7 @@ struct page *empty_zero_page; > EXPORT_SYMBOL(empty_zero_page); > > pgprot_t pgprot_default; > +pgprot_t pgprot_device; > EXPORT_SYMBOL(pgprot_default); >

Re: [PATCH v2 02/30] arm64: KVM: define HYP and Stage-2 translation page flags

2013-03-27 Thread Marc Zyngier
On 27/03/13 14:11, Catalin Marinas wrote: > On Tue, Mar 26, 2013 at 05:00:57PM +, Marc Zyngier wrote: >> --- a/arch/arm64/mm/mmu.c >> +++ b/arch/arm64/mm/mmu.c >> @@ -44,6 +44,7 @@ struct page *empty_zero_page; >> EXPORT_SYMBOL(empty_zero_page); >> >> pgprot_t pgprot_default; >> +pgprot_t p

Re: [PATCH 1/7 v2] KVM: PPC: e500: Expose MMU registers via ONE_REG

2013-03-27 Thread Alexander Graf
On 26.03.2013, at 23:05, Mihai Caraman wrote: > MMU registers were exposed to user-space using sregs interface. Add them > to ONE_REG interface and use kvmppc_get_one_reg/kvmppc_set_one_reg delegation > interface introduced by book3s. > > Signed-off-by: Mihai Caraman > --- > v2: > - Restrict se

Re: [PATCH 4/7 v2] KVM: PPC: e500: Add support for EPTCFG register

2013-03-27 Thread Alexander Graf
On 26.03.2013, at 23:05, Mihai Caraman wrote: > EPTCFG register defined by E.PT is accessed unconditionally by Linux guests > in the presence of MAV 2.0. Support it now. > > Signed-off-by: Mihai Caraman > --- > v2: > - Use has_feature() function > > Documentation/virtual/kvm/api.txt |1 +

Re: [PATCH 5/7 v2] KVM: PPC: e500: Remove E.PT and E.HV.LRAT categories from VCPUs

2013-03-27 Thread Alexander Graf
On 26.03.2013, at 23:05, Mihai Caraman wrote: > Embedded.Page Table (E.PT) category in VMs requires indirect tlb entries > emulation which is not supported yet. Configure TLBnCFG to remove E.PT > and E.HV.LRAT categories from VCPUs. > > Signed-off-by: Mihai Caraman > --- > v2: > - Remove E.HV.L

Re: [PATCH 5/7 v2] KVM: PPC: e500: Remove E.PT and E.HV.LRAT categories from VCPUs

2013-03-27 Thread Scott Wood
On 03/26/2013 05:05:10 PM, Mihai Caraman wrote: Embedded.Page Table (E.PT) category in VMs requires indirect tlb entries emulation which is not supported yet. Configure TLBnCFG to remove E.PT and E.HV.LRAT categories from VCPUs. Signed-off-by: Mihai Caraman --- v2: - Remove E.HV.LRAT from vc

Re: [PATCH] ARM: KVM: promote vfp_host pointer to generic host cpu context

2013-03-27 Thread Marc Zyngier
On Tue, 26 Mar 2013 16:37:22 +, Marc Zyngier wrote: > We use the vfp_host pointer to store the host VFP context, should > the guest start using VFP itself. > > Actually, we can use this pointer in a more generic way to store > CPU speficic data, and arm64 is using it to dump the whole host >

Re: Asus P8B75-B and Intel I3-3220T IOMMU

2013-03-27 Thread Matthias Fechner
Hi, Am 17.03.2013 15:17, schrieb Matthias Fechner: > If I replace the CPU by a Intel Core i5-3550 do you think the IOMMU will > work then with the mainboard? for the archive, I replaced the i3 now by an i5-3550 and IOMMU works perfectly. Thanks again. Matthias -- To unsubscribe from this list:

Re: [PATCH 2/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Nicholas A. Bellinger
On Mon, 2013-03-25 at 13:16 +0200, Michael S. Tsirkin wrote: > On Fri, Mar 22, 2013 at 10:52:28AM +0800, Asias He wrote: > > This patch fixes guest hang when booting seabios and guest. > > > > [0.576238] scsi0 : Virtio SCSI HBA > > [0.616754] virtio_scsi virtio1: request:id 0 is not a

Re: [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-27 Thread Nicholas A. Bellinger
On Wed, 2013-03-20 at 11:51 +0200, Michael S. Tsirkin wrote: > On Tue, Mar 19, 2013 at 06:57:08PM -0700, Nicholas A. Bellinger wrote: > > On Tue, 2013-03-19 at 09:40 +0100, Stefan Hajnoczi wrote: > > > On Tue, Mar 19, 2013 at 08:34:45AM +0800, Asias He wrote: > > > > --- > > > > hw/vhost.c | 2 ++

Re: [PATCH 2/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 01:29:15PM -0700, Nicholas A. Bellinger wrote: > On Mon, 2013-03-25 at 13:16 +0200, Michael S. Tsirkin wrote: > > On Fri, Mar 22, 2013 at 10:52:28AM +0800, Asias He wrote: > > > This patch fixes guest hang when booting seabios and guest. > > > > > > [0.576238] scsi0 :

Re: [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 02:31:27PM -0700, Nicholas A. Bellinger wrote: > On Wed, 2013-03-20 at 11:51 +0200, Michael S. Tsirkin wrote: > > On Tue, Mar 19, 2013 at 06:57:08PM -0700, Nicholas A. Bellinger wrote: > > > On Tue, 2013-03-19 at 09:40 +0100, Stefan Hajnoczi wrote: > > > > On Tue, Mar 19, 20

[PATCH 0/3] Add support for vhost-scsi-pci

2013-03-27 Thread Nicholas A. Bellinger
From: Nicholas Bellinger Hi QEMU folks, The following is the patch series to support vhost-scsi-pci within the upstream QEMU tree. This includes the refactoring of existing virtio-scsi code from Paolo to allow a VirtIOSCSICommon structure that is shared amoungst existing virtio-scsi-pci device

[PATCH 3/3] vhost: Only call vhost_verify_ring_mappings when adding vhost memory

2013-03-27 Thread Nicholas A. Bellinger
From: Nicholas Bellinger This patch fixes a bug where vhost_verify_ring_mappings() could be called after vhost_dev_unassign_memory() has been called during vhost_set_memory(..., add=false). This appears to be fallout from when dev->started = true assignment occurs before vhost_set_memory(..., ad

[PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
From: Paolo Bonzini The WWPN specified in configfs is passed to "-device vhost-scsi-pci". The tgpt field of the SET_ENDPOINT ioctl is obsolete now, so it is not available from the QEMU command-line. Instead, I hardcode it to zero. Changes in V4: - Set event_idx=off by default (nab, thanks as

[PATCH 1/3] virtio-scsi: create VirtIOSCSICommon

2013-03-27 Thread Nicholas A. Bellinger
From: Paolo Bonzini This patch refactors existing virtio-scsi code into VirtIOSCSICommon in order to allow virtio_scsi_init_common() to be used by both internal virtio_scsi_init() and external vhost-scsi-pci code. Signed-off-by: Paolo Bonzini Cc: Michael S. Tsirkin Cc: Asias He Signed-off-by:

Re: [PATCH 3/3] vhost: Only call vhost_verify_ring_mappings when adding vhost memory

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 09:59:46PM +, Nicholas A. Bellinger wrote: > From: Nicholas Bellinger > > This patch fixes a bug where vhost_verify_ring_mappings() could be > called after vhost_dev_unassign_memory() has been called during > vhost_set_memory(..., add=false). > Sorry, I NAK'd this al

Re: [PATCH 0/3] Add support for vhost-scsi-pci

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 09:59:43PM +, Nicholas A. Bellinger wrote: > From: Nicholas Bellinger > > Hi QEMU folks, > > The following is the patch series to support vhost-scsi-pci within the > upstream QEMU tree. > > This includes the refactoring of existing virtio-scsi code from Paolo > to al

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 09:59:45PM +, Nicholas A. Bellinger wrote: > From: Paolo Bonzini > > The WWPN specified in configfs is passed to "-device vhost-scsi-pci". > The tgpt field of the SET_ENDPOINT ioctl is obsolete now, so it is not > available from the QEMU command-line. Instead, I hardc

Re: [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-27 Thread Nicholas A. Bellinger
On Wed, 2013-03-27 at 23:56 +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 02:31:27PM -0700, Nicholas A. Bellinger wrote: > > On Wed, 2013-03-20 at 11:51 +0200, Michael S. Tsirkin wrote: > > > On Tue, Mar 19, 2013 at 06:57:08PM -0700, Nicholas A. Bellinger wrote: > > > > On Tue, 2013-03

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
On Thu, 2013-03-28 at 00:28 +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 09:59:45PM +, Nicholas A. Bellinger wrote: > > From: Paolo Bonzini > > > > The WWPN specified in configfs is passed to "-device vhost-scsi-pci". > > The tgpt field of the SET_ENDPOINT ioctl is obsolete now,

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > On Thu, 2013-03-28 at 00:28 +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 27, 2013 at 09:59:45PM +, Nicholas A. Bellinger wrote: > > > From: Paolo Bonzini > > > > > > The WWPN specified in configfs is passed to "-devi

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Michael S. Tsirkin
On Thu, Mar 28, 2013 at 12:50:21AM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > > On Thu, 2013-03-28 at 00:28 +0200, Michael S. Tsirkin wrote: > > > On Wed, Mar 27, 2013 at 09:59:45PM +, Nicholas A. Bellinger wrote: > > > > From: Pa

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
On Thu, 2013-03-28 at 00:50 +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > > On Thu, 2013-03-28 at 00:28 +0200, Michael S. Tsirkin wrote: > > > On Wed, Mar 27, 2013 at 09:59:45PM +, Nicholas A. Bellinger wrote: > > > > From: Paolo Bon

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
On Thu, 2013-03-28 at 00:53 +0200, Michael S. Tsirkin wrote: > On Thu, Mar 28, 2013 at 12:50:21AM +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > > > On Thu, 2013-03-28 at 00:28 +0200, Michael S. Tsirkin wrote: > > > > On Wed, Mar 27, 20

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
On Wed, 2013-03-27 at 16:24 -0700, Nicholas A. Bellinger wrote: > On Thu, 2013-03-28 at 00:53 +0200, Michael S. Tsirkin wrote: > > On Thu, Mar 28, 2013 at 12:50:21AM +0200, Michael S. Tsirkin wrote: > > > On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > > > > On Thu, 2013-03

[PATCH] tcm_vhost: Avoid VIRTIO_RING_F_EVENT_IDX feature bit

2013-03-27 Thread Nicholas A. Bellinger
From: Nicholas Bellinger This patch adds a VHOST_TCM_FEATURES mask minus VIRTIO_RING_F_EVENT_IDX so that vhost-scsi-pci userspace will strip this feature bit once GET_FEATURES reports it as being unsupported on the host. This is to avoid a bug where ->handle_kicks() are missed when EVENT_IDX is

[PATCH V2 0/2] tcm_vhost endpoint

2013-03-27 Thread Asias He
Reordered the patch, so patch 1 can go to v3.9. Asias He (2): tcm_vhost: Initialize vq->last_used_idx when set endpoint tcm_vhost: Use vq->private_data to indicate if the endpoint is setup drivers/vhost/tcm_vhost.c | 47 +-- 1 file changed, 41 inse

[PATCH V2 1/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Asias He
This patch fixes guest hang when booting seabios and guest. [0.576238] scsi0 : Virtio SCSI HBA [0.616754] virtio_scsi virtio1: request:id 0 is not a head! vq->last_used_idx is initialized only when /dev/vhost-scsi is opened or closed. vhost_scsi_open -> vhost_dev_init() -> vhost_v

[PATCH V2 2/2] tcm_vhost: Use vq->private_data to indicate if the endpoint is setup

2013-03-27 Thread Asias He
Currently, vs->vs_endpoint is used indicate if the endpoint is setup or not. It is set or cleared in vhost_scsi_set_endpoint() or vhost_scsi_clear_endpoint() under the vs->dev.mutex lock. However, when we check it in vhost_scsi_handle_vq(), we ignored the lock. Instead of using the vs->vs_endpoint

Re: [PATCH 2/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Asias He
On Wed, Mar 27, 2013 at 11:45:59PM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 01:29:15PM -0700, Nicholas A. Bellinger wrote: > > On Mon, 2013-03-25 at 13:16 +0200, Michael S. Tsirkin wrote: > > > On Fri, Mar 22, 2013 at 10:52:28AM +0800, Asias He wrote: > > > > This patch fixes gues

Re: [PATCH V7 0/5] virtio-scsi multiqueue

2013-03-27 Thread Wanlong Gao
On 03/23/2013 07:28 PM, Wanlong Gao wrote: > This series implements virtio-scsi queue steering, which gives > performance improvements of up to 50% (measured both with QEMU and > tcm_vhost backends). > > This version rebased on Rusty's virtio ring rework patches, which > has already gone into virt

Re: [PATCH V2 1/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Nicholas A. Bellinger
Hi Asias, On Thu, 2013-03-28 at 10:17 +0800, Asias He wrote: > This patch fixes guest hang when booting seabios and guest. > > [0.576238] scsi0 : Virtio SCSI HBA > [0.616754] virtio_scsi virtio1: request:id 0 is not a head! > > vq->last_used_idx is initialized only when /dev/vhost-sc

Re: [PATCH V2 1/2] tcm_vhost: Initialize vq->last_used_idx when set endpoint

2013-03-27 Thread Asias He
On Wed, Mar 27, 2013 at 07:54:07PM -0700, Nicholas A. Bellinger wrote: > Hi Asias, > > On Thu, 2013-03-28 at 10:17 +0800, Asias He wrote: > > This patch fixes guest hang when booting seabios and guest. > > > > [0.576238] scsi0 : Virtio SCSI HBA > > [0.616754] virtio_scsi virtio1: requ

Re: [PATCH] tcm_vhost: Avoid VIRTIO_RING_F_EVENT_IDX feature bit

2013-03-27 Thread Michael S. Tsirkin
On Thu, Mar 28, 2013 at 12:27:10AM +, Nicholas A. Bellinger wrote: > From: Nicholas Bellinger > > This patch adds a VHOST_TCM_FEATURES mask minus VIRTIO_RING_F_EVENT_IDX > so that vhost-scsi-pci userspace will strip this feature bit once > GET_FEATURES reports it as being unsupported on the h

Re: [PATCH V2 2/2] tcm_vhost: Use vq->private_data to indicate if the endpoint is setup

2013-03-27 Thread Michael S. Tsirkin
On Thu, Mar 28, 2013 at 10:17:28AM +0800, Asias He wrote: > Currently, vs->vs_endpoint is used indicate if the endpoint is setup or > not. It is set or cleared in vhost_scsi_set_endpoint() or > vhost_scsi_clear_endpoint() under the vs->dev.mutex lock. However, when > we check it in vhost_scsi_handl

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Michael S. Tsirkin
On Wed, Mar 27, 2013 at 04:24:22PM -0700, Nicholas A. Bellinger wrote: > On Thu, 2013-03-28 at 00:53 +0200, Michael S. Tsirkin wrote: > > On Thu, Mar 28, 2013 at 12:50:21AM +0200, Michael S. Tsirkin wrote: > > > On Wed, Mar 27, 2013 at 03:45:42PM -0700, Nicholas A. Bellinger wrote: > > > > On Thu,

Re: [PATCH] tcm_vhost: Avoid VIRTIO_RING_F_EVENT_IDX feature bit

2013-03-27 Thread Nicholas A. Bellinger
On Thu, 2013-03-28 at 08:04 +0200, Michael S. Tsirkin wrote: > On Thu, Mar 28, 2013 at 12:27:10AM +, Nicholas A. Bellinger wrote: > > From: Nicholas Bellinger > > > > This patch adds a VHOST_TCM_FEATURES mask minus VIRTIO_RING_F_EVENT_IDX > > so that vhost-scsi-pci userspace will strip this f

Re: [PATCH 2/3] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-03-27 Thread Nicholas A. Bellinger
On Thu, 2013-03-28 at 08:19 +0200, Michael S. Tsirkin wrote: > On Wed, Mar 27, 2013 at 04:24:22PM -0700, Nicholas A. Bellinger wrote: > > On Thu, 2013-03-28 at 00:53 +0200, Michael S. Tsirkin wrote: > > > On Thu, Mar 28, 2013 at 12:50:21AM +0200, Michael S. Tsirkin wrote: > > > > On Wed, Mar 27, 20

Re: [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-27 Thread Nicholas A. Bellinger
On Wed, 2013-03-27 at 15:33 -0700, Nicholas A. Bellinger wrote: > On Wed, 2013-03-27 at 23:56 +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 27, 2013 at 02:31:27PM -0700, Nicholas A. Bellinger wrote: > > > On Wed, 2013-03-20 at 11:51 +0200, Michael S. Tsirkin wrote: > > > > On Tue, Mar 19, 2013 a