SeaBIOS May 2013

seabios@seabios.org

29 participants
56 discussions

another iasl update breaks seabios compilation again
by Michael Tokarev 13 May '13

13 May '13

[Resending after being subscribed to the list] I'm not sure what happened yet, but I noticed that current seabios does not build with 20130214-32 version of iasl. Apparently, the listing file (-l) produced now does not contain any comments from the original source, so tools/acpi_extract.py produce nothing from these files. Here's the diff between listing produced by two versions of iasl for ssdt-pcihp: ---------------- --- out/ssdt-pcihp.lst-20100528 2013-05-09 17:55:43.872160687 +0400 +++ out/ssdt-pcihp.lst-20130214-32 2013-05-09 17:57:45.222803783 +0400 @@ -1,84 +1,72 @@ Intel ACPI Component Architecture -ASL Optimizing Compiler version 20100528 [Jul 2 2010] -Copyright (c) 2000 - 2010 Intel Corporation -Supports ACPI Specification Revision 4.0a - -Compilation of "out/ssdt-pcihp.dsl.i" - Thu May 9 17:55:43 2013 - - 1.... - 2..../* ACPI_EXTRACT_ALL_CODE ssdp_pcihp_aml */ - 3.... - 4....DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) - -00000000....53 53 44 54 58 00 00 00 "SSDTX..." -00000008....01 77 42 58 50 43 00 00 ".wBXPC.." -00000010....42 58 53 53 44 54 50 43 "BXSSDTPC" -00000018....01 00 00 00 49 4E 54 4C "....INTL" -00000020....28 05 10 20 ............ "(.. " - - 5....{ - 6.... External(\_SB.PCI0, DeviceObj) - 7.... External(\_SB.PCI0.PCEJ, MethodObj) - 8.... Scope(\_SB.PCI0) { - -00000024....10 33 5C 2E 5F 53 42 5F ".3\._SB_" -0000002C....50 43 49 30 ............ "PCI0" - - 9.... - 10..../* ACPI_EXTRACT_DEVICE_START ssdt_pcihp_start */ - 11.... - 12.... - 13..../* ACPI_EXTRACT_DEVICE_END ssdt_pcihp_end */ - 14.... - 15.... - 16..../* ACPI_EXTRACT_DEVICE_STRING ssdt_pcihp_name */ - 17.... - 18.... Device(SAA) { - -00000030....5B 82 26 53 41 41 5F ... "[.&SAA_" - - 19.... - 20..../* ACPI_EXTRACT_NAME_BYTE_CONST ssdt_pcihp_id */ - 21.... - 22.... Name(_SUN, 0xAA) - -00000037....08 5F 53 55 4E 0A AA ... "._SUN.." - - 23.... - 24..../* ACPI_EXTRACT_NAME_DWORD_CONST ssdt_pcihp_adr */ - 25.... - 26.... Name(_ADR, 0xAA0000) - -0000003E....08 5F 41 44 52 0C 00 00 "._ADR..." -00000046....AA 00 .................. ".." - - 27.... - 28..../* ACPI_EXTRACT_METHOD_STRING ssdt_pcihp_ej0 */ - 29.... - 30.... Method(_EJ0, 1) { - -00000048....14 0F 5F 45 4A 30 01 ... ".._EJ0." - - 31.... Return (PCEJ(_SUN)) - -0000004F....A4 ..................... "." -00000050....50 43 45 4A 5F 53 55 4E "PCEJ_SUN" - 32.... } - 33.... } - 34.... } - 35....} - 36.... +ASL Optimizing Compiler version 20130214-32 [Apr 6 2013] +Copyright (c) 2000 - 2013 Intel Corporation + +Compilation of "out/ssdt-pcihp.dsl.i" - Thu May 9 17:57:45 2013 + + 4: DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) + +00000000: 53 53 44 54 58 00 00 00 "SSDTX..." +00000008: 01 8B 42 58 50 43 00 00 "..BXPC.." +00000010: 42 58 53 53 44 54 50 43 "BXSSDTPC" +00000018: 01 00 00 00 49 4E 54 4C "....INTL" +00000020: 14 02 13 20 ............ "... " + + 5: { + 6: External(\_SB.PCI0, DeviceObj) + 7: External(\_SB.PCI0.PCEJ, MethodObj) + 8: Scope(\_SB.PCI0) { + +00000024: 10 33 5C 2E 5F 53 42 5F ".3\._SB_" +0000002C: 50 43 49 30 ............ "PCI0" + + 18: Device(SAA) { + +00000030: 5B 82 26 53 41 41 5F ... "[.&SAA_" + + 22: Name(_SUN, 0xAA) + +00000037: 08 5F 53 55 4E 0A AA ... "._SUN.." + + 26: Name(_ADR, 0xAA0000) + +0000003E: 08 5F 41 44 52 0C 00 00 "._ADR..." +00000046: AA 00 .................. ".." + + 30: Method(_EJ0, 1) { + +00000048: 14 0F 5F 45 4A 30 01 ... ".._EJ0." + + 31: Return (PCEJ(_SUN)) + +[****iasl****] +out/ssdt-pcihp.dsl.i 31: Return (PCEJ(_SUN)) +Warning 1104 - ^ Reserved method should not return a value (_EJ0) + + + +0000004F: A4 ..................... "." +00000050: 50 43 45 4A 5F 53 55 4E "PCEJ_SUN" + 32: } + 33: } + 34: } + 35: } + Summary of errors and warnings +out/ssdt-pcihp.dsl.i 31: Return (PCEJ(_SUN)) +Warning 1104 - ^ Reserved method should not return a value (_EJ0) -ASL Optimizing Compiler version 20100528 [Jul 2 2010] -ASL Input: out/ssdt-pcihp.dsl.i - 37 lines, 767 bytes, 6 keywords -AML Output: out/ssdt-pcihp.aml - 88 bytes, 5 named objects, 1 executable opcodes +ASL Optimizing Compiler version 20130214-32 [Apr 6 2013] -Compilation complete. 0 Errors, 0 Warnings, 0 Remarks, 0 Optimizations +ASL Input: out/ssdt-pcihp.dsl.i - 37 lines, 359 bytes, 6 keywords +AML Output: out/ssdt-pcihp.aml - 88 bytes, 5 named objects, 1 executable opcodes +Listing File: out/ssdt-pcihp.lst - 2016 bytes +Hex Dump: out/ssdt-pcihp.hex - 228 bytes +Compilation complete. 0 Errors, 1 Warnings, 0 Remarks, 0 Optimizations ---------------- As you can see, there's no code comments anymore, so nothing to output by acpi_extract.py I haven't digged further yet -- just a heads-up for now. Thanks, /mjt

2 2

another iasl update breaks seabios compilation again
by Michael Tokarev 09 May '13

09 May '13

I'm not sure what happened yet, but I noticed that current seabios does not build with 20130214-32 version of iasl. Apparently, the listing file (-l) produced now does not contain any comments from the original source, so tools/acpi_extract.py produce nothing from these files. Here's the diff between listing produced by two versions of iasl for ssdt-pcihp: ---------------- --- out/ssdt-pcihp.lst-20100528 2013-05-09 17:55:43.872160687 +0400 +++ out/ssdt-pcihp.lst-20130214-32 2013-05-09 17:57:45.222803783 +0400 @@ -1,84 +1,72 @@ Intel ACPI Component Architecture -ASL Optimizing Compiler version 20100528 [Jul 2 2010] -Copyright (c) 2000 - 2010 Intel Corporation -Supports ACPI Specification Revision 4.0a - -Compilation of "out/ssdt-pcihp.dsl.i" - Thu May 9 17:55:43 2013 - - 1.... - 2..../* ACPI_EXTRACT_ALL_CODE ssdp_pcihp_aml */ - 3.... - 4....DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) - -00000000....53 53 44 54 58 00 00 00 "SSDTX..." -00000008....01 77 42 58 50 43 00 00 ".wBXPC.." -00000010....42 58 53 53 44 54 50 43 "BXSSDTPC" -00000018....01 00 00 00 49 4E 54 4C "....INTL" -00000020....28 05 10 20 ............ "(.. " - - 5....{ - 6.... External(\_SB.PCI0, DeviceObj) - 7.... External(\_SB.PCI0.PCEJ, MethodObj) - 8.... Scope(\_SB.PCI0) { - -00000024....10 33 5C 2E 5F 53 42 5F ".3\._SB_" -0000002C....50 43 49 30 ............ "PCI0" - - 9.... - 10..../* ACPI_EXTRACT_DEVICE_START ssdt_pcihp_start */ - 11.... - 12.... - 13..../* ACPI_EXTRACT_DEVICE_END ssdt_pcihp_end */ - 14.... - 15.... - 16..../* ACPI_EXTRACT_DEVICE_STRING ssdt_pcihp_name */ - 17.... - 18.... Device(SAA) { - -00000030....5B 82 26 53 41 41 5F ... "[.&SAA_" - - 19.... - 20..../* ACPI_EXTRACT_NAME_BYTE_CONST ssdt_pcihp_id */ - 21.... - 22.... Name(_SUN, 0xAA) - -00000037....08 5F 53 55 4E 0A AA ... "._SUN.." - - 23.... - 24..../* ACPI_EXTRACT_NAME_DWORD_CONST ssdt_pcihp_adr */ - 25.... - 26.... Name(_ADR, 0xAA0000) - -0000003E....08 5F 41 44 52 0C 00 00 "._ADR..." -00000046....AA 00 .................. ".." - - 27.... - 28..../* ACPI_EXTRACT_METHOD_STRING ssdt_pcihp_ej0 */ - 29.... - 30.... Method(_EJ0, 1) { - -00000048....14 0F 5F 45 4A 30 01 ... ".._EJ0." - - 31.... Return (PCEJ(_SUN)) - -0000004F....A4 ..................... "." -00000050....50 43 45 4A 5F 53 55 4E "PCEJ_SUN" - 32.... } - 33.... } - 34.... } - 35....} - 36.... +ASL Optimizing Compiler version 20130214-32 [Apr 6 2013] +Copyright (c) 2000 - 2013 Intel Corporation + +Compilation of "out/ssdt-pcihp.dsl.i" - Thu May 9 17:57:45 2013 + + 4: DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) + +00000000: 53 53 44 54 58 00 00 00 "SSDTX..." +00000008: 01 8B 42 58 50 43 00 00 "..BXPC.." +00000010: 42 58 53 53 44 54 50 43 "BXSSDTPC" +00000018: 01 00 00 00 49 4E 54 4C "....INTL" +00000020: 14 02 13 20 ............ "... " + + 5: { + 6: External(\_SB.PCI0, DeviceObj) + 7: External(\_SB.PCI0.PCEJ, MethodObj) + 8: Scope(\_SB.PCI0) { + +00000024: 10 33 5C 2E 5F 53 42 5F ".3\._SB_" +0000002C: 50 43 49 30 ............ "PCI0" + + 18: Device(SAA) { + +00000030: 5B 82 26 53 41 41 5F ... "[.&SAA_" + + 22: Name(_SUN, 0xAA) + +00000037: 08 5F 53 55 4E 0A AA ... "._SUN.." + + 26: Name(_ADR, 0xAA0000) + +0000003E: 08 5F 41 44 52 0C 00 00 "._ADR..." +00000046: AA 00 .................. ".." + + 30: Method(_EJ0, 1) { + +00000048: 14 0F 5F 45 4A 30 01 ... ".._EJ0." + + 31: Return (PCEJ(_SUN)) + +[****iasl****] +out/ssdt-pcihp.dsl.i 31: Return (PCEJ(_SUN)) +Warning 1104 - ^ Reserved method should not return a value (_EJ0) + + + +0000004F: A4 ..................... "." +00000050: 50 43 45 4A 5F 53 55 4E "PCEJ_SUN" + 32: } + 33: } + 34: } + 35: } + Summary of errors and warnings +out/ssdt-pcihp.dsl.i 31: Return (PCEJ(_SUN)) +Warning 1104 - ^ Reserved method should not return a value (_EJ0) -ASL Optimizing Compiler version 20100528 [Jul 2 2010] -ASL Input: out/ssdt-pcihp.dsl.i - 37 lines, 767 bytes, 6 keywords -AML Output: out/ssdt-pcihp.aml - 88 bytes, 5 named objects, 1 executable opcodes +ASL Optimizing Compiler version 20130214-32 [Apr 6 2013] -Compilation complete. 0 Errors, 0 Warnings, 0 Remarks, 0 Optimizations +ASL Input: out/ssdt-pcihp.dsl.i - 37 lines, 359 bytes, 6 keywords +AML Output: out/ssdt-pcihp.aml - 88 bytes, 5 named objects, 1 executable opcodes +Listing File: out/ssdt-pcihp.lst - 2016 bytes +Hex Dump: out/ssdt-pcihp.hex - 228 bytes +Compilation complete. 0 Errors, 1 Warnings, 0 Remarks, 0 Optimizations ---------------- As you can see, there's no code comments anymore, so nothing to output by acpi_extract.py I haven't digged further yet -- just a heads-up for now. Thanks, /mjt

1 0

[PATCH RFC 0/3] seabios: move acpi table formatting out of bios
by Michael S. Tsirkin 09 May '13

09 May '13

Untested yet, but I thought I'd share the BIOS bits so we can agree on direction. In particular check out ROM sizes: - Before patchset with DSDT enabled Total size: 127880 Fixed: 59060 Free: 3192 (used 97.6% of 128KiB rom) - Before patchset with DSDT disabled Total size: 122844 Fixed: 58884 Free: 8228 (used 93.7% of 128KiB rom) - After patchset: Total size: 128776 Fixed: 59100 Free: 2296 (used 98.2% of 128KiB rom) - Legacy disabled at build time: Total size: 119836 Fixed: 58996 Free: 11236 (used 91.4% of 128KiB rom) As can be seen from this, most size savings come from dropping DSDT, but we do save a bit by removing other tables. Of course the real reason to move tables to QEMU is so that ACPI can better match hardware. This patchset adds an option to move all code for formatting acpi tables out of BIOS. With this, QEMU has full control over the table layout. All tables are loaded from the new "/etc/acpi/" directory. Any entries in this directory cause BIOS to disable ACPI table generation completely. A generic linker script, controlled by QEMU, is loaded from "/etc/linker-script". It is used to patch in table pointers and checksums. BIOS still has limited ability to parse the tables, for the following purposes: - locate resume vector - allocate RSDP in FSEG - allocate FACS at an aligned address -- MST Michael S. Tsirkin (3): linker: utility to patch in-memory ROM files acpi: load and link tables from /etc/acpi/ acpi: add an option to disable builtin tables Makefile | 2 +- src/Kconfig | 12 +++++++- src/acpi.c | 67 +++++++++++++++++++++++++++++++++++++++++++- src/linker.c | 90 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ src/linker.h | 50 +++++++++++++++++++++++++++++++++ src/util.h | 1 + 6 files changed, 219 insertions(+), 3 deletions(-) create mode 100644 src/linker.c create mode 100644 src/linker.h -- MST

4 19

[PATCH 0/2] accept MADT over fw_cfg
by Laszlo Ersek 08 May '13

08 May '13

The first patch cleans up some field assignments in build_madt(). The second one handles the case when MADT is provided by qemu over fw_cfg. I'll soon post the qemu-side series as well. I cross-tested {patched, unpatched} qemu with {patched, unpatched} seabios. Inside a RHEL-6 guest I dumped the RSDT, DSDT, and APIC (MADT) in all four cases, and compared the results. Patched qemu with unpatched seabios don't play nicely together (two APIC (MADT) tables present). Otherwise everything seemed fine to me. The CPU topology was set with -smp 3,maxcpus=16,sockets=2,cores=4,threads=2 The table dumps / disassemblies are too big to include here, please find them at <http://people.redhat.com/~lersek/acpi_move/madt_tests.tar.gz>. Laszlo Ersek (2): build_madt(): "fix" intsrcovr->{gsi,flags} and local_nmi->flags byte order accept MADT over fw_cfg src/acpi.c | 31 +++++++++++++++++++++++-------- 1 files changed, 23 insertions(+), 8 deletions(-)

7 64

Re: [SeaBIOS] How to modify qemu-kvm bios.bin?
by Peter Stuge 07 May '13

07 May '13

li peter wrote: > maybe qemu-kvm use non-coreboot uses, Correct. qemu-kvm unfortunately uses a special-case build of SeaBIOS without coreboot. > How do I do to reach my goal？ I am not sure. If SeaBIOS also has bootsplash support then you should see options in the SeaBIOS build configuration. If there are no options to enter e.g. a splash filename then probably there is no support. SeaBIOS also runs only for a rather short time, so splash support is not very meaningful. I think it would barely be noticeable when the VM starts. //Peter

2 1

How to modify qemu-kvm bios.bin?
by li peter 06 May '13

06 May '13

I know qemu-kvm use seabios as pc bios.i just want let vm created by qemu-kvm show a custom JPEG image during bootup. How to modify the file bios.bin? I see the following command on coreboot website，but i can't find coreboot.rom file in /usr/share/qemu-kvm/ . maybe qemu-kvm use non-coreboot uses,How do I do to reach my goal？ ./build/cbfstool build/coreboot.rom add -f /path/to/image.jpg -n bootsplash.jpg -t raw

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS May 2013