[coreboot-gerrit] Change in coreboot[master]: [RFC]cpu/x86/mp_init.c: Copy BSP chip_info to AP devices

[Angel Pons (Code Review)]

Attention is currently required from: Arthur Heymans, Raul Rangel, Nico Huber.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58603 )

Change subject: [RFC]cpu/x86/mp_init.c: Copy BSP chip_info to AP devices
......................................................................

Patch Set 1:

(1 comment)

File src/cpu/x86/mp_init.c:

https://review.coreboot.org/c/coreboot/+/58603/comment/8d4c06d8_636a1207 
PS1, Line 392: new->chip_info = info->cpu->chip_info;
> I was more wondering if you should check `new->chip_info == NULL` before overriding the value.

I analyzed the cases where `new->chip_info` would be non-null, and concluded that it's very unusual and unlikely to happen anytime soon. I think adding the check is a good idea, and it wouldn't make a difference most (if not all) cases.