[PATCH 1/8] kvm tools: add init/exit automatic calls

2012-09-17 Thread Sasha Levin
This adds a method to call init/exit functions similar to the kernel's init functions. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/include/kvm/util-init.h | 51 + tools/kvm/util/init.c | 69 +++ 2

[PATCH 1/8] kvm tools: add init/exit automatic calls

2012-09-17 Thread Sasha Levin
This adds a method to call init/exit functions similar to the kernel's init functions. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/include/kvm/util-init.h | 51 + tools/kvm/util/init.c | 69 +++ 2

[PATCH 3/8] kvm tools: pass kvm ptr directly to timer injection

2012-09-17 Thread Sasha Levin
This will help us get rid of the global kvm object. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/builtin-run.c | 10 -- tools/kvm/kvm.c | 1 + 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-run.c

[PATCH 2/8] kvm tools: use init/exit where possible

2012-09-17 Thread Sasha Levin
Switch to using init/exit calls instead of the repeating call blocks in builtin-run. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/Makefile | 1 + tools/kvm/builtin-run.c | 246 +--- tools/kvm/disk/core.c | 2 +

[PATCH 4/8] kvm tools: remove global kvm object

2012-09-17 Thread Sasha Levin
This was ugly, and now we get rid of it. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/builtin-run.c| 48 +++--- tools/kvm/hw/i8042.c | 4 ++-- tools/kvm/hw/pci-shmem.c | 2 +- tools/kvm/hw/rtc.c | 10

[PATCH 5/8] kvm tools: initialize the threadpool job iterator before using

2012-09-17 Thread Sasha Levin
This would fix a bug where the exit function of the threadpool would hang if no jobs were processed yet and a request to exit was received. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/util/threadpool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 6/8] kvm tools: don't exit on debug ioport write

2012-09-17 Thread Sasha Levin
While it shouldn't happen on regular guests, we sometimes hit it when fuzzing within the guest, which would cause the lkvm process to exit - which is undesired. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/x86/ioport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 7/8] kvm tools: fix SMP

2012-09-17 Thread Sasha Levin
We accidently broke SMP when we moved mptable init to before we initialize the vcpu count, that means that we always built smptable which was not properly initialized for the given configuration. Instead of initializing mptable as part of the kvm arch initialization, let it be initialized on

[PATCH 8/8] kvm tools: fix build optimization

2012-09-17 Thread Sasha Levin
I've accidently changed optimization level to -O0 when testing one of the patches and commited that. Revert it back to -O2. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/kvm/Makefile

Re: [RFC][PATCH] Improving directed yield scalability for PLE handler

2012-09-17 Thread Andrew Jones
On Fri, Sep 14, 2012 at 04:34:24PM -0400, Konrad Rzeszutek Wilk wrote: The concern I have is that even though we have gone through changes to help reduce the candidate vcpus we yield to, we still have a very poor idea of which vcpu really needs to run. The result is high cpu usage in the

Re: [RFC][PATCH] Improving directed yield scalability for PLE handler

2012-09-17 Thread Andrew Jones
On Sun, Sep 16, 2012 at 11:55:28AM +0300, Avi Kivity wrote: On 09/14/2012 12:30 AM, Andrew Theurer wrote: The concern I have is that even though we have gone through changes to help reduce the candidate vcpus we yield to, we still have a very poor idea of which vcpu really needs to run.

[PATCH v8 0/3] KVM: perf: kvm events analysis tool

2012-09-17 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Changelog: the changes from Avi's comments: - move the definition of x86 fault vector to asm/kvm.h - drop the second patch which introduced new tracpoints the changes from David's comments: - use scnprintf instead of snprintf - drop exclusive

[PATCH v8 1/3] KVM: x86: export svm/vmx exit code and vector code to userspace

