On Wed, Nov 11, 2009 at 8:57 PM, Glauber Costa <glom...@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 <aurel...@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.