KVM mce test , host and guest can't get mce information

2014-11-30 Thread Liu, RongrongX
Hi , I test mce test on kvm, according to the link : https://kernel.googlesource.com/pub/scm/utils/cpu/mce/mce-test/+/61252a69e3c16bf26adeeb6754c3714016228a18/kvm/README Kernel version:3.18.0-rc3 Qemu version: 1.1.0 + p2v.patch Mce-test source: https://kernel.googlesource.com/pub/scm/utils/cpu/m

Re: [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h

2014-11-30 Thread Jason Wang
On Sun, Nov 30, 2014 at 1:04 PM, David Miller wrote: From: Jason Wang Date: Thu, 27 Nov 2014 14:41:21 +0800 Signed-off-by: Jason Wang I don't think generic vhost patches should go via my tree. If you disagree, let me know why, thanks :) Agree. Michael, could you pls pick this into v

Re: [PATCH v7 28/46] vhost: make features 64 bit

2014-11-30 Thread Ben Hutchings
On Sun, 2014-11-30 at 18:44 +0300, Sergei Shtylyov wrote: > Hello. > > On 11/30/2014 6:11 PM, Michael S. Tsirkin wrote: > > > We need to use bit 32 for virtio 1.0 > > > Signed-off-by: Michael S. Tsirkin > > Reviewed-by: Jason Wang > > --- > > drivers/vhost/vhost.h | 4 ++-- > > 1 file chang

Re: [Qemu-devel] How does qemu know the virtual memory of the guest os?

2014-11-30 Thread Richard W.M. Jones
On Fri, Nov 28, 2014 at 04:17:10PM -0800, Jidong Xiao wrote: > Hi, > > I notice that Qemu supports dump virtual memory of Guest OS. As this > page suggests: > > > http://doc.opensuse.org/products/draft/SLES/SLES-kvm_sd_draft/cha.qemu.monitor.html > > To save the content of the virtual m

Re: [PATCH v7 28/46] vhost: make features 64 bit

2014-11-30 Thread Sergei Shtylyov
Hello. On 11/30/2014 6:11 PM, Michael S. Tsirkin wrote: We need to use bit 32 for virtio 1.0 Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang --- drivers/vhost/vhost.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/vhost/vhost.h b/drivers/vhos

[PATCH v7 28/46] vhost: make features 64 bit

2014-11-30 Thread Michael S. Tsirkin
We need to use bit 32 for virtio 1.0 Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang --- drivers/vhost/vhost.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h index 3eda654..c624b09 100644 --- a/drivers/vhost/vhost.

[PATCH v7 29/46] vhost: add memory access wrappers

2014-11-30 Thread Michael S. Tsirkin
Add guest memory access wrappers to handle virtio endianness conversions. Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang --- drivers/vhost/vhost.h | 31 +++ 1 file changed, 31 insertions(+) diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h index

[PATCH v7 30/46] vhost/net: force len for TX to host endian

2014-11-30 Thread Michael S. Tsirkin
vhost/net keeps a copy of some used ring but (ab)uses length field for internal house-keeping. This works because for tx used length is always 0. Suppress sparse errors: we use native endian-ness internally but never expose it to guest. Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang -

[PATCH v7 33/46] vhost/net: larger header for virtio 1.0

2014-11-30 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang --- drivers/vhost/net.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index cae22f9..1ac58d0 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c @@ -1027,7 +1027,8 @

[PATCH v7 31/46] vhost: virtio 1.0 endian-ness support

2014-11-30 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- drivers/vhost/vhost.c | 93 +++ 1 file changed, 56 insertions(+), 37 deletions(-) diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c index c90f437..4d379ed 100644 --- a/drivers/vhost/vhost.c +++ b/drive

[PATCH v7 32/46] vhost/net: virtio 1.0 byte swap

2014-11-30 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- drivers/vhost/net.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index dce5c58..cae22f9 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c @@ -416,7 +416,7 @@ static void ha

[PATCH v7 35/46] vhost/net: enable virtio 1.0

2014-11-30 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- drivers/vhost/net.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index 1ac58d0..984242e 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c @@ -61,7 +61,8 @@ MODULE_PARM_DESC(experiment

[PATCH v7 36/46] vhost/net: suppress compiler warning

2014-11-30 Thread Michael S. Tsirkin
len is always initialized since function is called with size > 0. Signed-off-by: Michael S. Tsirkin --- drivers/vhost/net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index 984242e..54ffbb0 100644 --- a/drivers/vhost/net.c +++ b/

[PATCH v7 45/46] vhost/scsi: partial virtio 1.0 support

2014-11-30 Thread Michael S. Tsirkin
Include all endian conversions as required by virtio 1.0. Don't set virtio 1.0 yet, since that requires ANY_LAYOUT which we don't yet support. Signed-off-by: Michael S. Tsirkin Acked-by: Paolo Bonzini --- drivers/vhost/scsi.c | 22 -- 1 file changed, 12 insertions(+), 10 del

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Chen Gang
On 11/30/14 19:08, Christoffer Dall wrote: > On Mon, Dec 01, 2014 at 10:04:32PM +0800, Chen Gang wrote: >> On 11/30/2014 06:07 PM, Christoffer Dall wrote: >>> On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: Use one looping instead of two, so can let code more simpler and get a

Re: [PATCH v3 6/8] KVM: kvm-vfio: wrapper to VFIO external API device helpers

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:58PM +0100, Eric Auger wrote: > Provide wrapper functions that allow KVM-VFIO device code to > interact with a vfio device: > - kvm_vfio_device_get_external_user gets a handle to a struct > vfio_device from the vfio device file descriptor and increments > its refer

Re: [PATCH v3 4/8] KVM: kvm-vfio: User API for IRQ forwarding

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:56PM +0100, Eric Auger wrote: > This patch adds and document a new KVM_DEV_VFIO_DEVICE group documents > and 2 device attributes: KVM_DEV_VFIO_DEVICE_FORWARD_IRQ, > KVM_DEV_VFIO_DEVICE_UNFORWARD_IRQ. The purpose is to be able > to set a VFIO device IRQ as forwarded o

Re: [PATCH v3 3/8] VFIO: platform: forwarded state tested when selecting IRQ handler

2014-11-30 Thread Christoffer Dall
The subject reads strangely, perhaps just: VFIO: platform: test forward state when selecting IRQ handler On Sun, Nov 23, 2014 at 07:35:55PM +0100, Eric Auger wrote: > In case the IRQ is forwarded, the VFIO platform IRQ handler does not > need to disable the IRQ anymore. > > When setting the IRQ

Re: [PATCH v3 2/8] KVM: arm64: Enable the KVM-VFIO device

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:54PM +0100, Eric Auger wrote: > Used by KVM-enabled VFIO-based device passthrough support in QEMU. > > Signed-off-by: Joel Schopp > Signed-off-by: Eric Auger > > --- > > Extracted from [RFC PATCH] arm64: KVM: add irqfd support > http://www.spinics.net/lists/kvm-ar

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Christoffer Dall
On Mon, Dec 01, 2014 at 10:04:32PM +0800, Chen Gang wrote: > On 11/30/2014 06:07 PM, Christoffer Dall wrote: > > On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: > >> Use one looping instead of two, so can let code more simpler and get a > >> little better performance. > >> > >> Signed-of

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Chen Gang
On 11/30/2014 06:07 PM, Christoffer Dall wrote: > On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: >> Use one looping instead of two, so can let code more simpler and get a >> little better performance. >> >> Signed-off-by: Chen Gang > > Performance? Please, this is vgic_create >

Re: [PATCH 7/7] KVM: arm64: guest debug, HW assisted debug support

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:05PM +, Alex Bennée wrote: > This adds support for userspace to control the HW debug registers for > guest debug. We'll only copy the $ARCH defined number across as that's > all that hyp.S will use anyway. I've moved some helper functions into > the hw_breakpoint.h

Re: [PATCH 6/7] KVM: arm64: re-factor hyp.S debug register code

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:04PM +, Alex Bennée wrote: > This is a pre-cursor to sharing the code with the guest debug support. > This replaces the big macro that fishes data out of a fixed location > with a more general helper macro to restore a set of debug registers. It > uses macro substi

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:03PM +, Alex Bennée wrote: > This adds support for single-stepping the guest. As userspace can and > will manipulate guest registers before restarting any tweaking of the > registers has to occur just before control is passed back to the guest. > Furthermore while

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Peter Maydell
On 30 November 2014 at 10:10, Christoffer Dall wrote: > In any case, I think it was related to how userspace observes the state > of the CPU, because when you do the MMIO operation emulation in > userspace, currently if you observe the PC though GET_ONE_REG, you'll > see a PC pointing to the next

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Christoffer Dall
On Wed, Nov 26, 2014 at 07:27:06PM +, Peter Maydell wrote: > On 25 November 2014 at 16:10, Alex Bennée wrote: > > This adds support for single-stepping the guest. As userspace can and > > will manipulate guest registers before restarting any tweaking of the > > registers has to occur just befo

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Christoffer Dall
On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: > Use one looping instead of two, so can let code more simpler and get a > little better performance. > > Signed-off-by: Chen Gang Performance? Please, this is vgic_create This does nothing else than introduce churn. -Christoffer