Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32090 )
Change subject: smbios: Add and adapt fields for version 3.2 ......................................................................
Patch Set 5:
|> Still, you could fix the cpuid code first, then add SMBIOS 3.2 stuff. I don't see why it would have to be part of the same commit. Removing the new core/tread procedure here and keeping everything else, along with the old core count in this commit would achieve the same goal, wouldn't it? If that is your request please confirm and I will do just that.
You can do that. My only request is that you don't fix existing fields and call that required for the SMBIOS update.
|> I was mostly referring to the added fields that were left untouched (at 0) where 0 isn't a valid value. Sorry for the confusion. There are 3 added field which are set to 0, and 0 indeed is a valid value for each of them. Please see these lines from the SMBIOS spec: " A value of 0000h indicates the Memory Subsystem Controller Product ID is unknown" " A value of 0000h indicates the Memory Subsystem Controller Manufacturer ID is unknown" " A value of 0000h indicates the Module Product ID is unknown" The FSP code have not yet been upgraded to 3.2 to popolate those memory fields appropriately. Hence their "unknown" 0 values.
That's in the past, see above "I *was* mosly referring". You fixed that already, I know.