<div dir="ltr">That approach sounds correct to me and should be a fun little project. <div><br></div><div>And for historical reference: <a href="http://bayoubobs.com/contact/">http://bayoubobs.com/contact/</a><br></div><div><br></div><div>Marc</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 25, 2016 at 3:39 AM Antonello Dettori <<a href="mailto:dev@dettori.io">dev@dettori.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  

    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <p>Hello, I'm currently taking a look at modifying the bayou payload
      in order to make it work on a modern coreboot system.<br>
      The conversion seems pretty straightforward except for a few
      points that I'm unsure about since I don't know what the situation
      was during the coreboot-v3/LAR era or what was the payload's
      design process like.</p>
    <p><br>
    </p>
    <p>1) Bayou is currently designed this way:</p>
    <ul>
      <li>Take user-written xml configuration</li>
      <li>Use pbuilder utility to parse the config</li>
      <li>pbuilder then generates a single blob containing every payload
        specified (it also performs elf->self conversions, adds the
        payloads to the blob, etc.)<br>
      </li>
      <li>Bayou (when executed) takes a compressed blob</li>
      <li>Reads the configuration from it<br>
      </li>
      <li>Parses through the blob looking for each payload specified<br>
      </li>
      <li>Follows the rest of the configuration<br>
      </li>
    </ul>
    <p>This design choice seems a bit inappropriate given that we are
      nowadays used to store multiple secondary payloads directly inside
      the cbfs (with the re-introduction of bayou "secondary payloads"
      could also be expanded to contain all the payloads).</p>
    <p>I'm instead thinking of doing the following changes:<br>
    </p>
    <ul>
      <li>Doing away with the blob</li>
      <li>Storing the configuration inside the cbfs as a cbfs_file</li>
      <li>Parse the configuration directly from the cbfs</li>
      <li>Use cbfstool to handle all addition/removal of payloads to the
        cbfs</li>
      <li>Eliminate pbuilder<br>
      </li>
    </ul>
    <p>This seems to be a much cleaner solution than the current one.<br>
      I currently don't see any downsides to it but I would like to hear
      other opinions on the matter.</p>
  </div>

--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="https://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://www.coreboot.org/mailman/listinfo/coreboot</a></blockquote></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr"><a href="http://marcjonesconsulting.com">http://marcjonesconsulting.com</a></div></div>