SeaBIOS

seabios@seabios.org

3770 discussions

Re: [SeaBIOS] [Qemu-devel] [PATCH 1/2] pci: Automatically patch PCI device id in PCI ROM
by Anthony Liguori 21 Oct '10

21 Oct '10

On 10/15/2010 03:51 PM, Stefan Weil wrote: > PCI device with different device ids sometimes share > the same rom code. Only the device id and the checksum > differ in a boot rom for such devices. > BTW, SeaBIOS doesn't reject ROMs when they're loaded via rombar, only when they're loaded via romfile. Maybe it's better to use fw_cfg to explicitly tell SeaBIOS to ignore the PCI device id in the rom header for a certain device? Regards, Anthony Liguori > The i825xx ethernet controller family is a typical example > which is implemented in hw/eepro100.c. It uses at least > 3 different device ids, so normally 3 boot roms would be needed. > > By automatically patching the device id (and the checksum) > in qemu, all emulated family members can share the same > boot rom. > > Cc: Markus Armbruster<armbru(a)redhat.com> > Cc: Michael S. Tsirkin<mst(a)redhat.com> > Signed-off-by: Stefan Weil<weil(a)mail.berlios.de> > --- > hw/pci.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 53 insertions(+), 0 deletions(-) > > diff --git a/hw/pci.c b/hw/pci.c > index 1280d4d..c1f8218 100644 > --- a/hw/pci.c > +++ b/hw/pci.c > @@ -1797,6 +1797,57 @@ static void pci_map_option_rom(PCIDevice *pdev, int region_num, pcibus_t addr, p > cpu_register_physical_memory(addr, size, pdev->rom_offset); > } > > +/* Patch the PCI device id in a PCI rom image if necessary. > + This is needed for an option rom which is used for more than one device. */ > +static void pci_patch_device_id(PCIDevice *pdev, uint8_t *ptr, int size) > +{ > + uint16_t vendor_id; > + uint16_t device_id; > + uint16_t rom_vendor_id; > + uint16_t rom_device_id; > + uint16_t rom_magic; > + uint16_t pcir_offset; > + > + /* Words in rom data are little endian (like in PCI configuration), > + so they can be read / written with pci_get_word / pci_set_word. */ > + > + /* Only a valid rom will be patched. */ > + rom_magic = pci_get_word(ptr); > + if (rom_magic != 0xaa55) { > + PCI_DPRINTF("Bad ROM magic %04x\n", rom_magic); > + return; > + } > + pcir_offset = pci_get_word(ptr + 0x18); > + if (pcir_offset + 8>= size || memcmp(ptr + pcir_offset, "PCIR", 4)) { > + PCI_DPRINTF("Bad PCIR offset 0x%x or signature\n", pcir_offset); > + return; > + } > + > + vendor_id = pci_get_word(pdev->config + PCI_VENDOR_ID); > + device_id = pci_get_word(pdev->config + PCI_DEVICE_ID); > + rom_vendor_id = pci_get_word(ptr + pcir_offset + 4); > + rom_device_id = pci_get_word(ptr + pcir_offset + 6); > + > + /* Don't patch a rom with wrong vendor id (might be changed if needed). */ > + if (vendor_id != rom_vendor_id) { > + return; > + } > + > + PCI_DPRINTF("ROM id %04x%04x / PCI id %04x%04x\n", > + vendor_id, device_id, rom_vendor_id, rom_device_id); > + > + if (device_id != rom_device_id) { > + /* Patch device id and checksum (at offset 6 for etherboot roms). */ > + uint8_t checksum; > + checksum = ptr[6]; > + checksum += (uint8_t)rom_device_id + (uint8_t)(rom_device_id>> 8); > + checksum -= (uint8_t)device_id + (uint8_t)(device_id>> 8); > + PCI_DPRINTF("ROM checksum %02x / %02x\n", ptr[6], checksum); > + ptr[6] = checksum; > + pci_set_word(ptr + pcir_offset + 6, device_id); > + } > +} > + > /* Add an option rom for the device */ > static int pci_add_option_rom(PCIDevice *pdev) > { > @@ -1849,6 +1900,8 @@ static int pci_add_option_rom(PCIDevice *pdev) > load_image(path, ptr); > qemu_free(path); > > + pci_patch_device_id(pdev, ptr, size); > + > pci_register_bar(pdev, PCI_ROM_SLOT, size, > 0, pci_map_option_rom); > >

3 5

[PATCH 0/2] pciinit: fix overflow when bar allocation
by Isaku Yamahata 19 Oct '10

19 Oct '10

This patch set fixes PCI bar allocation when bar overflow occured. I checked if pmm_alloc facility can be used, but it doesn't suit for pci bar allocation. So I resulted in new API, pci_region which encapsulates region allocation and overflow checks. The first patch introduces pci_region, and the second patch fixes the overflow case with pci_region. Isaku Yamahata (2): pci: introduce pci_region to manage pci io/memory/prefmemory regions. pciinit: use pci_region functions. Makefile | 3 +- src/pci_region.c | 70 +++++++++++++++++++++++++++++++ src/pciinit.c | 122 ++++++++++++++++++++++++++--------------------------- src/util.h | 15 +++++++ 4 files changed, 147 insertions(+), 63 deletions(-) create mode 100644 src/pci_region.c

2 4

Re: [SeaBIOS] [PATCH 1/2] pci: introduce pci_region to manage pci io/memory/prefmemory regions.
by Michael S. Tsirkin 19 Oct '10

19 Oct '10

On Mon, Oct 18, 2010 at 06:34:22PM +0900, Isaku Yamahata wrote: > This patch adds helper functions to manage pci area. > > Signed-off-by: Isaku Yamahata <yamahata(a)valinux.co.jp> > --- > Makefile | 3 +- > src/pci_region.c | 70 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ > src/util.h | 15 +++++++++++ > 3 files changed, 87 insertions(+), 1 deletions(-) > create mode 100644 src/pci_region.c > > diff --git a/Makefile b/Makefile > index 9d412f1..1663a5d 100644 > --- a/Makefile > +++ b/Makefile > @@ -19,7 +19,8 @@ SRCBOTH=misc.c pmm.c stacks.c output.c util.c block.c floppy.c ata.c mouse.c \ > SRC16=$(SRCBOTH) system.c disk.c font.c > SRC32FLAT=$(SRCBOTH) post.c shadow.c memmap.c coreboot.c boot.c \ > acpi.c smm.c mptable.c smbios.c pciinit.c optionroms.c mtrr.c \ > - lzmadecode.c bootsplash.c jpeg.c usb-hub.c paravirt.c dev-i440fx.c > + lzmadecode.c bootsplash.c jpeg.c usb-hub.c paravirt.c dev-i440fx.c \ > + pci_region.c > SRC32SEG=util.c output.c pci.c pcibios.c apm.c stacks.c > > cc-option = $(shell if test -z "`$(1) $(2) -S -o /dev/null -xc \ > diff --git a/src/pci_region.c b/src/pci_region.c > new file mode 100644 > index 0000000..a4e71d9 > --- /dev/null > +++ b/src/pci_region.c > @@ -0,0 +1,70 @@ > +// helper functions to manage pci io/memory/prefetch memory region > +// > +// Copyright (C) 2009 Isaku Yamahata <yamahata at valinux co jp> > +// > +// This file may be distributed under the terms of the GNU LGPLv3 license. > +// > +// > + > +#include "util.h" > + > +#define PCI_REGION_DISABLED (-1) > + > +void pci_region_init(struct pci_region *r, u32 start, u32 end) > +{ > + r->start = start; > + r->end = end; > + > + r->cur_end = start; > +} > + > +static u32 pci_region_alloc_align(struct pci_region *r, u32 size, u32 align) > +{ > + if (r->cur_end == PCI_REGION_DISABLED) { > + return 0; > + } So is special value PCI_REGION_DISABLED or cur_end? > + > + u32 s = ALIGN(r->cur_end, align); > + if (s > r->end || s < r->cur_end) { > + return 0; > + } > + u32 e = s + size; > + if (e > r->end || e < s) { > + return 0; > + } > + r->cur_end = e; > + return s; > +} > + > +u32 pci_region_alloc(struct pci_region *r, u32 size) > +{ > + return pci_region_alloc_align(r, size, size); > +} > + > +u32 pci_region_align(struct pci_region *r, u32 align) > +{ > + return pci_region_alloc_align(r, 0, align); > +} > + > +void pci_region_revert(struct pci_region *r, u32 addr) > +{ > + r->cur_end = addr; > +} > + > +u32 pci_region_disable(struct pci_region *r) > +{ > + return r->cur_end = PCI_REGION_DISABLED; > +} > + > +u32 pci_region_addr(const struct pci_region *r) > +{ > + if (r->cur_end == PCI_REGION_DISABLED){ > + return r->end; > + } > + return r->cur_end; > +} > + > +u32 pci_region_size(const struct pci_region *r) > +{ > + return r->end - r->start; > +} > diff --git a/src/util.h b/src/util.h > index 5cc9f17..ecd1c16 100644 > --- a/src/util.h > +++ b/src/util.h > @@ -344,6 +344,21 @@ void qemu_prep_reset(void); > void smm_save_and_copy(void); > void smm_relocate_and_restore(void); > > +// pci_region.c > +struct pci_region { > + u32 start; > + u32 end; > + > + u32 cur_end; > +}; > +void pci_region_init(struct pci_region *r, u32 start, u32 end); > +u32 pci_region_alloc(struct pci_region *r, u32 size); > +u32 pci_region_align(struct pci_region *r, u32 align); > +void pci_region_revert(struct pci_region *r, u32 addr); > +u32 pci_region_disable(struct pci_region *r); > +u32 pci_region_addr(const struct pci_region *r); > +u32 pci_region_size(const struct pci_region *r); > + Please document structure fields here and functions in the .c file. > // pciinit.c > extern const u8 pci_irqs[4]; > void pci_bios_allocate_regions(u16 bdf, void *arg); > -- > 1.7.1.1

2 1

Re: [SeaBIOS] system_powerdown not working for qemu-kvm 0.12.4?
by Avi Kivity 15 Oct '10

15 Oct '10

On 10/11/2010 07:53 PM, Ruben Kerkhof wrote: > On Mon, Oct 11, 2010 at 18:57, Avi Kivity<avi(a)redhat.com> wrote: > > On 10/11/2010 06:29 PM, Ruben Kerkhof wrote: > >> > >> Hi Avi, > >> > >> On Sun, Aug 15, 2010 at 13:00, Avi Kivity<avi(a)redhat.com> wrote: > >> > >> > I'm betting 73b48d914f9 is the cause, but let's see the full bisect. > >> > >> system_powerdown with FreeBSD 8.1 works on 73b48d914f9. > >> > >> I've bisected the issue to commit 76e617c. > >> > >> Let me know if I should do some other tests. > > > > It works on 73b48d914f9 but fails on 76e617c due to the seabios change. > > > > Can you try 76e617c but with SeaBIOS 0.5.0? If that works, keep qemu-kvm on > > 76e617c but bisect SeaBIOS from 0.5.0 to 0.5.1 to see which commit broke > > freebsd. > > > > Sure, > > 5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 is the first bad commit > commit 5c99b6c984682ddb1d4543a7e27a1f4ca633e6a6 > Author: Kevin O'Connor<kevin(a)koconnor.net> > Date: Wed Dec 30 12:36:22 2009 -0500 > > Commit compiled dsdt file; misc comment updates. > > Commit new dsdt with recent changes. > Add some misc comments. > Also, fix uninitialized warning in mptable code. > > Regards, Ruben Gleb, Kevin, any ideas? (summary: qemu-kvm doesn't acpi shutdown freebsd 8.1 with this commit; qemu.git does. May be due to interrupt polarity which kvm implements but qemu does not) -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.

5 11

[RFC] Passing boot order from qemu to seabios
by Gleb Natapov 13 Oct '10

13 Oct '10

Currently if VM is started with multiple disks it is almost impossible to guess which one of them will be used as boot device especially if there is a mix of ATA/virtio/SCSI devices. Essentially BIOS decides the order and without looking into the code you can't tell what the order will be (and in qemu-kvm if boot=on is used it brings even more havoc). We should allow fine-grained control of boot order from qemu command line, or as a minimum control what device will be used for booting. To do that along with inventing syntax to specify boot order on qemu command line we need to communicate boot order to seabios via fw_cfg interface. For that we need to have a way to unambiguously specify a disk from qemu to seabios. PCI bus address is not enough since not all devices are PCI (do we care about them?) and since one PCI device may control more then one disk (ATA slave/master, SCSI LUNs). We can do what EDD specification does. Describe disk as: bus type (isa/pci), address on a bus (16 bit base address for isa, b/s/f for pci) device type (ATA/SCSI/VIRTIO) device path (slave/master for ATA, LUN for SCSI, nothing for virtio) Will it cover all use cased? Any other ideas? Any ideas about qemu command line syntax? May be somebody whats to implement it? :) -- Gleb.

11 54

Re: [SeaBIOS] [PATCH] Allow rom to grow to 256K.
by H. Peter Anvin 27 Sep '10

27 Sep '10

On 09/25/2010 09:55 AM, Kevin O'Connor wrote: > Enable rom size to be over 128K. > > Apparently, only the first 128K of the rom are mirrored to the > 0xc0000-0xfffff area under qemu, so make sure to copy the rom (via > make_bios_writable) in the non-init part of the startup code. This > ensure the copy code is itself available. This is pretty typical... the BIOS is supposed to fit within the 0xe0000..0xfffff area during actual execution. -hpa

1 0

[PATCH] Cleanup - it's no longer necessary to manually reset global variables.
by Kevin O'Connor 25 Sep '10

25 Sep '10

Now that a soft-reboot forces a hard-reboot, it is no longer necessary to manually reset global variables. --- src/block.c | 11 ----------- src/cdrom.c | 2 -- src/clock.c | 1 - src/coreboot.c | 6 ------ src/disk.h | 1 - src/memmap.c | 7 ------- src/memmap.h | 1 - src/optionroms.c | 5 +---- src/pmm.c | 6 ------ src/post.c | 3 --- src/smp.c | 8 -------- src/stacks.c | 2 -- src/usb-hid.c | 9 --------- src/usb-hid.h | 1 - src/usb.c | 2 -- src/util.h | 1 - src/vgahooks.c | 5 ++--- 17 files changed, 3 insertions(+), 68 deletions(-) diff --git a/src/block.c b/src/block.c index b6b1902..3f4b13f 100644 --- a/src/block.c +++ b/src/block.c @@ -329,14 +329,3 @@ send_disk_op(struct disk_op_s *op) return stack_hop((u32)op, GET_SEG(SS), __send_disk_op); } - - -/**************************************************************** - * Setup - ****************************************************************/ - -void -drive_setup(void) -{ - memset(&Drives, 0, sizeof(Drives)); -} diff --git a/src/cdrom.c b/src/cdrom.c index 655ee00..349e401 100644 --- a/src/cdrom.c +++ b/src/cdrom.c @@ -109,8 +109,6 @@ cdemu_setup(void) { if (!CONFIG_CDROM_EMU) return; - cdemu_drive_gf = NULL; - cdemu_buf_fl = NULL; if (!Drives.cdcount) return; diff --git a/src/clock.c b/src/clock.c index f6a43e9..fcdc698 100644 --- a/src/clock.c +++ b/src/clock.c @@ -211,7 +211,6 @@ timer_setup(void) u32 ticks = (hours * 60 + minutes) * 60 + seconds; ticks = ((u64)ticks * PIT_TICK_RATE) / PIT_TICK_INTERVAL; SET_BDA(timer_counter, ticks); - SET_BDA(timer_rollover, 0); enable_hwirq(0, FUNC16(entry_08)); enable_hwirq(8, FUNC16(entry_70)); diff --git a/src/coreboot.c b/src/coreboot.c index db0063b..50d300c 100644 --- a/src/coreboot.c +++ b/src/coreboot.c @@ -126,8 +126,6 @@ coreboot_fill_map(void) { dprintf(3, "Attempting to find coreboot table\n"); - CBMemTable = NULL; - // Find coreboot table. struct cb_header *cbh = find_cb_header(0, 0x1000); if (!cbh) @@ -290,10 +288,6 @@ coreboot_copy_biostable(void) dprintf(3, "Relocating coreboot bios tables\n"); - // Init variables set in coreboot table memory scan. - PirOffset = 0; - RsdpAddr = 0; - // Scan CB_MEM_TABLE areas for bios tables. int i, count = MEM_RANGE_COUNT(cbm); for (i=0; i<count; i++) { diff --git a/src/disk.h b/src/disk.h index 9e5b083..55a5da3 100644 --- a/src/disk.h +++ b/src/disk.h @@ -234,7 +234,6 @@ void map_hd_drive(struct drive_s *drive_g); void map_cd_drive(struct drive_s *drive_g); int process_op(struct disk_op_s *op); int send_disk_op(struct disk_op_s *op); -void drive_setup(void); // floppy.c extern struct floppy_ext_dbt_s diskette_param_table2; diff --git a/src/memmap.c b/src/memmap.c index 84bc4fa..ea05953 100644 --- a/src/memmap.c +++ b/src/memmap.c @@ -119,13 +119,6 @@ add_e820(u64 start, u64 size, u32 type) //dump_map(); } -// Prep for memmap stuff - init bios table locations. -void -memmap_setup(void) -{ - e820_count = 0; -} - // Report on final memory locations. void memmap_finalize(void) diff --git a/src/memmap.h b/src/memmap.h index d4154c1..01c7ddb 100644 --- a/src/memmap.h +++ b/src/memmap.h @@ -17,7 +17,6 @@ struct e820entry { }; void add_e820(u64 start, u64 size, u32 type); -void memmap_setup(void); void memmap_finalize(void); // A typical OS page size diff --git a/src/optionroms.c b/src/optionroms.c index ceb4060..854c33f 100644 --- a/src/optionroms.c +++ b/src/optionroms.c @@ -72,7 +72,7 @@ struct pnp_data { #define PCIROM_CODETYPE_X86 0 // The end of the last deployed rom. -u32 RomEnd; +u32 RomEnd = BUILD_ROM_START; /**************************************************************** @@ -423,9 +423,6 @@ optionrom_setup(void) void vga_setup(void) { - VGAbdf = -1; - RomEnd = BUILD_ROM_START; - if (! CONFIG_OPTIONROMS) return; diff --git a/src/pmm.c b/src/pmm.c index bb90ff0..e0770ac 100644 --- a/src/pmm.c +++ b/src/pmm.c @@ -193,12 +193,6 @@ malloc_setup(void) ASSERT32FLAT(); dprintf(3, "malloc setup\n"); - ZoneLow.info = ZoneHigh.info = ZoneFSeg.info = NULL; - ZoneTmpLow.info = ZoneTmpHigh.info = NULL; - - // Clear memory in 0xf0000 area. - memset(BiosTableSpace, 0, CONFIG_MAX_BIOSTABLE); - // Populate temp high ram u32 highram = 0; int i; diff --git a/src/post.c b/src/post.c index a72b8cd..4e42731 100644 --- a/src/post.c +++ b/src/post.c @@ -211,7 +211,6 @@ maininit(void) mathcp_setup(); // Initialize mtrr - smp_probe_setup(); mtrr_setup(); // Initialize pci @@ -220,7 +219,6 @@ maininit(void) // Initialize internal tables boot_setup(); - drive_setup(); // Start hardware initialization (if optionrom threading) if (CONFIG_THREADS && CONFIG_THREAD_OPTIONROMS) @@ -327,7 +325,6 @@ void VISIBLE32INIT post(void) { // Detect ram and setup internal malloc. - memmap_setup(); qemu_cfg_port_probe(); ram_probe(); malloc_setup(); diff --git a/src/smp.c b/src/smp.c index 38e117e..40f5451 100644 --- a/src/smp.c +++ b/src/smp.c @@ -127,11 +127,3 @@ smp_probe(void) dprintf(1, "Found %d cpu(s) max supported %d cpu(s)\n", readl(&CountCPUs), MaxCountCPUs); } - -// Reset variables to zero -void -smp_probe_setup(void) -{ - CountCPUs = 0; - smp_mtrr_count = 0; -} diff --git a/src/stacks.c b/src/stacks.c index dade3af..d541a4f 100644 --- a/src/stacks.c +++ b/src/stacks.c @@ -195,8 +195,6 @@ thread_setup(void) { MainThread.next = &MainThread; MainThread.pprev = &MainThread.next; - MainThread.stackpos = NULL; - CanPreempt = 0; } // Return the 'struct thread_info' for the currently running thread. diff --git a/src/usb-hid.c b/src/usb-hid.c index b88d684..168b7fa 100644 --- a/src/usb-hid.c +++ b/src/usb-hid.c @@ -130,15 +130,6 @@ usb_hid_init(struct usb_pipe *pipe return -1; } -void -usb_hid_setup(void) -{ - if (CONFIG_USB_KEYBOARD) - keyboard_pipe = NULL; - if (CONFIG_USB_MOUSE) - mouse_pipe = NULL; -} - /**************************************************************** * Keyboard events diff --git a/src/usb-hid.h b/src/usb-hid.h index d8199b9..7fbcf4b 100644 --- a/src/usb-hid.h +++ b/src/usb-hid.h @@ -6,7 +6,6 @@ struct usb_interface_descriptor; struct usb_pipe; int usb_hid_init(struct usb_pipe *pipe , struct usb_interface_descriptor *iface, int imax); -void usb_hid_setup(void); inline int usb_kbd_active(void); inline int usb_kbd_command(int command, u8 *param); inline int usb_mouse_active(void); diff --git a/src/usb.c b/src/usb.c index cefc5f3..aa8d72c 100644 --- a/src/usb.c +++ b/src/usb.c @@ -423,8 +423,6 @@ usb_setup(void) dprintf(3, "init usb\n"); - usb_hid_setup(); - // Look for USB controllers int ehcibdf = -1; int count = 0; diff --git a/src/util.h b/src/util.h index c27037a..1823fd4 100644 --- a/src/util.h +++ b/src/util.h @@ -358,7 +358,6 @@ extern u32 CountCPUs; extern u32 MaxCountCPUs; void wrmsr_smp(u32 index, u64 val); void smp_probe(void); -void smp_probe_setup(void); // coreboot.c struct cbfs_file; diff --git a/src/vgahooks.c b/src/vgahooks.c index 14678be..eb4dfa8 100644 --- a/src/vgahooks.c +++ b/src/vgahooks.c @@ -13,7 +13,7 @@ #include "config.h" // CONFIG_* // The Bus/Dev/Fn of the primary VGA device. -int VGAbdf VAR16VISIBLE; +int VGAbdf VAR16VISIBLE = -1; // Coreboot board detected. int CBmainboard VAR16VISIBLE; @@ -305,11 +305,10 @@ vgahook_setup(const char *vendor, const char *part) if (! CONFIG_VGAHOOKS) return; - CBmainboard = 0; for (i=0; i<(sizeof(mainboard_list) / sizeof(mainboard_list[0])); i++) { if (!strcmp(vendor, mainboard_list[i].vendor) && !strcmp(part, mainboard_list[i].device)) { - printf("Found mainboard %s %s\n", vendor, part); + printf("Found mainboard %s %s\n", vendor, part); CBmainboard = mainboard_list[i].type; break; } -- 1.7.2.3

1 0

[PATCH] Allow rom to grow to 256K.
by Kevin O'Connor 25 Sep '10

25 Sep '10

Enable rom size to be over 128K. Apparently, only the first 128K of the rom are mirrored to the 0xc0000-0xfffff area under qemu, so make sure to copy the rom (via make_bios_writable) in the non-init part of the startup code. This ensure the copy code is itself available. --- src/post.c | 13 ++++++------- tools/checkrom.py | 2 ++ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/post.c b/src/post.c index 4a75f51..a72b8cd 100644 --- a/src/post.c +++ b/src/post.c @@ -321,18 +321,11 @@ reloc_init(void) func(); } -static int HaveRunPost; - // Start of Power On Self Test (POST) - the BIOS initilization phase. // This function sets up for and attempts relocation of the init code. void VISIBLE32INIT post(void) { - // Allow writes to modify bios area (0xf0000) - make_bios_writable(); - - HaveRunPost = 1; - // Detect ram and setup internal malloc. memmap_setup(); qemu_cfg_port_probe(); @@ -347,6 +340,8 @@ post(void) * POST entry point ****************************************************************/ +static int HaveRunPost; + // Attempt to invoke a hard-reboot. static void tryReboot(void) @@ -385,6 +380,10 @@ _start(void) // This is a soft reboot - invoke a hard reboot. tryReboot(); + // Allow writes to modify bios area (0xf0000) + make_bios_writable(); + HaveRunPost = 1; + // Perform main setup code. post(); } diff --git a/tools/checkrom.py b/tools/checkrom.py index 42cf2df..6309528 100755 --- a/tools/checkrom.py +++ b/tools/checkrom.py @@ -24,6 +24,8 @@ def main(): finalsize = 64*1024 if datasize > 64*1024: finalsize = 128*1024 + if datasize > 128*1024: + finalsize = 256*1024 # Sanity checks start = symbols['code32flat_start'].offset -- 1.7.2.3

1 0

[PATCH] Fix typo preventing relocated space from being used for option roms.
by Kevin O'Connor 25 Sep '10

25 Sep '10

--- src/optionroms.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/optionroms.c b/src/optionroms.c index f26a3aa..ceb4060 100644 --- a/src/optionroms.c +++ b/src/optionroms.c @@ -171,7 +171,7 @@ get_pci_rom(struct rom_header *rom) // Return start of code in 0xc0000-0xf0000 space. static inline u32 _max_rom(void) { extern u8 code32flat_start[], code32init_end[]; - return CONFIG_RELOCATE_INIT ? (u32)code32flat_start : (u32)code32init_end; + return CONFIG_RELOCATE_INIT ? (u32)code32init_end : (u32)code32flat_start; } // Return the memory position up to which roms may be located. static inline u32 max_rom(void) { -- 1.7.2.3

1 0

Pending patch summary
by Kevin O'Connor 25 Sep '10

25 Sep '10

Patches pending: Eduard - Gabriel Munteanu: 20100828: iommu: introduce AMD IOMMU support, initialize it * Likely for post v0.6.1 release Isaku Yamahata: 20100722: acpi: allow qemu to load dsdt as external acpi table. * Needs resolution on acpi patches Isaku Yamahata: 20100819: seabios: acpi: add mcfg table. * Needs resolution on acpi patches Patches pending with author: Isaku Yamahata: 20100726: seabios: pciinit: fix overflow when bar allocation. * Isaku was looking at another approach Sebastian Herbszt: 20100731: EDD fixes * Initial patch looked like it may break backwards compatibility; I haven't seen a new patch. Patches not applied: Arseny Klimovsky: 20100621: Added ACPI SLIC table * There was no explicit patch sent, and approach (eg, recompile seabios) doesn't look scalable. Eduardo Habkost: 20100618: Allow acpi and smbios default strings to be set on config.h * Should no longer be needed now that smbios strings can be passed in via fwcfg. Christopher Gautier: 20100723: Impossible to distinguish between two '/' keys * No explicit patch sent, and I didn't understand the description. Gleb Natapov: 20100614: load hpet info for HPET ACPI table from qemu * Gleb withdrew this patch due to a bug. Concept is pending ACPI resolution. Please let me know if I've missed any thing. -Kevin

2 2

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS