Attention is currently required from: Arthur Heymans, Johnny Lin. Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55005 )
Change subject: mb/ocp/deltalake: Add BIOS checksum value to SMBIOS ......................................................................
Patch Set 2:
(1 comment)
This change is ready for review.
Commit Message:
https://review.coreboot.org/c/coreboot/+/55005/comment/006f74ea_3980d944 PS2, Line 7: mb/ocp/deltalake: Add BIOS checksum value to SMBIOS Another approach is to do this in build process: a. During build, calculate checksum for of the flash image. Write the checksum to either UPD variable or cbfs. c. During boot time smbios type 1 generation, the value in the cbfs or UPD (if existing) is used to populate the value. But there is a chicken and egg problem. If a utility which checks the checksum, gets checksum of the flash image, it would not match what's reported in smbios type 1. Some research is needed to solve this chicken and egg problem.
Another approach is: a. During u-root uinit, calculate the flash image checksum. b. Update smbios type 1. I do not think we update any smbios field at u-root phase at this moment, so some infrastructure work is needed for this approach to work.
Since this work may turn out to have to drag the release preparation, I am okay with us deferring this work to CraterLake.
-
Jonathan Zhang (Code Review)