SeaBIOS

seabios@seabios.org

4 participants
3801 discussions

Re: [SeaBIOS] [Qemu-devel] insmod virtio-blk is broken in qemu 1.0
by Anthony Liguori 20 Dec '11

20 Dec '11

On 12/17/2011 09:25 AM, Richard W.M. Jones wrote: > On Sat, Dec 17, 2011 at 09:22:45AM -0600, Anthony Liguori wrote: >> I've even further narrowed it down to the presents or lack of '-vga >> cirrus'. If you add '-vga cirrus' to the above command line, the >> guest will boot successfully. > > Confirmed: Adding -vga cirrus to the command line cures it too. > > That's a strange one :-) vga sticks out a bit because it's one of the few places where we treat device memory as ram as a performance optimization. The only time vga has been touched in between v0.15 and v1.0 was during the introduction of the memory API. It's this commit: commit d67c3f2cd92aed2247bfa8a9da61a902b7b2ff09 Author: Gerd Hoffmann <kraxel(a)redhat.com> Date: Wed Aug 10 17:34:13 2011 +0200 seabios: update to master commit 8e301472e324b6d6496d8b4ffc66863e99d7a505 user visible changes in seabios: * ahci is enabled by default (and thus in this build). * bootorder support for ahci. * two-pass pci allocator (orders bars by size for better packing). Signed-off-by: Gerd Hoffmann <kraxel(a)redhat.com> :040000 040000 76eb0c81b76563b55cb2bb5c484ccd48b8cfcded 5ec0d65d3a763a5566fe1f4c86269cad6d671020 M pc-bios :040000 040000 a5a7ea6e297c1e7490b0a2c28a06ce56e5be9449 78adb664d3ea82f1a4dd5ec239887ac5b0168a7f M roms It can be reproduced by using virtio and -vga none with a number of PCI devices. The line below is what I used to bisect and reproduce 100% of the time. It's a 64-bit Fedora 15 guest. $ qemu-system-x86_64 -drive file=/home/anthony/images/fedora.img,if=none,snapshot=on,id=hd0 -device virtio-balloon-pci,addr=03.0 -device virtio-blk-pci,addr=04.0,drive=hd0 -kernel ~/vmlinuz-2.6.38.6-26.rc1.fc15.x86_64 -initrd ~/initramfs-2.6.38.6-26.rc1.fc15.x86_64.img -append "root=/dev/mapper/VolGroup-lv_root rd_LVM_LV=VolGroup/lv_root rd_LVM_LV=VolGroup/lv_swap ro console=ttyS0 selinux=0" -nographic -nodefconfig -m 1G -no-reboot -no-hpet -device virtio-serial -chardev socket,path=/tmp/foo.sock,id=channel0,server,nowait -device virtserialport,chardev=channel0,name=org.libguestfs.channel.0 -nodefaults -serial stdio -enable-kvm My guess it that it has something to do with the changes to the PCI allocator. I've confirmed reverting this commit fixes the problem. Regards, Anthony Liguori > > Rich. >

6 19

Cirrus VGA driver work
by Kevin O'Connor 19 Dec '11

19 Dec '11

