[M] Change in coreboot[master]: [WIP] Documentation/measured_boot.md: fix SRTM/DRTM explanations
Attention is currently required from: Daniel P. Smith, Michał Żygowski, Maciej Pijanowski, Krystian Hebel.
Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68751 )
Change subject: [WIP] Documentation/measured_boot.md: fix SRTM/DRTM explanations ......................................................................
Patch Set 2:
(6 comments)
Patchset:
PS1:
Apologies for providing entire rewrites, but I think it is important to get this precise to help peo […]
Thanks for taking the time to do the rewrite :)
File Documentation/security/vboot/measured_boot.md:
https://review.coreboot.org/c/coreboot/+/68751/comment/98368e85_4b56c9a3 PS1, Line 8: The "Initial Boot Block" or "Core Root of Trust for Measurement" is the first
I would recommend the follow rewrite of the paragraph. […]
Done
https://review.coreboot.org/c/coreboot/+/68751/comment/31ab522b_5b8fb440 PS1, Line 14:
I would recommend adding this section here, […]
Done
https://review.coreboot.org/c/coreboot/+/68751/comment/67afe61d_2de7d136 PS1, Line 22: ## SRTM
I would recommend moving this up before IBB section, and here is my suggestion for wording, […]
Done
https://review.coreboot.org/c/coreboot/+/68751/comment/81cf14bd_61efa4aa PS1, Line 34: SRTM measurements are done starting with the IBB as root of trust.
Suggested rewording, […]
Done
https://review.coreboot.org/c/coreboot/+/68751/comment/f2061bb5_ee0c2f2d PS1, Line 112: ## DRTM
Recommend rewrite, […]
Done
-
Sergii Dmytruk (Code Review)