SeaBIOS

seabios@seabios.org

4 participants
3801 discussions

serial console : no keystroke
by Maxime de Roucy 20 Aug '15

20 Aug '15

Hello, I am trying vanilla coreboot/SeaBios on an pc-engines apu1d4 : http://www.pcengines.ch/apu1d4.htm The board natively use a modified version of coreboot, the sources are available at http://www.pcengines.ch/howto.php#CoreBoot (coreboot_14090 8.tar.gz) I use a usb-SR232 adapter to connect on the board. I use minicom and picocom. Both are working fine. BUT When I use coreboot vanilla (master) with SeaBios vanilla (master, I also tried 1.7.5), when I get to "Press ESC for boot menu" I can't send keystroke to SeaBios through the serial console. I have to plug an USB keyboard and press ESC on it. I tried scancode (F10, F12, Q (0x10)) same problem. I disabled hardware and software flow control on minicom. In fact, SeaBios doesn't get ANY keystroke from the serial port. I can press any keyboard key in minicom/picocom the message "Press ESC for boot menu." stay until timeout (DEFAULT_BOOTMENU_WAIT) (I set etc/boot-menu-wait to 90000 to verify that… so I am pretty sure of it). Furthermore, after I enter the boot menu via usb keyboard I can't choose anything from the menu through minicom/picocom. I have to use an USB keyboard. When I get in the OS (tinycore Linux), minicom/picocom input works. I don't have the problem with the modified coreboot/SeaBios from PC -Engines. At first I thought it's a SeaBios problem but the keystroke code part (src/boot.c) are the same in vanilla SeaBios and PC-Engines SeaBios. PC-Engines sources don't have .config files (nor for coreboot neither for SeaBios) but have build/config.h and out/autoconf.h. I tried to look for some interesting difference between vanilla and PC -Engines sources for those file but my coreboot/SeaBios knowledge weren't enough to spot anything. Can anyone help me ? Since I don't know if the problem relies on coreboot or SeaBios I post this on both mailing-list. I attached to this mail : * my .config coreboot file * my build/config.h coreboot file * my payloads/external/SeaBIOS/seabios/out/autoconf.h SeaBios file * PC-Engines build/config.h * PC-Engines SeaBios autoconf.h -- Regards Maxime

3 3

: USB 3.0 port failed if a USB 3.0 device is attached when power up
by Zheng Bao 20 Aug '15

20 Aug '15

hi, Kevin, I am debugging the coreboot on AMD board. I found if a device is populated on USB 3.0 port during system power up, the USB3.0 function fail after OS boot up. If a USB 2.0 device is plugged, the port is OK. Can you take a look at this? Here is the debug message. ------------ SeaBIOS (version rel-1.8.0-26-g92f9b91-dirty-20150805_022528-supersrv) XHCI init on dev 00:10.0: regs @ 0xf0b8a000, 8 ports, 32 slots, 32 byte contexts XHCI extcap 0x1 @ f0b8a500 XHCI protocol USB 3.00, 4 ports (offset 1), def 0 XHCI protocol USB 2.00, 4 ports (offset 5), def 18 XHCI extcap 0xa @ f0b8a540 configure_xhci: resetting configure_xhci: setup 8 scratch pad buffers xhci_hub_reset port #5: 0x000202e1, powered, pls 7, speed 0 [ - ] XHCI port #5: 0x00200e03, powered, enabled, pls 0, speed 3 [High] xhci_alloc_pipe: usbdev 0xbff52960, ring 0xbff9e900, slotid 0, epid 1 xhci_cmd_enable_slot: xhci_trb_queue: ring 0xbff9ed00 [nidx 1, len 0] xhci_process_events: status change port #5 xhci_process_events: status change port #1 xhci_process_events: status change port #5 xhci_process_events: ring 0xbff9ed00 [trb 0xbff9ed00, evt 0xbff9ee00, type 33, eidx 1, cc 1] xhci_alloc_pipe: enable slot: got slotid 1 xhci_cmd_address_device: slotid 1 xhci_trb_queue: ring 0xbff9ed00 [nidx 2, len 0] WARNING - Timeout at xhci_event_wait:692! xhci_alloc_pipe: address device: failed (cc -1) XHCI no devices found WARNING - Timeout at wait_bit:310! Joe

3 4

[RFC 0/2] QEMU fw_cfg dma interface
by Marc Marí 19 Aug '15

19 Aug '15

These patches, with the support in QEMU (patches in the QEMU mailing list) make the Linux kernel setup and boot approximately 10x faster. TODO: * Other ways to boot when detecting fw_cfg DMA interface? * Is 0xfef00000 a good address? Is there any better address? Is it possible to not make it hardcoded? Marc Marí (2): Add QEMU fw_cfg dma interface support Add Linux boot code for the fw_cfg dma interface src/boot.c | 14 ++- src/config.h | 1 + src/fw/paravirt.c | 298 +++++++++++++++++++++++++++++++++++++++++++----------- src/fw/paravirt.h | 50 +++++++++ src/romlayout.S | 20 ++++ 5 files changed, 321 insertions(+), 62 deletions(-) -- 2.4.3

3 11

[PATCH] rtc: Support disabling the RTC timer irq support
by Kevin O'Connor 18 Aug '15

18 Aug '15

Add a build time config option to remove support for RTC timer interrupts along with the associated bios calls requiring that support. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/Kconfig | 9 +++++++++ src/clock.c | 13 ++++++++++++- src/hw/rtc.c | 6 ++++++ src/stacks.c | 2 +- 4 files changed, 28 insertions(+), 2 deletions(-) diff --git a/src/Kconfig b/src/Kconfig index 6256242..56a1b2f 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -299,6 +299,15 @@ menu "Hardware support" default y help Support parallel ports. This also enables int 17 parallel port calls. + config RTC_TIMER + bool "Real Time Clock (RTC) scheduling" + default y + help + Support MC146818 Real Time Clock chip timer + interrupts. This also enables int 1583 and int 1586 calls. + + Disabling this support does not disable access to the RTC + cmos registers. config USE_SMM depends on QEMU diff --git a/src/clock.c b/src/clock.c index ee2b53f..2bb5209 100644 --- a/src/clock.c +++ b/src/clock.c @@ -56,7 +56,8 @@ clock_setup(void) } enable_hwirq(0, FUNC16(entry_08)); - enable_hwirq(8, FUNC16(entry_70)); + if (CONFIG_RTC_TIMER) + enable_hwirq(8, FUNC16(entry_70)); } @@ -374,6 +375,10 @@ clear_usertimer(void) void handle_1586(struct bregs *regs) { + if (!CONFIG_RTC_TIMER) { + set_code_unimplemented(regs, RET_EUNSUPPORTED); + return; + } // Use the rtc to wait for the specified time. u8 statusflag = 0; u32 count = (regs->cx << 16) | regs->dx; @@ -417,6 +422,10 @@ handle_1583XX(struct bregs *regs) void handle_1583(struct bregs *regs) { + if (!CONFIG_RTC_TIMER) { + handle_1583XX(regs); + return; + } switch (regs->al) { case 0x00: handle_158300(regs); break; case 0x01: handle_158301(regs); break; @@ -430,6 +439,8 @@ handle_1583(struct bregs *regs) void VISIBLE16 handle_70(void) { + if (!CONFIG_RTC_TIMER) + return; debug_isr(DEBUG_ISR_70); // Check which modes are enabled and have occurred. diff --git a/src/hw/rtc.c b/src/hw/rtc.c index 628d542..570a3ad 100644 --- a/src/hw/rtc.c +++ b/src/hw/rtc.c @@ -62,6 +62,8 @@ rtc_updating(void) void rtc_setup(void) { + if (!CONFIG_RTC_TIMER) + return; rtc_write(CMOS_STATUS_A, 0x26); // 32,768Khz src, 976.5625us updates rtc_mask(CMOS_STATUS_B, ~RTC_B_DSE, RTC_B_24HR); rtc_read(CMOS_STATUS_C); @@ -73,6 +75,8 @@ int RTCusers VARLOW; void rtc_use(void) { + if (!CONFIG_RTC_TIMER) + return; int count = GET_LOW(RTCusers); SET_LOW(RTCusers, count+1); if (count) @@ -84,6 +88,8 @@ rtc_use(void) void rtc_release(void) { + if (!CONFIG_RTC_TIMER) + return; int count = GET_LOW(RTCusers); SET_LOW(RTCusers, count-1); if (count != 1) diff --git a/src/stacks.c b/src/stacks.c index 0c5f79a..e67aeb6 100644 --- a/src/stacks.c +++ b/src/stacks.c @@ -574,7 +574,7 @@ thread_setup(void) int threads_during_optionroms(void) { - return CONFIG_THREADS && ThreadControl == 2 && in_post(); + return CONFIG_THREADS && CONFIG_RTC_TIMER && ThreadControl == 2 && in_post(); } // Switch to next thread stack. -- 1.9.3

2 2

Re: [SeaBIOS] QEMU fw_cfg DMA interface
by Marc Marí 18 Aug '15

18 Aug '15

On Thu, 6 Aug 2015 13:27:16 +0100 Stefan Hajnoczi <stefanha(a)gmail.com> wrote: > On Thu, Aug 6, 2015 at 12:00 PM, Marc Marí <markmb(a)redhat.com> wrote: > > When running a Linux guest on top of QEMU, using the -kernel > > options, this is the timing improvement for x86: > > > > QEMU commit 2be4f242b50a8 and SeaBIOS commit 908a58c1d5ff > > QEMU startup time: .078 > > BIOS startup time: .060 > > Kernel setup time: .578 > > Total time: .716 > > > > QEMU with this patch series and SeaBIOS with this patch series > > QEMU startup time: .080 > > BIOS startup time: .039 > > Kernel setup time: .002 > > Total time: .121 > > Impressive results! > > Is this a fully-featured QEMU build or did you disable things? > > Is this the default SeaBIOS build or did you disable things? > This is the default QEMU configuration I get for my system. It's not a fully-featured QEMU, but it has a lot of things enabled. SeaBIOS has a default configuration (with debugging disabled). The QEMU configuration is: [...] tcg debug enabled no gprof enabled no sparse enabled no strip binaries yes profiler no static build no pixman system SDL support yes GTK support yes GNUTLS support yes GNUTLS hash yes GNUTLS gcrypt no GNUTLS nettle yes (2.7.1) VTE support yes curses support yes curl support yes mingw32 support no Audio drivers oss Block whitelist (rw) Block whitelist (ro) VirtFS support yes VNC support yes VNC TLS support yes VNC SASL support yes VNC JPEG support yes VNC PNG support yes xen support no brlapi support yes bluez support yes Documentation no GUEST_BASE yes PIE yes vde support yes netmap support no Linux AIO support yes ATTR/XATTR support yes Install blobs yes KVM support yes RDMA support yes TCG interpreter no fdt support yes preadv support yes fdatasync yes madvise yes posix_madvise yes sigev_thread_id yes uuid support yes libcap-ng support yes vhost-net support yes vhost-scsi support yes Trace backends nop spice support yes (0.12.7/0.12.5) rbd support yes xfsctl support no nss used yes libusb yes usb net redir yes OpenGL support no libiscsi support yes libnfs support no build guest agent yes QGA VSS support no QGA w32 disk info no seccomp support yes coroutine backend ucontext coroutine pool yes GlusterFS support yes Archipelago support no gcov gcov gcov enabled no TPM support yes libssh2 support yes TPM passthrough yes QOM debugging yes vhdx yes lzo support yes snappy support yes bzip2 support yes NUMA host support yes tcmalloc support no Marc

4 5

[PATCH 1/2] edd: Pass the segment/offset from int 1348 calls using a 'struct segoff_s'
by Kevin O'Connor 17 Aug '15

17 Aug '15

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/block.c | 16 +++++++++------- src/block.h | 2 +- src/disk.c | 2 +- 3 files changed, 11 insertions(+), 9 deletions(-) diff --git a/src/block.c b/src/block.c index a1054ac..cd732c1 100644 --- a/src/block.c +++ b/src/block.c @@ -288,10 +288,12 @@ map_floppy_drive(struct drive_s *drive) ****************************************************************/ static int -fill_generic_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf +fill_generic_edd(struct segoff_s edd, struct drive_s *drive_gf , u32 dpte_so, char *iface_type , int bdf, u8 channel, u16 iobase, u64 device_path) { + u16 seg = edd.seg; + struct int13dpt_s *param_far = (void*)(edd.offset+0); u16 size = GET_FARVAR(seg, param_far->size); u16 t13 = size == 74; @@ -394,7 +396,7 @@ fill_generic_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf struct dpte_s DefaultDPTE VARLOW; static int -fill_ata_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf) +fill_ata_edd(struct segoff_s edd, struct drive_s *drive_gf) { if (!CONFIG_ATA) return DISK_RET_EPARAM; @@ -446,24 +448,24 @@ fill_ata_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf) SET_LOW(DefaultDPTE.checksum, -sum); return fill_generic_edd( - seg, param_far, drive_gf, SEGOFF(SEG_LOW, (u32)&DefaultDPTE).segoff + edd, drive_gf, SEGOFF(SEG_LOW, (u32)&DefaultDPTE).segoff , "ATA ", bdf, channel, iobase1, slave); } int noinline -fill_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf) +fill_edd(struct segoff_s edd, struct drive_s *drive_gf) { switch (GET_GLOBALFLAT(drive_gf->type)) { case DTYPE_ATA: case DTYPE_ATA_ATAPI: - return fill_ata_edd(seg, param_far, drive_gf); + return fill_ata_edd(edd, drive_gf); case DTYPE_VIRTIO_BLK: case DTYPE_VIRTIO_SCSI: return fill_generic_edd( - seg, param_far, drive_gf, 0xffffffff + edd, drive_gf, 0xffffffff , "SCSI ", GET_GLOBALFLAT(drive_gf->cntl_id), 0, 0, 0); default: - return fill_generic_edd(seg, param_far, drive_gf, 0, NULL, 0, 0, 0, 0); + return fill_generic_edd(edd, drive_gf, 0, NULL, 0, 0, 0, 0); } } diff --git a/src/block.h b/src/block.h index 702f357..2ff359f 100644 --- a/src/block.h +++ b/src/block.h @@ -110,7 +110,7 @@ void map_floppy_drive(struct drive_s *drive); void map_hd_drive(struct drive_s *drive); void map_cd_drive(struct drive_s *drive); struct int13dpt_s; -int fill_edd(u16 seg, struct int13dpt_s *param_far, struct drive_s *drive_gf); +int fill_edd(struct segoff_s edd, struct drive_s *drive_gf); int default_process_op(struct disk_op_s *op); int process_op(struct disk_op_s *op); int send_disk_op(struct disk_op_s *op); diff --git a/src/disk.c b/src/disk.c index 91d939a..3854d00 100644 --- a/src/disk.c +++ b/src/disk.c @@ -522,7 +522,7 @@ disk_1347(struct bregs *regs, struct drive_s *drive_gf) static void disk_1348(struct bregs *regs, struct drive_s *drive_gf) { - int ret = fill_edd(regs->ds, (void*)(regs->si+0), drive_gf); + int ret = fill_edd(SEGOFF(regs->ds, regs->si), drive_gf); disk_ret(regs, ret); } -- 1.9.3