I was reviewing some of my past work on the Cirrus VGA BIOS port. I rechecked the work and figured I'd post what I had. The code is completely untested. What makes this interesting, though, is that the Cirrus driver is only really lacking VBE support after this. With Julian's recent work, maybe Cirrus support isn't far off. -Kevin diff --git a/vgasrc/clext.c b/vgasrc/clext.c index 36ccad3..225292e 100644 --- a/vgasrc/clext.c +++ b/vgasrc/clext.c @@ -9,6 +9,11 @@ #include "biosvar.h" // GET_GLOBAL #include "util.h" // dprintf + +/**************************************************************** + * tables + ****************************************************************/ + struct cirrus_mode_s { /* + 0 */ u16 mode; @@ -289,6 +294,11 @@ static struct cirrus_mode_s cirrus_modes[] VAR16 = { 0xff,0,0,0,0,0,0,0,0}, }; + +/**************************************************************** + * helper functions + ****************************************************************/ + static struct cirrus_mode_s * cirrus_get_modeentry(u8 mode) { @@ -348,19 +358,63 @@ cirrus_switch_mode(struct cirrus_mode_s *table) vgahw_set_single_palette_reg(0x10, v); } -void +static u8 +cirrus_get_memsize(void) +{ + // get DRAM band width + outb(0x0f, VGAREG_SEQU_ADDRESS); + u8 v = inb(VGAREG_SEQU_DATA); + u8 x = (v >> 3) & 0x03; + if (x == 0x03) { + if (v & 0x80) + // 4MB + return 0x40; + // 2MB + return 0x20; + } + return 0x04 << x; +} + +static void +cirrus_enable_16k_granularity(void) +{ + outb(0x0b, VGAREG_GRDC_ADDRESS); + u8 v = inb(VGAREG_GRDC_DATA); + outb(v | 0x20, VGAREG_GRDC_DATA); +} + +static void +cirrus_clear_vram(u16 param) +{ + cirrus_enable_16k_granularity(); + u8 count = cirrus_get_memsize() * 4; + u8 i; + for (i=0; i<count; i++) { + outw((i<<8) | 0x09, VGAREG_GRDC_ADDRESS); + memset16_far(0xa000, (void*)0, param, 16 * 1024); + } + outw(0x0009, VGAREG_GRDC_ADDRESS); +} + +int cirrus_set_video_mode(u8 mode) { dprintf(1, "cirrus mode %d\n", mode); SET_BDA(vbe_mode, 0); struct cirrus_mode_s *table_g = cirrus_get_modeentry(mode & 0x7f); if (table_g) { - //XXX - cirrus_set_video_mode_extended(table); - return; + cirrus_switch_mode(table_g); + // XXX - pass in mode without 0x7f mask + if (mode & 0x80) + cirrus_clear_vram(0xffff); + SET_BDA(video_mode, mode & 0x7f); + // XXX - return early from caller. + return 0; } table_g = cirrus_get_modeentry(0xfe); cirrus_switch_mode(table_g); dprintf(1, "cirrus mode switch regular\n"); + return -1; } static int @@ -370,6 +424,243 @@ cirrus_check(void) return inb(VGAREG_SEQU_DATA) == 0x12; } + +/**************************************************************** + * extbios + ****************************************************************/ + +static void +cirrus_extbios_80h(struct bregs *regs) +{ + u16 crtc_addr = cirrus_get_crtc(); + outb(0x27, crtc_addr); + u8 v = inb(crtc_addr + 1); + if (v == 0xa0) + // 5430 + regs->ax = 0x0032; + else if (v == 0xb8) + // 5446 + regs->ax = 0x0039; + else + regs->ax = 0x00ff; + regs->bx = 0x00; + return; +} + +static void +cirrus_extbios_81h(struct bregs *regs) +{ + // XXX + regs->ax = 0x0100; +} + +static void +cirrus_extbios_82h(struct bregs *regs) +{ + u16 crtc_addr = cirrus_get_crtc(); + outb(0x27, crtc_addr); + regs->al = inb(crtc_addr + 1) & 0x03; + regs->ah = 0xAF; +} + +static void +cirrus_extbios_85h(struct bregs *regs) +{ + regs->al = cirrus_get_memsize(); +} + +static void +cirrus_extbios_9Ah(struct bregs *regs) +{ + regs->ax = 0x4060; + regs->cx = 0x1132; +} + +extern void a0h_callback(void); +ASM16( + // fatal: not implemented yet + "a0h_callback:" + "cli\n" + "hlt\n" + "retf"); + +static void +cirrus_extbios_A0h(struct bregs *regs) +{ + struct cirrus_mode_s *table_g = cirrus_get_modeentry(regs->al & 0x7f); + regs->ah = (table_g ? 1 : 0); + regs->si = 0xffff; + regs->di = regs->ds = regs->es = regs->bx = (u32)a0h_callback; +} + +static void +cirrus_extbios_A1h(struct bregs *regs) +{ + regs->bx = 0x0e00; // IBM 8512/8513, color +} + +static void +cirrus_extbios_A2h(struct bregs *regs) +{ + regs->al = 0x07; // HSync 31.5 - 64.0 kHz +} + +static void +cirrus_extbios_AEh(struct bregs *regs) +{ + regs->al = 0x01; // High Refresh 75Hz +} + +void +cirrus_extbios(struct bregs *regs) +{ + // XXX - regs->bl < 0x80 or > 0xaf call regular handlers. + switch (regs->bl) { + case 0x80: cirrus_extbios_80h(regs); break; + case 0x81: cirrus_extbios_81h(regs); break; + case 0x82: cirrus_extbios_82h(regs); break; + case 0x85: cirrus_extbios_85h(regs); break; + case 0x9a: cirrus_extbios_9Ah(regs); break; + case 0xa0: cirrus_extbios_A0h(regs); break; + case 0xa1: cirrus_extbios_A1h(regs); break; + case 0xa2: cirrus_extbios_A2h(regs); break; + case 0xae: cirrus_extbios_AEh(regs); break; + default: break; + } +} + + +/**************************************************************** + * vesa calls + ****************************************************************/ + +#if 0 +static u16 +cirrus_vesamode_to_mode(u16 vesamode) +{ + // XXX - convert assembler + return 0; +} + +static u8 +cirrus_get_bpp_bytes(void) +{ + // XXX - convert assembler + return 0; +} + +static void +cirrus_set_line_offset(u16 new_line_offset) +{ + // XXX - convert assembler +} + +static u16 +cirrus_get_line_offset(void) +{ + // XXX - convert assembler + return 0; +} + +static u16 +cirrus_get_line_offset_entry(void *table) +{ + // XXX - convert assembler + return 0; +} + +static void +cirrus_set_start_addr(void *addr) +{ + // XXX - convert assembler +} + +static void * +cirrus_get_start_addr(void) +{ + // XXX - convert assembler + return NULL; +} +#endif + +static void +cirrus_vesa_00h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_01h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_02h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_03h(struct bregs *regs) +{ + // XXX - convert assembler +} + +// XXX - add cirrus_vesa_05h_farentry to vgaentry.S + +static void +cirrus_vesa_05h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_06h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_07h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_10h(struct bregs *regs) +{ + // XXX - convert assembler +} + +static void +cirrus_vesa_not_handled(struct bregs *regs) +{ + debug_stub(regs); + regs->ax = 0x014f; +} + +void +cirrus_vesa(struct bregs *regs) +{ + switch (regs->al) { + case 0x00: cirrus_vesa_00h(regs); break; + case 0x01: cirrus_vesa_01h(regs); break; + case 0x02: cirrus_vesa_02h(regs); break; + case 0x03: cirrus_vesa_03h(regs); break; + case 0x05: cirrus_vesa_05h(regs); break; + case 0x06: cirrus_vesa_06h(regs); break; + case 0x07: cirrus_vesa_07h(regs); break; + case 0x10: cirrus_vesa_10h(regs); break; + default: cirrus_vesa_not_handled(regs); break; + } +} + + +/**************************************************************** + * init + ****************************************************************/ + void cirrus_init(void) { diff --git a/vgasrc/vgatables.h b/vgasrc/vgatables.h index 1e76b3a..48bf828 100644 --- a/vgasrc/vgatables.h +++ b/vgasrc/vgatables.h @@ -205,7 +205,7 @@ void vgahw_enable_video_addressing(u8 disable); void vgahw_init(void); // clext.c -void cirrus_set_video_mode(u8 mode); +int cirrus_set_video_mode(u8 mode); void cirrus_init(void); // vbe.c -- not implemented yet.

1 0

Re: [SeaBIOS] [PATCH RFC 6/7] Partially implement VBE interface, create VBE driver skeleton
by Kevin O'Connor 18 Dec '11

18 Dec '11

On Sat, Dec 17, 2011 at 08:06:13PM +0000, Julian Pidancet wrote: > On Sat, Dec 17, 2011 at 4:17 PM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > On Fri, Dec 16, 2011 at 02:15:12PM +0000, Julian Pidancet wrote: > > [...] > >> +struct VbeInfoBlock > >> +{ > >> + u8 signature[4]; > >> + u16 version; > >> + u16 oem_string_off; > >> + u16 oem_string_seg; > > > > Looks like this is the same as "struct vesa_info" defined in > > src/bootsplash.c. It doesn't really matter which struct is used, but > > both areas of code should share the same definition. > > > > Also, I think it would be better to use "struct segoff_s" and to use > > "u32 signature". > > > > They indeed seems to be the same structures. Should I move vbe.h in > src/ and make the definitions common for both the VGA ROM and the BIOS > ? Yes - I think having them use a shared definition would be best. -Kevin

1 0

Re: [SeaBIOS] [patch] Skip the bootmenu if only one device is found
by Sebastian Herbszt 18 Dec '11

18 Dec '11

Marc Jones wrote: > The bootmenu is only useful if there are multiple devices. We can > speed up theboot by a few seconds if there is only a single device to > try to boot from. > Signed-off-by: Marc Jones <marcj303(a)gmail.com> Isn't the bootmenu also useful to check if the bios detected other boot devices? Please check the previous discussion [1]. [1] http://www.seabios.org/pipermail/seabios/2011-April/001656.html Sebastian

3 2

Re: [SeaBIOS] Analysis of the current state and proposed future path for SeaBIOS.
by Peter Stuge 18 Dec '11

18 Dec '11

Fred . wrote: > I do not know low-level programming. > So I just come up with ideas that I posted here, and mention stuff > that needs to be documented. > And say that website needs to be improved. Well my point is you can contribute also without programming. The suggestions are good, but if you want to actually work on improving the documentation that's even better! //Peter

2 1

Re: [SeaBIOS] [PATCH RFC 1/7] Only use -fomit-frame-pointer with 32bit flat code
by Julian Pidancet 18 Dec '11

18 Dec '11

On Sat, Dec 17, 2011 at 3:50 PM, Julian Pidancet <julian.pidancet(a)gmail.com> wrote: > On Sat, Dec 17, 2011 at 3:38 PM, Kevin O'Connor <kevin(a)koconnor.net> wrote: >> On Fri, Dec 16, 2011 at 02:15:07PM +0000, Julian Pidancet wrote: >>> The real-mode emulator that Windows 7 uses to execute the vga bios rom >>> becomes very confused when the code tries to dereference the stack >>> pointer instead of using the frame pointer, leading to corruption of >>> parameters passed on the stack from one function to another. >>> >>> Signed-off-by: Julian Pidancet <julian.pidancet(a)gmail.com> >> >> Wow - I can only imagine how hard it was to find that. >> >> I'm okay with turning off omit-frame-pointer for the vga bios. >> However, I'd prefer not to turn it off for the main seabios build. >> Does the emulator also try to emulate parts of the BIOS, which would >> make that a problem? >> > > Besides the functions which are common to both the BIOS and the VGA > ROM ( src/output.c and src/util.c), it doesn't seem to make any > particular problem with the rest of the BIOS. > The VGA ROM and the BIOS currently share the same CFLAGS. Due to the way the Makefile is written at the moment, I can hardly see how to modify it to compile both binaries with different CFLAGS without making a mess in the Makefile. Do you have any recommendations ? -- Julian

1 0

Re: [SeaBIOS] [PATCH RFC 6/7] Partially implement VBE interface, create VBE driver skeleton
by Kevin O'Connor 17 Dec '11

17 Dec '11

On Fri, Dec 16, 2011 at 02:15:12PM +0000, Julian Pidancet wrote: [...] > +struct VbeInfoBlock > +{ > + u8 signature[4]; > + u16 version; > + u16 oem_string_off; > + u16 oem_string_seg; Looks like this is the same as "struct vesa_info" defined in src/bootsplash.c. It doesn't really matter which struct is used, but both areas of code should share the same definition. Also, I think it would be better to use "struct segoff_s" and to use "u32 signature". -Kevin

2 1

[PATCH RFC 0/7] VGA ROM improvements
by Julian Pidancet 17 Dec '11

17 Dec '11

This patch set contain ground work for a working VBE implementation. It adds several things: * A configuration menu in seabios Kconfig * The ability to generate a PCI ROM * PMM facilities that the ROM can use to allocate memory * A skeleton driver to implement the VBE interface As a bonus, it fixes an issue when real-mode VGA ROM code is emulated by Windows 7, and is unable to cope with code generated with the -fomit-frame-pointer option. It should make pretty easy to implement the BOCHS DISPI interface. Also, I use these patchs to implement VBE for an experimental gfx device emulation. So far I managed to get Windows 7 to boot and show up a 1280x1024 desktop. Julian Pidancet (7): Only use -fomit-frame-pointer with 32bit flat code Rename CONFIG_VBE in CONFIG_VGA_BOCHS and CONFIG_CIRRUS in CONFIG_VGA_CIRRUS Add configuration menu for the VGA ROM buildrom.py: Handle PCI header checksum Add PMM memory allocation facilities for VGA option ROMs Partially implement VBE interface, create VBE driver skeleton Add vbe_flag field in BDA Makefile | 7 +- src/Kconfig | 35 +++++++ src/biosvar.h | 3 +- tools/buildrom.py | 5 + vgasrc/pmm.c | 50 +++++++++ vgasrc/pmm.h | 16 +++ vgasrc/vbe.c | 135 ++++++++++++++++++++++++ vgasrc/vbe.h | 194 +++++++++++++++++++++++++++++++++++ vgasrc/vga.c | 290 ++++++++++++++++++++++++++++++++++++++++++++-------- vgasrc/vgaentry.S | 47 ++++++++- vgasrc/vgatables.h | 6 - 11 files changed, 731 insertions(+), 57 deletions(-) create mode 100644 vgasrc/pmm.c create mode 100644 vgasrc/pmm.h create mode 100644 vgasrc/vbe.c create mode 100644 vgasrc/vbe.h -- Julian Pidancet

4 11

Re: [SeaBIOS] how to boot seabios with a raid controller?
by Rudolf Marek 17 Dec '11

17 Dec '11

Hi, I tried with silicon Image option rom and it was working as expected. Not sure what is your problem here. Maybe the option needs big real mode... Thanks Rudolf

2 1

Re: [SeaBIOS] how to boot seabios with a raid controller?
by Kevin O'Connor 17 Dec '11

17 Dec '11

On Fri, Dec 16, 2011 at 03:39:56PM -0700, Marc Jones wrote: > I'm trying coreboot + seabios on a system with a raid controller. What > is the expected behavior in seabios for booting a sata raid > controller? The controller supports raid and shows up as a raid class > device. The raid option rom is loaded, but doesn't find any raid > arrays. Later, seabios doesn't find the sata drive. I would expect > that seabios or the optionrom (int13?) would still be able to support > a normal sata drive. Thoughts, comments? It should work. Can you post the seabios log, along with any additional info the raid option rom reports on the screen? Are the drives you are looking to boot from in RAID mode (and thus should boot from the raid option rom)? Is this a built-in device or a card? -Kevin

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS