Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40523 )
Change subject: soc/xeon_sp: Read PPIN MSR and save to an array for each CPU
......................................................................
Patch Set 1:
Reading MSR for a remote CPU socket seems not available in coreboot, so piggyback on xeon_sp_core_init() that gets to run on each CPU core and read their MSRs. May not be the best way
to do it, maybe rely on FSP to report PPIN via HOB is another option.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/40523
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5e1de8bcb651fb8ae8b106db1978235b0dd84c47
Gerrit-Change-Number: 40523
Gerrit-PatchSet: 1
Gerrit-Owner: Johnny Lin
Gerrit-Reviewer: Andrey Petrov
anpetrov@fb.com
Gerrit-Reviewer: David Hendricks
david.hendricks@gmail.com
Gerrit-Reviewer: Jingle Hsu
Gerrit-Reviewer: Jonathan Zhang
jonzhang@fb.com
Gerrit-Reviewer: Morgan Jang
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Comment-Date: Mon, 20 Apr 2020 11:42:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment