Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47173 )
Change subject: soc/intel/xeon_sp: Don't add MC resource twice ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
In this case the values are actually coming from FSP, and not directly read from hardware. They are somewhat arbitrarily tied to the MC device (of which there is one per CPU) because it makes sense to have memory sizing be owned by the memory controller. But in this instance there is no actual hardware touched.
I don't have the specs to this memory controller in front of me, but normally these sorts of systems have to have each processor configured with the same mappings (that is what we used to do on the original AMD64 multi processor systems, too)
https://review.coreboot.org/c/coreboot/+/47173/1/src/soc/intel/xeon_sp/uncor... File src/soc/intel/xeon_sp/uncore.c:
https://review.coreboot.org/c/coreboot/+/47173/1/src/soc/intel/xeon_sp/uncor... PS1, Line 186: meomoy memory