[PATCH] Allow kvm to use more than 4 VCPUs

[PATCH] Allow kvm to use more than 4 VCPUs

[Anthony Liguori]

This patch actually allows KVM to be used with more than 4 VCPUs.  The change
in qemu-kvm.c was pretty difficult to find because it was using an open coded
array size of 4.  I changed that array to be 256 since that's the real maximum
on x86 and the additional storage space is a pretty trivial cost.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

diff --git a/libkvm/kvm-common.h b/libkvm/kvm-common.h
index f4040be..d4df1a4 100644
--- a/libkvm/kvm-common.h
+++ b/libkvm/kvm-common.h
@@ -19,7 +19,7 @@
 /* FIXME: share this number with kvm */
 /* FIXME: or dynamically alloc/realloc regions */
 #define KVM_MAX_NUM_MEM_REGIONS 8u
-#define MAX_VCPUS 4
+#define MAX_VCPUS 16
 
 /* kvm abi verison variable */
 extern int kvm_abi;
diff --git a/qemu/qemu-kvm.c b/qemu/qemu-kvm.c
index ffc59d5..1b7cac3 100644
--- a/qemu/qemu-kvm.c
+++ b/qemu/qemu-kvm.c
@@ -47,7 +47,7 @@ struct vcpu_info {
     int signalled;
     int stop;
     int stopped;
-} vcpu_info[4];
+} vcpu_info[256];
 
 CPUState *qemu_kvm_cpu_env(int index)
 {

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

Re: [PATCH] Allow kvm to use more than 4 VCPUs

[Avi Kivity]

Anthony Liguori wrote:
> This patch actually allows KVM to be used with more than 4 VCPUs.  The change
> in qemu-kvm.c was pretty difficult to find because it was using an open coded
> array size of 4.  I changed that array to be 256 since that's the real maximum
> on x86 and the additional storage space is a pretty trivial cost.
>
>   

Applied, thanks.  Please keep libkvm and qemu patches separate (when 
possible).


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

Re: [PATCH] Allow kvm to use more than 4 VCPUs

[Alexey Eremenko]

On Sun, Feb 24, 2008 at 8:16 AM, Avi Kivity <avi@qumranet.com> wrote:
> Anthony Liguori wrote:
>  > This patch actually allows KVM to be used with more than 4 VCPUs.  The change
>  > in qemu-kvm.c was pretty difficult to find because it was using an open coded
>  > array size of 4.  I changed that array to be 256 since that's the real maximum
>  > on x86 and the additional storage space is a pretty trivial cost.
>  >
>  >
>
>  Applied, thanks.  Please keep libkvm and qemu patches separate (when
>  possible).
Very nice !

After I test it, I will close this SMP bug:
http://sourceforge.net/tracker/index.php?func=detail&aid=1850037&group_id=180599&atid=893831

-- 
-Alexey Eremenko "Technologov"

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/