SeaBIOS

seabios@seabios.org

4 participants
3801 discussions

seabios release?
by Gerd Hoffmann 13 May '14

13 May '14

Hi, I think it's time to start planning the next seabios release. First, because a bunch of changes have piled up in master. And second, because of the smbios changes. They are a step forward in making seabios less dependent on qemu internals and I want have that in qemu soonish. So, how about this plan: (1) merge smbios patches in qemu (patchset is close to final now). (2) merge smbios patch in seabios (3) start freeze (4) tag release candidate (early may is realistic for that I think). (5) pull release candidate into qemu for wider testing (6) merge bugfix patches if needed (7) tag final release 2-3 weeks after release candidate (end of may probably). (8) pull final release into qemu. Comments? cheers, Gerd

4 8

[PATCH 0/2] use full io address space on q35
by Gerd Hoffmann 13 May '14

13 May '14

Hi, This patch adds a config option to move PORT_ACPI_PM_BASE, then use the full I/O address space on q35. Patches have been sent already a few months back, but IIRC we didn't came to a final conclusion. Resending rebased + with minor help text update to re-kick discussion. cheers, Gerd Gerd Hoffmann (2): Add config option to move PORT_ACPI_PM_BASE out of the way Allow using full io region on q35 src/Kconfig | 7 +++++++ src/fw/paravirt.h | 9 +++++++-- src/fw/pciinit.c | 16 +++++++++------- 3 files changed, 23 insertions(+), 9 deletions(-) -- 1.8.3.1

3 5

Re: [SeaBIOS] [PATCH] acpi: Fix acpi dsdt compile warning.
by Kevin O'Connor 12 May '14

12 May '14

On Fri, May 09, 2014 at 09:20:21AM -0400, Kevin O'Connor wrote: > The _EJ0 method should not return a value. FYI, I have pushed this patch along with the three other patches I sent on Thursday. -Kevin

1 0

[PATCH] acpi: Fix acpi dsdt compile warning.
by Kevin O'Connor 09 May '14

09 May '14

The _EJ0 method should not return a value. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/fw/acpi-dsdt.dsl | 1 - src/fw/ssdt-pcihp.dsl | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/fw/acpi-dsdt.dsl b/src/fw/acpi-dsdt.dsl index 56243c3..3556dca 100644 --- a/src/fw/acpi-dsdt.dsl +++ b/src/fw/acpi-dsdt.dsl @@ -139,7 +139,6 @@ DefinitionBlock ( Method(PCEJ, 1, NotSerialized) { // _EJ0 method - eject callback Store(ShiftLeft(1, Arg0), B0EJ) - Return (0x0) } /* Hotplug notification method supplied by SSDT */ diff --git a/src/fw/ssdt-pcihp.dsl b/src/fw/ssdt-pcihp.dsl index 67e485f..cb24c11 100644 --- a/src/fw/ssdt-pcihp.dsl +++ b/src/fw/ssdt-pcihp.dsl @@ -29,7 +29,7 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) Name(_ADR, 0xAA0000) ACPI_EXTRACT_METHOD_STRING ssdt_pcihp_ej0 Method(_EJ0, 1) { - Return (PCEJ(_SUN)) + PCEJ(_SUN) } } } -- 1.9.0

1 0

Re: [SeaBIOS] [PATCH] If an int 1587 call is made from an option rom, stay in bigreal mode.
by Kevin O'Connor 09 May '14

09 May '14

On Sat, Apr 19, 2014 at 12:30:01PM -0400, Kevin O'Connor wrote: > Modify the int 1587 handler to check if the POST phase is still > running. If it is, use bigreal mode segment limits so that the caller > remains in bigreal mode when the 1587 handler completes. This helps > with SeaVGABIOS' use of "direct" framebuffer accesses (an option rom > may attempt to display text during its option rom execution which can > cause SeaVGABIOS to make the int 1587 calls). FYI, I have committed this patch. -Kevin

1 0

[PATCH v2] ld: don't hardcode "-melf_i386"
by Roger Pau Monne 09 May '14

09 May '14

On FreeBSD, the name of the emulation is "-melf_i386_fbsd", so allow SeaBIOS to fetch the emulation to use from the environment variables. This allows SeaBIOS to compile on FreeBSD with gcc. Signed-off-by: Roger Pau Monné <roger.pau(a)citrix.com> Cc: Ian Campbell <ian.campbell(a)citrix.com> --- Changes in v2: - Rebase on top of master --- Makefile | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index e567ba1..4338f94 100644 --- a/Makefile +++ b/Makefile @@ -24,6 +24,7 @@ STRIP=$(CROSS_PREFIX)strip PYTHON=python CPP=cpp IASL:=iasl +LD32BIT_FLAG:=-melf_i386 # Source files SRCBOTH=misc.c stacks.c output.c string.c x86.c block.c cdrom.c mouse.c kbd.c \ @@ -154,8 +155,8 @@ $(OUT)romlayout16.lds: $(OUT)ccode32flat.o $(OUT)code32seg.o $(OUT)ccode16.o $(O @echo " Building ld scripts" $(Q)./scripts/buildversion.sh $(OUT)version.c $(Q)$(CC) $(CFLAGS32FLAT) -c $(OUT)version.c -o $(OUT)version.o - $(Q)$(LD) -melf_i386 -r $(OUT)ccode32flat.o $(OUT)version.o -o $(OUT)code32flat.o - $(Q)$(LD) -melf_i386 -r $(OUT)ccode16.o $(OUT)romlayout.o -o $(OUT)code16.o + $(Q)$(LD) $(LD32BIT_FLAG) -r $(OUT)ccode32flat.o $(OUT)version.o -o $(OUT)code32flat.o + $(Q)$(LD) $(LD32BIT_FLAG) -r $(OUT)ccode16.o $(OUT)romlayout.o -o $(OUT)code16.o $(Q)$(OBJDUMP) -thr $(OUT)code32flat.o > $(OUT)code32flat.o.objdump $(Q)$(OBJDUMP) -thr $(OUT)code32seg.o > $(OUT)code32seg.o.objdump $(Q)$(OBJDUMP) -thr $(OUT)code16.o > $(OUT)code16.o.objdump -- 1.7.7.5 (Apple Git-26)

4 4

[PATCH] Fix int 1589 calls when CONFIG_ENTRY_EXTRASTACK is enabled.
by Kevin O'Connor 09 May '14

09 May '14

The int 1589 call is entered in real mode and returns in protected mode. However, the code to use the "extra stack" does not support that. Fix this by never using the "extra stack" on int 1589 calls. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/post.c | 2 +- src/romlayout.S | 10 ++++++++++ src/system.c | 4 ++-- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/src/post.c b/src/post.c index 0d9e66f..0fdd28e 100644 --- a/src/post.c +++ b/src/post.c @@ -61,7 +61,7 @@ ivt_init(void) SET_IVT(0x12, FUNC16(entry_12)); SET_IVT(0x13, FUNC16(entry_13_official)); SET_IVT(0x14, FUNC16(entry_14)); - SET_IVT(0x15, FUNC16(entry_15)); + SET_IVT(0x15, FUNC16(entry_15_official)); SET_IVT(0x16, FUNC16(entry_16)); SET_IVT(0x17, FUNC16(entry_17)); SET_IVT(0x18, FUNC16(entry_18)); diff --git a/src/romlayout.S b/src/romlayout.S index f6b7979..57e8bcc 100644 --- a/src/romlayout.S +++ b/src/romlayout.S @@ -378,6 +378,12 @@ entry_pcibios16: ENTRY_ARG handle_pcibios iretw +// int 1589 entry point + DECLFUNC entry_1589 +entry_1589: + ENTRY_ARG handle_1589 + iretw + // BIOS32 support .code32 DECLFUNC entry_bios32 @@ -682,6 +688,10 @@ entry_10_0x0f: IRQ_ENTRY_ARG 11 ORG 0xf859 + .global entry_15_official +entry_15_official: + cmpb $0x89, %ah + je entry_1589 // 1589 calls return in protected mode IRQ_ENTRY_ARG 15 // 0xfa6e - vgafont8 in font.c diff --git a/src/system.c b/src/system.c index 3cb2228..756dc31 100644 --- a/src/system.c +++ b/src/system.c @@ -191,9 +191,10 @@ handle_1588(struct bregs *regs) } // Switch to protected mode -static void +void VISIBLE16 handle_1589(struct bregs *regs) { + debug_enter(regs, DEBUG_HDL_15); set_a20(1); pic_reset(regs->bl, regs->bh); @@ -355,7 +356,6 @@ handle_15(struct bregs *regs) case 0x86: handle_1586(regs); break; case 0x87: handle_1587(regs); break; case 0x88: handle_1588(regs); break; - case 0x89: handle_1589(regs); break; case 0x90: handle_1590(regs); break; case 0x91: handle_1591(regs); break; case 0xc0: handle_15c0(regs); break; -- 1.9.0

1 0

[PATCH] build: Make sure romlayout.o is recompiled on Kconfig change.
by Kevin O'Connor 09 May '14

09 May '14

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e567ba1..608d033 100644 --- a/Makefile +++ b/Makefile @@ -146,7 +146,7 @@ $(OUT)code32seg.o: $(OUT)autoconf.h $(patsubst %.c, $(OUT)src/%.o,$(SRC32SEG)) ; $(OUT)ccode32flat.o: $(OUT)autoconf.h $(patsubst %.c, $(OUT)src/%.o,$(SRC32FLAT)) ; $(call whole-compile, $(CFLAGS32FLAT), $(addprefix src/, $(SRC32FLAT)),$@) -$(OUT)romlayout.o: src/romlayout.S $(OUT)asm-offsets.h +$(OUT)romlayout.o: src/romlayout.S $(OUT)autoconf.h $(OUT)asm-offsets.h @echo " Compiling (16bit) $@" $(Q)$(CC) $(CFLAGS16) -c -D__ASSEMBLY__ $< -o $@ -- 1.9.0

1 0

[PATCH] xhci: Use msleep() instead of mdelay() for port status delay.
by Kevin O'Connor 09 May '14

09 May '14

Use msleep() so that interrupts and other threads can occur during the delay. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/hw/usb-xhci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/hw/usb-xhci.c b/src/hw/usb-xhci.c index f64f562..ca1fe25 100644 --- a/src/hw/usb-xhci.c +++ b/src/hw/usb-xhci.c @@ -705,7 +705,7 @@ configure_xhci(void *data) writel(&xhci->op->usbcmd, reg); // FIXME: try find a more elegant way than a fixed delay - mdelay(100); + msleep(100); usb_enumerate(&xhci->hub); // XXX - should walk list of pipes and free unused pipes. -- 1.9.0

1 0

Re: [SeaBIOS] [Xen-devel] [PATCH v3] Hvmloader: Modify ACPI to only supply _EJ0 methods for PCIslots that support hotplug by runtime patching
by Kevin O'Connor 08 May '14

08 May '14

On Thu, May 08, 2014 at 07:29:15AM +0000, Gonglei (Arei) wrote: > > > > --- /dev/null > > > > +++ b/tools/firmware/hvmloader/tools/acpi_extract.py > > > > @@ -0,0 +1,308 @@ > > > > +#!/usr/bin/python > > > > +# Copyright (C) 2011 Red Hat, Inc., Michael S. Tsirkin <mst(a)redhat.com> > > > > +# > > > > +# This file may be distributed under the terms of the GNU GPLv3 license. > > > > > > Well, Xen code is under GPLv2 so I don't think you can do this? > > > > > > Unless Michael is willing to release it under a different licence? > > > > I'd be fine with "GPLv2 or later" so code can flow back to seabios > > as well - but you will have to track down everyone else who > > changed this file since I wrote it and get their permission. > > > Thanks, Michael. I search the git log of above two python files, and > find two person has changed those two files, Johannes and Kevin. > > CC'ing Johannes and Kevin for their permission, which using those > two files in Xen code. Thanks in advance! I'm okay with those two python scripts being used as "GPLv2 or later". -Kevin

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS