SeaBIOS

seabios@seabios.org

5 participants
3802 discussions

[PATCH] vgabios: Make sure exported structs use PACKED.
by Kevin O'Connor 06 Feb '14

06 Feb '14

Commit ca668640 introduced structs for data returned from the handle_101b and handle_101c calls. However, the structs were not declared as packed and that caused incorrect behavior for handle_101b. This was seen to break some old DOS programs (Win3 setup and DISPLAY.SYS). This patch adds in the necessary PACKED declaration to the structs. The 'struct saveBDAstate' remains un-packed, as it does not appear that its contents are documented. (And, its format already differs from what was in the previous "lgpl vgabios".) Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- vgasrc/stdvga.c | 4 ++-- vgasrc/vgabios.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/vgasrc/stdvga.c b/vgasrc/stdvga.c index e0661f1..c94ec06 100644 --- a/vgasrc/stdvga.c +++ b/vgasrc/stdvga.c @@ -338,7 +338,7 @@ struct saveVideoHardware { u8 grdc_regs[9]; u16 crtc_addr; u8 plane_latch[4]; -}; +} PACKED; static void stdvga_save_hw_state(u16 seg, struct saveVideoHardware *info) @@ -412,7 +412,7 @@ struct saveDACcolors { u8 pelmask; u8 dac[768]; u8 color_select; -}; +} PACKED; static void stdvga_save_dac_state(u16 seg, struct saveDACcolors *info) diff --git a/vgasrc/vgabios.c b/vgasrc/vgabios.c index 57f1c1f..f70b2b6 100644 --- a/vgasrc/vgabios.c +++ b/vgasrc/vgabios.c @@ -1114,7 +1114,7 @@ struct funcInfo { u8 save_flags; u8 disp_info; u8 reserved_34[12]; -}; +} PACKED; static void handle_101b(struct bregs *regs) -- 1.8.5.3

1 0

Intermittent USB keyboard freeze
by Gabriel L. Somlo 05 Feb '14

05 Feb '14

Hi Kevin, I'm running into an intermittent (roughly 50% of the time) USB keyboard freeze, using the following command line: bin/qemu-system-x86_64 -enable-kvm -m 2048 -cpu core2duo -M q35 \ -usb -device usb-kbd -device usb-mouse \ -device ide-drive,bus=ide.2,drive=HDD \ -drive id=HDD,if=none,snapshot=on,file=./Fedora-Live-Desktop-x86_64-20-1.iso \ -bios bios-test.bin where both qemu and bios-test.bin are built from their latest currently available git masters. The symptom is that, roughly half the time I can't navigate the grub splash screen to select an entry via the up/down arrow keys. After a few keystrokes, I start getting console errors like this: usb-kbd: warning: key event queue full usb-kbd: warning: key event queue full usb-kbd: warning: key event queue full ... It's not specific to grub/Fedora either, I get the same behavior with Chameleon when trying to boot OS X. The other 50% of the time, things work just fine. After a bisect on Seabios, I *think* the culprit is commit 4ea2fa9620ec4c0c6da53d534d6a69c8a0db27cd I couldn't easily reverse that patch due to other churn in the code, so I wasn't able to build a seabios with everything but that commit to test for sure. Also, one has to fire up qemu several (at least 10) times to adequately make sure the problem isn't present, so take my bisect with a grain of salt in case the content of the patch doesn't make sense with respect to the error behavior :) Thanks, --Gabriel

3 10

[PATCH 2/2] SMBIOS: Add Type 2 (Baseboard) struct
by Gabriel L. Somlo 04 Feb '14

04 Feb '14

