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.

Nico,
The fix for the existing field have been removed.
Please take a look.

View Change

To view, visit change 32090. To unsubscribe, or for help writing mail filters, visit settings.