[coreboot] [PATCH] put coreboot table in high tables area

Stefan Reinauer stepan at coresystems.de
Wed Mar 18 13:53:04 CET 2009

On 18.03.2009 12:12 Uhr, Peter Stuge wrote:
> Stefan Reinauer wrote:
>> +++ i386/coreboot.c	(working copy)
> ..
>> +		case CB_TAG_FORWARD:
>> +			return cb_parse_header((void *)(unsigned long)((struct cb_forward *)rec)->forward, len, info);
>> +			continue;
> Are these semantics correct? Will this recurse? Should it also
> continue parsing after a forward tag?
> I guess we should have decided on these things before committing
> but.. :)
My word is code decides, not the comitee ;-)

> If the intent is to never have more than one forward tag and never
> have anything but the forward tag in one table if there is a forward
> tag at all I think we should at least document it but ideally codify
> it, so that we don't end up with a situation where someone tries to
> "misuse" the structures, or fail to parse them?

Unless someone else is going to create a Firmware that produces a
coreboot table in a different way than it is right now, we might want to
revisit. In the current scenario this would merely be academic.


coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090318/d9fdeb19/attachment.html>

More information about the coreboot mailing list