September 2010 - OpenBIOS - mail.coreboot.org

starting
by G 3 Oct. 18, 2010

Oct. 18, 2010

I have begun doing work with OpenBIOS. I was wondering if it is possible to compile OpenBIOS for PowerPC on Mac OS X? If not, what Linux distro should I use? Any advice would really help.

6 17

Re: [OpenBIOS] PReP PowerPC "emulation"
by Massimo Montecchi Oct. 12, 2010

Oct. 12, 2010

Until r828 was in this way ... then somethink was wrong.... 2010/9/13 Laurent Vivier <Laurent(a)vivier.eu> > > Hi, > > the last time I tried to boot AIX (6 or 7 monthes ago...), OpenBIOS was > able to load the bootloader but as RTAS is not provided by OpenBIOS, > bootloader failed and didn't go further. I think it was with the CHRP > emulation. > > Regards, > Laurent > > > > > Hi, > > I have tried qemu from 0.11 to latest git (as of 11/09/2010) with latest > openbios inside, but if i want to use PREP PPC "machine" the error is the > same: > > qemu: hardware error: PowerPC 601 / 620 / 970 need a 1MB BIOS > > CPU #0: > NIP 00000000 LR 00000000 CTR 00000000 XER 00000000 > MSR 00000000 HID0 00000000 HF 00000000 idx 0 > TB 00000000 00000000 DECR ffffffff > GPR00 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR04 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR08 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR12 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR16 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR20 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR24 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > GPR28 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > CR 00000000 [ - - - - - - - - ] RES 00000000 > FPR00 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR04 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR08 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR12 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR16 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR20 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR24 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPR28 0000000000000000 0000000000000000 0000000000000000 0000000000000000 > FPSCR 00000000 > SRR0 00000000 SRR1 00000000 SDR1 00000000 > > How can I use openbios to start an AIX with qemu ? > > -- > Cordiali Saluti/Best Regards > > Massimo Montecchi > Modena - Italy > > > > > -- > --------------------- Laurent(a)vivier.eu --------------------- > "Tout ce qui est impossible reste à accomplir" Jules Verne > "Things are only impossible until they're not" Jean-Luc Picard > > -- > OpenBIOS http://openbios.org/ > Mailinglist: http://lists.openbios.org/mailman/listinfo > Free your System - May the Forth be with you > -- Cordiali Saluti/Best Regards Massimo Montecchi Modena - Italy

4 12

[PATCH] Darwin/ppc host needs fake int128_t for sparc64
by Andreas Färber Oct. 3, 2010

Oct. 3, 2010

Otherwise it chokes on ducell in include/kernel/stack.h. Signed-off-by: Andreas Färber <andreas.faerber(a)web.de> --- It seems I only mentioned this on the list but never sent a proper patch. http://www.openfirmware.info/pipermail/openbios/2009-November/004192.html Not sure about Linux/ppc or Darwin/x86, so I've limited it to Darwin/ppc for now. Regards, Andreas config/scripts/switch-arch | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/config/scripts/switch-arch b/config/scripts/switch-arch index c1158fa..8b6563d 100755 --- a/config/scripts/switch-arch +++ b/config/scripts/switch-arch @@ -71,6 +71,8 @@ crosscflags() if test "$host" = "x86" -a "$targetlongbits" = "64"; then cflags="$cflags -DNEED_FAKE_INT128_T" + elif test "$host" = "ppc" -a "$targetlongbits" = "64" -a `uname -s` = "Darwin"; then + cflags="$cflags -DNEED_FAKE_INT128_T" fi CROSSCFLAGS=$cflags -- 1.7.0.4

2 3

OpenBIOS and ppc64?
by Andreas Färber Oct. 1, 2010

Oct. 1, 2010

Hello, I'm investigating the idea of a new QEMU chrp machine derived from mac99. The background is that there are ofmem issues when running Haiku on the g3beige machine, whereas running AIX on mac99 fails with the fix in place that makes Haiku go further. I assume this is due to differences in memory layout so that I need to move OpenBIOS to another location in RAM and anything else that clobbers the memory locations AIX assumes to be able to claim for itself. Not having (low-level) access to an IBM Power System / System p / pSeries, the best source I could find was a JS20 device tree [1]. (Some older RS/6000 ones for comparison are available at [2].) Interestingly the nvram is at 0x00000100ff800000 and pci at 0x08000000f8000000, i.e. above 32-bit address space. With sizeof(void*) == 4, drivers/macio.c:nvram is unable to save and access such an address. ofmem cannot deal with 64-bit virtual/physical addresses either, since it uses 32-bit ucell in translation_t and range_t. The MMU translations on mac99 don't match Apple's OpenFirmware on my G5 size- wise. What's the plan for fixing this? Will we need an openbios-ppc64 after all, or is there a way to fix these issues in openbios-ppc using uint64_t and some assembler magic? The 970 init (arch/ppc/qemu/ init.c:ppc64_patch_handlers) apparently does something clever for the interrupt handlers and seems able to use 64-bit ld/std instructions in start.S. That still wouldn't help with ucell based structs and C functions of course... Regards, Andreas [1] http://www.ibm.com/developerworks/forums/thread.jspa?threadID=73270 [2] http://www.tibit.org/ppc/

5 14

[commit] r868 - trunk/openbios-devel/libopenbios
by repository service Sept. 30, 2010

Sept. 30, 2010

Author: mcayland Date: Wed Sep 29 18:27:42 2010 New Revision: 868 URL: http://tracker.coreboot.org/trac/openbios/changeset/868 Log: When DEBUG_CIF is defined, a serial message of the form "method(...) = " is emitted by dump_service(). After successful execution the return value and a linebreak are emitted by dump_return(). If however the method fails, nothing is emitted, so that subsequent method calls end up in place of the return value. Observed for getproplen with AIX. Emit an error indication and a linebreak when DEBUG_CIF is defined. Signed-off-by: Andreas Färber <andreas.faerber(a)web.de> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)siriusit.co.uk> Modified: trunk/openbios-devel/libopenbios/client.c Modified: trunk/openbios-devel/libopenbios/client.c ============================================================================== --- trunk/openbios-devel/libopenbios/client.c Tue Sep 28 20:59:47 2010 (r867) +++ trunk/openbios-devel/libopenbios/client.c Wed Sep 29 18:27:42 2010 (r868) @@ -306,6 +306,10 @@ if( val == -1 ) printk("Unimplemented service %s ([%ld] -- [%ld])\n", pb->service, pb->nargs, pb->nret ); +#ifdef DEBUG_CIF + else + printk("ERROR!\n"); +#endif return -1; }

5 4

[PATCH 1/2] iso9660: Fix Forth return value for iso9660_files_open()
by Andreas Färber Sept. 30, 2010

Sept. 30, 2010

There's a mixup of 0 and -1. Don't return failure if iso9660_open() succeeded. Don't return success in case iso9660_open() or iso9660_mount() failed. This resolves both a hang when trying to boot cd:,\\:tbxi and failure to boot either cd:,\ppc\bootinfo.txt or cd:,\ppc\chrp\bootfile.exe despite present on the AIX 6.1 disk. Cc: Mark Cave-Ayland <mark.cave-ayland(a)siriusit.co.uk> Signed-off-by: Andreas Färber <andreas.faerber(a)web.de> --- fs/iso9660/iso9660_fs.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/iso9660/iso9660_fs.c b/fs/iso9660/iso9660_fs.c index 5714d6d..3fb5c16 100644 --- a/fs/iso9660/iso9660_fs.c +++ b/fs/iso9660/iso9660_fs.c @@ -49,7 +49,7 @@ iso9660_files_open( iso9660_info_t *mi ) if ( mi->volume == NULL ) { free( path ); close_io( fd ); - RET( -1 ); + RET( 0 ); } mi->common->dir = iso9660_opendir( mi->volume, path ); @@ -59,11 +59,11 @@ iso9660_files_open( iso9660_info_t *mi ) iso9660_umount( mi->volume ); close_io( fd ); free( path ); - RET( -1 ); + RET( 0 ); } mi->common->type = FILE; free( path ); - RET( 0 ); + RET( -1 ); } mi->common->type = DIR; free( path ); -- 1.7.3

3 7

Re: [OpenBIOS] x86 multiboot header?
by Nathan Kunkee Sept. 30, 2010

Sept. 30, 2010

> Date: Tue, 21 Sep 2010 10:07:19 +0100 > From: mark.cave-ayland(a)siriusit.co.uk > To: openbios(a)openbios.org > Subject: Re: [OpenBIOS] x86 multiboot header? > > Nathan Kunkee wrote: > > > Hi all, > > > > As of revision 861, I am unable to build a working multiboot image. I am working on Ubuntu 10.4, gcc 4.4. Although OpenSolaris b134, gcc4, also doesn't work. > > > > Grub 1.95 doesn't recognize the file as a multiboot file. QEMU crashes when passed this via -kernel (something about running code near 0xa000). Sure enough, I can find the second header for 0x2BAD but not 0x1BAD. Is the linker script that prefixes the header off? > > > > Thank you in advance for any assistance! > > Nathan > > Interesting. Did you find r861 as the result of an SVN bisect? If so, > posting a link to your test image so that someone could debug this would > be very helpful. > > > ATB, > > Mark. Hello, I should have said "As of the latest commit (r861)..." since I wanted to convey that I'd tried the latest sources, not that a particular revision introduced this, nor that I found it with a bisect. I found the following change enabled the multiboot image to load with the embedded utils/iso/boot/grub: Index: arch/x86/multiboot.c =================================================================== --- arch/x86/multiboot.c (revision 872) +++ arch/x86/multiboot.c (working copy) @@ -15,8 +15,10 @@ unsigned int magic, flags, checksum; }; -static const struct mbheader multiboot_header - __attribute__((section (".hdr"))) = +//static +const struct mbheader multiboot_header + __attribute__ ((section (".hdr"))) += { MULTIBOOT_HEADER_MAGIC, MULTIBOOT_HEADER_FLAGS, However, even with this, qemu doesn't successfully load the multiboot image: nathan@fangorn:~/openbios-devel/obj-x86$ /usr/local/build/qemu/i386-softmmu/qemu -L /usr/local/src/qemu/pc-bios/ -kernel openbios.multiboot -initrd openbios-x86.dict qemu: I believe we found a multiboot image! multiboot: mh_header_addr = 0 multiboot: mh_load_addr = 0x8 multiboot: mh_load_end_addr = 0x9 multiboot: mh_bss_end_addr = 0x1 qemu: loading multiboot kernel (0x14d28 bytes) at 0x8 multiboot loading module: openbios-x86.dict mod00: 00016008 - 0002dd44 mod_start: 0x9b73cd0 mod_end: 0x9b8ba0c cmdline: 00015018 multiboot: mh_entry_addr = 0x42464c45 mb_buf_phys = 00000008 mod_start = 00016008 mb_mods_count = 1 qemu: fatal: Trying to execute code outside RAM or ROM at 0x42464c45 I've uploaded a zip (openbios-multi.zip) to http://filebin.ca/boqpcs/openbios-multi.zip [filebin.ca] if you'd like to look at my test image. Thank you for your assistance! Nathan

2 1

[commit] r872 - in trunk/openbios-devel: arch/ppc/mol arch/ppc/pearpc arch/sparc32 arch/x86/xbox include/libopenbios libopenbios packages
by repository service Sept. 29, 2010

Sept. 29, 2010

