Another thought, if we want to get rid of this legacy, x86 location, but keep the id, how about adding a simple file to CBFS as standard from now on, and keeping the legacy id in the bootblock for a few releases until users adapted?

That's also my preferred option. (I would be fine with keeping the old .id in parallel for a bit longer too, but I don't know if Furquan had any specific pressure to get rid of it right now.)

From an academic viewpoint, I would probably put the version into the CBFS header and board/vendor into the FMAP header ;) Not sure how feasible that is.

If you actually mean header, I don't think there's any space for those kind of things in there, and I don't think we'd want to mess with these well-established structures.

Looking at a current CBFS, we already have a `revision` file... which is not very machine readable, though. And what is still missing is the board/vendor id, but that really shouldn't differ between multiple CBFS's inside a single image.

Yeah, it would probably make sense to either merge this into the revision file or supersede that file with whatever new format that contains all that info we'd make up.

View Change

To view, visit change 40377. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibe1aaf36bec00b4cffc46a3640a5a0d1e4bb8067
Gerrit-Change-Number: 40377
Gerrit-PatchSet: 2
Gerrit-Owner: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu>
Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com>
Gerrit-Reviewer: Frans Hendriks <fhendriks@eltan.com>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Werner Zeh <werner.zeh@siemens.com>
Gerrit-Reviewer: Wim Vervoorn <wvervoorn@eltan.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Fri, 24 Apr 2020 21:11:56 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment