Re: [Qemu-devel] Powerpc regressions?

[Aurelien Jarno <aurelien@aurel32.net>]

On Sat, Jul 11, 2009 at 01:42:52AM +0200, Aurelien Jarno wrote:
> On Tue, Jul 07, 2009 at 05:48:02PM -0500, Rob Landley wrote:
> > If you grab this tarball:
> > 
> > http://impactlinux.com/fwl/downloads/binaries/system-image/system-image-powerpc.tar.bz2
> > 
> > Extract it, and ./run-emulator.sh.
> > 
> > This ran fine under svn 6657 (which is git 2d18e637e5ec).  The next commit screwed up openbios, but 
> > reverting openbios worked for a while.
> > 
> > In the last couple months, two problems have cropped up:
> > 
> > 1) -hda sets /dev/hdc instead of /dev/hda (which is the cdrom).
> 
> Wrong. -hda sets the first hard drive, that is on the internal PowerMAC
> controller. -hdc sets the first drive of the add-on IDE card that is
> used in this emulation to connect the CD-ROM, as the PowerMAC IDE
> controller emulation has still some bugs.
> 
> Then the Linux kernel decide to call the cdrom hda and the hard-disk
> hdc. You will get exactly the same result if you put an add-on card
> on a real PowerMAC machine. If you consider that a bug, you should
> report the bug to the Linux kernel.
> 
> > 2) The kernel panics running init:
> 
> This is a bug I don't reproduce with my Debian installation, but that I
> am able to reproduce with your image. 
> 
> You listed commit 6657 as the culprit, but is it only for both 1) and/or
> 2). It would be nice to know the first bad commit for 2).
> 
> If the problem lies in OpenBIOS, then the solution is to bisect on the
> OpenBIOS side.
> 

After some more tries, I am able to reproduce a similar problem using my
Debian image. The problem has been caused by this commit, which is not 
directly related to PowerPC. Something is wrong with the irq handling
there.

10c4c98ab7dc18169b37b76f6ea5e60ebe65222b is first bad commit
commit 10c4c98ab7dc18169b37b76f6ea5e60ebe65222b
Author: Gerd Hoffmann <kraxel@redhat.com>
Date:   Tue Jun 30 14:12:08 2009 +0200

    qdev: replace bus_type enum with bus_info struct.

    BusInfo is filled with name and size (pretty much like I did for
    DeviceInfo as well).  There is also a function pointer to print
    bus-specific device information to the monitor.  sysbus is hooked
    up there, I've also added a print function for PCI.

    Device creation is slightly modified as well:  The device type search
    loop now also checks the bus type while scanning the list instead of
    complaining thereafter in case of a mismatch.  This effectively gives
    each bus a private namespace for device names.

    Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
    Signed-off-by: Paul Brook <paul@codesourcery.com>

:040000 040000 10e33f4d7e538e49ef5de43c1a519b374e2f8678 69eaad1e16e461a411a69d4ed2b5d97e7633adff M      hw


-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net