<p style="white-space: pre-wrap; word-wrap: break-word;">I am not sure what you want to do with this, so I'm gonna go with cautious disagreement unless you convince me that this is really needed for some reason.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Stage compression and full content compression are two different things. They use the same algorithms under the hood, but the way they're packaged is different and we'd better not confuse the two. The compression attribute is only used for the full content version, and it's more than informational... it actually tells coreboot how to treat the whole contents of that CBFS file. So if you set that on a stage or payload, that would mean that coreboot should try to decompress that full file content (including the stage or payload header) with the given algorithm, which will obviously not work. (Whether the current CBFS code would actually try to do that or not I'm not sure, but regardless I think that would be the most logical interpretation of the data.)</p><p style="white-space: pre-wrap; word-wrap: break-word;">cbfstool can already show you the stage and payload compression algorithms, you just have to call print with -v so it will show you individual stage/payload components. If your only gripe here is that cbfstool print without -v shows 'none' on those files, I'd be fine with just changing the output to use the algorithm from the stage or the biggest payload section there. (Still, technically, 'none' is correct because the stage or payload header is always uncompressed. Stages and especially payloads are essentially container formats. Parts of the container may be internally compressed, but we never compress the whole thing.)</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://review.coreboot.org/27115">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/27115">change 27115</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/27115"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I03927dcbd486602f54210a344d5cc7af2265c965 </div>
<div style="display:none"> Gerrit-Change-Number: 27115 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Julius Werner <jwerner@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Jun 2018 23:43:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>