Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ArmPkg/ProcessorSubClassDxe: Limit CoreCount to 0xFF #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jbrasen
Copy link
Contributor

@jbrasen jbrasen commented Jan 31, 2024

The CoreCount and EnabledCore counts should be set to 0xFF if value is
greater than 255 per the SMBIOS specification.

Change-Id: I8e3e6e519d4444575a4573d24ac0d291aec532f0 Reviewed-on: https://git-master.nvidia.com/r/c/3rdparty/edk2/+/2828322
Reviewed-by: svcacv [email protected]
Reviewed-by: Nick Ramirez [email protected]
GVS: Gerrit_Virtual_Submit [email protected]
(cherry picked from commit 2a5998a59dada5d04824ce28f8d039c3ba5a6a30) Reviewed-on: https://git-master.nvidia.com/r/c/3rdparty/edk2/+/2832413
Reviewed-by: svc-sw-mobile-l4t [email protected]
Reviewed-by: Ashish Singhal [email protected]
Tested-by: Ashish Singhal [email protected]

@jbrasen jbrasen added Pending EDK2 Review EDK2 maintainer review needed Pending EDK2 merge Change is not in tianocore/master Pending EDK2 stable release Change is not in most recent edk2-stable tag In NVIDIA repo Changes are part of our builds prior to stable tag labels Jan 31, 2024
 The CoreCount, EnabledCore and ThreadCount counts should be set to 0xFF
 if value is greater than 255 per the SMBIOS specification.

Signed-off-by: Jeff Brasen <[email protected]>
Change-Id: I8e3e6e519d4444575a4573d24ac0d291aec532f0
Reviewed-on: https://git-master.nvidia.com/r/c/3rdparty/edk2/+/2828322
Reviewed-by: svcacv <[email protected]>
Reviewed-by: Nick Ramirez <[email protected]>
GVS: Gerrit_Virtual_Submit <[email protected]>
(cherry picked from commit 2a5998a59dada5d04824ce28f8d039c3ba5a6a30)
Reviewed-on: https://git-master.nvidia.com/r/c/3rdparty/edk2/+/2832413
Reviewed-by: svc-sw-mobile-l4t <[email protected]>
Reviewed-by: Ashish Singhal <[email protected]>
Tested-by: Ashish Singhal <[email protected]>
@jbrasen jbrasen force-pushed the Upstream/ProcessorCoreCount branch from 944835f to 4268474 Compare February 6, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In NVIDIA repo Changes are part of our builds prior to stable tag Pending EDK2 merge Change is not in tianocore/master Pending EDK2 Review EDK2 maintainer review needed Pending EDK2 stable release Change is not in most recent edk2-stable tag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant