[coreboot-gerrit] [M] Change in coreboot[master]: soc/intel/xeon_sp: Don't sort struct device cpus for numa

[Arthur Heymans (Code Review)]

Attention is currently required from: Lance Zhao, Arthur Heymans, Anjaneya "Reddy" Chagam, Jonathan Zhang, Johnny Lin, Tim Wawrzynczak, Christian Walter, Angel Pons, Shuming Chu (Shuming), TangYiwei.

Hello Lance Zhao, build bot (Jenkins), Anjaneya "Reddy" Chagam, Jonathan Zhang, Johnny Lin, Paul Menzel, Tim Wawrzynczak, Christian Walter, Angel Pons, TangYiwei,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/68912

to look at the new patch set (#29).

Change subject: soc/intel/xeon_sp: Don't sort struct device cpus for numa
......................................................................

soc/intel/xeon_sp: Don't sort struct device cpus for numa

Currently the xeon_sp code reassigns struct devices apic_id so that srat
entries can be added in a certain order.

This is not a good idea as it breaks thread local storage which contains
a pointer to its struct device cpu.

This moves the sorting of the lapic_ids to the srat table generation
and adds the numa node id in each core init entry. Now it is done in
parallel too as a bonus.

Change-Id: I372bcea1932d28e9bf712cc712f19a76fe3199b1
Signed-off-by: Arthur Heymans 
---
M src/soc/intel/xeon_sp/cpx/cpu.c
M src/soc/intel/xeon_sp/include/soc/util.h
M src/soc/intel/xeon_sp/skx/cpu.c
M src/soc/intel/xeon_sp/uncore_acpi.c
M src/soc/intel/xeon_sp/util.c
5 files changed, 59 insertions(+), 120 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/68912/29