May 2009 - OpenBIOS - mail.coreboot.org

QEMU OpenBIOS booting?
by Steven Noonan May 24, 2010

May 24, 2010

Hi folks, I've been trying to use OpenBIOS with QEMU, and I've had trouble getting it to boot any PowerPC-based OS discs (Linux, Mac OS X, etc). OpenBIOS just doesn't seem to find anything bootable. If I try booting QEMU with -nographic, I get this output (I get the same with a self-built openbios-qemu.elf from the OpenBIOS SVN as well): ---- snip ---- >> ============================================================= >> OpenBIOS 1.0 [Apr 12 2009 00:55] >> Configuration device id QEMU version 1 machine id 1 >> CPUs: 1 >> Memory: 128M >> UUID: 00000000-0000-0000-0000-000000000000 >> CPU type PowerPC,970FX Welcome to OpenBIOS v1.0 built on Apr 12 2009 00:55 >> ELF - yaboot_startup: Entering boot, no path >> ELF - try_bootinfo: Trying hd:0,ppc\bootinfo.txt >> ELF - try_bootinfo: Can't open hd:0,ppc\bootinfo.txt >> ELF - try_path: Trying hd:2,\ofclient >> ELF - try_path: Can't open hd:2,\ofclient >> ELF - try_path: Trying hd:2,\yaboot conf=hd:2,\yaboot.conf >> ELF - try_path: Can't open hd:2,\yaboot >> *** Boot failure! No secondary bootloader specified *** ---- snip ---- Laurent Vivier suggested in this conversation in February (http://lists.openbios.org/pipermail/openbios/2009-February/003476.html) that there could be a bug in partition map decoding. Has such a bug been confirmed? I was trying to find the bug myself (I am an experienced C/C++ programmer), but the bug seems to be subtle, and my lack of familiarity with the code base doesn't help much. I've been trying to figure this out for 7 hours straight now... Any advice or other help would be much appreciated! - Steven

13 55

[PATCH] sparc64: clear unused registers when calling client image
by Igor Kovalenko May 30, 2009

May 30, 2009

This change seems to fix the problem here with silo not being able to deal with bss data. The problem is that silo does access to an address which is calculated from %i3 + bss offset, so %i3 must be somehow initialized. To do this I created start_client_image() helper which initializes clear context, fills it with stack pointer and client interface handler pointer then switches to it. According to docs all registers not used to convey data to client code must be cleared. Since my sparc64-elf-gcc cross compiler managed to discard stores to __context in switch_to() I had to make __context a volatile pointer. Signed-off-by: Igor Kovalenko <igor.v.kovalenko(a)gmail.com> -- Kind regards, Igor V. Kovalenko

2 3

r500 - trunk/openbios-devel/arch/sparc64
by svn＠openbios.org May 30, 2009

May 30, 2009

Author: blueswirl Date: 2009-05-30 09:43:58 +0200 (Sat, 30 May 2009) New Revision: 500 Modified: trunk/openbios-devel/arch/sparc64/aoutload.c Log: sparc64: clear unused registers when calling client image (Igor Kovalenko) Modified: trunk/openbios-devel/arch/sparc64/aoutload.c =================================================================== --- trunk/openbios-devel/arch/sparc64/aoutload.c 2009-05-30 07:41:48 UTC (rev 499) +++ trunk/openbios-devel/arch/sparc64/aoutload.c 2009-05-30 07:43:58 UTC (rev 500) @@ -132,6 +132,9 @@ extern int of_client_interface( int *params ); entry = (void *) addr_fixup(start); + + __asm__ __volatile__("clr %i3\n"); + image_retval = entry(0, 0, 0, 0, (unsigned long)&of_client_interface); } #endif

1 0

PATH: Fix b(field) Fcode evaluation
by Mark Cave-Ayland May 30, 2009

May 30, 2009

Hi everyone, This patch fixes the Fcode evaluation of b(field) which was implemented incorrectly. With this patch applied, corruption seen in several of the package words for Milax/Martux has now disappeared. Included below are the Fcode test cases I used (courtesy of Milax): \ Test Fcode struct...field...constant (no headers) \ Forth: struct /x field >filesize /x field >offset /x field >block0 constant /file-record 0 >offset . here cc c, \ offset16 a5 c, \ 0 02 c, 48 c, \ /x b5 c, 08 c, 00 c, \ new-token be c, \ b(field) 02 c, 48 c, \ /x b5 c, 08 c, 01 c, \ new-token be c, \ b(field) 02 c, 48 c, \ /x b5 c, 08 c, 02 c, \ new-token be c, \ b(field) b5 c, 08 c, 03 c, \ new-token ba c, \ b(constant) a5 c, \ 0 08 c, 01 c, \ Fcode #801 9d c, \ Display stack 00 c, \ End \ Headers version here cc c, \ offset16 a5 c, \ 0 02 c, 48 c, \ /x b6 c, 09 c, 3e c, 66 c, 69 c, 6c c, 65 c, 73 c, 69 c, 7a c, 65 c, \ named-token (>filesize) 08 c, 00 c, be c, \ b(field) 02 c, 48 c, \ /x b6 c, 07 c, 3e c, 6f c, 66 c, 66 c, 73 c, 65 c, 74 c, \ named-token (>offset) 08 c, 01 c, be c, \ b(field) 02 c, 48 c, \ /x b6 c, 07 c, 3e c, 62 c, 6c c, 6f c, 63 c, 6b c, 30 c, \ named-token (>block0) 08 c, 02 c, be c, \ b(field) b6 c, 0c c, 2f c, 66 c, 69 c, 6c c, 65 c, 2d c, 72 c, 65 c, 63 c, 6f c, \ named-token (/file-record) 72 c, 64 c, 08 c, 03 c, ba c, \ b(constant) a5 c, \ 0 08 c, 01 c, \ Fcode #801 9d c, \ Display stack 00 c, \ End 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

2 1

r499 - trunk/openbios-devel/forth/device
by svn＠openbios.org May 30, 2009

May 30, 2009

