OpenBIOS March 2013

openbios@openbios.org

10 participants
31 discussions

Re: [OpenBIOS] Booting SunOS from OpenBIOS
by Tarl Neustaedter 26 Mar '13

26 Mar '13

On 2013-Mar-26 04:33 , Artyom Tarasenko wrote: > Not a bootable ELF image > Loading a.out image... > Loaded 7680 bytes > entry point is 0x4000 > bootpath: /iommu/sbus/espdma/esp/sd@0,0 > > Jumping to entry point 00004000 for type 00000005... > switching to new context: > Unhandled Exception 0x00000007 > PC = 0x00401a04 NPC = 0x00401a08 > Stopping execution > > > Is the OpenBIOS guess correct and SunOS 4.x actually does have an a.out loader? It shouldn't be a.out. The bootloader (on blocks 1-15) of a Solaris disk is an fcode file (compiled forth).

2 1

Re: [OpenBIOS] Booting SunOS from OpenBIOS
by Mark Cave-Ayland 26 Mar '13

26 Mar '13

On 26/03/13 08:33, Artyom Tarasenko wrote: > Hi Mark, > > [adding the OpenBIOS mailing list, hoping to get some more feedback] > > by the way, currently Jean Michel has to use the proprietary OBP. > There seems to be a problem with loading the SunOS 4.x under OpenBIOS: > > $ sparc-softmmu/qemu-system-sparc -L pc-bios -nographic -hda disk-solaris-1.1.2 > > Configuration device id QEMU version 1 machine id 32 > CPUs: 1 x FMI,MB86904 > UUID: 00000000-0000-0000-0000-000000000000 > Welcome to OpenBIOS v1.0 built on Aug 19 2012 13:36 > Type 'help' for detailed information > Trying disk... > Not a bootable ELF image > Loading a.out image... > Loaded 7680 bytes > entry point is 0x4000 > bootpath: /iommu/sbus/espdma/esp/sd@0,0 > > Jumping to entry point 00004000 for type 00000005... > switching to new context: > Unhandled Exception 0x00000007 > PC = 0x00401a04 NPC = 0x00401a08 > Stopping execution > > > Is the OpenBIOS guess correct and SunOS 4.x actually does have an a.out loader? Yes, that looks good to me. From memory the Solaris 8 bootloader is an a.out bootloader based at 0x4000, and the kernel loads at 0x400000 so you're definitely getting into the kernel image. The quick and easy way to get a rough idea as to where the failure lies is to copy the kernel ELF image from your Solaris disk image and run objdump on it to find out the name of the function where the exception occurs based upon the exception address. If you're already in the kernel panic() function, then use a sparc-linux-gdb attached to QEMU with a breakpoint to get a stack backtrace at the exception point, which again should give you an idea of where the issue is based upon the function names. HTH, Mark.

2 1

Booting SunOS from OpenBIOS
by Artyom Tarasenko 26 Mar '13

26 Mar '13

Hi Mark, [adding the OpenBIOS mailing list, hoping to get some more feedback] by the way, currently Jean Michel has to use the proprietary OBP. There seems to be a problem with loading the SunOS 4.x under OpenBIOS: $ sparc-softmmu/qemu-system-sparc -L pc-bios -nographic -hda disk-solaris-1.1.2 Configuration device id QEMU version 1 machine id 32 CPUs: 1 x FMI,MB86904 UUID: 00000000-0000-0000-0000-000000000000 Welcome to OpenBIOS v1.0 built on Aug 19 2012 13:36 Type 'help' for detailed information Trying disk... Not a bootable ELF image Loading a.out image... Loaded 7680 bytes entry point is 0x4000 bootpath: /iommu/sbus/espdma/esp/sd@0,0 Jumping to entry point 00004000 for type 00000005... switching to new context: Unhandled Exception 0x00000007 PC = 0x00401a04 NPC = 0x00401a08 Stopping execution Is the OpenBIOS guess correct and SunOS 4.x actually does have an a.out loader? Artyom -- Regards, Artyom Tarasenko solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu

1 0

[patch] Add Mac OS X building directions to readme file
by Programmingkid 22 Mar '13

22 Mar '13

This patch gives the user a link in the README file, to directions on how to build on Mac OS X. The current directions in the README file are not enough. signed-off-by: John Arbuckle <programmingkidx(a)gmail.com> Index: README =================================================================== --- README (revision 1102) +++ README (working copy) @@ -35,6 +35,10 @@ make run - run unix example. + +*Please refer to these directions when building on Mac OS X: +http://www.openfirmware.info/How_to_build_OpenBIOS_on_Mac_OS_X + How OpenBIOS works ------------------

1 0

Re: [OpenBIOS] [PATCH 1/2] loader: implement new loader type for CHRP/Apple partition bootcode (%BOOT)
by Programmingkid 22 Mar '13

22 Mar '13

On Mar 22, 2013, at 2:05 PM, Mark Cave-Ayland wrote: > On 22/03/13 17:54, Programmingkid wrote: > >> I did a "svn revert -R ." to make it completely like the online repo. Then I copied and paste your patch from gmail to a text file. Then I applied it using the patch command like this: "patch -p2< patch.txt". > > Maybe your mailer is mangling the raw patchset? Does the attached work? > > > ATB, > > Mark. > <0001-loader-implement-new-loader-type-for-CHRP-Apple-part.patch><0002-PPC-Add-the-old-world-bootcode-device-dev-BOOT-to-th.patch> Thanks for the files. They successfully patched my repo. I was able to boot from my quik enabled debian hard drive image file and from a Debian and Apple cd. The only issue I saw was with this message being printed in the serial console: >> Not a bootable ELF image. Other than that the patches look good.

1 0

[PATCH V2 1/5] allow powerpc-unknown-linux-gnu- toolchain prefix
by Amadeusz Sławiński 22 Mar '13

22 Mar '13

crossdev on gentoo builds crosscompiler with powerpc-unknown-linux-gnu- prefix Signed-off-by: Amadeusz Sławiński <amade(a)asmblr.net> --- config/scripts/switch-arch | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/scripts/switch-arch b/config/scripts/switch-arch index a9c911b..28b3b45 100755 --- a/config/scripts/switch-arch +++ b/config/scripts/switch-arch @@ -97,7 +97,7 @@ archname() select_prefix() { - for TARGET in ${1}-linux-gnu- ${1}-linux- ${1}-elf- ${1}-eabi- + for TARGET in ${1}-unknown-linux-gnu- ${1}-linux-gnu- ${1}-linux- ${1}-elf- ${1}-eabi- do if type ${TARGET}gcc > /dev/null 2>&1 then -- 1.8.1.5

1 4

[PATCH] ppc: Fix is_apple() for PReP
by Andreas Färber 22 Mar '13

22 Mar '13

It was returning true for ARCH_PREP although not an Apple device. Limit it to Old World and New World Macs. Signed-off-by: Andreas Färber <andreas.faerber(a)web.de> --- arch/ppc/qemu/init.c | 2 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/arch/ppc/qemu/init.c b/arch/ppc/qemu/init.c index a8fc320..8efbeeb 100644 --- a/arch/ppc/qemu/init.c +++ b/arch/ppc/qemu/init.c @@ -76,7 +76,7 @@ enum { int is_apple(void) { - return 1; + return is_oldworld() || is_newworld(); } int is_oldworld(void) -- 1.7.10.4

2 1

[RFC PATCH 0/2] implement device nodes needed by xnu kernel
by Amadeusz Sławiński 22 Mar '13

22 Mar '13

Following patches implement /uni-n and /pci/mac-io/via-cuda/power-mgt With them xnu kernel no longer panics when recognizing machine type. Amadeusz

3 6

[PATCH 1/2] loader: implement new loader type for CHRP/Apple partition bootcode (%BOOT)
by Mark Cave-Ayland 22 Mar '13

22 Mar '13

One of the primary reasons that issues with the quik bootloader were not detected earlier was because arch/ppc/qemu/main.c has a separate code path for forcing an old-world boot when -boot c is passed to QEMU. This commit implements the bootcode loader as a core OpenBIOS loader which enables old-world payloads such as quik to be executed using: load hd:,%BOOT go Note that we also fix a bug in mac-parts.c to ensure that we don't try and interpose a filesystem package when %BOOT is passed as a filename. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk> CC: Alexander Graf <agraf(a)suse.de> CC: Andreas Färber <afaerber(a)suse.de> --- openbios-devel/config/examples/ppc_config.xml | 1 + openbios-devel/forth/debugging/client.fs | 4 ++ openbios-devel/include/libopenbios/bootcode_load.h | 22 ++++++ openbios-devel/libopenbios/bootcode_load.c | 71 ++++++++++++++++++++ openbios-devel/libopenbios/build.xml | 1 + openbios-devel/libopenbios/load.c | 13 ++++ openbios-devel/packages/mac-parts.c | 16 ++--- 7 files changed, 119 insertions(+), 9 deletions(-) create mode 100644 openbios-devel/include/libopenbios/bootcode_load.h create mode 100644 openbios-devel/libopenbios/bootcode_load.c diff --git a/openbios-devel/config/examples/ppc_config.xml b/openbios-devel/config/examples/ppc_config.xml index ccb616d..5bb789f 100644 --- a/openbios-devel/config/examples/ppc_config.xml +++ b/openbios-devel/config/examples/ppc_config.xml @@ -26,6 +26,7 @@ <option name="CONFIG_VGA_DEPTH" type="integer" value="8"/> <option name="CONFIG_LOADER_AOUT" type="boolean" value="false"/> <option name="CONFIG_LOADER_BOOTINFO" type="boolean" value="true"/> + <option name="CONFIG_LOADER_BOOTCODE" type="boolean" value="true"/> <option name="CONFIG_LOADER_ELF" type="boolean" value="true"/> <option name="CONFIG_LOADER_FCODE" type="boolean" value="false"/> <option name="CONFIG_LOADER_FORTH" type="boolean" value="false"/> diff --git a/openbios-devel/forth/debugging/client.fs b/openbios-devel/forth/debugging/client.fs index 24b1ec7..9da9b33 100644 --- a/openbios-devel/forth/debugging/client.fs +++ b/openbios-devel/forth/debugging/client.fs @@ -27,6 +27,9 @@ create saved-program-state saved-program-state.size allot variable state-valid 0 state-valid ! +variable want-bootcode +0 want-bootcode ! + variable file-size : !load-size file-size ! ; @@ -44,6 +47,7 @@ variable file-size 5 constant aout 10 constant fcode 11 constant forth +12 constant bootcode : init-program ( -- ) diff --git a/openbios-devel/include/libopenbios/bootcode_load.h b/openbios-devel/include/libopenbios/bootcode_load.h new file mode 100644 index 0000000..147783a --- /dev/null +++ b/openbios-devel/include/libopenbios/bootcode_load.h @@ -0,0 +1,22 @@ +/* + * Creation Date: <2010/03/22 18:00:00 mcayland> + * Time-stamp: <2010/03/22 18:00:00 mcayland> + * + * <bootcode_load.h> + * + * Raw bootcode (%BOOT) loader + * + * Copyright (C) 2013 Mark Cave-Ayland (mark.cave-ayland(a)ilande.co.uk) + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation + * + */ + +#ifndef _H_BOOTCODELOAD +#define _H_BOOTCODELOAD + +extern int bootcode_load(ihandle_t dev); + +#endif /* _H__H_BOOTCODELOAD */ diff --git a/openbios-devel/libopenbios/bootcode_load.c b/openbios-devel/libopenbios/bootcode_load.c new file mode 100644 index 0000000..3644997 --- /dev/null +++ b/openbios-devel/libopenbios/bootcode_load.c @@ -0,0 +1,71 @@ +/* + * Raw bootcode loader (CHRP/Apple %BOOT) + * Written by Mark Cave-Ayland 2013 + */ + +#include "config.h" +#include "kernel/kernel.h" +#include "libopenbios/bindings.h" +#include "libopenbios/bootcode_load.h" +#include "libc/diskio.h" +#include "drivers/drivers.h" +#define printf printk +#define debug printk + +#define OLDWORLD_BOOTCODE_BASEADDR (0x3f4000) + +int +bootcode_load(ihandle_t dev) +{ + int retval = -1, count = 0, fd; + unsigned long bootcode, loadbase, offset; + + /* Mark the saved-program-state as invalid */ + feval("0 state-valid !"); + + fd = open_ih(dev); + if (fd == -1) { + goto out; + } + + /* Default to loading at load-base */ + fword("load-base"); + loadbase = POP(); + +#ifdef CONFIG_PPC + /* However Old World Macs need to load to a different address */ + if (is_oldworld()) { + loadbase = OLDWORLD_BOOTCODE_BASEADDR; + } +#endif + + bootcode = loadbase; + offset = 0; + + while(1) { + if (seek_io(fd, offset) == -1) + break; + count = read_io(fd, (void *)bootcode, 512); + offset += count; + bootcode += count; + } + + /* If we didn't read anything then exit */ + if (!count) { + goto out; + } + + /* Initialise saved-program-state */ + PUSH(loadbase); + feval("saved-program-state >sps.entry !"); + PUSH(offset); + feval("saved-program-state >sps.file-size !"); + feval("bootcode saved-program-state >sps.file-type !"); + + feval("-1 state-valid !"); + +out: + close_io(fd); + return retval; +} + diff --git a/openbios-devel/libopenbios/build.xml b/openbios-devel/libopenbios/build.xml index 04e3800..8c1fe3f 100644 --- a/openbios-devel/libopenbios/build.xml +++ b/openbios-devel/libopenbios/build.xml @@ -3,6 +3,7 @@ <library name="openbios" type="static" target="target"> <object source="aout_load.c" condition="LOADER_AOUT"/> <object source="bindings.c"/> + <object source="bootcode_load.c" condition="LOADER_BOOTCODE"/> <object source="bootinfo_load.c" condition="LOADER_BOOTINFO"/> <object source="client.c"/> <object source="console_common.c"/> diff --git a/openbios-devel/libopenbios/load.c b/openbios-devel/libopenbios/load.c index 2a2a7f9..1d50a82 100644 --- a/openbios-devel/libopenbios/load.c +++ b/openbios-devel/libopenbios/load.c @@ -36,6 +36,10 @@ #include "libopenbios/forth_load.h" #endif +#ifdef CONFIG_LOADER_BOOTCODE +#include "libopenbios/bootcode_load.h" +#endif + struct sys_info sys_info; void *elf_boot_notes = NULL; @@ -92,4 +96,13 @@ void load(ihandle_t dev) } #endif +#ifdef CONFIG_LOADER_BOOTCODE + /* Check for a "raw" %BOOT bootcode payload */ + feval("want-bootcode @"); + valid = POP(); + if (valid) { + bootcode_load(dev); + } +#endif + } diff --git a/openbios-devel/packages/mac-parts.c b/openbios-devel/packages/mac-parts.c index 85d53f8..c29b554 100644 --- a/openbios-devel/packages/mac-parts.c +++ b/openbios-devel/packages/mac-parts.c @@ -89,8 +89,12 @@ macparts_open( macparts_info_t *di ) parnum = atol(parstr); /* Detect if we are looking for the bootcode */ - if (strcmp(argstr, "%BOOT") == 0) + if (strcmp(argstr, "%BOOT") == 0) { want_bootcode = 1; + feval("1 want-bootcode !"); + } else { + feval("0 want-bootcode !"); + } } DPRINTF("parstr: %s argstr: %s parnum: %d\n", parstr, argstr, parnum); @@ -285,16 +289,10 @@ macparts_open( macparts_info_t *di ) set_property(chosen_ph, "bootpath", bootpath, strlen(bootpath) + 1); } - - /* If the filename was %BOOT then it's not a real filename, so clear argstr before - attempting interpose */ - if (want_bootcode) { - argstr = strdup(""); - } - + /* If we have been asked to open a particular file, interpose the filesystem package with the passed filename as an argument */ - if (strlen(argstr)) { + if (!want_bootcode && strlen(argstr)) { push_str( argstr ); PUSH_ph( ph ); fword("interpose"); -- 1.7.10.4

1 1

[commit] r1103 - in trunk/openbios-devel: forth/debugging packages
by repository service 22 Mar '13

22 Mar '13

Author: mcayland Date: Fri Mar 22 15:16:46 2013 New Revision: 1103 URL: http://tracker.coreboot.org/trac/openbios/changeset/1103 Log: mac-parts.c: Fix auto-partition selection for the dir word. Commit r1100 fixed the quik bootloader but broke automatic partition selection for the dir word. Differentiate between the package having no arguments (argstr == NULL) and the package having empty arguments (argstr == ""), and alter the dir word to use the latter version to invoke the automatic partition selection code when required. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk> CC: Alexander Graf <agraf(a)suse.de> Modified: trunk/openbios-devel/forth/debugging/client.fs trunk/openbios-devel/packages/mac-parts.c Modified: trunk/openbios-devel/forth/debugging/client.fs ============================================================================== --- trunk/openbios-devel/forth/debugging/client.fs Sat Mar 9 00:37:44 2013 (r1102) +++ trunk/openbios-devel/forth/debugging/client.fs Fri Mar 22 15:16:46 2013 (r1103) @@ -126,7 +126,7 @@ : dir ( "{paths}<cr>" -- ) linefeed parse - ascii , left-split + ascii , split-after 2dup open-dev dup 0= if drop cr ." Unable to locate device " type Modified: trunk/openbios-devel/packages/mac-parts.c ============================================================================== --- trunk/openbios-devel/packages/mac-parts.c Sat Mar 9 00:37:44 2013 (r1102) +++ trunk/openbios-devel/packages/mac-parts.c Fri Mar 22 15:16:46 2013 (r1103) @@ -79,6 +79,10 @@ } else { argstr = str; } + + /* Make sure argstr is not null */ + if (argstr == NULL) + argstr = strdup(""); /* Convert the id to a partition number */ if (parstr && strlen(parstr)) @@ -89,10 +93,6 @@ want_bootcode = 1; } - /* Make sure argstr is not null */ - if (argstr == NULL) - argstr = strdup(""); - DPRINTF("parstr: %s argstr: %s parnum: %d\n", parstr, argstr, parnum); DPRINTF("want_bootcode %d\n", want_bootcode); @@ -126,7 +126,7 @@ * Implement partition selection as per the PowerPC Microprocessor CHRP bindings */ - if (str == NULL || parnum == 0) { + if (argstr == NULL || parnum == 0) { /* According to the spec, partition 0 as well as no arguments means the whole disk */ offs = (long long)0; size = (long long)__be32_to_cpu(dmap.sbBlkCount) * bs;

1 0