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 7:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40523/5/src/soc/intel/xeon_sp/skx/c... File src/soc/intel/xeon_sp/skx/cpu.c:
https://review.coreboot.org/c/coreboot/+/40523/5/src/soc/intel/xeon_sp/skx/c... PS5, Line 75: /* If socket_index is 0 then all PPIN have been saved. */
You are right, but I can't find a way to only read the MSR on a core in the 2nd socket without runni […]
I found mp_run_on_aps() can run on the assigned CPU core only, after enabling config PARALLEL_MP_AP_WORK, mp_run_on_aps(function, NULL, x , 100 * USECS_PER_MSEC) can run function() on logical CPU #x only, so it should be more efficient and simpler. With CPU number being get_platform_thread_count() - 1, this core should always be in the last socket? I tried this method works.