Author: blueswirl Date: Wed Sep 29 22:30:51 2010 New Revision: 872 URL: http://tracker.coreboot.org/trac/openbios/changeset/872 Log: Avoid a lot of malloc/free traffic Each console write caused temporary buffer allocation. Avoid allocations by changing console_draw_str() to use Forth string parameters, which are usually readily available. Signed-off-by: Blue Swirl <blauwirbel(a)gmail.com> Modified: trunk/openbios-devel/arch/ppc/mol/methods.c trunk/openbios-devel/arch/ppc/mol/mol.h trunk/openbios-devel/arch/ppc/pearpc/methods.c trunk/openbios-devel/arch/ppc/pearpc/pearpc.h trunk/openbios-devel/arch/sparc32/console.c trunk/openbios-devel/arch/x86/xbox/methods.c trunk/openbios-devel/include/libopenbios/console.h trunk/openbios-devel/libopenbios/console_common.c trunk/openbios-devel/packages/video.c Modified: trunk/openbios-devel/arch/ppc/mol/methods.c ============================================================================== --- trunk/openbios-devel/arch/ppc/mol/methods.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/ppc/mol/methods.c Wed Sep 29 22:30:51 2010 (r872) @@ -85,14 +85,9 @@ { int len = POP(); char *addr = (char*)POP(); - char *s = malloc( len + 1 ); - - strncpy_nopad( s, addr, len ); - s[len]=0; /* printk( "%s", s ); */ - console_draw_str( s ); - free( s ); + console_draw_fstr(addr, len); PUSH( len ); } Modified: trunk/openbios-devel/arch/ppc/mol/mol.h ============================================================================== --- trunk/openbios-devel/arch/ppc/mol/mol.h Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/ppc/mol/mol.h Wed Sep 29 22:30:51 2010 (r872) @@ -24,7 +24,7 @@ extern void set_color( int index, unsigned long color ); /* console.c */ -extern int console_draw_str( const char *str ); +extern int console_draw_fstr(const char *str, int len); extern void console_close( void ); /* pseudodisk.c */ Modified: trunk/openbios-devel/arch/ppc/pearpc/methods.c ============================================================================== --- trunk/openbios-devel/arch/ppc/pearpc/methods.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/ppc/pearpc/methods.c Wed Sep 29 22:30:51 2010 (r872) @@ -70,16 +70,10 @@ { int len = POP(); char *addr = (char*)POP(); - char *s = malloc( len + 1 ); - - strncpy_nopad( s, addr, len ); - s[len]=0; printk( "%s", s ); //vfd_draw_str( s ); - console_draw_str( s ); - - free( s ); + console_draw_fstr(addr, len); PUSH( len ); } Modified: trunk/openbios-devel/arch/ppc/pearpc/pearpc.h ============================================================================== --- trunk/openbios-devel/arch/ppc/pearpc/pearpc.h Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/ppc/pearpc/pearpc.h Wed Sep 29 22:30:51 2010 (r872) @@ -19,7 +19,7 @@ extern int vfd_draw_str( const char *str ); extern void vfd_close( void ); -extern int console_draw_str( const char *str ); +extern int console_draw_fstr(const char *str, int len); #include "kernel.h" Modified: trunk/openbios-devel/arch/sparc32/console.c ============================================================================== --- trunk/openbios-devel/arch/sparc32/console.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/sparc32/console.c Wed Sep 29 22:30:51 2010 (r872) @@ -32,12 +32,11 @@ static void video_putchar(int c) { - char buf[2]; + char buf; - buf[0] = c & 0xff; - buf[1] = 0; + buf = c & 0xff; - console_draw_str(buf); + console_draw_fstr(&buf, 1); } static void video_cls(void) Modified: trunk/openbios-devel/arch/x86/xbox/methods.c ============================================================================== --- trunk/openbios-devel/arch/x86/xbox/methods.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/arch/x86/xbox/methods.c Wed Sep 29 22:30:51 2010 (r872) @@ -35,16 +35,10 @@ { int len = POP(); char *addr = (char*)POP(); - char *s = malloc( len + 1 ); - - strncpy_nopad( s, addr, len ); - s[len]=0; printk( "%s", s ); //vfd_draw_str( s ); - console_draw_str( s ); - - free( s ); + console_draw_fstr(addr, len); PUSH( len ); } Modified: trunk/openbios-devel/include/libopenbios/console.h ============================================================================== --- trunk/openbios-devel/include/libopenbios/console.h Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/include/libopenbios/console.h Wed Sep 29 22:30:51 2010 (r872) @@ -2,7 +2,7 @@ #define VIDEO_CONSOLE_H /* libopenbios/console_common.c */ -int console_draw_str(const char *str); +int console_draw_fstr(const char *str, int len); int console_init(void); void console_close(void); Modified: trunk/openbios-devel/libopenbios/console_common.c ============================================================================== --- trunk/openbios-devel/libopenbios/console_common.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/libopenbios/console_common.c Wed Sep 29 22:30:51 2010 (r872) @@ -382,12 +382,12 @@ } int -console_draw_str( const char *str ) +console_draw_fstr(const char *str, int len) { unsigned int y, x; unsigned char ch; - if (!str) { + if (!str || len <= 0) { return 0; } @@ -395,7 +395,7 @@ return -1; show_cursor(0); - while( (ch=*str++) ) { + while((ch = *str++) && len--) { do_con_trol(ch); if( cons.x >= cons.w ) { Modified: trunk/openbios-devel/packages/video.c ============================================================================== --- trunk/openbios-devel/packages/video.c Wed Sep 29 20:34:41 2010 (r871) +++ trunk/openbios-devel/packages/video.c Wed Sep 29 22:30:51 2010 (r872) @@ -288,11 +288,10 @@ char *addr; int len; - len = GETTOS(); - addr = pop_fstr_copy(); + len = POP(); + addr = (char *)POP(); - console_draw_str(addr); - free(addr); + console_draw_fstr(addr, len); PUSH(len); }

1 0

[commit] r871 - trunk/openbios-devel/packages
by repository service Sept. 29, 2010

Sept. 29, 2010

Author: blueswirl Date: Wed Sep 29 20:34:41 2010 New Revision: 871 URL: http://tracker.coreboot.org/trac/openbios/changeset/871 Log: video: improve boundary checking Add checks to fill_rect() and video_scroll(). Signed-off-by: Blue Swirl <blauwirbel(a)gmail.com> Modified: trunk/openbios-devel/packages/video.c Modified: trunk/openbios-devel/packages/video.c ============================================================================== --- trunk/openbios-devel/packages/video.c Wed Sep 29 19:06:31 2010 (r870) +++ trunk/openbios-devel/packages/video.c Wed Sep 29 20:34:41 2010 (r871) @@ -142,7 +142,8 @@ char *pp; unsigned long col = get_color(col_ind); - if( !video.has_video || x < 0 || y < 0 || x+w > video.fb.w || y+h > video.fb.h ) + if (!video.has_video || x < 0 || y < 0 || w <= 0 || h <= 0 || + x + w > video.fb.w || y + h > video.fb.h) return; pp = (char*)video.fb.mphys + video.fb.rb * y; @@ -203,6 +204,9 @@ { int i, offs, size, *dest, *src; + if (height <= 0 || height >= video.fb.h) { + return; + } offs = video.fb.rb * height; size = (video.fb.h * video.fb.rb - offs)/16; dest = (int*)video.fb.mphys;

1 0

[commit] r870 - trunk/openbios-devel/libopenbios
by repository service Sept. 29, 2010

Sept. 29, 2010

Author: blueswirl Date: Wed Sep 29 19:06:31 2010 New Revision: 870 URL: http://tracker.coreboot.org/trac/openbios/changeset/870 Log: console: fix off-by-one errror The maximum allowed value for x axis is one less than cons.w. Signed-off-by: Blue Swirl <blauwirbel(a)gmail.com> Modified: trunk/openbios-devel/libopenbios/console_common.c Modified: trunk/openbios-devel/libopenbios/console_common.c ============================================================================== --- trunk/openbios-devel/libopenbios/console_common.c Wed Sep 29 18:28:28 2010 (r869) +++ trunk/openbios-devel/libopenbios/console_common.c Wed Sep 29 19:06:31 2010 (r870) @@ -346,7 +346,7 @@ draw_line(cons.y); return; case 2: /* erase whole line */ - for (i = 0; i <= cons.w; i++) + for (i = 0; i < cons.w; i++) cons.buf[cons.y * cons.w + i] = ' '; draw_line(cons.y); return;

1 0