[Qemu-devel] Re: [OpenBIOS] [commit] r791 - in trunk/openbios-devel: fs/grubfs include/fs packages

[Qemu-devel] Re: [OpenBIOS] [commit] r791 - in trunk/openbios-devel: fs/grubfs include/fs packages

[Aurelien Jarno]

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

Re: [Qemu-devel] Re: [OpenBIOS] [commit] r791 - in trunk/openbios-devel: fs/grubfs include/fs packages

[Mark Cave-Ayland]

Aurelien Jarno wrote:

> 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]

Ah okay, that'll be part of my recent tidy-up work - let me take a quick 
look. Thanks for the test image as that will really help :)


ATB,

Mark.

-- 
Mark Cave-Ayland - Senior Technical Architect
PostgreSQL - PostGIS
Sirius Corporation plc - control through freedom
http://www.siriusit.co.uk
t: +44 870 608 0063

Sirius Labs: http://www.siriusit.co.uk/labs

Re: [Qemu-devel] Re: [OpenBIOS] [commit] r791 - in trunk/openbios-devel: fs/grubfs include/fs packages

[Mark Cave-Ayland]

Aurelien Jarno wrote:

> 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]

Thanks for the report - this should now be fixed as of OpenBIOS r821.


ATB,

Mark.

-- 
Mark Cave-Ayland - Senior Technical Architect
PostgreSQL - PostGIS
Sirius Corporation plc - control through freedom
http://www.siriusit.co.uk
t: +44 870 608 0063

Sirius Labs: http://www.siriusit.co.uk/labs

Re: [Qemu-devel] Re: [OpenBIOS] [commit] r791 - in trunk/openbios-devel: fs/grubfs include/fs packages

[Aurelien Jarno]

On Tue, Jul 13, 2010 at 05:45:14PM +0100, Mark Cave-Ayland wrote:
> Aurelien Jarno wrote:
>
>> 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]
>
> Thanks for the report - this should now be fixed as of OpenBIOS r821.
>

That was fast ;-) Thanks for the fix, I'll commit this new version to
QEMU.

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