Feature #420
openUse standard format of TPM event log
33%
Description
Currently coreboot uses proprietary format for TPM event log. TCG has standardized log formats, differently for TPM1.2 (aka legacy or SHA1) [1] and TPM2.0 (aka crypto agile) [2], both of which can be parsed by Linux kernel and exposed in sysfs. I don't know of any tool outside of cbmem which can parse coreboot format; this includes payloads which may be interested in continuing chain of trust started by coreboot.
Another incompatibility is caused by vboot's assignment of PCRs. Roles of PCRs are roughly specified by TCG in both of mentioned documents, they are more or less compatible with each other, but not with current coreboot code.
These changes could break assumptions made by existing platforms, so they should be made as Kconfig options.
This is a tracking issue to collect subtasks that need to be done in order to support standard event log formats.
Related links
Updated by Arthur Heymans about 2 years ago
https://review.coreboot.org/c/coreboot/+/51710 Implements the TCG one. The coreboot implementation is not a 'proprietary' format. That would imply that there is a license restriction on using it which there is not.
A lot of the TCG spec simply does not make a lot of sense for coreboot which is why it's not implemented.
Updated by Michał Żygowski about 2 years ago
- Related links updated (diff)
Arthur Heymans wrote in #note-2:
https://review.coreboot.org/c/coreboot/+/51710 Implements the TCG one. The coreboot implementation is not a 'proprietary' format. That would imply that there is a license restriction on using it which there is not.
A lot of the TCG spec simply does not make a lot of sense for coreboot which is why it's not implemented.
Yes, it is not "proprietary", "custom" would be a better word here. Anyway, having an option for TCG compliant event log will not hurt.
Thank you for pointing the patch, adding to Related Links.
Updated by Michał Żygowski about 2 years ago
- Related to Cleanup #421: Change API of functions taking hash as an argument added
Updated by Michał Żygowski about 2 years ago
- Related to Feature #422: Create Kconfig menu for TPM event log format added
Updated by Michał Żygowski about 2 years ago
- Related to Feature #423: Implement legacy and crypto agile TPM event log formats added
Updated by Michał Żygowski about 2 years ago
- Related to Feature #424: Create and implement option to choose either TCG or vboot PCR assignment added
Updated by Michał Żygowski about 2 years ago
- Related to Feature #425: Add parsing of new TPM event log formats to cbmem utility added
Updated by Michał Żygowski about 2 years ago
- Related to Documentation #426: Document existing and added TPM event log formats and PCR usage added
Updated by Michał Żygowski about 2 years ago
Adding subtask seems to be a new issue creation. In order to avoid duplication, I have added these issues from related links to the Related Issues section.
Updated by Michał Żygowski about 2 years ago
- Related links updated (diff)
Michał Żygowski wrote in #note-10:
Adding subtask seems to be a new issue creation. In order to avoid duplication, I have added these issues from related links to the Related Issues section.
NVM, I can assign Parent Task to the other issues and it will automatically link as subtask.
Updated by Sergii Dmytruk 4 months ago
2 out of 6 subtasks are closed, but really all of them are done by now, including this one.