Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Konrad Rzeszutek Wilk
On Wed, Nov 12, 2014 at 12:10:02PM +0100, Juergen Gross wrote: > Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to > cpupool0 before destroying it) introduced an error in the accounting > of cpupools regarding the number of domains. The number of domains > is nor adjusted when a domai

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Konrad Rzeszutek Wilk
On Wed, Nov 12, 2014 at 11:11:15AM +, George Dunlap wrote: > On Wed, Nov 12, 2014 at 11:10 AM, Juergen Gross wrote: > > Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to > > cpupool0 before destroying it) introduced an error in the accounting > > of cpupools regarding the number

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Juergen Gross
On 11/12/2014 12:10 PM, George Dunlap wrote: On Wed, Nov 12, 2014 at 10:40 AM, Juergen Gross wrote: Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to cpupool0 before destroying it) introduced an error in the accounting of cpupools regarding the number of domains. The number of dom

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread George Dunlap
On Wed, Nov 12, 2014 at 11:10 AM, Juergen Gross wrote: > Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to > cpupool0 before destroying it) introduced an error in the accounting > of cpupools regarding the number of domains. The number of domains > is nor adjusted when a domain is mo

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread George Dunlap
On Wed, Nov 12, 2014 at 10:40 AM, Juergen Gross wrote: > Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to > cpupool0 before destroying it) introduced an error in the accounting > of cpupools regarding the number of domains. The number of domains > is nor adjusted when a domain is mo

[Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Juergen Gross
Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to cpupool0 before destroying it) introduced an error in the accounting of cpupools regarding the number of domains. The number of domains is nor adjusted when a domain is moved to cpupool0 in kill_domain(). Correct this by introducing a

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Jürgen Groß
On 11/12/2014 11:46 AM, Andrew Cooper wrote: On 12/11/14 10:40, Juergen Gross wrote: Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to cpupool0 before destroying it) introduced an error in the accounting of cpupools regarding the number of domains. The number of domains is nor adju

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Jan Beulich
>>> On 12.11.14 at 11:46, wrote: > On 12/11/14 10:40, Juergen Gross wrote: >> --- a/xen/common/cpupool.c >> +++ b/xen/common/cpupool.c >> @@ -225,6 +225,35 @@ static int cpupool_destroy(struct cpupool *c) >> } >> >> /* >> + * Move domain to another cpupool >> + */ >> +static int cpupool_move_d

Re: [Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Andrew Cooper
On 12/11/14 10:40, Juergen Gross wrote: > Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to > cpupool0 before destroying it) introduced an error in the accounting > of cpupools regarding the number of domains. The number of domains > is nor adjusted when a domain is moved to cpupool0

[Xen-devel] [PATCH] Adjust number of domains in cpupools when destroying domain

2014-11-12 Thread Juergen Gross
Commit bac6334b51d9bcfe57ecf4a4cb5288348fcf044a (move domain to cpupool0 before destroying it) introduced an error in the accounting of cpupools regarding the number of domains. The number of domains is nor adjusted when a domain is moved to cpupool0 in kill_domain(). Correct this by introducing a