OS X (10.7/Lion and 10.8/MountainLion) guests crash during boot if a Type 2 (Baseboard) structure is not found in the SMBIOS table. Signed-off-by: Gabriel Somlo <somlo(a)cmu.edu> --- src/fw/smbios.c | 36 ++++++++++++++++++++++++++++++++++++ src/std/smbios.h | 16 ++++++++++++++++ 2 files changed, 52 insertions(+) diff --git a/src/fw/smbios.c b/src/fw/smbios.c index b2d7f14..5b76468 100644 --- a/src/fw/smbios.c +++ b/src/fw/smbios.c @@ -251,6 +251,41 @@ smbios_init_type_1(void *start) return end; } +/* Type 2 -- Base Board */ +static void * +smbios_init_type_2(void *start) +{ + struct smbios_type_2 *p = (struct smbios_type_2 *)start; + char *end = (char *)start + sizeof(struct smbios_type_2); + size_t size; + int str_index = 0; + + p->header.type = 2; + p->header.length = sizeof(struct smbios_type_2); + p->header.handle = 0x200; + + load_str_field_with_default(2, manufacturer_str, BUILD_APPNAME); + load_str_field_or_skip(2, product_str); + load_str_field_or_skip(2, version_str); + load_str_field_or_skip(2, serial_number_str); + load_str_field_or_skip(2, asset_tag_number_str); + load_str_field_or_skip(2, location_str); + + set_field_with_default(2, feature_flags, 0x01); /* Motherboard */ + set_field_with_default(2, chassis_handle, 0x300); /* T3 System Enclosure */ + set_field_with_default(2, board_type, 0x0a); /* Motherboard */ + set_field_with_default(2, contained_element_count, 0); + + *end = 0; + end++; + if (!str_index) { + *end = 0; + end++; + } + + return end; +} + /* Type 3 -- System Enclosure */ static void * smbios_init_type_3(void *start) @@ -546,6 +581,7 @@ smbios_setup(void) add_struct(0, p); add_struct(1, p); + add_struct(2, p); add_struct(3, p); int cpu_num; diff --git a/src/std/smbios.h b/src/std/smbios.h index ba0e95d..86a4c57 100644 --- a/src/std/smbios.h +++ b/src/std/smbios.h @@ -59,6 +59,22 @@ struct smbios_type_1 { u8 family_str; } PACKED; +/* SMBIOS type 2 - Base Board */ +struct smbios_type_2 { + struct smbios_structure_header header; + u8 manufacturer_str; + u8 product_str; + u8 version_str; + u8 serial_number_str; + u8 asset_tag_number_str; + u8 feature_flags; + u8 location_str; + u16 chassis_handle; + u8 board_type; + u8 contained_element_count; + // contained elements follow +} PACKED; + /* SMBIOS type 3 - System Enclosure (v2.3) */ struct smbios_type_3 { struct smbios_structure_header header; -- 1.8.1.4

1 0

PC-BSD installer does not boot with 1.7.4 (bisected)
by Daniele Di Proietto 04 Feb '14

04 Feb '14

Hi, I am stepping in to add that this bug seems to affect also FreeBSD 9.1 kvm guests. Thanks, Daniele

1 0

[PATCH] pci: improve io address space allocation
by Gerd Hoffmann 03 Feb '14

03 Feb '14

This patch improves the io address space allocation. It adds a check that the region above 0xc000 which is traditionally used for pci io is actually big enougth. If it isn't it tries the larger window at 0x1000. If that is to small too it errors out. When creating guests with multiple pci-pci bridges (and devices with io regions behind them) the 0xc000 -> 0xffff region quickly becomes too small. While being at it document the io address space layout used by qemu/seabios. Signed-off-by: Gerd Hoffmann <kraxel(a)redhat.com> --- src/fw/pciinit.c | 45 ++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 40 insertions(+), 5 deletions(-) diff --git a/src/fw/pciinit.c b/src/fw/pciinit.c index b0d01ca..64f1d41 100644 --- a/src/fw/pciinit.c +++ b/src/fw/pciinit.c @@ -705,10 +705,42 @@ static int pci_bios_check_devices(struct pci_bus *busses) ****************************************************************/ // Setup region bases (given the regions' size and alignment) -static int pci_bios_init_root_regions(struct pci_bus *bus) -{ - bus->r[PCI_REGION_TYPE_IO].base = 0xc000; +static int pci_bios_init_root_regions_io(struct pci_bus *bus) +{ + /* + * QEMU I/O address space usage: + * 0000 - 0fff legacy isa, pci config, pci root bus, ... + * 1000 - 9fff free + * a000 - afff hotplug (cpu, pci via acpi, i440fx/piix only) + * b000 - bfff power management (PORT_ACPI_PM_BASE) + * [ qemu 1.4+ implements pci config registers + * properly so guests can place the registers + * where they want, on older versions its fixed ] + * c000 - ffff free, traditionally used for pci io + */ + struct pci_region *r_io = &bus->r[PCI_REGION_TYPE_IO]; + u64 sum = pci_region_sum(r_io); + if (sum < 0x4000) { + /* traditional region is big enougth, use it */ + r_io->base = 0xc000; + } else if (sum < 0x9000) { + /* use the larger region at 0x1000 */ + r_io->base = 0x1000; + } else { + /* + * Not enougth io address space -> error out. + * + * TODO: on q35 we can move PORT_ACPI_PM_BASE out of + * the way, then use the whole 1000 -> ffff region. + */ + return -1; + } + dprintf(1, "PCI: IO: %4llx - %4llx\n", r_io->base, r_io->base + sum - 1); + return 0; +} +static int pci_bios_init_root_regions_mem(struct pci_bus *bus) +{ struct pci_region *r_end = &bus->r[PCI_REGION_TYPE_PREFMEM]; struct pci_region *r_start = &bus->r[PCI_REGION_TYPE_MEM]; @@ -786,8 +818,11 @@ static void pci_region_map_entries(struct pci_bus *busses, struct pci_region *r) static void pci_bios_map_devices(struct pci_bus *busses) { + if (pci_bios_init_root_regions_io(busses)) + panic("PCI: out of I/O address space\n"); + dprintf(1, "PCI: 32: %016llx - %016llx\n", pcimem_start, pcimem_end); - if (pci_bios_init_root_regions(busses)) { + if (pci_bios_init_root_regions_mem(busses)) { struct pci_region r64_mem, r64_pref; r64_mem.list.first = NULL; r64_pref.list.first = NULL; @@ -796,7 +831,7 @@ static void pci_bios_map_devices(struct pci_bus *busses) pci_region_migrate_64bit_entries(&busses[0].r[PCI_REGION_TYPE_PREFMEM], &r64_pref); - if (pci_bios_init_root_regions(busses)) + if (pci_bios_init_root_regions_mem(busses)) panic("PCI: out of 32bit address space\n"); u64 sum_mem = pci_region_sum(&r64_mem); -- 1.8.3.1

3 3

[RFC][PATCH] coreboot: Add support for a "links" file to have aliases in CBFS.
by Kevin O'Connor 30 Jan '14

30 Jan '14

The "links" file is a newline separated list where each line is a "link name" followed by a "destination name" separated by a space character. For each line, SeaBIOS will consider each "link name" to be a CBFS file that has the contents found in the CBFS file with "destination name". Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- This patch may be useful for those using intel option roms where the same rom is used on a number of VGA devices with different vendor/device ids. With this patch, one can make a "links" file with the list of devices (eg, "pci8086,0a06.rom") that all point to the same physical rom (eg, "pci8086,0406.rom"). It's possible to implement this aliasing approach in a number of different ways. The approach taken in this patch is a single file with all the link names. This was done based on feedback that there could be 20 or more aliases for a single rom and creating individual CBFS files for each link would be unwieldy. That said, I'm not sure this is the best approach. I'm circulating it for comments. --- src/fw/coreboot.c | 34 ++++++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/src/fw/coreboot.c b/src/fw/coreboot.c index df8fca8..270d01a 100644 --- a/src/fw/coreboot.c +++ b/src/fw/coreboot.c @@ -408,6 +408,40 @@ coreboot_cbfs_init(void) fhdr = (void*)ALIGN((u32)cfile->data + cfile->rawsize , be32_to_cpu(hdr->align)); } + + // Process CBFS links file. The links file is a newline separated + // file where each line has a "link name" and a destination name + // separated by a space character. + char *links = romfile_loadfile("links", NULL); + if (!links) + return; + char *p = links; + for (;;) { + char *linkname = p; + char *end = strchr(linkname, '\n'); + if (!end) + break; + *end = '\0'; + p = end + 1; + char *linkend = strchr(linkname, ' '); + if (!linkend || linkname == linkend) + continue; + *linkend = '\0'; + struct romfile_s *ufile = romfile_find(linkend+1); + if (!ufile) + continue; + struct cbfs_romfile_s *cufile + = container_of(ufile, struct cbfs_romfile_s, file); + struct cbfs_romfile_s *cfile = malloc_tmp(sizeof(*cfile)); + if (!cfile) { + warn_noalloc(); + break; + } + memcpy(cfile, cufile, sizeof(*cfile)); + strtcpy(cfile->file.name, linkname, sizeof(cfile->file.name)); + romfile_add(&cfile->file); + } + free(links); } struct cbfs_payload_segment { -- 1.8.5.3

1 0

Re: [SeaBIOS] [RFC] vgahooks: Add Google stumpy, lumpy, and link boards to intel vgahook list.
by Kevin O'Connor 30 Jan '14

30 Jan '14

On Fri, Jan 17, 2014 at 04:50:27PM -0800, Stefan Reinauer wrote: > Yes, I think it's worth having them in upstream SeaBIOS! > > I've been thinking about creating a way of passing this information between > coreboot and SeaBIOS in the future, so we don't have to keep the handlers > around twice. One way of doing that would be to have the handler live in > SMM and have SeaBIOS do an SMI command to get the right answers. Would you > be open to something like that? That seems over kill to me. The Intel VGA stuff seems to be pretty stable and it's only a few fields that need to be passed back. What about placing the required info in a coreboot table - then SeaBIOS can just pass back the desired fields and not have to have board by board specific logic. BTW, I didn't port over "Alex" as it seems to be totally different from the other boards. -Kevin

2 1

Re: [SeaBIOS] [PATCH 1/3] usb: Move default pipe max packet size code from xhci to main code.
by Kevin O'Connor 29 Jan '14

29 Jan '14

On Mon, Jan 27, 2014 at 12:29:11PM -0500, Kevin O'Connor wrote: > Also, the max packet size for super speed devices is 512 bytes (not > 256 bytes). > > Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> > --- > > Some additional patches on my xhci-testing branch. Also available at: > > https://github.com/KevinOConnor/seabios/tree/xhci-testing FYI, I have pushed the xhci-testing branch to the seabios master branch. -Kevin

1 0

[PATCH 3/3] xhci: Add copyright notice to usb-xhci.c
by Kevin O'Connor 27 Jan '14

27 Jan '14

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/hw/usb-xhci.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/hw/usb-xhci.c b/src/hw/usb-xhci.c index b307c5e..e3acc39 100644 --- a/src/hw/usb-xhci.c +++ b/src/hw/usb-xhci.c @@ -1,3 +1,10 @@ +// Code for handling XHCI "Super speed" USB controllers. +// +// Copyright (C) 2013 Gerd Hoffmann <kraxel(a)redhat.com> +// Copyright (C) 2014 Kevin O'Connor <kevin(a)koconnor.net> +// +// This file may be distributed under the terms of the GNU LGPLv3 license. + #include "biosvar.h" // GET_LOWFLAT #include "config.h" // CONFIG_* #include "malloc.h" // memalign_low -- 1.8.5.3

1 0

[PATCH 2/3] xhci: Use the same endpoint initialization code in xhci_alloc_pipe()
by Kevin O'Connor 27 Jan '14

27 Jan '14

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/hw/usb-xhci.c | 74 ++++++++++++++++++++++--------------------------------- 1 file changed, 29 insertions(+), 45 deletions(-) diff --git a/src/hw/usb-xhci.c b/src/hw/usb-xhci.c index 3366f09..b307c5e 100644 --- a/src/hw/usb-xhci.c +++ b/src/hw/usb-xhci.c @@ -804,7 +804,7 @@ static struct usbhub_op_s xhci_hub_ops = { static struct xhci_inctx * -xhci_alloc_inctx(struct usbdevice_s *usbdev) +xhci_alloc_inctx(struct usbdevice_s *usbdev, int maxepid) { struct usb_xhci_s *xhci = container_of( usbdev->hub->cntl, struct usb_xhci_s, usb); @@ -816,9 +816,8 @@ xhci_alloc_inctx(struct usbdevice_s *usbdev) } memset(in, 0, size); - in->add = 0x01; struct xhci_slotctx *slot = (void*)&in[1 << xhci->context64]; - slot->ctx[0] |= (1 << 27); // context entries + slot->ctx[0] |= maxepid << 27; // context entries slot->ctx[0] |= speed_to_xhci[usbdev->speed] << 20; // Set high-speed hub flags. @@ -859,9 +858,10 @@ static int xhci_config_hub(struct usbhub_s *hub) if ((hdslot->ctx[3] >> 27) == 3) // Already configured return 0; - struct xhci_inctx *in = xhci_alloc_inctx(hub->usbdev); + struct xhci_inctx *in = xhci_alloc_inctx(hub->usbdev, 1); if (!in) return -1; + in->add = 0x01; struct xhci_slotctx *slot = (void*)&in[1 << xhci->context64]; slot->ctx[0] |= 1 << 26; slot->ctx[1] |= hub->portcount << 24; @@ -911,6 +911,23 @@ xhci_alloc_pipe(struct usbdevice_s *usbdev if (eptype == USB_ENDPOINT_XFER_INT) pipe->buf = malloc_low(pipe->pipe.maxpacket); + // Allocate input context and initialize endpoint info. + struct xhci_inctx *in = xhci_alloc_inctx(usbdev, epid); + if (!in) + goto fail; + in->add = 0x01 | (1 << epid); + struct xhci_epctx *ep = (void*)&in[(pipe->epid+1) << xhci->context64]; + if (eptype == USB_ENDPOINT_XFER_INT) + ep->ctx[0] = (usb_getFrameExp(usbdev, epdesc) + 3) << 16; + ep->ctx[1] |= eptype << 3; + if (epdesc->bEndpointAddress & USB_DIR_IN + || eptype == USB_ENDPOINT_XFER_CONTROL) + ep->ctx[1] |= 1 << 5; + ep->ctx[1] |= pipe->pipe.maxpacket << 16; + ep->deq_low = (u32)&pipe->reqs.ring[0]; + ep->deq_low |= 1; // dcs + ep->length = pipe->pipe.maxpacket; + dprintf(3, "%s: usbdev %p, ring %p, slotid %d, epid %d\n", __func__, usbdev, &pipe->reqs, pipe->slotid, pipe->epid); if (pipe->epid == 1) { @@ -920,7 +937,7 @@ xhci_alloc_pipe(struct usbdevice_s *usbdev if (ret) goto fail; } - // Enable slot and send set_address command. + // Enable slot. u32 size = (sizeof(struct xhci_slotctx) * 32) << xhci->context64; struct xhci_slotctx *dev = memalign_high(1024 << xhci->context64, size); if (!dev) { @@ -934,65 +951,32 @@ xhci_alloc_pipe(struct usbdevice_s *usbdev goto fail; } dprintf(3, "%s: enable slot: got slotid %d\n", __func__, slotid); - memset(dev, 0, size); pipe->slotid = usbdev->slotid = slotid; xhci->devs[slotid].ptr_low = (u32)dev; xhci->devs[slotid].ptr_high = 0; - struct xhci_inctx *in = xhci_alloc_inctx(usbdev); - if (!in) - goto fail; - in->add |= (1 << 1); - - struct xhci_epctx *ep = (void*)&in[2 << xhci->context64]; - ep->ctx[0] |= (3 << 16); // interval: 1ms - ep->ctx[1] |= (4 << 3); // control pipe - ep->ctx[1] |= (pipe->pipe.maxpacket << 16); - - ep->deq_low = (u32)&pipe->reqs.ring[0]; - ep->deq_low |= 1; // dcs - ep->deq_high = 0; - ep->length = pipe->pipe.maxpacket; - + // Send set_address command. int cc = xhci_cmd_address_device(xhci, slotid, in); - free(in); if (cc != CC_SUCCESS) { dprintf(1, "%s: address device: failed (cc %d)\n", __func__, cc); goto fail; } } else { - struct xhci_inctx *in = xhci_alloc_inctx(usbdev); - if (!in) - goto fail; - in->add |= (1 << pipe->epid); - struct xhci_slotctx *slot = (void*)&in[1 << xhci->context64]; - slot->ctx[0] = (slot->ctx[0] & ~0xf8000000) | (pipe->epid << 27); - - struct xhci_epctx *ep = (void*)&in[(pipe->epid+1) << xhci->context64]; - if (eptype == USB_ENDPOINT_XFER_INT) - ep->ctx[0] = (usb_getFrameExp(usbdev, epdesc) + 3) << 16; - ep->ctx[1] |= (eptype << 3); - if (epdesc->bEndpointAddress & USB_DIR_IN) - ep->ctx[1] |= (1 << 5); - ep->ctx[1] |= (pipe->pipe.maxpacket << 16); - ep->deq_low = (u32)&pipe->reqs.ring[0]; - ep->deq_low |= 1; // dcs - ep->deq_high = 0; - ep->length = pipe->pipe.maxpacket; - pipe->slotid = usbdev->slotid; + // Send configure command. int cc = xhci_cmd_configure_endpoint(xhci, pipe->slotid, in); - free(in); if (cc != CC_SUCCESS) { dprintf(1, "%s: configure endpoint: failed (cc %d)\n", __func__, cc); goto fail; } } - + free(in); return &pipe->pipe; + fail: free(pipe); + free(in); return NULL; } @@ -1010,11 +994,11 @@ xhci_update_pipe(struct usbdevice_s *usbdev, struct usb_pipe *upipe dprintf(3, "%s: usbdev %p, ring %p, slotid %d, epid %d\n", __func__, usbdev, &pipe->reqs, pipe->slotid, pipe->epid); if (eptype == USB_ENDPOINT_XFER_CONTROL && - pipe->pipe.maxpacket != epdesc->wMaxPacketSize) { + pipe->pipe.maxpacket != epdesc->wMaxPacketSize) { dprintf(1, "%s: reconf ctl endpoint pkt size: %d -> %d\n", __func__, pipe->pipe.maxpacket, epdesc->wMaxPacketSize); pipe->pipe.maxpacket = epdesc->wMaxPacketSize; - struct xhci_inctx *in = xhci_alloc_inctx(usbdev); + struct xhci_inctx *in = xhci_alloc_inctx(usbdev, 1); if (!in) return upipe; in->add = (1 << 1); -- 1.8.5.3

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS