Re: [Qemu-ppc] standard test image not booting with qemu-system-ppc

[Mark Cave-Ayland]

On 03/02/14 20:02, Nitin Srivastava wrote:

> Hi ,
> I downloaded the latest qemu source from git and compiled it on my
> centos 6.5 machine.
> According to the following e-mail from this mailing list I tried the
> following
> _http://comments.gmane.org/gmane.comp.emulators.qemu/181171_
> Test case: download the squeeze standard image from
> _http://people.debian.org/~aurel32/qemu/powerpc/_
> <http://people.debian.org/%7Eaurel32/qemu/powerpc/>
> and run with
> qemu-system-ppc -hda debian_squeeze_powerpc_standard.qcow2
> but I get following error
> VNC server running on `::1:5900'
> > > =============================================================
> > > OpenBIOS 1.1 [Oct 2 2013 22:57]
> > > Configuration device id QEMU version 1 machine id 2
> > > CPUs: 1
> > > Memory: 128M
> > > UUID: 00000000-0000-0000-0000-000000000000
> > > CPU type PowerPC,750
> > > Not a bootable ELF image
> qemu: terminating on signal 2
> address@hidden:~:117#
> also please note that my qemu-system-ppc is latest, as its built from
> source.
> address@hidden:~:117#qemu-system-ppc -version
> QEMU emulator version 1.7.50, Copyright (c) 2003-2008 Fabrice Bellard
> address@hidden:~:118#
> Please help.
> Regds.
> Nitin

Hi Nitin,

Having just updated to git master, I now see this issue too. A quick 
session with git bisect shows the culprit is this commit:


address@hidden:~/src/qemu/git/qemu$ git bisect bad
360e607b88a23d378f6efaa769c76d26f538234d is the first bad commit
commit 360e607b88a23d378f6efaa769c76d26f538234d
Author: Stefano Stabellini <address@hidden>
Date:   Thu Jan 30 12:46:05 2014 +0000

    address_space_translate: do not cross page boundaries

The following commit:

commit 149f54b53b7666a3facd45e86eece60ce7d3b114
Author: Paolo Bonzini <address@hidden>
Date:   Fri May 24 12:59:37 2013 +0200

    memory: add address_space_translate

breaks Xen support in QEMU, in particular the Xen mapcache. The effect
is that one Windows XP installation out of ten would end up with BSOD.

The reason is that after this commit l in address_space_rw can span a
page boundary, however qemu_get_ram_ptr still calls xen_map_cache asking
to map a single page (if block->offset == 0).

Fix the issue by reverting to the previous behaviour: do not return a
length from address_space_translate_internal that can span a page
boundary.

Also in address_space_translate do not ignore the length returned by
address_space_translate_internal.

This patch should be backported to QEMU 1.6.x.

Signed-off-by: Stefano Stabellini <address@hidden>
Signed-off-by: Anthony Perard <address@hidden>
Tested-by: Paolo Bonzini <address@hidden>
Acked-by: Paolo Bonzini <address@hidden>
Cc: address@hidden


Stefano/Alex, is there any reason why this would break qemu-system-ppc?


ATB,

Mark.