From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44545) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bDmux-0002TO-1o for qemu-devel@nongnu.org; Fri, 17 Jun 2016 02:06:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bDmur-0001nz-Pz for qemu-devel@nongnu.org; Fri, 17 Jun 2016 02:06:09 -0400 Date: Fri, 17 Jun 2016 16:07:18 +1000 From: David Gibson Message-ID: <20160617060718.GC19581@voom.fritz.box> References: <1465912676-6089-1-git-send-email-thuth@redhat.com> <20160615031018.GD4882@voom.fritz.box> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LwW0XdcUbUexiWVK" Content-Disposition: inline In-Reply-To: <20160615031018.GD4882@voom.fritz.box> Subject: Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Thomas Huth Cc: agraf@suse.de, Mark Cave-Ayland , Markus Armbruster , qemu-devel@nongnu.org, blauwirbel@gmail.com, qemu-ppc@nongnu.org, Artyom Tarasenko --LwW0XdcUbUexiWVK Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 15, 2016 at 01:10:18PM +1000, David Gibson wrote: > On Tue, Jun 14, 2016 at 03:57:56PM +0200, Thomas Huth wrote: > > Since the mac99 and g3beige PowerPC machines recently broke without > > being noticed, it would be good to have a tester for "make check" > > that detects such issues immediately. A simple way to test the firmware > > of these machines is to use the "-prom-env" parameter of QEMU. This > > parameter can be used to put some Forth code into the 'boot-command' > > firmware variable which then can signal success to the tester by > > writing a magic value to a known memory location. And since some of the > > Sparc machines are also using OpenBIOS, they are now tested with this > > prom-env-tester, too. > >=20 > > Reviewed-by: Markus Armbruster > > Signed-off-by: Thomas Huth > > --- > > v2: Removed unnecessary include statements (as suggested by Markus) >=20 > Beautiful, I've applied this to ppc-for-2.7, assuming I don't get an > objection to taking this through my tree. Ugh.. turns out this fails on sparc64 target on a 32-bit x86 host. Specifically it trips the tcg_abort() at the end of tcg_reg_alloc() (tcg/tcg.c). I'm reasonably confident this is a pre-existing bug, just triggered by this test, but in the interests of getting this up and running on the platforms where it is working, I've disabled the testcase on sparc64 for now. Mark, if you could debug the sparc64-on-i386 failure at some point, that would be helpful. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --LwW0XdcUbUexiWVK Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXY5OWAAoJEGw4ysog2bOSyCoP/086A/dQBPkFZrrdeWpnJ7FD KR5ptYOg18lELpfYXclqefnOFkkR5PjHHUZrfSdkaXkU0HrGxQPv2VRt/mMVcCtX T29lwTdZUc+Iy9uZZyL3PmOoYyxruzijnVVdwoLUYuq+i7ifG9557jJDFnhb9qIP rv+SJFa38uOhYKWM9O/VDz2D2uplyN+mF4Dht8eQpVrOCN1+P1FH98NiOVqHoM3b 325FnZzlIVemqipeC8MrxL+AAGBg1W+rk+wtZdFsd7E8ag+DF18QVgu6wgLxw4nE wYB4fGUFfQmy+mHDhbhFOjOg5UjRB7Feu3OuZ6w4nKBOS66KDKbM8o8qX3fxg460 yFoEA/iyPy0uYxUb3CFiRIO5QSuzdcqUiUK0dttTF87HU6g2tuQsEIp/02NpY0Ma ZlqREtgr0PyHbWo8pgeufI9cW9W8H1ac4dueti0yHYL8P3WGXz80mYGdB5bx0gqi xQYOosnB4q/Hd257Z7Q53LOFxg1dmXBIW3PMCEjLnGllbpOI/zQhGJ3bcHhZ2I2O Wwg7rlX/JcLpPeZCmF4sWZ3fOQ+rMUlrzIKYiZmdrid+q/8p87mtVzxP4BpAmFPo +p3ZUxfbXlPTGyIAFGfkKS+/czFnCQWEIQ065nd+aY6Y2Wfb3xLBvDXWliLcoSG6 e/FWNrbFZ0Q5iTENVbsS =v3rj -----END PGP SIGNATURE----- --LwW0XdcUbUexiWVK--