Author: blueswirl Date: 2009-05-30 09:41:48 +0200 (Sat, 30 May 2009) New Revision: 499 Modified: trunk/openbios-devel/forth/device/fcode.fs Log: Fix b(field) Fcode evaluation (Mark Cave-Ayland) Modified: trunk/openbios-devel/forth/device/fcode.fs =================================================================== --- trunk/openbios-devel/forth/device/fcode.fs 2009-05-24 11:15:48 UTC (rev 498) +++ trunk/openbios-devel/forth/device/fcode.fs 2009-05-30 07:41:48 UTC (rev 499) @@ -307,7 +307,9 @@ \ Defines type of new FCode function as field. : b(field) - $create + 6 , + ['] noop , + reveal over , + does>

1 0

PATCH: Fix b(case)...b(of)...b(endof)...b(endcase)
by Mark Cave-Ayland May 25, 2009

May 25, 2009

Hi everyone, It seems that b(case)...b(of)...b(endof)...b(endcase) was broken in exactly the same way as the b(do)...b(loop) commands during Fcode evaluation. This patch fixes this, which now allows the Martux/Milax CDROM images to fail gracefully rather than segfaulting: OpenBIOS for Sparc64 Configuration device id QEMU version 1 machine id 0 CPUs: 1 x SUNW,UltraSPARC-II UUID: 00000000-0000-0000-0000-000000000000 Welcome to OpenBIOS v1.0 built on May 24 2009 10:37 Type 'help' for detailed information [sparc64] Booting file 'disk' with parameters '' Not a bootable ELF image Not a Linux kernel image Not a bootable a.out image Loading FCode image... Loaded 7392 bytes entry point is 0x4000 Evaluating FCode... isn't unique. isn't unique. claim virt = ffffffffffffffff size = 0 align = 0 seek failed Can't mount root byte-load: exception caught! 0 > 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

2 2

r498 - trunk/openbios-devel/forth/bootstrap
by svn＠openbios.org May 24, 2009

May 24, 2009

Author: blueswirl Date: 2009-05-24 13:15:48 +0200 (Sun, 24 May 2009) New Revision: 498 Modified: trunk/openbios-devel/forth/bootstrap/bootstrap.fs Log: Fix b(case)...b(of)...b(endof)...b(endcase) (Mark Cave-Ayland) Modified: trunk/openbios-devel/forth/bootstrap/bootstrap.fs =================================================================== --- trunk/openbios-devel/forth/bootstrap/bootstrap.fs 2009-05-24 11:04:06 UTC (rev 497) +++ trunk/openbios-devel/forth/bootstrap/bootstrap.fs 2009-05-24 11:15:48 UTC (rev 498) @@ -318,7 +318,7 @@ ['] then execute loop execute-tmp-comp -; compile-only +; immediate : of 1 + >r @@ -327,13 +327,13 @@ ['] if execute ['] drop , r> - ; compile-only + ; immediate : endof >r ['] else execute r> - ; compile-only + ; immediate \

1 0

PATCH: Implement push-package/pop-package
by Mark Cave-Ayland May 24, 2009

May 24, 2009

Hi everyone, This patch implements the missing push-package/pop-package words which seem to be included in various Fcode bootloaders (even though they are not part of the official spec). With this patch applied, I now get the following on my Solaris image/Qemu boot: OpenBIOS for Sparc64 Configuration device id QEMU version 1 machine id 0 CPUs: 1 x SUNW,UltraSPARC-II UUID: 00000000-0000-0000-0000-000000000000 Welcome to OpenBIOS v1.0 built on May 24 2009 10:08 Type 'help' for detailed information [sparc64] Booting file 'disk' with parameters '' Not a bootable ELF image Not a Linux kernel image Not a bootable a.out image Loading FCode image... Loaded 5936 bytes entry point is 0x4000 Evaluating FCode... Loading package 1.4 04 Aug 1995 13:02:54. open isn't unique. FCode UFS Reader 1.12 00/07/17 15:48:16. Loading: /platform//ufsboot claim virt = 1 size = 200 align = 0 Loading: /platform/sun4u/ufsboot Boot load failed. ok 0 > So it looks as if we're now at the point where we need to start implementing the client interface claim/release functions. 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

3 5

r497 - trunk/openbios-devel/forth/device
by svn＠openbios.org May 24, 2009

May 24, 2009

Author: blueswirl Date: 2009-05-24 13:04:06 +0200 (Sun, 24 May 2009) New Revision: 497 Modified: trunk/openbios-devel/forth/device/package.fs trunk/openbios-devel/forth/device/table.fs Log: Implement push-package/pop-package (Mark Cave-Ayland) Modified: trunk/openbios-devel/forth/device/package.fs =================================================================== --- trunk/openbios-devel/forth/device/package.fs 2009-05-23 16:03:20 UTC (rev 496) +++ trunk/openbios-devel/forth/device/package.fs 2009-05-24 11:04:06 UTC (rev 497) @@ -252,3 +252,32 @@ then s" map-out" $call-parent ; + + +\ Deprecated functions (required for compatibility with older loaders) + +variable package-stack-pos 0 package-stack-pos ! +create package-stack 8 cells allot + +: push-package ( phandle -- ) + \ Throw an error if we attempt to push a full stack + package-stack-pos @ 8 >= if + ." cannot push-package onto full stack" cr + -99 throw + then + active-package + package-stack-pos @ /n * package-stack + ! + package-stack-pos @ 1 + package-stack-pos ! + active-package! + ; + +: pop-package ( -- ) + \ Throw an error if we attempt to pop an empty stack + package-stack-pos @ 0 = if + ." cannot pop-package from empty stack" cr + -99 throw + then + package-stack-pos @ 1 - package-stack-pos ! + package-stack-pos @ /n * package-stack + @ + active-package! + ; Modified: trunk/openbios-devel/forth/device/table.fs =================================================================== --- trunk/openbios-devel/forth/device/table.fs 2009-05-23 16:03:20 UTC (rev 496) +++ trunk/openbios-devel/forth/device/table.fs 2009-05-24 11:04:06 UTC (rev 497) @@ -280,7 +280,8 @@ ['], ms ['], finish-device ['], decode-phys \ 128 - 2 n['], reserved-fcode + ['], push-package + ['], pop-package ['], interpose \ extension (recommended practice) 4 n['], reserved-fcode ['], map-low

1 0

r496 - trunk/openbios-devel/arch/sparc64
by svn＠openbios.org May 23, 2009

May 23, 2009

Author: blueswirl Date: 2009-05-23 18:03:20 +0200 (Sat, 23 May 2009) New Revision: 496 Modified: trunk/openbios-devel/arch/sparc64/lib.c Log: Add claim and release client methods Modified: trunk/openbios-devel/arch/sparc64/lib.c =================================================================== --- trunk/openbios-devel/arch/sparc64/lib.c 2009-05-23 15:31:31 UTC (rev 495) +++ trunk/openbios-devel/arch/sparc64/lib.c 2009-05-23 16:03:20 UTC (rev 496) @@ -325,6 +325,7 @@ align = POP(); size = POP(); virt = POP(); + printk("claim virt = %lx size = %lx align = %lx\n", virt, size, align); PUSH(virt); // XXX } @@ -339,6 +340,7 @@ size = POP(); virt = POP(); + printk("release virt = %lx size = %lx\n", virt, size); // XXX } @@ -504,4 +506,9 @@ push_str("translations"); fword("property"); + + push_str("/openprom/client-services"); + fword("find-device"); + bind_func("claim", mmu_claim); + bind_func("release", mmu_release); }

1 0