1 2

[PATCH 1/2] Move CanInterrupt check to check_irqs() and wait_irq()
by Kevin O'Connor 17 Aug '15

17 Aug '15

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/stacks.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/stacks.c b/src/stacks.c index 1ad9ef0..1cd0fde 100644 --- a/src/stacks.c +++ b/src/stacks.c @@ -661,6 +661,9 @@ fail: void VISIBLE16 check_irqs(void) { + if (!MODESEGMENT && !CanInterrupt) + // Can't enable interrupts (PIC and/or IVT not yet setup) + return; if (need_hop_back()) { extern void _cfunc16_check_irqs(void); stack_hop_back(0, 0, _cfunc16_check_irqs); @@ -674,12 +677,11 @@ void yield(void) { if (MODESEGMENT || !CONFIG_THREADS) { - if (MODESEGMENT || CanInterrupt) - check_irqs(); + check_irqs(); return; } struct thread_info *cur = getCurThread(); - if (cur == &MainThread && CanInterrupt) + if (cur == &MainThread) // Permit irqs to fire check_irqs(); @@ -690,6 +692,9 @@ yield(void) void VISIBLE16 wait_irq(void) { + if (!MODESEGMENT && !CanInterrupt) + // Can't enable interrupts (PIC and/or IVT not yet setup) + return; if (need_hop_back()) { extern void _cfunc16_wait_irq(void); stack_hop_back(0, 0, _cfunc16_wait_irq); @@ -707,8 +712,7 @@ yield_toirq(void) yield(); return; } - if (MODESEGMENT || CanInterrupt) - wait_irq(); + wait_irq(); } // Wait for all threads (other than the main thread) to complete. -- 1.9.3

1 2

[PATCH 00/14] Initial sdcard support on real hardware
by Kevin O'Connor 17 Aug '15

17 Aug '15

This series further improves the SeaBIOS sdcard support. At the conclusion of the series the sdcard support is enabled on configurations beyond QEMU. This code was tested on an Asus C300 (a Baytrail style Chromebook) with both an external sdcard card and an internal eMMC drive. -Kevin Kevin O'Connor (14): sdcard: Fix typo - use sdcard_pio() instead of sdcard_pio_app() sdcard: Fill command bits according to spec sdcard: Support SDHCI v3.00 spec clock setting sdcard: Move power setup to new function sdcard_set_power() sdcard: Power controller up to maximum voltage supported sdcard: Power down controller on failure sdcard: The card should never be in a busy state at start of sdcard_pio() sdcard: Implement timeout on every block read in sdcard_pio_transfer() sdcard: Rename waitw() to sdcard_waitw() and simplify sdcard: Perform a controller reset at start of init sdcard: Check for error events during sdcard_pio() sdcard: Initial support for MMC cards sdcard: Allow the sdcard driver to run on real hardware sdcard: Allow sdcard addresses to be specified in CBFS files docs/Runtime_config.md | 1 + src/Kconfig | 2 +- src/hw/sdcard.c | 293 +++++++++++++++++++++++++++++++++---------------- 3 files changed, 202 insertions(+), 94 deletions(-) -- 1.9.3

1 15

[PATCH] optionroms: Don't run option rom on PCI bar if CBFS/fw_cfg version exists
by Kevin O'Connor 17 Aug '15

17 Aug '15

If there exists a "pci%04x,%04x.rom" file in CBFS/fw_cfg then the code should not attempt to run a rom found on the PCI device itself - even if the one in CBFS/fw_cfg is not a valid rom. This makes the code match the documentation. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/optionroms.c | 30 ++++++++++++------------------ 1 file changed, 12 insertions(+), 18 deletions(-) diff --git a/src/optionroms.c b/src/optionroms.c index a24c539..c81eff2 100644 --- a/src/optionroms.c +++ b/src/optionroms.c @@ -183,19 +183,6 @@ deploy_romfile(struct romfile_s *file) return rom; } -// Check if an option rom is at a hardcoded location or in CBFS. -static struct rom_header * -lookup_hardcode(struct pci_device *pci) -{ - char fname[17]; - snprintf(fname, sizeof(fname), "pci%04x,%04x.rom" - , pci->vendor, pci->device); - struct romfile_s *file = romfile_find(fname); - if (file) - return deploy_romfile(file); - return NULL; -} - // Run all roms in a given CBFS directory. static void run_file_roms(const char *prefix, int isvga, u64 *sources) @@ -324,21 +311,28 @@ fail: } // Attempt to map and initialize the option rom on a given PCI device. -static int +static void init_pcirom(struct pci_device *pci, int isvga, u64 *sources) { u16 bdf = pci->bdf; dprintf(4, "Attempting to init PCI bdf %02x:%02x.%x (vd %04x:%04x)\n" , pci_bdf_to_bus(bdf), pci_bdf_to_dev(bdf), pci_bdf_to_fn(bdf) , pci->vendor, pci->device); - struct rom_header *rom = lookup_hardcode(pci); - if (!rom && ((RunPCIroms > 1) || ((RunPCIroms == 1) && isvga))) + + char fname[17]; + snprintf(fname, sizeof(fname), "pci%04x,%04x.rom" + , pci->vendor, pci->device); + struct romfile_s *file = romfile_find(fname); + struct rom_header *rom = NULL; + if (file) + rom = deploy_romfile(file); + else if (RunPCIroms > 1 || (RunPCIroms == 1 && isvga)) rom = map_pcirom(pci); if (! rom) // No ROM present. - return -1; + return; setRomSource(sources, rom, RS_PCIROM | (u32)pci); - return init_optionrom(rom, bdf, isvga); + init_optionrom(rom, bdf, isvga); } -- 1.9.3

1 1

[PATCH] build: Require "make VERSION=x" parameter to be on command line
by Kevin O'Connor 13 Aug '15

13 Aug '15

Don't pull in VERSION from the build environment - there is too great a potential for a variable conflict. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile b/Makefile index e287530..220a9f7 100644 --- a/Makefile +++ b/Makefile @@ -26,6 +26,7 @@ PYTHON=python CPP=cpp IASL:=iasl LD32BIT_FLAG:=-melf_i386 +VERSION:= # Source files SRCBOTH=misc.c stacks.c output.c string.c block.c cdrom.c disk.c mouse.c kbd.c \ -- 1.9.3

2 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS