<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 18.03.2009 12:12 Uhr, Peter Stuge wrote:
<blockquote cite="mid:20090318111234.23456.qmail@stuge.se" type="cite">
  <pre wrap="">Stefan Reinauer wrote:
  </pre>
  <blockquote type="cite">
    <pre wrap="">+++ i386/coreboot.c    (working copy)
    </pre>
  </blockquote>
  <pre wrap=""><!---->..
  </pre>
  <blockquote type="cite">
    <pre wrap="">+              case CB_TAG_FORWARD:
+                       return cb_parse_header((void *)(unsigned long)((struct cb_forward *)rec)->forward, len, info);
+                       continue;
    </pre>
  </blockquote>
  <pre wrap=""><!---->
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.. :)
  </pre>
</blockquote>
My word is code decides, not the comitee ;-)<br>
<br>
<blockquote cite="mid:20090318111234.23456.qmail@stuge.se" type="cite">
  <pre wrap="">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?</pre>
</blockquote>
<br>
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.<br>
<br>
Stefan<br>
<br>
<br>
<pre class="moz-signature" cols="72">-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: <a class="moz-txt-link-abbreviated" href="mailto:info@coresystems.de">info@coresystems.de</a>  • <a class="moz-txt-link-freetext" href="http://www.coresystems.de/">http://www.coresystems.de/</a>
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
</pre>
</body>
</html>