Patch Set 1:

(1 comment)

Sorry for the revert. Please, split out the functional
changes (e.g. the cpuid stuff around core/thread count)
into their own patch (it's not related to the SMBIOS
update, is it?). This change does a lot of things, and
I haven't spotted anything yet that is *required* for

From the 3.2 spec, core counts and thread counts don't seem to be optional.
The newly added procedure is populating them to comply to one of your previous comment:
" please don't add fields without giving them a reasonable value"
I can remove it from this patch if you require it and set the values to 0. Because the count generated by the code before this upgrade was not accurate, at least not on the GKL Sparky platform.


View Change

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1171c20ac85a2231d949d4ac1c3e3c544ba3e5ef
Gerrit-Change-Number: 32090
Gerrit-PatchSet: 5
Gerrit-Owner: Francois Toguo Fotso <>
Gerrit-Reviewer: Francois Toguo Fotso <>
Gerrit-Reviewer: Nico Huber <>
Gerrit-Reviewer: Patrick Georgi <>
Gerrit-Reviewer: Patrick Rudolph <>
Gerrit-Reviewer: Paul Menzel <>
Gerrit-Reviewer: build bot (Jenkins) <>
Gerrit-CC: Francois Toguo Fotso <>
Gerrit-CC: Lijian Zhao <>
Gerrit-CC: Patrick Rudolph <>
Gerrit-Comment-Date: Thu, 28 Mar 2019 21:50:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment