[coreboot-gerrit] Change in coreboot[master]: cpu/x86: Make MP init timeout configurable

[David Hendricks (Code Review)]

David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38546 )

Change subject: cpu/x86: Make MP init timeout configurable
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38546/1/src/cpu/x86/mp_init.c 
File src/cpu/x86/mp_init.c:

https://review.coreboot.org/c/coreboot/+/38546/1/src/cpu/x86/mp_init.c@525 
PS1, Line 525: 	const int timeout_us = CONFIG_FLIGHT_PLAN_TIMEOUT_US;
> It is oaky for me to simply increase the timeout for all targets, if this is fine for the community. […]

We have num_aps defined below, maybe we can use that as a multiplier? So for example, 100ms per AP with a minimum of 1s:
int num_aps = mp_params->num_cpus - 1;
int timeout_us = min(1000000, 100000 * num_aps);

That removes the need for yet another Kconfig variable and is also consistent with what the comment above suggests about time potentially increasing due to contention of resources such as UART.