Change in coreboot[master]: soc/intel/cannonlake: set LT_LOCK_MEMORY at end of POST

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36356 )

Change subject: soc/intel/cannonlake: set LT_LOCK_MEMORY at end of POST
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36356/2/src/soc/intel/cannonlake/fi... 
File src/soc/intel/cannonlake/finalize.c:

https://review.coreboot.org/c/coreboot/+/36356/2/src/soc/intel/cannonlake/fi... 
PS2, Line 116: mp_run_on_all_cpus
> given the scope of LT_LOCK_MEMORY (package), this seems unnecessary

There is no official/public documentation on this. Multiple leaks set it per-thread, FSP sets it once per package. Experiments at time of writing the patches showed, that setting it per package does not lock TSEGMB and PAVPC properly but only settig it per-thread does.

I'd be more than happy to find out what the reason for this behaviour is, but atm this seems to be a valid workaround, that doesn't hurt much.