From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=46982 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1OYZlf-0002Ia-LZ
	for qemu-devel@nongnu.org; Tue, 13 Jul 2010 03:19:04 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <aurelien@aurel32.net>) id 1OYZle-00078A-Bl
	for qemu-devel@nongnu.org; Tue, 13 Jul 2010 03:19:03 -0400
Received: from hall.aurel32.net ([88.191.82.174]:54862)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <aurelien@aurel32.net>) id 1OYZle-000781-5N
	for qemu-devel@nongnu.org; Tue, 13 Jul 2010 03:19:02 -0400
Date: Tue, 13 Jul 2010 09:19:00 +0200
From: Aurelien Jarno <aurelien@aurel32.net>
Message-ID: <20100713071900.GB18814@hall.aurel32.net>
References: <E1OM5t7-0006Sr-2p@ra.coresystems.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-15
Content-Disposition: inline
In-Reply-To: <E1OM5t7-0006Sr-2p@ra.coresystems.de>
Sender: Aurelien Jarno <aurelien@aurel32.net>
Subject: [Qemu-devel] Re: [OpenBIOS] [commit] r791 - in
	trunk/openbios-devel: fs/grubfs include/fs packages
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.nongnu.org/archive/html/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: The OpenBIOS Mailinglist <openbios@openbios.org>
Cc: qemu-devel@nongnu.org

On Tue, Jun 08, 2010 at 10:59:09PM +0200, repository service wrote:
> Author: mcayland
> Date: Tue Jun  8 22:59:08 2010
> New Revision: 791
> URL: http://tracker.coreboot.org/trac/openbios/changeset/791
> 
> Log:
> As documented on the mailing lists, change the interposition order of the disk packages from:
> 
> cdrom -> deblocker -> disk-label -> (misc-files | sun-parts | pc-parts | mac-parts) 
> 
> to:
> 
> cdrom -> deblocker -> disk-label -> (sun-parts | pc-parts | mac-parts) -> misc-files 
> 
> This makes the overall code much simpler, since instead of passing get-info structures between the different layers to work out 
> the current device offset, each package can simply maintain its own offsets and bubble them back up to the parent.
> 
> There are also many, many bugfixes present is patch, some of which are given below:
> 
> - Add debugging in /packages/misc-files, augment debugging in all other disk packages
> - Fix alignment issue in /packages/pc-parts so it works correctly on strictly align archs, e.g. SPARC64
> - Fix parsing of [<id>,][filename] arguments to load (in preparation for next patch set)
> - Fix byte-swapping issues in /packages/mac-parts (in theory it should now work regardless of endian)
> 

Unfortunately this commit has broken PowerPC boot using the Quik
bootloader. CD-ROM boot is not affected. This can easily be reproduced
using the image from:

http://people.debian.org/~aurel32/qemu/powerpc/debian_lenny_powerpc_small.qcow2

I have tried with the latest available revision, and the problem is
still there. Does anyone has an idea of the problem before I start
debugging it?

[CCed: qemu-devel@nongnu.org as this version is now used in QEMU]

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