<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>