SeaBIOS

seabios@seabios.org

2 participants
3799 discussions

Re: [SeaBIOS] [PATCH 3/9] Don't access drive_g->desc from boot_cdrom().
by Kevin O'Connor 29 Dec '10

29 Dec '10

On Wed, Dec 29, 2010 at 12:47:14PM -0500, Kevin O'Connor wrote: > The drive description is allocated with malloc_tmp() and is thus only > available during the POST phase - boot_cdrom() is called during the > boot phase. Odd - two patches got merged into one here. -Kevin

1 0

[PATCH] fix inline build warning
by Lin Ming 29 Dec '10

29 Dec '10

# make Working around non-functional -combine Compiling whole program out/ccode.16.s In file included from out/ccode.16.s.tmp.c:5: out/../src/util.h:204: warning: ‘memcpy_fl’ declared inline after being called out/../src/util.h:204: warning: previous declaration of ‘memcpy_fl’ was here Compiling (16bit) out/code16.o Compiling whole program out/ccode32flat.o In file included from out/ccode32flat.o.tmp.c:5: out/../src/util.h:204: warning: ‘memcpy_fl’ declared inline after being called out/../src/util.h:204: warning: previous declaration of ‘memcpy_fl’ was here Compiling whole program out/code32seg.o This patch fix above warnings. Signed-off-by: Lin Ming <ming.m.lin(a)intel.com> --- src/util.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/util.h b/src/util.h index f5b9446..cadebdb 100644 --- a/src/util.h +++ b/src/util.h @@ -201,7 +201,7 @@ void *memset(void *s, int c, size_t n); void memset_fl(void *ptr, u8 val, size_t size); inline void memcpy_far(u16 d_seg, void *d_far , u16 s_seg, const void *s_far, size_t len); -void memcpy_fl(void *d_fl, const void *s_fl, size_t len); +inline void memcpy_fl(void *d_fl, const void *s_fl, size_t len); void *memcpy(void *d1, const void *s1, size_t len); #if MODESEGMENT == 0 #define memcpy __builtin_memcpy

2 2

[PATCH 1/2] Record boot priority for each BAID/BCV/BEV.
by Gleb Natapov 28 Dec '10

28 Dec '10

For each BAID/BCV/BEV lookup its boot priority in boot order file. If there is no entry for a device there assign it the lowest boot priority. Signed-off-by: Gleb Natapov <gleb(a)redhat.com> --- src/ahci.c | 4 ++-- src/ata.c | 13 ++++++++++--- src/boot.c | 47 +++++++++++++++++++++++++++++++++++------------ src/boot.h | 16 ++++++++++------ src/coreboot.c | 2 +- src/floppy.c | 8 +++++++- src/optionroms.c | 18 +++++++++++++++--- src/usb-msc.c | 4 ++-- src/virtio-blk.c | 2 +- 9 files changed, 83 insertions(+), 31 deletions(-) diff --git a/src/ahci.c b/src/ahci.c index 03e39ad..a7f458f 100644 --- a/src/ahci.c +++ b/src/ahci.c @@ -382,7 +382,7 @@ ahci_port_init(struct ahci_ctrl_s *ctrl, u32 pnr) // Setup disk geometry translation. setup_translation(&port->drive); // Register with bcv system. - boot_add_hd(&port->drive); + boot_add_hd(&port->drive, -1); } else { // found cdrom (atapi) port->drive.blksize = CDROM_SECTOR_SIZE; @@ -397,7 +397,7 @@ ahci_port_init(struct ahci_ctrl_s *ctrl, u32 pnr) // fill cdidmap if (iscd) - boot_add_cd(&port->drive); + boot_add_cd(&port->drive, -1); } dprintf(1, "%s\n", port->drive.desc); diff --git a/src/ata.c b/src/ata.c index e01f842..872c5e3 100644 --- a/src/ata.c +++ b/src/ata.c @@ -783,8 +783,12 @@ init_drive_atapi(struct atadrive_s *dummy, u16 *buffer) dprintf(1, "%s\n", adrive_g->drive.desc); // fill cdidmap - if (iscd) - boot_add_cd(&adrive_g->drive); + if (iscd) { + int prio = bootprio_find_ata_device(adrive_g->chan_gf->pci_bdf, + adrive_g->chan_gf->chanid, + adrive_g->slave); + boot_add_cd(&adrive_g->drive, prio); + } return adrive_g; } @@ -833,8 +837,11 @@ init_drive_ata(struct atadrive_s *dummy, u16 *buffer) // Setup disk geometry translation. setup_translation(&adrive_g->drive); + int prio = bootprio_find_ata_device(adrive_g->chan_gf->pci_bdf, + adrive_g->chan_gf->chanid, + adrive_g->slave); // Register with bcv system. - boot_add_hd(&adrive_g->drive); + boot_add_hd(&adrive_g->drive, prio); return adrive_g; } diff --git a/src/boot.c b/src/boot.c index 73ac5c8..e43d1a5 100644 --- a/src/boot.c +++ b/src/boot.c @@ -152,51 +152,54 @@ bootentry_add(int type, int prio, u32 data, const char *desc) *pprev = be; } +#define P(P, DP) (((P) == -1) ? (DP) : (P)) + // Add a BEV vector for a given pnp compatible option rom. void -boot_add_bev(u16 seg, u16 bev, u16 desc) +boot_add_bev(u16 seg, u16 bev, u16 desc, int prio) { - bootentry_add(IPL_TYPE_BEV, DefaultBEVPrio, SEGOFF(seg, bev).segoff + bootentry_add(IPL_TYPE_BEV, P(prio, DefaultBEVPrio), SEGOFF(seg, bev).segoff , desc ? MAKE_FLATPTR(seg, desc) : "Unknown"); DefaultBEVPrio = DEFAULT_PRIO; } // Add a bcv entry for an expansion card harddrive or legacy option rom void -boot_add_bcv(u16 seg, u16 ip, u16 desc) +boot_add_bcv(u16 seg, u16 ip, u16 desc, int prio) { - bootentry_add(IPL_TYPE_BCV, DEFAULT_PRIO, SEGOFF(seg, ip).segoff + bootentry_add(IPL_TYPE_BCV, P(prio, DEFAULT_PRIO), SEGOFF(seg, ip).segoff , desc ? MAKE_FLATPTR(seg, desc) : "Legacy option rom"); } void -boot_add_floppy(struct drive_s *drive_g) +boot_add_floppy(struct drive_s *drive_g, int prio) { - bootentry_add(IPL_TYPE_FLOPPY, DefaultFloppyPrio, (u32)drive_g + bootentry_add(IPL_TYPE_FLOPPY, P(prio, DefaultFloppyPrio), (u32)drive_g , drive_g->desc); } void -boot_add_hd(struct drive_s *drive_g) +boot_add_hd(struct drive_s *drive_g, int prio) { - bootentry_add(IPL_TYPE_HARDDISK, DefaultHDPrio, (u32)drive_g + bootentry_add(IPL_TYPE_HARDDISK, P(prio, DefaultHDPrio), (u32)drive_g , drive_g->desc); } void -boot_add_cd(struct drive_s *drive_g) +boot_add_cd(struct drive_s *drive_g, int prio) { - bootentry_add(IPL_TYPE_CDROM, DefaultCDPrio, (u32)drive_g + bootentry_add(IPL_TYPE_CDROM, P(prio, DefaultCDPrio), (u32)drive_g , drive_g->desc); } // Add a CBFS payload entry void -boot_add_cbfs(void *data, const char *desc) +boot_add_cbfs(void *data, const char *desc, int prio) { - bootentry_add(IPL_TYPE_CBFS, DEFAULT_PRIO, (u32)data, desc); + bootentry_add(IPL_TYPE_CBFS, P(prio, DEFAULT_PRIO), (u32)data, desc); } +#undef P /**************************************************************** * Boot menu and BCV execution @@ -489,3 +492,23 @@ handle_19(void) SET_EBDA(boot_sequence, 0); do_boot(0); } + +int bootprio_find_pci_device(int bdf) +{ + return -1; +} + +int bootprio_find_ata_device(int bdf, int chanid, int slave) +{ + return -1; +} + +int bootprio_find_fdc_device(int bfd, int port, int fdid) +{ + return -1; +} + +int bootprio_find_named_rom(const char *name) +{ + return -1; +} diff --git a/src/boot.h b/src/boot.h index f1a428e..dd3cc49 100644 --- a/src/boot.h +++ b/src/boot.h @@ -35,14 +35,18 @@ struct ipl_s { // boot.c extern struct ipl_s IPL; void boot_setup(void); -void boot_add_bev(u16 seg, u16 bev, u16 desc); -void boot_add_bcv(u16 seg, u16 ip, u16 desc); +void boot_add_bev(u16 seg, u16 bev, u16 desc, int prio); +void boot_add_bcv(u16 seg, u16 ip, u16 desc, int prio); struct drive_s; -void boot_add_floppy(struct drive_s *drive_g); -void boot_add_hd(struct drive_s *drive_g); -void boot_add_cd(struct drive_s *drive_g); -void boot_add_cbfs(void *data, const char *desc); +void boot_add_floppy(struct drive_s *drive_g, int prio); +void boot_add_hd(struct drive_s *drive_g, int prio); +void boot_add_cd(struct drive_s *drive_g, int prio); +void boot_add_cbfs(void *data, const char *desc, int prio); void boot_prep(void); +int bootprio_find_pci_device(int bdf); +int bootprio_find_ata_device(int bdf, int chanid, int slave); +int bootprio_find_fdc_device(int bfd, int port, int fdid); +int bootprio_find_named_rom(const char *name); #endif // __BOOT_H diff --git a/src/coreboot.c b/src/coreboot.c index dab0a54..503a6d2 100644 --- a/src/coreboot.c +++ b/src/coreboot.c @@ -608,7 +608,7 @@ register_cbfs_payload(void) if (!desc) break; snprintf(desc, MAXDESCSIZE, "Payload [%s]", &filename[4]); - boot_add_cbfs(file, desc); + boot_add_cbfs(file, desc, -1); } } diff --git a/src/floppy.c b/src/floppy.c index 8986e39..c59b2fe 100644 --- a/src/floppy.c +++ b/src/floppy.c @@ -14,6 +14,9 @@ #include "pic.h" // eoi_pic1 #include "bregs.h" // struct bregs #include "boot.h" // boot_add_floppy +#include "pci.h" // pci_to_bdf +#include "pci_ids.h" // PCI_CLASS_BRIDGE_ISA + #define FLOPPY_SIZE_CODE 0x02 // 512 byte sectors #define FLOPPY_DATALEN 0xff // Not used - because size code is 0x02 @@ -117,7 +120,10 @@ addFloppy(int floppyid, int ftype, int driver) memcpy(&drive_g->lchs, &FloppyInfo[ftype].chs , sizeof(FloppyInfo[ftype].chs)); - boot_add_floppy(drive_g); + int bdf = pci_find_class(PCI_CLASS_BRIDGE_ISA); /* isa-to-pci bridge */ + int prio = bootprio_find_fdc_device(bdf, 0x3f0, floppyid); + + boot_add_floppy(drive_g, prio); return drive_g; } diff --git a/src/optionroms.c b/src/optionroms.c index 2ac4325..87e645f 100644 --- a/src/optionroms.c +++ b/src/optionroms.c @@ -220,6 +220,16 @@ setRomSource(u64 *sources, struct rom_header *rom, u64 source) sources[((u32)rom - BUILD_ROM_START) / OPTION_ROM_ALIGN] = source; } +static int +getRomPriority(u64 *sources, struct rom_header *rom) +{ + u64 source = sources[((u32)rom - BUILD_ROM_START) / OPTION_ROM_ALIGN]; + if (!source) + return -1; + if (source & RS_PCIROM) + return bootprio_find_pci_device(source); + return bootprio_find_named_rom(romfile_name(source)); +} /**************************************************************** * Roms in CBFS @@ -408,21 +418,23 @@ optionrom_setup(void) pos += OPTION_ROM_ALIGN; continue; } + int prio = getRomPriority(sources, rom); pos += ALIGN(rom->size * 512, OPTION_ROM_ALIGN); struct pnp_data *pnp = get_pnp_rom(rom); if (! pnp) { // Legacy rom. - boot_add_bcv(FLATPTR_TO_SEG(rom), OPTION_ROM_INITVECTOR, 0); + boot_add_bcv(FLATPTR_TO_SEG(rom), OPTION_ROM_INITVECTOR, 0, prio); continue; } // PnP rom. if (pnp->bev) // Can boot system - add to IPL list. - boot_add_bev(FLATPTR_TO_SEG(rom), pnp->bev, pnp->productname); + boot_add_bev(FLATPTR_TO_SEG(rom), pnp->bev, pnp->productname, prio); else // Check for BCV (there may be multiple). while (pnp && pnp->bcv) { - boot_add_bcv(FLATPTR_TO_SEG(rom), pnp->bcv, pnp->productname); + boot_add_bcv(FLATPTR_TO_SEG(rom), pnp->bcv, pnp->productname, + prio); pnp = get_pnp_next(rom, pnp); } } diff --git a/src/usb-msc.c b/src/usb-msc.c index 378143e..2a6c31a 100644 --- a/src/usb-msc.c +++ b/src/usb-msc.c @@ -143,7 +143,7 @@ setup_drive_cdrom(struct disk_op_s *op) { op->drive_g->blksize = CDROM_SECTOR_SIZE; op->drive_g->sectors = (u64)-1; - boot_add_cd(op->drive_g); + boot_add_cd(op->drive_g, -1); return 0; } @@ -171,7 +171,7 @@ setup_drive_hd(struct disk_op_s *op) setup_translation(op->drive_g); // Register with bcv system. - boot_add_hd(op->drive_g); + boot_add_hd(op->drive_g, -1); return 0; } diff --git a/src/virtio-blk.c b/src/virtio-blk.c index def8313..834523a 100644 --- a/src/virtio-blk.c +++ b/src/virtio-blk.c @@ -156,7 +156,7 @@ init_virtio_blk(u16 bdf) vdrive_g->drive.desc = desc; setup_translation(&vdrive_g->drive); - boot_add_hd(&vdrive_g->drive); + boot_add_hd(&vdrive_g->drive, bootprio_find_pci_device(bdf)); vp_set_status(ioaddr, VIRTIO_CONFIG_S_ACKNOWLEDGE | VIRTIO_CONFIG_S_DRIVER | VIRTIO_CONFIG_S_DRIVER_OK); -- 1.7.2.3

2 2

[PATCH 0/2] Simplify boot ordering mechanisms
by Kevin O'Connor 27 Dec '10

27 Dec '10

This is a patch series to simplify the current boot prioritization code. The basic idea is to replace the BCV list with a list that contains all bootable objects. It's then easier to make that list authoritative and do all ordering on it. This is a code simplification even without the new "bootorder" file reading code. -Kevin Kevin O'Connor (2): Populate drive_g->desc prior to calling add_bcv_internal(). Simplify boot ordering by building an inclusive boot list. src/ahci.c | 11 +- src/ata.c | 13 +- src/block.c | 20 +--- src/boot.c | 425 ++++++++++++++++++++++-------------------------------- src/boot.h | 20 +-- src/coreboot.c | 20 +++ src/floppy.c | 5 +- src/optionroms.c | 6 +- src/usb-msc.c | 11 +- src/virtio-blk.c | 10 +- 10 files changed, 226 insertions(+), 315 deletions(-) -- 1.7.3.4

2 3

Re: [SeaBIOS] [PATCH 2/2] Simplify boot ordering by building an inclusive boot list.
by Kevin O'Connor 27 Dec '10

27 Dec '10

On Sun, Dec 26, 2010 at 11:33:42PM -0500, Kevin O'Connor wrote: > Replace the bcv list with a full list of all "bootable objects". All > ordering can then be done on this list. The final boot menu, drive > mapping, and BEV list generation is then driven from this > authoritative list. > > Move "Floppy" and "DVD/CD" description prefixes into drive description > generation code to simplify boot menu generation. > > Rework QEMU's CMOS defined bootorder to work with priority scheme in > new boot list. > > Have every CBFS entry create it's own BEV entry (instead of one entry > for all CBFS payloads). Move CBFS payload detection code into > coreboot.c. Hrmm - I should probably have done the following (incremental patch) to ensure no conflict with bootorder file reading: diff --git a/src/boot.c b/src/boot.c index f7cc8d5..73ac5c8 100644 --- a/src/boot.c +++ b/src/boot.c @@ -70,12 +70,12 @@ loadBootOrder(void) } while(f); } -#define MAX_BOOT_PRIO 0xff +#define DEFAULT_PRIO 9999 -int DefaultFloppyPrio = 1; -int DefaultCDPrio = 2; -int DefaultHDPrio = 3; -int DefaultBEVPrio = 4; +static int DefaultFloppyPrio = 101; +static int DefaultCDPrio = 102; +static int DefaultHDPrio = 103; +static int DefaultBEVPrio = 104; void boot_setup(void) @@ -92,9 +92,10 @@ boot_setup(void) IPL.checkfloppysig = 0; u32 bootorder = (inb_cmos(CMOS_BIOS_BOOTFLAG2) | ((inb_cmos(CMOS_BIOS_BOOTFLAG1) & 0xf0) << 4)); - DefaultFloppyPrio = DefaultCDPrio = DefaultHDPrio = MAX_BOOT_PRIO; + DefaultFloppyPrio = DefaultCDPrio = DefaultHDPrio + = DefaultBEVPrio = DEFAULT_PRIO; int i; - for (i=1; i<4; i++) { + for (i=101; i<104; i++) { u32 val = bootorder & 0x0f; bootorder >>= 4; switch (val) { @@ -157,14 +158,14 @@ boot_add_bev(u16 seg, u16 bev, u16 desc) { bootentry_add(IPL_TYPE_BEV, DefaultBEVPrio, SEGOFF(seg, bev).segoff , desc ? MAKE_FLATPTR(seg, desc) : "Unknown"); - DefaultBEVPrio = MAX_BOOT_PRIO; + DefaultBEVPrio = DEFAULT_PRIO; } // Add a bcv entry for an expansion card harddrive or legacy option rom void boot_add_bcv(u16 seg, u16 ip, u16 desc) { - bootentry_add(IPL_TYPE_BCV, MAX_BOOT_PRIO, SEGOFF(seg, ip).segoff + bootentry_add(IPL_TYPE_BCV, DEFAULT_PRIO, SEGOFF(seg, ip).segoff , desc ? MAKE_FLATPTR(seg, desc) : "Legacy option rom"); } @@ -193,7 +194,7 @@ boot_add_cd(struct drive_s *drive_g) void boot_add_cbfs(void *data, const char *desc) { - bootentry_add(IPL_TYPE_CBFS, MAX_BOOT_PRIO, (u32)data, desc); + bootentry_add(IPL_TYPE_CBFS, DEFAULT_PRIO, (u32)data, desc); }

1 0

Re: [SeaBIOS] [PATCH] pciinit: fix off-by-one
by Kevin O'Connor 26 Dec '10

26 Dec '10

On Fri, Dec 24, 2010 at 10:51:08AM +0900, Isaku Yamahata wrote: > IO port ends at 64K - 1. not 64K. Thanks - I applied this. -Kevin

1 0

Re: [SeaBIOS] your mail
by Kevin O'Connor 26 Dec '10

26 Dec '10

I applied this change. -Kevin On Tue, Dec 21, 2010 at 10:29:27PM +0100, Stefan Reinauer wrote: > Hi, > > I made these two tiny changes to SeaBIOS when I was playing with the > HEAD version. > > The Makefile change prevents the test command from failing if CC > contains a space. > > The coreboot change prevents a warning on our gcc 4.5.2 in certain > circumstances when sizeof seems to be unsigned long int instead of > unsigned int. > > They're both unintrusive, shouldn't have potential for breaking > anything. > > Stefan > > > Signed-off-by: Stefan Reinauer <stepan(a)coreboot.org> > > --- Makefile 2010-12-21 13:21:07.179055000 -0800 > +++ Makefile 2010-12-21 13:21:52.129455000 -0800 > @@ -68,7 +68,7 @@ > ################ Build rules > > # Verify the gcc configuration and test if -fwhole-program works. > -TESTGCC:=$(shell CC=$(CC) tools/test-gcc.sh) > +TESTGCC:=$(shell CC="$(CC)" tools/test-gcc.sh) > ifeq "$(TESTGCC)" "-1" > $(error "Please upgrade GCC") > endif > --- src/coreboot.c 2010-12-21 13:21:07.278052000 -0800 > +++ src/coreboot.c 2010-12-21 13:23:03.782793000 -0800 > @@ -320,7 +320,7 @@ > u8 scratch[15980]; > int need = (LzmaGetNumProbs(&state.Properties) * sizeof(CProb)); > if (need > sizeof(scratch)) { > - dprintf(1, "LzmaDecode need %d have %d\n", need, sizeof(scratch)); > + dprintf(1, "LzmaDecode need %d have %d\n", need, (unsigned int)sizeof(scratch)); > return -1; > } > state.Probs = (CProb *)scratch; > > >

1 0

[PATCH 0/3] not yet applied boot priority patches rebased
by Gleb Natapov 26 Dec '10

26 Dec '10

Also changed the order of patches as you asked. Gleb Natapov (3): Record boot priority for each BAID/BCV/BEV. Sort IPLs according to their boot priority. Add functions for boot device path parsing. src/ahci.c | 4 +- src/ata.c | 10 ++- src/boot.c | 251 ++++++++++++++++++++++++++++++++++++++++++++++++++---- src/boot.h | 15 +++- src/floppy.c | 22 +++++- src/optionroms.c | 19 ++++- src/usb-msc.c | 2 +- src/virtio-blk.c | 2 +- 8 files changed, 292 insertions(+), 33 deletions(-) -- 1.7.2.3

2 4

[PATCH 0/9] My current Seabios patch queue.
by Gleb Natapov 26 Dec '10

26 Dec '10

Gleb Natapov (9): Provide full EDD 3.0 info for virtio disk Add BAID for ahci cdrom. Add strchr() function. Read bootorder file into memory. Add romfile_name() function. Add functions for boot device path parsing. Create mapping between rom address and boot priority. Record boot priority for each BAID/BCV/BEV. Sort IPLs according to their boot priority. src/ahci.c | 6 +- src/ata.c | 10 ++- src/boot.c | 296 ++++++++++++++++++++++++++++++++++++++++++++++++++---- src/boot.h | 17 +++- src/disk.c | 126 +++++++++++++---------- src/floppy.c | 22 ++++- src/optionroms.c | 53 +++++++++- src/paravirt.c | 8 ++ src/paravirt.h | 6 + src/usb-msc.c | 2 +- src/util.c | 11 ++ src/util.h | 1 + src/virtio-blk.c | 2 +- 13 files changed, 471 insertions(+), 89 deletions(-) -- 1.7.2.3

2 24

Re: [SeaBIOS] [PATCH] pciinit: fix off-by-one
by Peter Stuge 24 Dec '10

24 Dec '10

Isaku Yamahata wrote: > IO port ends at 64K - 1. not 64K. Is the parameter end of region, or size of region? //Peter

2 1

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS