* [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS @ 2013-09-14 12:18 Andrew Jones 2013-09-15 9:08 ` Gleb Natapov 0 siblings, 1 reply; 7+ messages in thread From: Andrew Jones @ 2013-09-14 12:18 UTC (permalink / raw) To: kvm; +Cc: gleb, pbonzini, linux-kernel Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. Signed-off-by: Andrew Jones <drjones@redhat.com> --- arch/x86/include/asm/kvm_host.h | 5 +++-- arch/x86/kvm/Kconfig | 10 ++++++++++ 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h index c76ff74a98f2e..e7e9b523a8f7e 100644 --- a/arch/x86/include/asm/kvm_host.h +++ b/arch/x86/include/asm/kvm_host.h @@ -31,8 +31,9 @@ #include <asm/msr-index.h> #include <asm/asm.h> -#define KVM_MAX_VCPUS 255 -#define KVM_SOFT_MAX_VCPUS 160 +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) + #define KVM_USER_MEM_SLOTS 125 /* memory slots that are not exposed to userspace */ #define KVM_PRIVATE_MEM_SLOTS 3 diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig index a47a3e54b964b..e9532c33527ee 100644 --- a/arch/x86/kvm/Kconfig +++ b/arch/x86/kvm/Kconfig @@ -52,6 +52,16 @@ config KVM If unsure, say N. +config KVM_MAX_VCPUS + int "Number maximum supported virtual CPUs per VM" + depends on KVM + default 255 + help + Static number of max supported virtual CPUs per VM. + + Set to a lower number to save some resources. Set to a higher + number to test scalability. + config KVM_INTEL tristate "KVM for Intel processors support" depends on KVM -- 1.8.1.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-14 12:18 [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS Andrew Jones @ 2013-09-15 9:08 ` Gleb Natapov 2013-09-16 8:28 ` Andrew Jones 0 siblings, 1 reply; 7+ messages in thread From: Gleb Natapov @ 2013-09-15 9:08 UTC (permalink / raw) To: Andrew Jones; +Cc: kvm, pbonzini, linux-kernel On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > Signed-off-by: Andrew Jones <drjones@redhat.com> > --- > arch/x86/include/asm/kvm_host.h | 5 +++-- > arch/x86/kvm/Kconfig | 10 ++++++++++ > 2 files changed, 13 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > index c76ff74a98f2e..e7e9b523a8f7e 100644 > --- a/arch/x86/include/asm/kvm_host.h > +++ b/arch/x86/include/asm/kvm_host.h > @@ -31,8 +31,9 @@ > #include <asm/msr-index.h> > #include <asm/asm.h> > > -#define KVM_MAX_VCPUS 255 > -#define KVM_SOFT_MAX_VCPUS 160 > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > + > #define KVM_USER_MEM_SLOTS 125 > /* memory slots that are not exposed to userspace */ > #define KVM_PRIVATE_MEM_SLOTS 3 > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > index a47a3e54b964b..e9532c33527ee 100644 > --- a/arch/x86/kvm/Kconfig > +++ b/arch/x86/kvm/Kconfig > @@ -52,6 +52,16 @@ config KVM > > If unsure, say N. > > +config KVM_MAX_VCPUS > + int "Number maximum supported virtual CPUs per VM" > + depends on KVM > + default 255 > + help > + Static number of max supported virtual CPUs per VM. > + > + Set to a lower number to save some resources. Set to a higher > + number to test scalability. > + Maximum this can save is around 2K per VM. This is pretty insignificant considering overall memory footprint even smallest VM has. > config KVM_INTEL > tristate "KVM for Intel processors support" > depends on KVM > -- > 1.8.1.4 -- Gleb. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-15 9:08 ` Gleb Natapov @ 2013-09-16 8:28 ` Andrew Jones 2013-09-16 8:47 ` Gleb Natapov 0 siblings, 1 reply; 7+ messages in thread From: Andrew Jones @ 2013-09-16 8:28 UTC (permalink / raw) To: Gleb Natapov; +Cc: kvm, pbonzini, linux-kernel On Sun, Sep 15, 2013 at 12:08:38PM +0300, Gleb Natapov wrote: > On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > --- > > arch/x86/include/asm/kvm_host.h | 5 +++-- > > arch/x86/kvm/Kconfig | 10 ++++++++++ > > 2 files changed, 13 insertions(+), 2 deletions(-) > > > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > > index c76ff74a98f2e..e7e9b523a8f7e 100644 > > --- a/arch/x86/include/asm/kvm_host.h > > +++ b/arch/x86/include/asm/kvm_host.h > > @@ -31,8 +31,9 @@ > > #include <asm/msr-index.h> > > #include <asm/asm.h> > > > > -#define KVM_MAX_VCPUS 255 > > -#define KVM_SOFT_MAX_VCPUS 160 > > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > > + > > #define KVM_USER_MEM_SLOTS 125 > > /* memory slots that are not exposed to userspace */ > > #define KVM_PRIVATE_MEM_SLOTS 3 > > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > > index a47a3e54b964b..e9532c33527ee 100644 > > --- a/arch/x86/kvm/Kconfig > > +++ b/arch/x86/kvm/Kconfig > > @@ -52,6 +52,16 @@ config KVM > > > > If unsure, say N. > > > > +config KVM_MAX_VCPUS > > + int "Number maximum supported virtual CPUs per VM" > > + depends on KVM > > + default 255 > > + help > > + Static number of max supported virtual CPUs per VM. > > + > > + Set to a lower number to save some resources. Set to a higher > > + number to test scalability. > > + > Maximum this can save is around 2K per VM. This is pretty insignificant > considering overall memory footprint even smallest VM has. Should I reword this, dropping all 'save resources' verbiage, in order to avoid sending a message that this option can affect resource consumption? Or just leave it as it is, because even though it's insignificant, it's still true and balances out the 'Set to a higher' part. drew > > > config KVM_INTEL > > tristate "KVM for Intel processors support" > > depends on KVM > > -- > > 1.8.1.4 > > -- > Gleb. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-16 8:28 ` Andrew Jones @ 2013-09-16 8:47 ` Gleb Natapov 2013-09-16 12:03 ` Andrew Jones 0 siblings, 1 reply; 7+ messages in thread From: Gleb Natapov @ 2013-09-16 8:47 UTC (permalink / raw) To: Andrew Jones; +Cc: kvm, pbonzini, linux-kernel On Mon, Sep 16, 2013 at 10:28:20AM +0200, Andrew Jones wrote: > On Sun, Sep 15, 2013 at 12:08:38PM +0300, Gleb Natapov wrote: > > On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > > > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > > > > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > > --- > > > arch/x86/include/asm/kvm_host.h | 5 +++-- > > > arch/x86/kvm/Kconfig | 10 ++++++++++ > > > 2 files changed, 13 insertions(+), 2 deletions(-) > > > > > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > > > index c76ff74a98f2e..e7e9b523a8f7e 100644 > > > --- a/arch/x86/include/asm/kvm_host.h > > > +++ b/arch/x86/include/asm/kvm_host.h > > > @@ -31,8 +31,9 @@ > > > #include <asm/msr-index.h> > > > #include <asm/asm.h> > > > > > > -#define KVM_MAX_VCPUS 255 > > > -#define KVM_SOFT_MAX_VCPUS 160 > > > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > > > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > > > + > > > #define KVM_USER_MEM_SLOTS 125 > > > /* memory slots that are not exposed to userspace */ > > > #define KVM_PRIVATE_MEM_SLOTS 3 > > > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > > > index a47a3e54b964b..e9532c33527ee 100644 > > > --- a/arch/x86/kvm/Kconfig > > > +++ b/arch/x86/kvm/Kconfig > > > @@ -52,6 +52,16 @@ config KVM > > > > > > If unsure, say N. > > > > > > +config KVM_MAX_VCPUS > > > + int "Number maximum supported virtual CPUs per VM" > > > + depends on KVM > > > + default 255 > > > + help > > > + Static number of max supported virtual CPUs per VM. > > > + > > > + Set to a lower number to save some resources. Set to a higher > > > + number to test scalability. > > > + > > Maximum this can save is around 2K per VM. This is pretty insignificant > > considering overall memory footprint even smallest VM has. > > Should I reword this, dropping all 'save resources' verbiage, in order to > avoid sending a message that this option can affect resource consumption? > Or just leave it as it is, because even though it's insignificant, it's > still true and balances out the 'Set to a higher' part. > I do not think config option is necessary. The overhead is so insignificant that there is no point in additional user visible knob, at least while only 255 vcpu are supported. Is there a reason for anyone to configure less them 255 vcpus here? OTOH what prevents someone from configuring more then 255 vcpus? -- Gleb. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-16 8:47 ` Gleb Natapov @ 2013-09-16 12:03 ` Andrew Jones 2013-09-16 14:16 ` Gleb Natapov 0 siblings, 1 reply; 7+ messages in thread From: Andrew Jones @ 2013-09-16 12:03 UTC (permalink / raw) To: Gleb Natapov; +Cc: kvm, pbonzini, linux-kernel On Mon, Sep 16, 2013 at 11:47:10AM +0300, Gleb Natapov wrote: > On Mon, Sep 16, 2013 at 10:28:20AM +0200, Andrew Jones wrote: > > On Sun, Sep 15, 2013 at 12:08:38PM +0300, Gleb Natapov wrote: > > > On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > > > > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > > > > > > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > > > --- > > > > arch/x86/include/asm/kvm_host.h | 5 +++-- > > > > arch/x86/kvm/Kconfig | 10 ++++++++++ > > > > 2 files changed, 13 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > > > > index c76ff74a98f2e..e7e9b523a8f7e 100644 > > > > --- a/arch/x86/include/asm/kvm_host.h > > > > +++ b/arch/x86/include/asm/kvm_host.h > > > > @@ -31,8 +31,9 @@ > > > > #include <asm/msr-index.h> > > > > #include <asm/asm.h> > > > > > > > > -#define KVM_MAX_VCPUS 255 > > > > -#define KVM_SOFT_MAX_VCPUS 160 > > > > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > > > > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > > > > + > > > > #define KVM_USER_MEM_SLOTS 125 > > > > /* memory slots that are not exposed to userspace */ > > > > #define KVM_PRIVATE_MEM_SLOTS 3 > > > > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > > > > index a47a3e54b964b..e9532c33527ee 100644 > > > > --- a/arch/x86/kvm/Kconfig > > > > +++ b/arch/x86/kvm/Kconfig > > > > @@ -52,6 +52,16 @@ config KVM > > > > > > > > If unsure, say N. > > > > > > > > +config KVM_MAX_VCPUS > > > > + int "Number maximum supported virtual CPUs per VM" > > > > + depends on KVM > > > > + default 255 > > > > + help > > > > + Static number of max supported virtual CPUs per VM. > > > > + > > > > + Set to a lower number to save some resources. Set to a higher > > > > + number to test scalability. > > > > + > > > Maximum this can save is around 2K per VM. This is pretty insignificant > > > considering overall memory footprint even smallest VM has. > > > > Should I reword this, dropping all 'save resources' verbiage, in order to > > avoid sending a message that this option can affect resource consumption? > > Or just leave it as it is, because even though it's insignificant, it's > > still true and balances out the 'Set to a higher' part. > > > I do not think config option is necessary. The overhead is so > insignificant that there is no point in additional user visible knob, > at least while only 255 vcpu are supported. Is there a reason for anyone > to configure less them 255 vcpus here? OTOH what prevents someone from > configuring more then 255 vcpus? The reason to configure less is to be able to compile a hard limit, without having to muck with the source. E.g. if you really don't want to allow more than KVM_SOFT_MAX_VCPUS to be configured, then you can compile with CONFIG_KVM_MAX_VCPUS == KVM_SOFT_MAX_VCPUS. Nothing prevents someone from configuring more than the max in userspace, but if they try to create/use more than the max (kvm_vm_ioctl_create_vcpu), it'll fail (EINVAL). I see this as a step towards getting rid of KVM_SOFT_MAX_VCPUS. I.e. compile with whatever maximum limit you want (can support), and return online/possible cpus for the recommended number. Only configure the kernel with more than your typical maximum (was KVM_SOFT_MAX_VCPUS) for development/testing purposes. drew ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-16 12:03 ` Andrew Jones @ 2013-09-16 14:16 ` Gleb Natapov 2013-09-16 15:36 ` Andrew Jones 0 siblings, 1 reply; 7+ messages in thread From: Gleb Natapov @ 2013-09-16 14:16 UTC (permalink / raw) To: Andrew Jones; +Cc: kvm, pbonzini, linux-kernel On Mon, Sep 16, 2013 at 02:03:33PM +0200, Andrew Jones wrote: > On Mon, Sep 16, 2013 at 11:47:10AM +0300, Gleb Natapov wrote: > > On Mon, Sep 16, 2013 at 10:28:20AM +0200, Andrew Jones wrote: > > > On Sun, Sep 15, 2013 at 12:08:38PM +0300, Gleb Natapov wrote: > > > > On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > > > > > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > > > > > > > > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > > > > --- > > > > > arch/x86/include/asm/kvm_host.h | 5 +++-- > > > > > arch/x86/kvm/Kconfig | 10 ++++++++++ > > > > > 2 files changed, 13 insertions(+), 2 deletions(-) > > > > > > > > > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > > > > > index c76ff74a98f2e..e7e9b523a8f7e 100644 > > > > > --- a/arch/x86/include/asm/kvm_host.h > > > > > +++ b/arch/x86/include/asm/kvm_host.h > > > > > @@ -31,8 +31,9 @@ > > > > > #include <asm/msr-index.h> > > > > > #include <asm/asm.h> > > > > > > > > > > -#define KVM_MAX_VCPUS 255 > > > > > -#define KVM_SOFT_MAX_VCPUS 160 > > > > > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > > > > > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > > > > > + > > > > > #define KVM_USER_MEM_SLOTS 125 > > > > > /* memory slots that are not exposed to userspace */ > > > > > #define KVM_PRIVATE_MEM_SLOTS 3 > > > > > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > > > > > index a47a3e54b964b..e9532c33527ee 100644 > > > > > --- a/arch/x86/kvm/Kconfig > > > > > +++ b/arch/x86/kvm/Kconfig > > > > > @@ -52,6 +52,16 @@ config KVM > > > > > > > > > > If unsure, say N. > > > > > > > > > > +config KVM_MAX_VCPUS > > > > > + int "Number maximum supported virtual CPUs per VM" > > > > > + depends on KVM > > > > > + default 255 > > > > > + help > > > > > + Static number of max supported virtual CPUs per VM. > > > > > + > > > > > + Set to a lower number to save some resources. Set to a higher > > > > > + number to test scalability. > > > > > + > > > > Maximum this can save is around 2K per VM. This is pretty insignificant > > > > considering overall memory footprint even smallest VM has. > > > > > > Should I reword this, dropping all 'save resources' verbiage, in order to > > > avoid sending a message that this option can affect resource consumption? > > > Or just leave it as it is, because even though it's insignificant, it's > > > still true and balances out the 'Set to a higher' part. > > > > > I do not think config option is necessary. The overhead is so > > insignificant that there is no point in additional user visible knob, > > at least while only 255 vcpu are supported. Is there a reason for anyone > > to configure less them 255 vcpus here? OTOH what prevents someone from > > configuring more then 255 vcpus? > > The reason to configure less is to be able to compile a hard limit, > without having to muck with the source. E.g. if you really don't want to > allow more than KVM_SOFT_MAX_VCPUS to be configured, then you can compile > with CONFIG_KVM_MAX_VCPUS == KVM_SOFT_MAX_VCPUS. > > Nothing prevents someone from configuring more than the max in > userspace, but if they try to create/use more than the max > (kvm_vm_ioctl_create_vcpu), it'll fail (EINVAL). > I was talking about configuring more then 255 vcus in kernel config. You should have "range 1 255" there. > I see this as a step towards getting rid of KVM_SOFT_MAX_VCPUS. I.e. > compile with whatever maximum limit you want (can support), and return > online/possible cpus for the recommended number. Only configure the kernel > with more than your typical maximum (was KVM_SOFT_MAX_VCPUS) for > development/testing purposes. > The idea behind soft/hard limit was to allow people easily check scalability without need to recompile the kernel. If custom build will be needed most people will not bother. -- Gleb. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS 2013-09-16 14:16 ` Gleb Natapov @ 2013-09-16 15:36 ` Andrew Jones 0 siblings, 0 replies; 7+ messages in thread From: Andrew Jones @ 2013-09-16 15:36 UTC (permalink / raw) To: Gleb Natapov; +Cc: kvm, pbonzini, linux-kernel On Mon, Sep 16, 2013 at 05:16:20PM +0300, Gleb Natapov wrote: > On Mon, Sep 16, 2013 at 02:03:33PM +0200, Andrew Jones wrote: > > On Mon, Sep 16, 2013 at 11:47:10AM +0300, Gleb Natapov wrote: > > > On Mon, Sep 16, 2013 at 10:28:20AM +0200, Andrew Jones wrote: > > > > On Sun, Sep 15, 2013 at 12:08:38PM +0300, Gleb Natapov wrote: > > > > > On Sat, Sep 14, 2013 at 02:18:49PM +0200, Andrew Jones wrote: > > > > > > Take CONFIG_KVM_MAX_VCPUS from arm32, but set the default to 255. > > > > > > > > > > > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > > > > > --- > > > > > > arch/x86/include/asm/kvm_host.h | 5 +++-- > > > > > > arch/x86/kvm/Kconfig | 10 ++++++++++ > > > > > > 2 files changed, 13 insertions(+), 2 deletions(-) > > > > > > > > > > > > diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h > > > > > > index c76ff74a98f2e..e7e9b523a8f7e 100644 > > > > > > --- a/arch/x86/include/asm/kvm_host.h > > > > > > +++ b/arch/x86/include/asm/kvm_host.h > > > > > > @@ -31,8 +31,9 @@ > > > > > > #include <asm/msr-index.h> > > > > > > #include <asm/asm.h> > > > > > > > > > > > > -#define KVM_MAX_VCPUS 255 > > > > > > -#define KVM_SOFT_MAX_VCPUS 160 > > > > > > +#define KVM_MAX_VCPUS CONFIG_KVM_MAX_VCPUS > > > > > > +#define KVM_SOFT_MAX_VCPUS min(160, KVM_MAX_VCPUS) > > > > > > + > > > > > > #define KVM_USER_MEM_SLOTS 125 > > > > > > /* memory slots that are not exposed to userspace */ > > > > > > #define KVM_PRIVATE_MEM_SLOTS 3 > > > > > > diff --git a/arch/x86/kvm/Kconfig b/arch/x86/kvm/Kconfig > > > > > > index a47a3e54b964b..e9532c33527ee 100644 > > > > > > --- a/arch/x86/kvm/Kconfig > > > > > > +++ b/arch/x86/kvm/Kconfig > > > > > > @@ -52,6 +52,16 @@ config KVM > > > > > > > > > > > > If unsure, say N. > > > > > > > > > > > > +config KVM_MAX_VCPUS > > > > > > + int "Number maximum supported virtual CPUs per VM" > > > > > > + depends on KVM > > > > > > + default 255 > > > > > > + help > > > > > > + Static number of max supported virtual CPUs per VM. > > > > > > + > > > > > > + Set to a lower number to save some resources. Set to a higher > > > > > > + number to test scalability. > > > > > > + > > > > > Maximum this can save is around 2K per VM. This is pretty insignificant > > > > > considering overall memory footprint even smallest VM has. > > > > > > > > Should I reword this, dropping all 'save resources' verbiage, in order to > > > > avoid sending a message that this option can affect resource consumption? > > > > Or just leave it as it is, because even though it's insignificant, it's > > > > still true and balances out the 'Set to a higher' part. > > > > > > > I do not think config option is necessary. The overhead is so > > > insignificant that there is no point in additional user visible knob, > > > at least while only 255 vcpu are supported. Is there a reason for anyone > > > to configure less them 255 vcpus here? OTOH what prevents someone from > > > configuring more then 255 vcpus? > > > > The reason to configure less is to be able to compile a hard limit, > > without having to muck with the source. E.g. if you really don't want to > > allow more than KVM_SOFT_MAX_VCPUS to be configured, then you can compile > > with CONFIG_KVM_MAX_VCPUS == KVM_SOFT_MAX_VCPUS. > > > > Nothing prevents someone from configuring more than the max in > > userspace, but if they try to create/use more than the max > > (kvm_vm_ioctl_create_vcpu), it'll fail (EINVAL). > > > I was talking about configuring more then 255 vcus in kernel config. You > should have "range 1 255" there. Ah, I assumed one could configure this higher if they wanted to test something higher. Other than requiring another word in a bitmap, I didn't see what would change going beyond that boundary. > > > I see this as a step towards getting rid of KVM_SOFT_MAX_VCPUS. I.e. > > compile with whatever maximum limit you want (can support), and return > > online/possible cpus for the recommended number. Only configure the kernel > > with more than your typical maximum (was KVM_SOFT_MAX_VCPUS) for > > development/testing purposes. > > > The idea behind soft/hard limit was to allow people easily check > scalability without need to recompile the kernel. If custom build will > be needed most people will not bother. Keeping the default 255 wouldn't have changed anything we have today, other than making it easier to adjust for scalability testing beyond 255 (assuming they could go beyond). However, as I just wrote in the other thread, you've convinced me to keep a constant soft limit definition, but since the soft limit is a custom concept, then I believe it belongs in the config. drew ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-09-16 15:36 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-09-14 12:18 [PATCH] x86: kvm: introduce CONFIG_KVM_MAX_VCPUS Andrew Jones 2013-09-15 9:08 ` Gleb Natapov 2013-09-16 8:28 ` Andrew Jones 2013-09-16 8:47 ` Gleb Natapov 2013-09-16 12:03 ` Andrew Jones 2013-09-16 14:16 ` Gleb Natapov 2013-09-16 15:36 ` Andrew Jones
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox