Change in coreboot[master]: soc/intel/common: Do bounds check on CPU core/thread number

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Anil Kumar K, Selma Bensaid, Paul Menzel, Patrick Rudolph.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61945 )

Change subject: soc/intel/common: Do bounds check on CPU core/thread number
......................................................................

Patch Set 7:

(1 comment)

Patchset:

PS7:
> Hi Tim. i added this assert at the root of problem , where the MSR is being read. […]

I understand what you're saying, but I think the real root is array access without bounds checking. Array accesses "have" to be bounds-checked in C, therefore the array accesses are where this should happen. I have no idea if all the hardware that this function is run on has CONFIG_MAX_CPUS already set up correctly to avoid hanging here.