SeaBIOS

seabios@seabios.org

3 participants
3799 discussions

Xen HVM Support
by Ian Campbell 22 Jul '11

22 Jul '11

The following enables support for running SeaBIOS as the virtual BIOS in a Xen HVM (fully-virtualised) guest. In this series SeaBIOS is loaded by hvmloader (the Xen early firmware) rather than directly as was previously tried. This approach turned out to be far less intrusive on the SeaBIOS side and leaves knowledge of the virtual hardware setup encapsulated within hvmloader which is updated in lock step with the hypervisor and associated tools. BIOS tables are built by hvmloader and passed to SeaBIOS via a simple in memory data structure. The tables are identified by scanning for their signatures which is hopefully extensible enough but the core structure can also be extended as necessary. The associated hvmloader series was only posted on xen-devel can be found at: http://marc.info/?l=xen-devel&m=130692187111154 the most interesting patch is probably: http://marc.info/?l=xen-devel&m=130692182011054 Ian.

5 8

[PATCH] xen: actually setup hypercalls.
by Ian Campbell 21 Jul '11

21 Jul '11

This was somehow dropped during the iterations of the original Xen patches. It's actually harmless at the moment since there are no users of hypercalls but patches are being written to support Xen PV block devices which need this. It's not clear exactly how early this needs to be but I think it needs to be at least before init_hw() (since that would detect disk devices). I put it first in maininit() since the only dependency is malloc(). Signed-off-by: Ian Campbell <ian.campbell(a)citrix.com> --- src/post.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/post.c b/src/post.c index d98941c..b2f6d51 100644 --- a/src/post.c +++ b/src/post.c @@ -210,6 +210,9 @@ startBoot(void) static void maininit(void) { + // Setup Xen hypercalls + xen_init_hypercalls(); + // Setup ivt/bda/ebda init_ivt(); init_bda(); -- 1.7.2.5

3 5

Re: [SeaBIOS] [PATCH 0/7] ahci update
by Kevin O'Connor 18 Jul '11

18 Jul '11

On Mon, Jul 18, 2011 at 12:26:41PM -0600, Marc Jones wrote: > On Sun, Jul 17, 2011 at 8:16 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > I've confirmed that with this patch series applied I can boot my > > Asrock e350m1 board in AHCI mode. > You tested that by changing SATA_MODE in cfg.h? Yes - I used the patch below to enable AHCI mode in coreboot. >If it is working and > commited, we should make ahci the default in coreboot. I have not yet committed the AHCI patch series. However, SeaBIOS can now handle the sb800 in AHCI mode with its regular ATA support. (A previous committed patch enables the the ATA driver to recognize that the AMD ahci controller can be driven as an ATA device when AHCI is disabled in SeaBIOS.) -Kevin diff --git a/src/southbridge/amd/cimx/sb800/cfg.c b/src/southbridge/amd/cimx/sb800/cfg.c index 0a09e11..36136b4 100644 --- a/src/southbridge/amd/cimx/sb800/cfg.c +++ b/src/southbridge/amd/cimx/sb800/cfg.c @@ -83,7 +83,7 @@ void sb800_cimx_config(AMDSBCFG *sb_config) sb_config->SATAMODE.SataMode.SataController = SATA_CONTROLLER; sb_config->SATAMODE.SataMode.SataIdeCombMdPriSecOpt = 0; //0 -IDE as primary, 1 -IDE as secondary. //TODO: set to secondary not take effect. - sb_config->SATAMODE.SataMode.SataIdeCombinedMode = 0; //IDE controlor exposed and combined mode enabled + sb_config->SATAMODE.SataMode.SataIdeCombinedMode = 1; //IDE controlor exposed and combined mode enabled sb_config->SATAMODE.SataMode.SATARefClkSel = SATA_CLOCK_SOURCE; /* Azalia HDA */ diff --git a/src/southbridge/amd/cimx/sb800/cfg.h b/src/southbridge/amd/cimx/sb800/cfg.h index 05db9ab..581fc66 100644 --- a/src/southbridge/amd/cimx/sb800/cfg.h +++ b/src/southbridge/amd/cimx/sb800/cfg.h @@ -109,7 +109,7 @@ * NOTE: DO NOT ALLOW SATA & IDE use same mode */ #ifndef SATA_MODE - #define SATA_MODE NATIVE_IDE_MODE + #define SATA_MODE AHCI_MODE #endif /**

1 0

Adding non standard VGA modes
by Bernhard Guillon 18 Jul '11

18 Jul '11

Hi all, I want to ask if it is possible to add non standard VGA modes like 800x480 to seabios vgabios implementation. I want to be able to use it for qemu. After looking at the code I am puzzeled about the "random numbers" e.g. [1]. Can you hint me to any documentation or books I can read to understand what they are doing? best regards, Bernhard 1 http://code.coreboot.org/p/seabios/source/tree/master/vgasrc/vgatables.c -- Scanned by MailScanner.

2 2

[PATCH 2/2] Remove 16bit wrappers from PMM code.
by Kevin O'Connor 16 Jul '11

16 Jul '11

Now that the PMM code is only run in 32bit mode, remove the GET/SET_PMMVAR macros and other 16bit wrappers. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/pmm.c | 131 +++++++++++++++++++++++++----------------------------------- 1 files changed, 55 insertions(+), 76 deletions(-) diff --git a/src/pmm.c b/src/pmm.c index 2e9d1ff..b812515 100644 --- a/src/pmm.c +++ b/src/pmm.c @@ -10,22 +10,6 @@ #include "farptr.h" // GET_FARVAR #include "biosvar.h" // GET_BDA - -#if MODESEGMENT -// The 16bit pmm entry points runs in "big real" mode, and can -// therefore read/write to the 32bit malloc variables. -#define GET_PMMVAR(var) ({ \ - SET_SEG(ES, 0); \ - __GET_VAR("addr32 ", ES, (var)); }) -#define SET_PMMVAR(var, val) do { \ - SET_SEG(ES, 0); \ - __SET_VAR("addr32 ", ES, (var), (val)); \ - } while (0) -#else -#define GET_PMMVAR(var) (var) -#define SET_PMMVAR(var, val) do { (var) = (val); } while (0) -#endif - // Information on a reserved area. struct allocinfo_s { struct allocinfo_s *next, **pprev; @@ -44,13 +28,9 @@ struct zone_s { struct allocinfo_s *info; }; -struct zone_s ZoneLow; -struct zone_s ZoneHigh; -struct zone_s ZoneFSeg; -struct zone_s ZoneTmpLow; -struct zone_s ZoneTmpHigh; +struct zone_s ZoneLow, ZoneHigh, ZoneFSeg, ZoneTmpLow, ZoneTmpHigh; -struct zone_s *Zones[] = { +static struct zone_s *Zones[] = { &ZoneTmpLow, &ZoneLow, &ZoneFSeg, &ZoneTmpHigh, &ZoneHigh }; @@ -64,24 +44,24 @@ static void * allocSpace(struct zone_s *zone, u32 size, u32 align, struct allocinfo_s *fill) { struct allocinfo_s *info; - for (info = GET_PMMVAR(zone->info); info; info = GET_PMMVAR(info->next)) { - void *dataend = GET_PMMVAR(info->dataend); - void *allocend = GET_PMMVAR(info->allocend); + for (info = zone->info; info; info = info->next) { + void *dataend = info->dataend; + void *allocend = info->allocend; void *newallocend = (void*)ALIGN_DOWN((u32)allocend - size, align); if (newallocend >= dataend && newallocend <= allocend) { // Found space - now reserve it. - struct allocinfo_s **pprev = GET_PMMVAR(info->pprev); + struct allocinfo_s **pprev = info->pprev; if (!fill) fill = newallocend; - SET_PMMVAR(fill->next, info); - SET_PMMVAR(fill->pprev, pprev); - SET_PMMVAR(fill->data, newallocend); - SET_PMMVAR(fill->dataend, newallocend + size); - SET_PMMVAR(fill->allocend, allocend); - - SET_PMMVAR(info->allocend, newallocend); - SET_PMMVAR(info->pprev, &fill->next); - SET_PMMVAR(*pprev, fill); + fill->next = info; + fill->pprev = pprev; + fill->data = newallocend; + fill->dataend = newallocend + size; + fill->allocend = allocend; + + info->allocend = newallocend; + info->pprev = &fill->next; + *pprev = fill; return newallocend; } } @@ -92,13 +72,13 @@ allocSpace(struct zone_s *zone, u32 size, u32 align, struct allocinfo_s *fill) static void freeSpace(struct allocinfo_s *info) { - struct allocinfo_s *next = GET_PMMVAR(info->next); - struct allocinfo_s **pprev = GET_PMMVAR(info->pprev); - SET_PMMVAR(*pprev, next); + struct allocinfo_s *next = info->next; + struct allocinfo_s **pprev = info->pprev; + *pprev = next; if (next) { - if (GET_PMMVAR(next->allocend) == GET_PMMVAR(info->data)) - SET_PMMVAR(next->allocend, GET_PMMVAR(info->allocend)); - SET_PMMVAR(next->pprev, pprev); + if (next->allocend == info->data) + next->allocend = info->allocend; + next->pprev = pprev; } } @@ -109,8 +89,8 @@ addSpace(struct zone_s *zone, void *start, void *end) // Find position to add space struct allocinfo_s **pprev = &zone->info, *info; for (;;) { - info = GET_PMMVAR(*pprev); - if (!info || GET_PMMVAR(info->data) < start) + info = *pprev; + if (!info || info->data < start) break; pprev = &info->next; } @@ -121,10 +101,9 @@ addSpace(struct zone_s *zone, void *start, void *end) tempdetail.datainfo.pprev = pprev; tempdetail.datainfo.data = tempdetail.datainfo.dataend = start; tempdetail.datainfo.allocend = end; - struct allocdetail_s *tempdetailp = MAKE_FLATPTR(GET_SEG(SS), &tempdetail); - SET_PMMVAR(*pprev, &tempdetailp->datainfo); + *pprev = &tempdetail.datainfo; if (info) - SET_PMMVAR(info->pprev, &tempdetailp->datainfo.next); + info->pprev = &tempdetail.datainfo.next; // Allocate final allocation info. struct allocdetail_s *detail = allocSpace( @@ -133,23 +112,21 @@ addSpace(struct zone_s *zone, void *start, void *end) detail = allocSpace(&ZoneTmpLow, sizeof(*detail) , MALLOC_MIN_ALIGN, NULL); if (!detail) { - SET_PMMVAR(*tempdetail.datainfo.pprev, tempdetail.datainfo.next); + *tempdetail.datainfo.pprev = tempdetail.datainfo.next; if (tempdetail.datainfo.next) - SET_PMMVAR(tempdetail.datainfo.next->pprev - , tempdetail.datainfo.pprev); + tempdetail.datainfo.next->pprev = tempdetail.datainfo.pprev; warn_noalloc(); return; } } // Replace temp alloc space with final alloc space - memcpy_fl(&detail->datainfo, &tempdetailp->datainfo - , sizeof(detail->datainfo)); - SET_PMMVAR(detail->handle, PMM_DEFAULT_HANDLE); + memcpy(&detail->datainfo, &tempdetail.datainfo, sizeof(detail->datainfo)); + detail->handle = PMM_DEFAULT_HANDLE; - SET_PMMVAR(*tempdetail.datainfo.pprev, &detail->datainfo); + *tempdetail.datainfo.pprev = &detail->datainfo; if (tempdetail.datainfo.next) - SET_PMMVAR(tempdetail.datainfo.next->pprev, &detail->datainfo.next); + tempdetail.datainfo.next->pprev = &detail->datainfo.next; } // Search all zones for an allocation obtained from allocSpace() @@ -158,10 +135,10 @@ findAlloc(void *data) { int i; for (i=0; i<ARRAY_SIZE(Zones); i++) { - struct zone_s *zone = GET_PMMVAR(Zones[i]); + struct zone_s *zone = Zones[i]; struct allocinfo_s *info; - for (info = GET_PMMVAR(zone->info); info; info = GET_PMMVAR(info->next)) - if (GET_PMMVAR(info->data) == data) + for (info = zone->info; info; info = info->next) + if (info->data == data) return info; } return NULL; @@ -171,11 +148,11 @@ findAlloc(void *data) static struct allocinfo_s * findLast(struct zone_s *zone) { - struct allocinfo_s *info = GET_PMMVAR(zone->info); + struct allocinfo_s *info = zone->info; if (!info) return NULL; for (;;) { - struct allocinfo_s *next = GET_PMMVAR(info->next); + struct allocinfo_s *next = info->next; if (!next) return info; info = next; @@ -244,6 +221,7 @@ malloc_fixupreloc(void) void malloc_finalize(void) { + ASSERT32FLAT(); dprintf(3, "malloc finalize\n"); // Reserve more low-mem if needed. @@ -273,9 +251,8 @@ relocate_ebda(u32 newebda, u32 oldebda, u8 ebda_size) // EBDA isn't at end of ram - give up. return -1; - // Do copy (this assumes memcpy copies forward - otherwise memmove - // is needed) - memcpy_fl((void*)newebda, (void*)oldebda, ebda_size * 1024); + // Do copy + memmove((void*)newebda, (void*)oldebda, ebda_size * 1024); // Update indexes dprintf(1, "ebda moved from %x to %x\n", oldebda, newebda); @@ -291,9 +268,9 @@ zonelow_expand(u32 size, u32 align) struct allocinfo_s *info = findLast(&ZoneLow); if (!info) return; - u32 oldpos = (u32)GET_PMMVAR(info->allocend); + u32 oldpos = (u32)info->allocend; u32 newpos = ALIGN_DOWN(oldpos - size, align); - u32 bottom = (u32)GET_PMMVAR(info->dataend); + u32 bottom = (u32)info->dataend; if (newpos >= bottom && newpos <= oldpos) // Space already present. return; @@ -317,8 +294,8 @@ zonelow_expand(u32 size, u32 align) // Update zone if (ebda_end == bottom) { - SET_PMMVAR(info->data, (void*)newbottom); - SET_PMMVAR(info->dataend, (void*)newbottom); + info->data = (void*)newbottom; + info->dataend = (void*)newbottom; } else addSpace(&ZoneLow, (void*)newbottom, (void*)ebda_end); } @@ -352,6 +329,7 @@ allocExpandSpace(struct zone_s *zone, u32 size, u32 align void * __malloc pmm_malloc(struct zone_s *zone, u32 handle, u32 size, u32 align) { + ASSERT32FLAT(); if (!size) return NULL; @@ -376,7 +354,7 @@ pmm_malloc(struct zone_s *zone, u32 handle, u32 size, u32 align) " ret=%p (detail=%p)\n" , zone, handle, size, align , data, detail); - SET_PMMVAR(detail->handle, handle); + detail->handle = handle; return data; } @@ -385,8 +363,9 @@ pmm_malloc(struct zone_s *zone, u32 handle, u32 size, u32 align) int pmm_free(void *data) { + ASSERT32FLAT(); struct allocinfo_s *info = findAlloc(data); - if (!info || data == (void*)info || data == GET_PMMVAR(info->dataend)) + if (!info || data == (void*)info || data == info->dataend) return -1; struct allocdetail_s *detail = container_of( info, struct allocdetail_s, datainfo); @@ -404,8 +383,8 @@ pmm_getspace(struct zone_s *zone) // XXX - results not reliable when CONFIG_THREAD_OPTIONROMS u32 maxspace = 0; struct allocinfo_s *info; - for (info = GET_PMMVAR(zone->info); info; info = GET_PMMVAR(info->next)) { - u32 space = GET_PMMVAR(info->allocend) - GET_PMMVAR(info->dataend); + for (info = zone->info; info; info = info->next) { + u32 space = info->allocend - info->dataend; if (space > maxspace) maxspace = space; } @@ -425,16 +404,15 @@ pmm_find(u32 handle) { int i; for (i=0; i<ARRAY_SIZE(Zones); i++) { - struct zone_s *zone = GET_PMMVAR(Zones[i]); + struct zone_s *zone = Zones[i]; struct allocinfo_s *info; - for (info = GET_PMMVAR(zone->info); info - ; info = GET_PMMVAR(info->next)) { - if (GET_PMMVAR(info->data) != (void*)info) + for (info = zone->info; info; info = info->next) { + if (info->data != (void*)info) continue; struct allocdetail_s *detail = container_of( info, struct allocdetail_s, detailinfo); - if (GET_PMMVAR(detail->handle) == handle) - return GET_PMMVAR(detail->datainfo.data); + if (detail->handle == handle) + return detail->datainfo.data; } } return NULL; @@ -561,6 +539,7 @@ handle_pmmXX(u16 *args) u32 VISIBLE32INIT handle_pmm(u16 *args) { + ASSERT32FLAT(); if (! CONFIG_PMM) return PMM_FUNCTION_NOT_SUPPORTED; -- 1.7.4.4

1 0

[PATCH 1/2] Run option rom visible PMM code in 32bit mode instead of 16bit mode.
by Kevin O'Connor 16 Jul '11

16 Jul '11

Use call32() to jump into handle_pmm(). This reduces the amount of 16bit code needed. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/pmm.c | 51 ++++++++++++++++++++++++++++++++++++++++----------- src/post.c | 3 +++ src/romlayout.S | 6 ++++-- src/stacks.c | 2 +- src/util.h | 2 ++ 5 files changed, 50 insertions(+), 14 deletions(-) diff --git a/src/pmm.c b/src/pmm.c index e0770ac..2e9d1ff 100644 --- a/src/pmm.c +++ b/src/pmm.c @@ -44,13 +44,13 @@ struct zone_s { struct allocinfo_s *info; }; -struct zone_s ZoneLow VAR32FLATVISIBLE; -struct zone_s ZoneHigh VAR32FLATVISIBLE; -struct zone_s ZoneFSeg VAR32FLATVISIBLE; -struct zone_s ZoneTmpLow VAR32FLATVISIBLE; -struct zone_s ZoneTmpHigh VAR32FLATVISIBLE; +struct zone_s ZoneLow; +struct zone_s ZoneHigh; +struct zone_s ZoneFSeg; +struct zone_s ZoneTmpLow; +struct zone_s ZoneTmpHigh; -struct zone_s *Zones[] VAR32FLATVISIBLE = { +struct zone_s *Zones[] = { &ZoneTmpLow, &ZoneLow, &ZoneFSeg, &ZoneTmpHigh, &ZoneHigh }; @@ -225,6 +225,22 @@ malloc_setup(void) } } +// Update pointers after code relocation. +void +malloc_fixupreloc(void) +{ + ASSERT32FLAT(); + if (!CONFIG_RELOCATE_INIT) + return; + dprintf(3, "malloc fixup reloc\n"); + + int i; + for (i=0; i<ARRAY_SIZE(Zones); i++) { + struct zone_s *zone = Zones[i]; + zone->info->pprev = &zone->info; + } +} + void malloc_finalize(void) { @@ -542,7 +558,7 @@ handle_pmmXX(u16 *args) return PMM_FUNCTION_NOT_SUPPORTED; } -u32 VISIBLE16 +u32 VISIBLE32INIT handle_pmm(u16 *args) { if (! CONFIG_PMM) @@ -551,12 +567,25 @@ handle_pmm(u16 *args) u16 arg1 = args[0]; dprintf(DEBUG_HDL_pmm, "pmm call arg1=%x\n", arg1); + int oldpreempt; + if (CONFIG_THREAD_OPTIONROMS) { + // Not a preemption event - don't wait in wait_preempt() + oldpreempt = CanPreempt; + CanPreempt = 0; + } + + u32 ret; switch (arg1) { - case 0x00: return handle_pmm00(args); - case 0x01: return handle_pmm01(args); - case 0x02: return handle_pmm02(args); - default: return handle_pmmXX(args); + case 0x00: ret = handle_pmm00(args); break; + case 0x01: ret = handle_pmm01(args); break; + case 0x02: ret = handle_pmm02(args); break; + default: ret = handle_pmmXX(args); break; } + + if (CONFIG_THREAD_OPTIONROMS) + CanPreempt = oldpreempt; + + return ret; } // romlayout.S diff --git a/src/post.c b/src/post.c index d98941c..6f7aa14 100644 --- a/src/post.c +++ b/src/post.c @@ -210,6 +210,9 @@ startBoot(void) static void maininit(void) { + // Running at new code address - do code relocation fixups + malloc_fixupreloc(); + // Setup ivt/bda/ebda init_ivt(); init_bda(); diff --git a/src/romlayout.S b/src/romlayout.S index fbc5cb3..5983a4a 100644 --- a/src/romlayout.S +++ b/src/romlayout.S @@ -264,8 +264,10 @@ entry_pmm: pushw %ds movw %ss, %cx // Move %ss to %ds movw %cx, %ds - leal 28(%esp), %eax // %eax points to start of args - calll handle_pmm + movl $_cfunc32flat_handle_pmm, %eax // Setup: call32(handle_pmm, args, -1) + leal 28(%esp), %edx // %edx points to start of args + movl $-1, %ecx + calll call32 movw %ax, 12(%esp) // Modify %ax:%dx to return %eax shrl $16, %eax movw %ax, 4(%esp) diff --git a/src/stacks.c b/src/stacks.c index 17495d5..17f1a4a 100644 --- a/src/stacks.c +++ b/src/stacks.c @@ -32,7 +32,7 @@ static inline void lgdt(struct descloc_s *desc) { } // Call a 32bit SeaBIOS function from a 16bit SeaBIOS function. -u32 +u32 VISIBLE16 call32(void *func, u32 eax, u32 errret) { ASSERT16(); diff --git a/src/util.h b/src/util.h index 5cdb483..174e94b 100644 --- a/src/util.h +++ b/src/util.h @@ -226,6 +226,7 @@ int get_keystroke(int msec); u32 call32(void *func, u32 eax, u32 errret); inline u32 stack_hop(u32 eax, u32 edx, void *func); extern struct thread_info MainThread; +extern int CanPreempt; struct thread_info *getCurThread(void); void yield(void); void wait_irq(void); @@ -415,6 +416,7 @@ void pnp_setup(void); // pmm.c extern struct zone_s ZoneLow, ZoneHigh, ZoneFSeg, ZoneTmpLow, ZoneTmpHigh; void malloc_setup(void); +void malloc_fixupreloc(void); void malloc_finalize(void); void *pmm_malloc(struct zone_s *zone, u32 handle, u32 size, u32 align); int pmm_free(void *data); -- 1.7.4.4

1 0

[PATCH] Fix compilation errors/warnings
by Stefan Berger 13 Jul '11

13 Jul '11

With gcc 4.6.0 (20110530) and ld version 2.21.52.0.1-5.fc16 I get the following compilation errors/warnings: Working around non-functional -combine Working around non-functional -combine Working around non-functional -combine Build Kconfig config file Compiling whole program out/ccode.16.s Compiling to assembler out/asm-offsets.s In file included from out/ccode.16.s.tmp.c:35:0: out/../src/ahci.c: In function ‘ahci_detect’: out/../src/ahci.c:411:25: warning: variable ‘port’ set but not used [-Wunused-but-set-variable] Compiling whole program out/ccode32flat.o Compiling whole program out/code32seg.o Generating offset file out/asm-offsets.h Compiling (16bit) out/code16.o In file included from out/ccode32flat.o.tmp.c:35:0: out/../src/ahci.c: In function ‘ahci_detect’: out/../src/ahci.c:411:25: warning: variable ‘port’ set but not used [-Wunused-but-set-variable] In file included from out/ccode32flat.o.tmp.c:57:0: out/../src/bmp.c: At top level: out/../src/bmp.c:15:16: error: redefinition of ‘struct tagBITMAPFILEHEADER’ out/../src/bmp.c:15:16: note: originally defined here out/../src/bmp.c:21:3: error: conflicting types for ‘BITMAPFILEHEADER’ out/../src/bmp.c:21:3: note: previous declaration of ‘BITMAPFILEHEADER’ was here out/../src/bmp.c:21:21: error: conflicting types for ‘tagBITMAPFILEHEADER’ out/../src/bmp.c:21:21: note: previous declaration of ‘tagBITMAPFILEHEADER’ was here out/../src/bmp.c:23:16: error: redefinition of ‘struct tagBITMAPINFOHEADER’ out/../src/bmp.c:23:16: note: originally defined here out/../src/bmp.c:35:3: error: conflicting types for ‘BITMAPINFOHEADER’ out/../src/bmp.c:35:3: note: previous declaration of ‘BITMAPINFOHEADER’ was here out/../src/bmp.c:35:21: error: conflicting types for ‘tagBITMAPINFOHEADER’ out/../src/bmp.c:35:21: note: previous declaration of ‘tagBITMAPINFOHEADER’ was here out/../src/bmp.c:37:16: error: redefinition of ‘struct tagRGBQUAD’ out/../src/bmp.h:6:12: note: originally defined here out/../src/bmp.c:42:3: error: conflicting types for ‘RGBQUAD’ out/../src/bmp.c:42:3: note: previous declaration of ‘RGBQUAD’ was here out/../src/bmp.c:42:12: error: conflicting types for ‘tagRGBQUAD’ out/../src/bmp.c:42:12: note: previous declaration of ‘tagRGBQUAD’ was here out/../src/bmp.c:50:13: error: redefinition of ‘raw_data_format_adjust_24bpp’ out/../src/bmp.c:50:13: note: previous definition of ‘raw_data_format_adjust_24bpp’ was here out/../src/bmp.c:61:21: error: redefinition of ‘bmp_alloc’ out/../src/bmp.c:61:21: note: previous definition of ‘bmp_alloc’ was here out/../src/bmp.c:67:5: error: redefinition of ‘bmp_decode’ out/../src/bmp.c:67:5: note: previous definition of ‘bmp_decode’ was here out/../src/bmp.c:86:6: error: redefinition of ‘bmp_get_size’ out/../src/bmp.c:86:6: note: previous definition of ‘bmp_get_size’ was here out/../src/bmp.c:93:5: error: redefinition of ‘bmp_show’ out/../src/bmp.c:93:5: note: previous definition of ‘bmp_show’ was here make: *** [out/ccode32flat.o] Error 1 make: *** Waiting for unfinished jobs.... cp: cannot stat `out/bios.bin': No such file or directory cp: cannot stat `out/bios.bin': No such file or directory cp: cannot stat `out/bios.bin': No such file or directory cp: cannot stat `out/bios.bin': No such file or directory cp: cannot stat `out/bios.bin': No such file or directory The following patch fixes them. Signed-off-by: Stefan Berger <stefanb(a)linux.vnet.ibm.com> diff --git a/Makefile b/Makefile index 2786c9e..a4ca497 100644 --- a/Makefile +++ b/Makefile @@ -20,7 +20,7 @@ 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 \ - pci_region.c biostables.c xen.c bmp.c + pci_region.c biostables.c xen.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/ahci.c b/src/ahci.c index fb4b70c..f7bd4b5 100644 --- a/src/ahci.c +++ b/src/ahci.c @@ -408,7 +408,6 @@ static void ahci_detect(void *data) { struct ahci_ctrl_s *ctrl = data; - struct ahci_port_s *port; u32 pnr, max; int rc; @@ -422,7 +421,7 @@ ahci_detect(void *data) dprintf(1, "AHCI/%d: link %s\n", pnr, rc == 0 ? "up" : "down"); if (rc != 0) continue; - port = ahci_port_init(ctrl, pnr); + ahci_port_init(ctrl, pnr); } }

2 1

Re: [SeaBIOS] [Qemu-devel] SeaBIOS error with Juniper FreeBSD kernel
by Kevin O'Connor 11 Jul '11

11 Jul '11

On Sun, Feb 21, 2010 at 04:18:38PM -0700, Brandon Bennett wrote: > > On Sat, Feb 20, 2010 at 9:05 PM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > >> Should a kernel fail during boot, I'd suspect it doesn't like one of > >> the apm/pcibios callbacks, or it doesn't like one of the > >> smbios/mptable/acpi tables. You could try compiling the SeaBIOS code > >> (see http://seabios.org/Download ) and increasing the debugging by > >> modifying src/config.h. Specifically, you could increase > >> CONFIG_DEBUG_LEVEL, and set DEBUG_HDL_pcibios32 and DEBUG_HDL_apm to > >> 1. Also, you could try disabling some of the features to see if that > >> prevents the fault (eg, disabling CONFIG_ACPI / CONFIG_SMBIOS / > >> CONFIG_MPTABLE). > > > > I have narrowed it down to SMBIOS. If I disable CONFIG_SMBIOS the > image boots up fine. Gleb, have you seen this thread? Some of the recent changes to smbios that look like possible culprits are: Make SMBIOS table pass MS SVVP test Use MaxCountCPUs during building of per cpu tables. Add malloc_high/fseg() and rework bios table creation to use them. There were other changes, but the comments indicate they were only ports of changes already in bochs. I suppose it's also possible the lack of smbios is turning off some other feature in the guest (eg, acpi) that's the real culprit. -Kevin

5 10

[PATCH] Simplify POST entry code by moving reboot logic from post.c to resume.c.
by Kevin O'Connor 11 Jul '11

11 Jul '11

Detect a resume/reboot by inspecting HaveRunPost instead of inspecting the cmos reset code. Inspecting a global variable is both simpler and safer. Move the reboot logic from post.c to resume.c - this makes the code in post.c simpler as it is now only called once on machine startup. This also makes it easier to ensure all POST initialization code resides in the relocatable "init" sections. Also, rename _start() to handle_post() so that it is more in keeping with the entry_xxx() and handle_xxx() function naming. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/post.c | 74 +++++++++--------------------------- src/resume.c | 111 ++++++++++++++++++++++++++++++++++++------------------- src/romlayout.S | 59 +++++------------------------ src/stacks.c | 5 -- src/util.h | 10 +++++ 5 files changed, 113 insertions(+), 146 deletions(-) diff --git a/src/post.c b/src/post.c index 813ff20..d98941c 100644 --- a/src/post.c +++ b/src/post.c @@ -277,7 +277,7 @@ maininit(void) /**************************************************************** - * Code relocation + * POST entry and code relocation ****************************************************************/ // Update given relocs for the code at 'dest' with a given 'delta' @@ -289,8 +289,7 @@ updateRelocs(void *dest, u32 *rstart, u32 *rend, u32 delta) *((u32*)(dest + *reloc)) += delta; } -// Start of Power On Self Test - the BIOS initilization. This -// function sets up for and attempts relocation of the init code. +// Relocate init code and then call maininit() at new address. static void reloc_init(void) { @@ -329,63 +328,23 @@ reloc_init(void) } // Start of Power On Self Test (POST) - the BIOS initilization phase. -// This function sets up for and attempts relocation of the init code. +// This function does the setup needed for code relocation, and then +// invokes the relocation and main setup code. void VISIBLE32INIT -post(void) +handle_post(void) { - // Detect ram and setup internal malloc. - qemu_cfg_port_probe(); - ram_probe(); - malloc_setup(); - - reloc_init(); -} - - -/**************************************************************** - * POST entry point - ****************************************************************/ - -static int HaveRunPost; - -// Attempt to invoke a hard-reboot. -static void -tryReboot(void) -{ - dprintf(1, "Attempting a hard reboot\n"); - - // Setup for reset on qemu. - if (! CONFIG_COREBOOT) { - qemu_prep_reset(); - if (HaveRunPost) - apm_shutdown(); - } - - // Try keyboard controller reboot. - i8042_reboot(); - - // Try PCI 0xcf9 reboot - pci_reboot(); + debug_serial_setup(); + dprintf(1, "Start bios (version %s)\n", VERSION); - // Try triple fault - asm volatile("int3"); + // Enable CPU caching + setcr0(getcr0() & ~(CR0_CD|CR0_NW)); - panic("Could not reboot"); -} + // Clear CMOS reboot flag. + outb_cmos(0, CMOS_RESET_CODE); -// 32-bit entry point. -void VISIBLE32FLAT -_start(void) -{ + // Make sure legacy DMA isn't running. init_dma(); - debug_serial_setup(); - dprintf(1, "Start bios (version %s)\n", VERSION); - - if (HaveRunPost) - // This is a soft reboot - invoke a hard reboot. - tryReboot(); - // Check if we are running under Xen. xen_probe(); @@ -393,6 +352,11 @@ _start(void) make_bios_writable(); HaveRunPost = 1; - // Perform main setup code. - post(); + // Detect ram and setup internal malloc. + qemu_cfg_port_probe(); + ram_probe(); + malloc_setup(); + + // Relocate initialization code and call maininit(). + reloc_init(); } diff --git a/src/resume.c b/src/resume.c index 20e2e3d..4390fb5 100644 --- a/src/resume.c +++ b/src/resume.c @@ -10,6 +10,12 @@ #include "biosvar.h" // struct bios_data_area_s #include "bregs.h" // struct bregs #include "acpi.h" // find_resume_vector +#include "ps2port.h" // i8042_reboot +#include "pci.h" // pci_reboot +#include "cmos.h" // inb_cmos + +// Indicator if POST phase has been run. +int HaveRunPost VAR16VISIBLE; // Reset DMA controller void @@ -26,34 +32,19 @@ init_dma(void) // Handler for post calls that look like a resume. void VISIBLE16 -handle_resume(u8 status) +handle_resume(void) { - init_dma(); - debug_serial_setup(); + int status = inb_cmos(CMOS_RESET_CODE); + outb_cmos(0, CMOS_RESET_CODE); dprintf(1, "In resume (status=%d)\n", status); + init_dma(); + switch (status) { - case 0xfe: - if (CONFIG_S3_RESUME) { - // S3 resume request. Jump to 32bit mode to handle the resume. - asm volatile( - "movw %w1, %%ss\n" - "movl %0, %%esp\n" - "movl $_cfunc32flat_s3_resume, %%edx\n" - "jmp transition32\n" - : : "i"(BUILD_S3RESUME_STACK_ADDR), "r"(0) - ); - break; - } - // NO BREAK - case 0x00: - case 0x0d ... 0xfd: - case 0xff: - // Normal post - now that status has been cleared a reset will - // run regular boot code.. - reset_vector(); - break; + case 0x01 ... 0x04: + case 0x06 ... 0x09: + panic("Unimplemented shutdown status: %02x\n", status); case 0x05: // flush keyboard (issue EOI) and jump via 40h:0067h @@ -88,19 +79,33 @@ handle_resume(u8 status) : : "m"(BDA_JUMP), "r"(SEG_BDA) ); break; + + default: + break; } - panic("Unimplemented shutdown status: %02x\n", status); + // Not a 16bit resume - do remaining checks in 32bit mode + asm volatile( + "movw %w1, %%ss\n" + "movl %0, %%esp\n" + "movl $_cfunc32flat_handle_resume32, %%edx\n" + "jmp transition32\n" + : : "i"(BUILD_S3RESUME_STACK_ADDR), "r"(0), "a"(status) + ); } -void VISIBLE32FLAT +// Handle an S3 resume event +static void s3_resume(void) { - ASSERT32FLAT(); if (!CONFIG_S3_RESUME) - panic("S3 resume support not compiled in.\n"); + return; - dprintf(1, "In 32bit resume\n"); + u32 s3_resume_vector = find_resume_vector(); + if (!s3_resume_vector) { + dprintf(1, "No resume vector set!\n"); + return; + } smm_init(); @@ -108,18 +113,48 @@ s3_resume(void) make_bios_readonly(); - u32 s3_resume_vector = find_resume_vector(); - // Invoke the resume vector. struct bregs br; memset(&br, 0, sizeof(br)); - if (s3_resume_vector) { - dprintf(1, "Jump to resume vector (%x)\n", s3_resume_vector); - br.code = FLATPTR_TO_SEGOFF((void*)s3_resume_vector); - } else { - dprintf(1, "No resume vector set!\n"); - // Jump to the post vector to restart with a normal boot. - br.code = SEGOFF(SEG_BIOS, (u32)reset_vector - BUILD_BIOS_ADDR); - } + dprintf(1, "Jump to resume vector (%x)\n", s3_resume_vector); + br.code = FLATPTR_TO_SEGOFF((void*)s3_resume_vector); call16big(&br); } + +// Attempt to invoke a hard-reboot. +static void +tryReboot(void) +{ + dprintf(1, "Attempting a hard reboot\n"); + + // Setup for reset on qemu. + if (! CONFIG_COREBOOT) { + qemu_prep_reset(); + if (HaveRunPost) + apm_shutdown(); + } + + // Try keyboard controller reboot. + i8042_reboot(); + + // Try PCI 0xcf9 reboot + pci_reboot(); + + // Try triple fault + asm volatile("int3"); + + panic("Could not reboot"); +} + +void VISIBLE32FLAT +handle_resume32(int status) +{ + ASSERT32FLAT(); + dprintf(1, "In 32bit resume\n"); + + if (status == 0xfe) + s3_resume(); + + // Must be a soft reboot - invoke a hard reboot. + tryReboot(); +} diff --git a/src/romlayout.S b/src/romlayout.S index f540c1e..f5c36fc 100644 --- a/src/romlayout.S +++ b/src/romlayout.S @@ -230,64 +230,25 @@ __call16big_from32: /**************************************************************** - * POST entry point + * Misc. entry points. ****************************************************************/ - DECLFUNC entry_post -entry_post: - // Enable cache - movl %cr0, %eax - andl $~(CR0_CD|CR0_NW), %eax - movl %eax, %cr0 - +// Resume (and reboot) entry point - called from entry_post + DECLFUNC entry_resume +entry_resume: // Disable interrupts cli cld - - // Check for restart indicator. - movl $CMOS_RESET_CODE|NMI_DISABLE_BIT, %eax - outb %al, $PORT_CMOS_INDEX - inb $PORT_CMOS_DATA, %al - cmpb $0x0, %al - jnz 1f - - // Normal entry point - ENTRY_INTO32 _cfunc32flat__start - - // Entry point when a post call looks like a resume. -1: - // Save old shutdown status. - movl %eax, %ebx - - // Clear shutdown status register. - movl $CMOS_RESET_CODE|NMI_DISABLE_BIT, %eax - outb %al, $PORT_CMOS_INDEX - xorl %eax, %eax - outb %al, $PORT_CMOS_DATA - // Use a stack in EBDA movw $SEG_BDA, %ax movw %ax, %ds movw BDA_ebda_seg, %ax - - cmpw $EBDA_SEGMENT_START, %ax - jle 2f - // EBDA segment doesn't look valid - use startup value. - movw $EBDA_SEGMENT_START, %ax - -2: movw %ax, %ds + movw %ax, %ds movw %ax, %ss movl $EBDA_OFFSET_TOP_STACK, %esp - // Call handler. - movl %ebx, %eax jmp handle_resume - -/**************************************************************** - * Misc. entry points. - ****************************************************************/ - // PMM entry point DECLFUNC entry_pmm entry_pmm: @@ -418,7 +379,7 @@ post32: movw %ax, %gs movw %ax, %ss movl $BUILD_STACK_ADDR, %esp - ljmpl $SEG32_MODE32_CS, $_cfunc32flat__start + ljmpl $SEG32_MODE32_CS, $_cfunc32flat_handle_post .code16gcc @@ -494,8 +455,10 @@ entry_18: .endm ORG 0xe05b -entry_post_official: - jmp entry_post +entry_post: + cmpl $0, %cs:HaveRunPost // Check for resume/reboot + jnz entry_resume + ENTRY_INTO32 _cfunc32flat_handle_post // Normal entry point ORG 0xe2c3 IRQ_ENTRY 02 @@ -578,7 +541,7 @@ entry_iret_official: ORG 0xfff0 // Power-up Entry Point .global reset_vector reset_vector: - ljmpw $SEG_BIOS, $entry_post_official + ljmpw $SEG_BIOS, $entry_post // 0xfff5 - BiosDate in misc.c diff --git a/src/stacks.c b/src/stacks.c index 53bf669..17495d5 100644 --- a/src/stacks.c +++ b/src/stacks.c @@ -24,11 +24,6 @@ struct thread_info VAR32FLATVISIBLE MainThread = { * Low level helpers ****************************************************************/ -static inline u32 getcr0(void) { - u32 cr0; - asm("movl %%cr0, %0" : "=r"(cr0)); - return cr0; -} static inline void sgdt(struct descloc_s *desc) { asm("sgdtl %0" : "=m"(*desc)); } diff --git a/src/util.h b/src/util.h index 303c524..4319f63 100644 --- a/src/util.h +++ b/src/util.h @@ -61,6 +61,15 @@ static inline void cpuid(u32 index, u32 *eax, u32 *ebx, u32 *ecx, u32 *edx) : "0" (index)); } +static inline u32 getcr0(void) { + u32 cr0; + asm("movl %%cr0, %0" : "=r"(cr0)); + return cr0; +} +static inline void setcr0(u32 cr0) { + asm("movl %0, %%cr0" : : "r"(cr0)); +} + static inline u64 rdmsr(u32 index) { u64 ret; @@ -424,6 +433,7 @@ void enable_bootsplash(void); void disable_bootsplash(void); // resume.c +extern int HaveRunPost; void init_dma(void); // pnpbios.c -- 1.7.4.4

1 0

[PATCH v2] Fix serial port flushing code.
by Kevin O'Connor 09 Jul '11

09 Jul '11

debug_serial() was doing a full flush while debug_serial_flush() was only doing a partial flush. Fix that. --- Changes from v1 - fix bit math so it actually checks the correct bit. --- src/output.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/output.c b/src/output.c index 5c91ae5..462ffb7 100644 --- a/src/output.c +++ b/src/output.c @@ -49,7 +49,7 @@ debug_serial(char c) if (!CONFIG_DEBUG_SERIAL) return; int timeout = DEBUG_TIMEOUT; - while ((inb(CONFIG_DEBUG_SERIAL_PORT+SEROFF_LSR) & 0x60) != 0x60) + while ((inb(CONFIG_DEBUG_SERIAL_PORT+SEROFF_LSR) & 0x20) != 0x20) if (!timeout--) // Ran out of time. return; @@ -63,7 +63,7 @@ debug_serial_flush(void) if (!CONFIG_DEBUG_SERIAL) return; int timeout = DEBUG_TIMEOUT; - while ((inb(CONFIG_DEBUG_SERIAL_PORT+SEROFF_LSR) & 0x40) != 0x40) + while ((inb(CONFIG_DEBUG_SERIAL_PORT+SEROFF_LSR) & 0x60) != 0x60) if (!timeout--) // Ran out of time. return; -- 1.7.4.4

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS