Re: [Xen-devel] [PATCH] tools/xl: refuse to set number of vcpus to 0 via xl vcpu-set

2018-09-03 Thread Juergen Gross
On 03/09/18 14:26, Jan Beulich wrote: On 03.09.18 at 13:59, wrote: >> --- a/tools/xl/xl_vcpu.c >> +++ b/tools/xl/xl_vcpu.c >> @@ -341,6 +341,10 @@ static int vcpuset(uint32_t domid, const char* >> nr_vcpus, int check_host) >> fprintf(stderr, "Error: Invalid argument.\n"); >>

Re: [Xen-devel] [PATCH] tools/xl: refuse to set number of vcpus to 0 via xl vcpu-set

2018-09-03 Thread Jan Beulich
>>> On 03.09.18 at 13:59, wrote: > --- a/tools/xl/xl_vcpu.c > +++ b/tools/xl/xl_vcpu.c > @@ -341,6 +341,10 @@ static int vcpuset(uint32_t domid, const char* nr_vcpus, > int check_host) > fprintf(stderr, "Error: Invalid argument.\n"); > return 1; > } > +if (nr_vcpus ==

[Xen-devel] [PATCH] tools/xl: refuse to set number of vcpus to 0 via xl vcpu-set

2018-09-03 Thread Juergen Gross
Trying to set the number of vcpus of a domain to 0 isn't refused. We should not allow that. Signed-off-by: Juergen Gross --- tools/xl/xl_vcpu.c | 4 1 file changed, 4 insertions(+) diff --git a/tools/xl/xl_vcpu.c b/tools/xl/xl_vcpu.c index aef486864c..45ac01beda 100644 ---