2012-09-17 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Exporting KVM exit information to userspace to be consumed by perf. [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by: Dong Hao haod...@linux.vnet.ibm.com Signed-off-by: Xiao Guangrong

[PATCH v8 3/3] KVM: perf: kvm events analysis tool

2012-09-17 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Add 'perf kvm stat' support to analyze kvm vmexit/mmio/ioport smartly Usage: - kvm stat run a command and gather performance counter statistics, it is the alias of perf stat - trace kvm events: perf kvm stat record, or, if other

[PATCH v8 2/3] perf: move stat related code to util/stat.c

2012-09-17 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Then, the code can be shared between kvm events and perf stat [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Signed-off-by: Dong Hao

Re: [Qemu-devel] Block Migration Assertion in qemu-kvm 1.2.0

2012-09-17 Thread Kevin Wolf
Am 16.09.2012 12:13, schrieb Peter Lieven: Hi, when trying to block migrate a VM from one node to another, the source VM crashed with the following assertion: block.c:3829: bdrv_set_in_use: Assertion `bs-in_use != in_use' failed. Is this sth already addresses/known? Not that I'm aware

[PATCH] kvm tools: fix SDL build error when libsdl isn't installed

2012-09-17 Thread Sasha Levin
We used wrong prototypes for sdl init/exit when libsdl wasn't installed when building. This would cause build errors. Reported-by: Kashyap Chamarthy kashyap...@gmail.com Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/include/kvm/sdl.h | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH] kvm tools: don't die if sdl wasn't compiled in and we don't try using it

2012-09-17 Thread Sasha Levin
If SDL isn't compiled in we shouldn't die unless we actually try using it. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/include/kvm/sdl.h | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tools/kvm/include/kvm/sdl.h b/tools/kvm/include/kvm/sdl.h

Re: [PATCH] kvm tools: fix SDL build error when libsdl isn't installed

2012-09-17 Thread Kashyap Chamarthy
On Mon, Sep 17, 2012 at 4:03 PM, Sasha Levin levinsasha...@gmail.com wrote: We used wrong prototypes for sdl init/exit when libsdl wasn't installed when building. This would cause build errors. Reported-by: Kashyap Chamarthy kashyap...@gmail.com Signed-off-by: Sasha Levin

RE: [PATCH 2/5]KVM:x86, apicv: adjust for virtual interrupt delivery

2012-09-17 Thread Li, Jiongxi
-Original Message- From: Li, Jiongxi Sent: Friday, September 14, 2012 10:16 PM To: 'Avi Kivity' Cc: kvm@vger.kernel.org Subject: RE: [PATCH 2/5]KVM:x86, apicv: adjust for virtual interrupt delivery Sorry for the late response -Original Message- From: Avi Kivity

RE: [PATCH 4/5]KVM:x86, apicv: add interface for poking EOI exit bitmap

2012-09-17 Thread Li, Jiongxi
-Original Message- From: Avi Kivity [mailto:a...@redhat.com] Sent: Friday, September 07, 2012 12:38 AM To: Li, Jiongxi Cc: kvm@vger.kernel.org Subject: Re: [PATCH 4/5]KVM:x86, apicv: add interface for poking EOI exit bitmap On 09/05/2012 08:41 AM, Li, Jiongxi wrote: With

Re: [PATCH] kvm tools: don't die if sdl wasn't compiled in and we don't try using it

2012-09-17 Thread Kashyap Chamarthy
On Mon, Sep 17, 2012 at 4:46 PM, Sasha Levin levinsasha...@gmail.com wrote: If SDL isn't compiled in we shouldn't die unless we actually try using it. Signed-off-by: Sasha Levin levinsasha...@gmail.com --- tools/kvm/include/kvm/sdl.h | 10 -- 1 file changed, 8 insertions(+), 2

Re: [PATCH -v2 2/2] make the compaction skip ahead logic robust

2012-09-17 Thread Mel Gorman
On Sat, Sep 15, 2012 at 04:55:24PM +0100, Richard Davies wrote: Hi Rik, Mel and Shaohua, Thank you for your latest patches. I attach my latest perf report for a slow boot with all of these applied. Thanks for testing. Mel asked for timings of the slow boots. It's very hard to give

Re: [PATCH] kvm tools: don't die if sdl wasn't compiled in and we don't try using it

2012-09-17 Thread Kashyap Chamarthy
On Mon, Sep 17, 2012 at 5:14 PM, Kashyap Chamarthy kashyap...@gmail.com wrote: On Mon, Sep 17, 2012 at 4:46 PM, Sasha Levin levinsasha...@gmail.com wrote: If SDL isn't compiled in we shouldn't die unless we actually try using it. Signed-off-by: Sasha Levin levinsasha...@gmail.com ---

Re: [PATCH v3] kvm/fpu: Enable fully eager restore kvm FPU

2012-09-17 Thread Marcelo Tosatti
On Mon, Sep 17, 2012 at 02:07:43AM +, Hao, Xudong wrote: -Original Message- From: Avi Kivity [mailto:a...@redhat.com] Sent: Friday, September 14, 2012 12:40 AM To: Marcelo Tosatti Cc: Hao, Xudong; kvm@vger.kernel.org; Zhang, Xiantao Subject: Re: [PATCH v3] kvm/fpu: Enable

[PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization CPUIDs. This is primarily done so that the guest will think it is running under vmware when hypervisor-vendor=vmware is specified as a property of a cpu. This depends on: http://lists.gnu.org/archive/html/qemu-devel/2012-09/msg01400.html As far as I know it is

[PATCH v3 00/17] *** SUBJECT HERE ***

2012-09-17 Thread Don Slutz
*** BLURB HERE *** Don Slutz (17): target-i386: Allow tsc-frequency to be larger then 2.147G target-i386: Add missing kvm bits. target-i386: Add Hypervisor level. target-i386: Add cpu object access routines for Hypervisor level. target-i386: Add x86_set_hyperv. target-i386: Use

Re: [PATCH 0/3] Prepare kvm for lto

2012-09-17 Thread Marcelo Tosatti
On Sun, Sep 16, 2012 at 03:10:56PM +0300, Avi Kivity wrote: vmx.c has an lto-unfriendly bit, fix it up. While there, clean up our asm code. v2: add missing .global in case vmx_return and vmx_set_constant_host_state() become separated by lto Avi Kivity (3): KVM: VMX: Make

[PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization CPUIDs. This is primarily done so that the guest will think it is running under vmware when hypervisor-vendor=vmware is specified as a property of a cpu. This depends on: http://lists.gnu.org/archive/html/qemu-devel/2012-09/msg01400.html As far as I know it is

[PATCH v3 03/17] target-i386: Add Hypervisor level.

2012-09-17 Thread Don Slutz
This is just the EAX value. This is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html VMware documention

[PATCH v3 08/17] target-i386: Add Hypervisor vendor.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization vendor. This is EBX, ECX, EDX data for 0x4000. QEMU knows this is KVM_CPUID_SIGNATURE (0x4000). This is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel

[PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization vendor. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization vendor. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index bfaee02..89a45b5 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1975,6 +1975,17 @@ void

[PATCH v3 15/17] target-i386: Add setting of Hypervisor leaf extra for known vmare4.

2012-09-17 Thread Don Slutz
This was taken from: http://article.gmane.org/gmane.comp.emulators.kvm.devel/22643 Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c

[PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 5afb188..1b3a472 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1842,9 +1842,9 @@ void

[PATCH v3 14/17] target-i386: Add cpu object access routines for Hypervisor leaf extra.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 66 + 1 files changed, 66 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 9ac3076..34d2291 100644 --- a/target-i386/cpu.c +++

[PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/kvm.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index b8789f2..17c72bc 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -454,6 +454,25 @@

[PATCH v3 08/17] target-i386: Add Hypervisor vendor.

2012-09-17 Thread Don Slutz
This is EBX, ECX, EDX data. This is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html VMware documention

[PATCH v3 13/17] target-i386: Add optional Hypervisor leaf extra.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.h |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.h b/target-i386/cpu.h index 47bc00c..a2d3588 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -799,6 +799,10 @@ typedef struct

[PATCH v3 12/17] target-i386: Add some known names to Hypervisor vendor.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 36 +++- target-i386/cpu.h | 12 2 files changed, 47 insertions(+), 1 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 1b3a472..9ac3076 100644 ---

[PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/kvm.c | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index bf27793..b8789f2 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -392,13 +392,15 @@ int

[PATCH v3 05/17] target-i386: Add x86_set_hyperv.

2012-09-17 Thread Don Slutz
This is used to set the cpu object's hypervisor level to the default for Microsoft's Hypervisor. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index

[PATCH v3 09/17] target-i386: Add cpu object access routines for Hypervisor vendor.

2012-09-17 Thread Don Slutz
These are modeled after x86_cpuid_set_vendor and x86_cpuid_get_vendor. Since kvm's vendor is shorter, the test for correct size is removed and zero padding is added. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 44 1 files

[PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
This does not provide vendor support in tcg yet. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 4120393..d3b9bd8 100644 ---

[PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G

2012-09-17 Thread Don Slutz
The check using INT_MAX (2147483647) is wrong in this case. Signed-off-by: Fred Oliveira folive...@cloudswitch.com Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c

[PATCH v3 04/17] target-i386: Add cpu object access routines for Hypervisor level.

2012-09-17 Thread Don Slutz
These are modeled after x86_cpuid_get_xlevel and x86_cpuid_set_xlevel. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index

[PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/kvm.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 895d848..bf27793 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -389,12 +389,12 @@ int

[PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 03/17] target-i386: Add Hypervisor level.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level or maximim cpuid function present in this leaf. This is just the EAX value for 0x4000. QEMU knows this is KVM_CPUID_SIGNATURE (0x4000). This is based on: Microsoft Hypervisor CPUID Leaves:

[PATCH v3 02/17] target-i386: Add missing kvm bits.

2012-09-17 Thread Don Slutz
Fix duplicate name (kvmclock = kvm_clock2) also. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 0313cf5..5f9866a 100644 --- a/target-i386/cpu.c +++

Re: [RFC][PATCH] Improving directed yield scalability for PLE handler

2012-09-17 Thread Andrew Jones
On Sat, Sep 15, 2012 at 09:38:54PM +0530, Raghavendra K T wrote: On 09/14/2012 10:40 PM, Andrew Jones wrote: On Thu, Sep 13, 2012 at 04:30:58PM -0500, Andrew Theurer wrote: On Thu, 2012-09-13 at 17:18 +0530, Raghavendra K T wrote: * Andrew Theurerhaban...@linux.vnet.ibm.com [2012-09-11

Re: [PATCH v3 00/17] *** SUBJECT HERE ***

2012-09-17 Thread Don Slutz
forgot to delete the backup versions. :( -Don On 09/17/12 09:39, Don Slutz wrote: *** BLURB HERE *** Don Slutz (17): target-i386: Allow tsc-frequency to be larger then 2.147G target-i386: Add missing kvm bits. target-i386: Add Hypervisor level. target-i386: Add cpu object access

Re: [PATCH -v2 2/2] make the compaction skip ahead logic robust

2012-09-17 Thread Rik van Riel
On 09/15/2012 11:55 AM, Richard Davies wrote: Hi Rik, Mel and Shaohua, Thank you for your latest patches. I attach my latest perf report for a slow boot with all of these applied. Mel asked for timings of the slow boots. It's very hard to give anything useful here! A normal boot would be a

Re: [PATCH v3 00/17] *** SUBJECT HERE ***

2012-09-17 Thread Don Slutz
On 09/17/12 09:49, Don Slutz wrote: forgot to delete the backup versions. :( -Don On 09/17/12 09:39, Don Slutz wrote: Here is the planned cover letter: From 7c0a80d8e870da981786b7235d3a968024c89abb Mon Sep 17 00:00:00 2001 In-Reply-To: 1346354435-21685-1-git-send-email-...@cloudswitch.com

[PATCH v2] target-i386: Report on error during cpu_x86_register().

2012-09-17 Thread Don Slutz
Send it to stderr before free of the details. Signed-off-by: Don Slutz d...@cloudswitch.com --- v2: Change __FUNCTION__ to __func__ target-i386/cpu.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 57c064f..760cca4 100644 ---

[PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.

2012-09-17 Thread Don Slutz
Resend with new id so the backup files are not included. Also known as Paravirtualization CPUIDs. This is primarily done so that the guest will think it is running under vmware when hypervisor-vendor=vmware is specified as a property of a cpu. This depends on:

[PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G

2012-09-17 Thread Don Slutz
The check using INT_MAX (2147483647) is wrong in this case. Signed-off-by: Fred Oliveira folive...@cloudswitch.com Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c

[PATCH v3 02/17] target-i386: Add missing kvm bits.

2012-09-17 Thread Don Slutz
Fix duplicate name (kvmclock = kvm_clock2) also. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 0313cf5..5f9866a 100644 --- a/target-i386/cpu.c +++

[PATCH v3 03/17] target-i386: Add Hypervisor level.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level or maximim cpuid function present in this leaf. This is just the EAX value for 0x4000. QEMU knows this is KVM_CPUID_SIGNATURE (0x4000). This is based on: Microsoft Hypervisor CPUID Leaves:

[PATCH v3 04/17] target-i386: Add cpu object access routines for Hypervisor level.

2012-09-17 Thread Don Slutz
These are modeled after x86_cpuid_get_xlevel and x86_cpuid_set_xlevel. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index

[PATCH v3 05/17] target-i386: Add x86_set_hyperv.

2012-09-17 Thread Don Slutz
This is used to set the cpu object's hypervisor level to the default for Microsoft's Hypervisor. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index

[PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization level. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 08/17] target-i386: Add Hypervisor vendor.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization vendor. This is EBX, ECX, EDX data for 0x4000. QEMU knows this is KVM_CPUID_SIGNATURE (0x4000). This is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel

[PATCH v3 09/17] target-i386: Add cpu object access routines for Hypervisor vendor.

2012-09-17 Thread Don Slutz
These are modeled after x86_cpuid_set_vendor and x86_cpuid_get_vendor. Since kvm's vendor is shorter, the test for correct size is removed and zero padding is added. Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 44 1 files

[PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Also known as Paravirtualization vendor. This change is based on: Microsoft Hypervisor CPUID Leaves: http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx Linux kernel change starts with: http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html

[PATCH v3 12/17] target-i386: Add some known names to Hypervisor vendor.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 36 +++- target-i386/cpu.h | 12 2 files changed, 47 insertions(+), 1 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 1b3a472..9ac3076 100644 ---

[PATCH v3 13/17] target-i386: Add optional Hypervisor leaf extra.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.h |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.h b/target-i386/cpu.h index 47bc00c..a2d3588 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -799,6 +799,10 @@ typedef struct

[PATCH v3 14/17] target-i386: Add cpu object access routines for Hypervisor leaf extra.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 66 + 1 files changed, 66 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 9ac3076..34d2291 100644 --- a/target-i386/cpu.c +++

[PATCH v3 15/17] target-i386: Add setting of Hypervisor leaf extra for known vmare4.

2012-09-17 Thread Don Slutz
This was taken from: http://article.gmane.org/gmane.comp.emulators.kvm.devel/22643 Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c

Re: [PATCH v2] target-i386: Report on error during cpu_x86_register().

2012-09-17 Thread Don Slutz
On 09/17/12 10:00, Don Slutz wrote: Send it to stderr before free of the details. Signed-off-by: Don Slutz d...@cloudswitch.com --- v2: Change __FUNCTION__ to __func__ target-i386/cpu.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c

[PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=tcg.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/cpu.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index bfaee02..89a45b5 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1975,6 +1975,17 @@ void

[PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=kvm.

2012-09-17 Thread Don Slutz
Signed-off-by: Don Slutz d...@cloudswitch.com --- target-i386/kvm.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index b8789f2..17c72bc 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -454,6 +454,25 @@

Re: [PATCH -v2 2/2] make the compaction skip ahead logic robust

2012-09-17 Thread Mel Gorman
On Mon, Sep 17, 2012 at 09:50:08AM -0400, Rik van Riel wrote: On 09/15/2012 11:55 AM, Richard Davies wrote: Hi Rik, Mel and Shaohua, Thank you for your latest patches. I attach my latest perf report for a slow boot with all of these applied. Mel asked for timings of the slow boots. It's

KVM call for agenda for Tuesday, September 18th

2012-09-17 Thread Juan Quintela
Hi Please send in any agenda items you are interested in covering. 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-devel] Block Migration Assertion in qemu-kvm 1.2.0

2012-09-17 Thread Peter Lieven
On 09/17/12 10:41, Kevin Wolf wrote: Am 16.09.2012 12:13, schrieb Peter Lieven: Hi, when trying to block migrate a VM from one node to another, the source VM crashed with the following assertion: block.c:3829: bdrv_set_in_use: Assertion `bs-in_use != in_use' failed. Is this sth already

Re: [PATCH v8 1/3] KVM: x86: export svm/vmx exit code and vector code to userspace

2012-09-17 Thread Arnaldo Carvalho de Melo
Em Mon, Sep 17, 2012 at 04:31:13PM +0800, Dong Hao escreveu: From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Exporting KVM exit information to userspace to be consumed by perf. [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by: Dong Hao

Re: [PATCH v9 2/2] kvm: On Ack, De-assert Notify KVM_IRQFD extension

2012-09-17 Thread Alex Williamson
On Wed, 2012-09-05 at 17:57 +0300, Avi Kivity wrote: On 08/21/2012 10:29 PM, Alex Williamson wrote: For VFIO based device assignment we'd like a mechanism to allow level triggered interrutps to be directly injected into KVM. KVM_IRQFD already allows this for edge triggered interrupts, but

Re: [PATCH] kvm: make processes waiting on vcpu mutex killable

2012-09-17 Thread Marcelo Tosatti
On Sun, Sep 16, 2012 at 11:50:30AM +0300, Michael S. Tsirkin wrote: vcpu mutex can be held for unlimited time so taking it with mutex_lock on an ioctl is wrong: one process could be passed a vcpu fd and call this ioctl on the vcpu used by another process, it will then be unkillable until the

Re: [PATCH] kvm: Fix kvmclock documentation to match reality

2012-09-17 Thread Marcelo Tosatti
On Sun, Sep 16, 2012 at 12:55:40PM +0200, Stefan Fritsch wrote: Hi, I found the kvmclock documentation to be rather unhelpful. This patch should fix it. Cheers, Stefan Author: Stefan Fritsch s...@sfritsch.de Date: Sun Sep 16 12:30:46 2012 +0200 kvm: Fix kvmclock documentation

[PATCH 09/18] perf stat: Move stats related code to util/stat.c

2012-09-17 Thread Arnaldo Carvalho de Melo
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Then, the code can be shared between kvm events and perf stat. Signed-off-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on acme's git tree ] Signed-off-by: Dong Hao

Re: [Qemu-devel] Block Migration and xbzrle

2012-09-17 Thread Orit Wasserman
On 09/16/2012 01:39 PM, Peter Lieven wrote: Hi, I remember that this was broken some time ago and currently with qemu-kvm 1.2.0 I am still not able to use block migration plus xbzrle. The migration fails if both are used together. XBZRLE without block migration works. Can someone

Re: [PATCH v9 0/2] kvm: level irqfd support

2012-09-17 Thread Alex Williamson
On Wed, 2012-08-22 at 11:25 +0300, Michael S. Tsirkin wrote: On Tue, Aug 21, 2012 at 07:28:15PM -0600, Alex Williamson wrote: On Wed, 2012-08-22 at 03:31 +0300, Michael S. Tsirkin wrote: On Tue, Aug 21, 2012 at 01:28:57PM -0600, Alex Williamson wrote: Here's the much anticipated re-write

Re: [Qemu-devel] [PATCH 0/9] [PULL] qemu-kvm.git uq/master queue

2012-09-17 Thread Anthony Liguori
Marcelo Tosatti mtosa...@redhat.com writes: The following changes since commit e0a1e32dbc41e6b2aabb436a9417dfd32177a3dc: Merge branch 'usb.64' of git://git.kraxel.org/qemu (2012-09-11 18:06:56 +0200) are available in the git repository at:

Re: [Qemu-devel] [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.

2012-09-17 Thread Eduardo Habkost
On Mon, Sep 17, 2012 at 10:00:50AM -0400, Don Slutz wrote: Resend with new id so the backup files are not included. Also known as Paravirtualization CPUIDs. This is primarily done so that the guest will think it is running under vmware when hypervisor-vendor=vmware is specified as a

[KVM] Guest Debugging Facilities in KVM

2012-09-17 Thread Dean Pucsek
Hello, For my Masters thesis I am investigating the usage of Intel VT-x and branch tracing in the domain of malware analysis. Essentially what I'm aiming to do is trace the execution of a guest VM and then pass that trace on to some other tools. I've been playing KVM for a couple weeks now

Re: [PATCHSET] kthread_worker: reimplement flush_kthread_work() to allow freeing during execution

2012-09-17 Thread Tejun Heo
On Fri, Sep 14, 2012 at 03:50:40PM -0700, Colin Cross wrote: This patch set fixes a reproducible crash I'm seeing on a 3.4.10 kernel. flush_kthread_worker (which is different from flush_kthread_work) is initializing a kthread_work and a completion on the stack, then queuing it and calling

Re: [Qemu-devel] Block Migration Assertion in qemu-kvm 1.2.0

2012-09-17 Thread Peter Lieven
On 09/17/12 10:41, Kevin Wolf wrote: Am 16.09.2012 12:13, schrieb Peter Lieven: Hi, when trying to block migrate a VM from one node to another, the source VM crashed with the following assertion: block.c:3829: bdrv_set_in_use: Assertion `bs-in_use != in_use' failed. Is this sth already

Re: [PATCHSET] kthread_worker: reimplement flush_kthread_work() to allow freeing during execution

2012-09-17 Thread Greg KH
On Mon, Sep 17, 2012 at 12:40:16PM -0700, Tejun Heo wrote: On Fri, Sep 14, 2012 at 03:50:40PM -0700, Colin Cross wrote: This patch set fixes a reproducible crash I'm seeing on a 3.4.10 kernel. flush_kthread_worker (which is different from flush_kthread_work) is initializing a kthread_work

[PATCH] kvm tools: support build-time checks

2012-09-17 Thread Sasha Levin
Support using build-time check tools when building lkvm. This allows using tools such as smatch with the same syntax used with kernel code. For example, to build with smatch checks, first make sure you have smatch installed, then run: make CHECK=smatch -p=kernel C=1 Signed-off-by: Sasha

Re: NIC emulation with built-in rate limiting?

2012-09-17 Thread Rick Jones
So, while the question includes the stability of how things get plumbed for a VM and whether moving some of that into the NIC emulation might help :) I've gone ahead and re-run the experiment with bare-iron. This time just for kicks I used 50 Mbit/s throttle inbound and outbound. The

RE: [PATCH v3] kvm/fpu: Enable fully eager restore kvm FPU

2012-09-17 Thread Hao, Xudong
-Original Message- From: Marcelo Tosatti [mailto:mtosa...@redhat.com] Sent: Monday, September 17, 2012 9:31 PM To: Hao, Xudong Cc: Avi Kivity; kvm@vger.kernel.org; Zhang, Xiantao Subject: Re: [PATCH v3] kvm/fpu: Enable fully eager restore kvm FPU On Mon, Sep 17, 2012 at

Re: [RFC][PATCH] Improving directed yield scalability for PLE handler

2012-09-17 Thread Andrew Theurer
On Sun, 2012-09-16 at 11:55 +0300, Avi Kivity wrote: On 09/14/2012 12:30 AM, Andrew Theurer wrote: The concern I have is that even though we have gone through changes to help reduce the candidate vcpus we yield to, we still have a very poor idea of which vcpu really needs to run. The

[PATCH v10 0/2] kvm: level irqfd support

2012-09-17 Thread Alex Williamson
Updated with suggestions from Michael and Avi, the OADN option is now a resample option. Re-working locking went surprisingly well, lockdep clean, and now allows us to use a single irq source ID for all resample irqfds. I hope we're close. Thanks, Alex --- Alex Williamson (2): kvm: Add

[PATCH v10 1/2] kvm: Provide pre-locked setup to irq ack notifier

2012-09-17 Thread Alex Williamson
This enables better integration into irqfd setup where we can adjust our lock ordering to hold irq_lock, making these callable and avoiding irq source ID races. Signed-off-by: Alex Williamson alex.william...@redhat.com --- include/linux/kvm_host.h |4 virt/kvm/irq_comm.c | 18

[PATCH v10 2/2] kvm: Add resampling irqfds for level triggered interrupts

2012-09-17 Thread Alex Williamson
To emulate level triggered interrupts, add a resample option to KVM_IRQFD. When specified, a new resamplefd is provided that notifies the user when the irqchip has been resampled by the VM. This may, for instance, indicate an EOI. Also in this mode, injection of an interrupt through an irqfd

Can't use USB devices in their designated applications on Windows Vista guest

2012-09-17 Thread Dirk Heinrichs
Hi, I have a really strange problem using USB devices in a Windows Vista (32bit) guest. After adding a USB device (an Adroid phone or a GPS navigation device) to the guest, Vista starts to install the driver(s) for the new device just as it would do on real hardware. However, when it comes to