On 17.03.2009 15:43 Uhr, Kevin O'Connor wrote:
One suggestion - instead of making the "floating table" be a sub-table of a coreboot-table, make it a stand alone data structure. As it stands, I think existing code might get confused when it finds a coreboot table without any real data in it.
No worries about that, almost all existing code is fixed up by now. Also, the existing implementations are not less confused if they don't find a table at all. The format was designed to be flexible enough to handle any number of subtables dynamically.
Attached is the patch against libpayload.