From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1N8KJK-0004gn-Iu
	for qemu-devel@nongnu.org; Wed, 11 Nov 2009 16:01:03 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1N8KJF-0004Xp-Tx
	for qemu-devel@nongnu.org; Wed, 11 Nov 2009 16:01:01 -0500
Received: from [199.232.76.173] (port=48570 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1N8KJF-0004XQ-7t
	for qemu-devel@nongnu.org; Wed, 11 Nov 2009 16:00:57 -0500
Received: from mail-yw0-f176.google.com ([209.85.211.176]:33429)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <blauwirbel@gmail.com>) id 1N8KJE-0003bu-QR
	for qemu-devel@nongnu.org; Wed, 11 Nov 2009 16:00:56 -0500
Received: by ywh6 with SMTP id 6so1335125ywh.4
	for <qemu-devel@nongnu.org>; Wed, 11 Nov 2009 13:00:56 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <20091111185714.GA11146@mothafucka.localdomain>
References: <20091110205040.GF9052@hall.aurel32.net>
	<f43fc5580911101319k612647eah7e5fbceec1278b10@mail.gmail.com>
	<20091111185714.GA11146@mothafucka.localdomain>
From: Blue Swirl <blauwirbel@gmail.com>
Date: Wed, 11 Nov 2009 23:00:36 +0200
Message-ID: <f43fc5580911111300m54a699c7re6b97e320aaa8b6d@mail.gmail.com>
Subject: Re: [Qemu-devel] arm, mips and mipsel broken
Content-Type: text/plain; charset=UTF-8
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Glauber Costa <glommer@redhat.com>
Cc: qemu-devel@nongnu.org, Aurelien Jarno <aurelien@aurel32.net>

On Wed, Nov 11, 2009 at 8:57 PM, Glauber Costa <glommer@redhat.com> wrote:
> On Tue, Nov 10, 2009 at 11:19:40PM +0200, Blue Swirl wrote:
>> On Tue, Nov 10, 2009 at 10:50 PM, Aurelien Jarno <aurelien@aurel32.net> wrote:
>> > Please note that at least qemu-system-arm, qemu-system-mips and
>> > qemu-system-mipsel are broken by this commit:
>>
>> Given that none of the devices touched by the commit should be used by
>> these targets, the breakage comes from just the single new call to
>> qemu_system_reset in vl.c. This means that the reset functions for
>> those boards and devices must be awfully buggy.
>>
>> I think the easiest solution is to surround the call by
>> #if defined(TARGET_I386) || defined(TARGET_PPC) || defined(TARGET_SPARC)
>> #endif
>> until the devices have been fixed.
>
> if we don't call that, how can we be sure that the devices are in fact reset?
>
> We need a call to qemu_system_reset somewhere before we can start the machine

Strange, reverting the commits make PPC crash on boot, but
system_reset starts the system:
qemu: warning: could not load VGA bios 'video.x'
invalid/unsupported opcode: 00 - 00 - 00 (00000000) 00000000 0
invalid/unsupported opcode: 00 - 00 - 00 (00000000) 00000700 0
QEMU 0.11.50 monitor - type 'help' for more information
(qemu) system_reset
(qemu)
>> =============================================================
>> OpenBIOS 1.0 [Oct 18 2009 12:04]
>> Configuration device id QEMU version 1 machine id 2
>> CPUs: 1
>> Memory: 128M
>> UUID: 00000000-0000-0000-0000-000000000000
>> CPU type PowerPC,750
Welcome to OpenBIOS v1.0 built on Oct 18 2009 12:04

I'm not sure how to fix this.