June 2010 - SeaBIOS - mail.coreboot.org

Pending patch summary
by Kevin O'Connor June 21, 2010

June 21, 2010

There are a bunch of patches pending. I've put together a summary below. If your patch isn't on this list, please ping me. -Kevin Misc: Isaku Yamahata - pci: multi pci bus support - pci express, etc. Stefan Reinauer - Bootsplash for SeaBIOS - Add jpeg bootsplash support ACPI/SMBIOS: Alex Williamson - smbios: Allow all fields to be set via qemu_cfg_smbios_load_field() - allow all smbios fields to be overwritten by qemu. Arseny Klimovsky - Added ACPI SLIC table - Add acpi slic table generation to seabios - Update slic table oem ids in seabios Eduardo Habkost - Allow acpi and smbios default strings to be set on config.h - Allow each oem id in every smbios/acpi table to be set independently Isaku Yamahata - acpi: allow qemu to load dsdt as external acpi table. - Allow loading of an external dsdt table Gleb Natapov - load hpet info for HPET ACPI table from qemu - Fix hpet by pulling correct info from qemu_cfg Liu, Jinsong - KVM vcpu hotplug seabios infrastructure - replace dynamic ssdt with two builtin ssdts (16/255 cpus) Jes Sorensen - SMBIOS strings - Not really a patch, but Jes wants to rework smbios oem ids

2 1

Added ACPI SLIC table
by Arseny Klimovsky June 21, 2010

June 21, 2010

Hello, in some cases Microsoft OEM licence can be used in virtual machine legally. http://www.microsoft.com/windowsserver2008/en/us/licensing-datacenter.aspx For Windows 7 it is not clear, but possibly it is legal. To make in technically possible, one should use its ACPI SLIC table from original BIOS in virtual BIOS. I've created a patch for some versions (including latest stable), it can be easily cherry-picked in master. http://gitorious.org/seabios/seabios Of course, the patch does not include this SLIC table, only a code and place to put it. Coreboot project have such support (at least for some motherboards). -- Klimovsky Arseny

4 5

Re: [SeaBIOS] [PATCH 4/8] seabios: pciinit: make pci bar assigner preferchable memory aware.
by Kevin O'Connor June 21, 2010

June 21, 2010

On Thu, Jun 17, 2010 at 08:03:12PM +0900, Isaku Yamahata wrote: > Make pci bar assigner preferchable memory aware. > This is needed for PCI bridge support because memory space and > prefetchable memory space is filtered independently based on > memory base/limit and prefetchable memory base/limit of pci bridge. > On bus 0, such a distinction isn't necessary so keep existing behavior > by checking bus=0. [...] > pci_bios_io_addr = 0xc000; > pci_bios_mem_addr = BUILD_PCIMEM_START; > > + /* pci_bios_mem_addr + <some value> */ > + pci_bios_prefmem_addr = pci_bios_mem_addr + 0x08000000; How can we be sure that the "mem" space doesn't overlap with the "prefmem" space? -Kevin

2 2

[PATCH] seabios: acpi: allow qemu to load dsdt as external acpi table.
by Isaku Yamahata June 21, 2010

June 21, 2010

allow qemu to load dsdt as external acpi table. Signed-off-by: Isaku Yamahata <yamahata(a)valinux.co.jp> --- src/acpi.c | 27 +++++++++++++++++++++++++-- 1 files changed, 25 insertions(+), 2 deletions(-) diff --git a/src/acpi.c b/src/acpi.c index 0559443..254e43b 100644 --- a/src/acpi.c +++ b/src/acpi.c @@ -126,6 +126,11 @@ struct fadt_descriptor_rev1 } PACKED; /* + * Differentiated System Descrition Table (DSDT) + */ +#define DSDT_SIGNATURE 0x54445344 // DSDT + +/* * MADT values and structures */ @@ -280,6 +285,11 @@ static inline u32 cpu_to_le32(u32 x) return x; } +static inline u32 le32_to_cpu(u32 x) +{ + return x; +} + static void build_header(struct acpi_table_header *h, u32 sig, int len, u8 rev) { @@ -634,7 +644,8 @@ acpi_bios_init(void) } while(0) // Add tables - ACPI_INIT_TABLE(build_fadt(bdf)); + struct fadt_descriptor_rev1 *fadt = build_fadt(bdf); + ACPI_INIT_TABLE(fadt); ACPI_INIT_TABLE(build_ssdt()); ACPI_INIT_TABLE(build_madt()); ACPI_INIT_TABLE(build_hpet()); @@ -649,7 +660,19 @@ acpi_bios_init(void) warn_noalloc(); continue; } - ACPI_INIT_TABLE(qemu_cfg_next_acpi_table_load(addr, len)); + struct acpi_table_header *header = + qemu_cfg_next_acpi_table_load(addr, len); + if (header->signature == DSDT_SIGNATURE) { + if (fadt) { + free((void*)le32_to_cpu(fadt->dsdt)); + fadt->dsdt = cpu_to_le32((u32)addr); + fadt->checksum -= checksum(fadt, sizeof(*fadt)); + header->checksum -= checksum(header, len); + dprintf(1, "ACPI DSDT=%p\n", addr); + } + } else { + ACPI_INIT_TABLE(header); + } if (tbl_idx == MAX_ACPI_TABLES) { warn_noalloc(); break; -- 1.6.6.1

2 1

Re: [SeaBIOS] [PATCH 8/8] seabios: pciinit: initialize pci bridge filtering registers.
by Kevin O'Connor June 21, 2010

June 21, 2010

On Thu, Jun 17, 2010 at 08:03:16PM +0900, Isaku Yamahata wrote: > initialize pci bridge filtering registers. [...] > +static void pci_bios_init_device_bridge(u16 bdf) > +{ > + u32 io_old; > + u32 mem_old; > + u32 prefmem_old; It would be preferable if C99 initializations were used here - especially since there are already C99 declarations introduced later in this function. BTW, my other comments/questions aside, the patch series looks good to me. -Kevin

1 0

Re: [SeaBIOS] [PATCH 1/8] seabios: pci: introduce foreachpci_in_bus() helper macro.
by Kevin O'Connor June 21, 2010

June 21, 2010

On Thu, Jun 17, 2010 at 08:03:09PM +0900, Isaku Yamahata wrote: > This patch introduces foreachpci_in_bus() helper macro for > depth first recursion. foreachpci() is for width first recursion. > The macro will be used later to initialize pci bridge > that requires depth first recursion. Instead of introducing pci_next_in_bus(), can't we just use the existing pci_next() call? Something like: #define foreachpci_in_bus(BDF, MAX, BUS) \ for (MAX=(BUS + 0x0100), BDF = BUS \ ; BDF >= 0 && BDF < (BUS + 0x0100) \ ; BDF = pci_next(BDF+1, &MAX)) -Kevin

1 0

Re: [SeaBIOS] [PATCH] KVM vcpu hotplug seabios infrastructure
by Isaku Yamahata June 20, 2010

June 20, 2010

Hi Jison. Can you check the changeset of 12cbb43b9dc8a16f712188cb308a0eb0321f3e20 and update Makefile changes? The rule for acpi-ssdt-15.hex and acpi-ssdt-anycpu.hex seems same, so it can be combined into a single generic rule. The rules removes the files at the last, but it would be more reliable to use Makefile intermediate and let make to remove them automatically. On Sun, Jun 20, 2010 at 11:40:02PM +0800, Liu, Jinsong wrote: > Kevin O'Connor wrote: > > On Wed, Jun 16, 2010 at 12:19:24PM +0800, Liu, Jinsong wrote: > >> Gleb Natapov wrote: > >>> FWIW looks good to me. This patch removes dynamic creation of > >>> Processor scope and supports only 15/255 cpus configuration, but > >>> given that new code is much more complex that old one I don't see > >>> other possibilities (unless we'll add aml compiler to seabios > >>> runtime). > >>> > > > > > I'm okay with the approach (two SSDTs selected by cpu count). The > > build changes don't look right (15.c is too cryptic). However, more > > importantly, I want to see where we end up going with the qemu/seabios > > acpi interface discussion before committing this. > > > > -Kevin > > > Kevin, > > Thanks for comments! > > In your last comments, you prefer not to use iasl to compile seabios. > Now I update the patch as attached (only need update Makefile): > 1. cancel cryptic name like 15.c from Makefile, and we give detailed description for cpu number in new version; > 2. user don't need iasl again to compile seabios, except if they want to change dsdt/ssdt itself; > > The process to apply patch and build in my desktop is: > $ git apply kvm-vcpu-hotplug-seabios.patch > $ git add src/acpi-ssdt.dsl > $ git add src/mk_ssdt.c > $ git add src/acpi-ssdt-15.hex > $ git add src/acpi-ssdt-anycpu.hex > $ git commit -a > $ make > > We test the updated patch, it works OK with linux2.6.32 hot-add and hot-remove. > > Thanks, > Jinsong > _______________________________________________ > SeaBIOS mailing list > SeaBIOS(a)seabios.org > http://www.seabios.org/mailman/listinfo/seabios -- yamahata

2 2

Re: [SeaBIOS] [PATCH] KVM vcpu hotplug seabios infrastructure
by Peter Stuge June 20, 2010

June 20, 2010

Liu, Jinsong wrote: > In our patch, how about use iasl (ssdt build rules) in Makefile, > and I will build ssdt in my local server, and then send the compile > result to you which contain acpi-ssdt[] aml array I don't think that makes sense. The source code will still be under some version control so it seems much better to simply keep it in the same project as all the other source code that is related. //Peter

2 1

Re: [SeaBIOS] [PATCH] KVM vcpu hotplug seabios infrastructure
by Kevin O'Connor June 20, 2010

June 20, 2010

On Wed, Jun 16, 2010 at 12:19:24PM +0800, Liu, Jinsong wrote: > Gleb Natapov wrote: > > FWIW looks good to me. This patch removes dynamic creation of > > Processor scope and supports only 15/255 cpus configuration, but > > given that new code is much more complex that old one I don't see > > other possibilities (unless we'll add aml compiler to seabios > > runtime). > > > > Thanks Gleb for comments! > > Kevin, what's your comments? I'm okay with the approach (two SSDTs selected by cpu count). The build changes don't look right (15.c is too cryptic). However, more importantly, I want to see where we end up going with the qemu/seabios acpi interface discussion before committing this. -Kevin

2 1

[PATCH 0/8] seabios: pci: multi pci bus support
by Isaku Yamahata June 18, 2010

June 18, 2010

This patch set allows seabios to initialize multi pci bus and 64bit BAR. Currently seabios is able to initialize only pci root bus. However multi pci bus support is wanted because - more pci bus is wanted in qemu for many slots - pci express support is commin in qemu which requires multi pci bus. those patches on Qemu part are under way, though. Isaku Yamahata (8): seabios: pci: introduce foreachpci_in_bus() helper macro. seabios: pciinit: factor out pci bar region allocation logic. seabios: pciinit: make pci memory space assignment 64bit aware. seabios: pciinit: make pci bar assigner preferchable memory aware. seabios: pciinit: factor out bar offset calculation. seabios: pciinit: make bar offset calculation pci bridge aware. seabios: pciinit: pci bridge bus initialization. seabios: pciinit: initialize pci bridge filtering registers. src/pci.c | 30 ++++++ src/pci.h | 11 ++ src/pciinit.c | 310 ++++++++++++++++++++++++++++++++++++++++++++++++-------- 3 files changed, 306 insertions(+), 45 deletions(-)

3 14