From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Re: [PATCH] Adjust number of domains in cpupools when destroying domain Date: Wed, 12 Nov 2014 12:21:04 +0100 Message-ID: <546342A0.2080501@suse.com> References: <1415788840-19497-1-git-send-email-jgross@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap , Dietmar Hahn Cc: lutz.dube@ts.fujitsu.com, Jan Beulich , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org 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 domains >> is nor adjusted when a domain is moved to cpupool0 in kill_domain(). >> >> Correct this by introducing a cpupool function doing the move >> instead of open coding it by calling sched_move_domain(). >> >> Signed-off-by: Juergen Gross >> Tested-by: Dietmar Hahn > > Juergen / Dietmar -- do either of you have a reasonably complete set > of tests for cpupools? It seems like even basic corner cases (like > shutting down a domain in a pool and then destroying a pool) aren't > being tested. > > It would be really good if someone could try to do a more thorough > test before the 4.5 release. It shouldn't be too hard to write a > script to test a lot of this functionality programmatically. For the xm toolstack we had some tests at Fujitsu. Dietmar, you could ask Lutz for advice. He might still have the scripts somewhere. They should be easily adaptable to xl. In case you don't have time to try them would you send them to me? Juergen