From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47234) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fiybg-0002bm-O7 for qemu-devel@nongnu.org; Fri, 27 Jul 2018 05:00:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fiybc-0004Uh-0j for qemu-devel@nongnu.org; Fri, 27 Jul 2018 05:00:16 -0400 Received: from 19.mo3.mail-out.ovh.net ([178.32.98.231]:35905) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fiybb-0004Rt-OX for qemu-devel@nongnu.org; Fri, 27 Jul 2018 05:00:11 -0400 Received: from player718.ha.ovh.net (unknown [10.109.159.157]) by mo3.mail-out.ovh.net (Postfix) with ESMTP id 38ADB1C367E for ; Fri, 27 Jul 2018 11:00:09 +0200 (CEST) Date: Fri, 27 Jul 2018 11:00:00 +0200 From: Greg Kurz Message-ID: <20180727110000.16f55218@bahia> In-Reply-To: References: <153252992640.319494.8451297710133862507.stgit@bahia.lan> <20180727052724.GJ3694@umbus.fritz.box> <20180727095452.45edee8b@bahia> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for-3.1] tests/cpu-plug-test: check CPU hotplug on ppc64 with KVM List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Thomas Huth Cc: David Gibson , qemu-devel@nongnu.org, =?UTF-8?B?Q8OpZHJpYw==?= Le Goater On Fri, 27 Jul 2018 10:18:14 +0200 Thomas Huth wrote: > On 07/27/2018 09:54 AM, Greg Kurz wrote: > > On Fri, 27 Jul 2018 15:27:24 +1000 > > David Gibson wrote: > > > >> On Wed, Jul 25, 2018 at 04:45:26PM +0200, Greg Kurz wrote: > >>> Commit b585395b655 fixed a regression introduced by some recent changes > >>> in the XICS code, that was causing QEMU to crash instantly during CPU > >>> hotplug with KVM. This is typically the kind of bug we'd like our > >>> test suite to detect before it gets merged. Unfortunately, the current > >>> tests run with '-machine accel=qtest' and don't exercise KVM specific > >>> paths in QEMU. > >>> > >>> This patch hence changes add_pseries_test_case() to launch QEMU with > >>> '-machine accel=kvm' if KVM is available. > >>> > >>> A notable consequence is that the guest will execute SLOF, but for some > >>> reasons SLOF sometimes hits a program exception. This causes the guest > >>> to loop forever and the test to be stuck. Since we don't need the guest > >>> to be truely running, let's pass -S to QEMU to avoid that. > >>> > >>> Also disable machine capabilities that could be unavailable in KVM, eg, > >>> when using PR KVM. > >>> > >>> Signed-off-by: Greg Kurz > >> > >> I'm pretty sure trying to change the accelerator on a qtest test just > >> doesn't make sense. We'd need a different approach for testing cpu > >> hotplug against kvm & tcg backends. > >> > > > > The test starts QEMU, triggers the CPU hotplug code with a QMP command > > and checks the command didn't fail (or QEMU didn't crash, as it would > > have before commit b585395b655a)... I really don't understand what > > is wrong with that... Please elaborate. > > For a "real" test, I think you'd need a guest OS that is reacting to the > hot plug events. So maybe this should rather be done in the avocado > framework instead? > The intent isn't a "real" test actually, but just to exercise the XICS-KVM paths in QEMU that get called during CPU hotplug. This cannot be achieved with the qtest accelerator. This patch would have revealed the regression in b585395b655a right away, with the simple 'make check' developers are expected to run before posting. Autotest has CPU hotplug tests, I don't know if avocado also has some, but in any case, this is a different story IMHO. Cheers, -- Greg > Thomas > >