[coreboot-gerrit] Change in coreboot[master]: arch/x86: Implement RESET_VECTOR_IN_RAM

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35035 )

Change subject: arch/x86: Implement RESET_VECTOR_IN_RAM
......................................................................

Patch Set 22:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35035/21/src/arch/x86/memlayout.ld 
File src/arch/x86/memlayout.ld:

https://review.coreboot.org/c/coreboot/+/35035/21/src/arch/x86/memlayout.ld@... 
PS21, Line 56: #if !CONFIG(RESET_VECTOR_IN_RAM)
> I have started a patch series here to move id completely out of .ld: https://review.coreboot. […]

A twoliner fix for this was suggested as solution in some previous patchset, together with some cases of external utilities and payloads that depend of current id pointer location near the reset vector. I pushed untested CB:40583 based on that.

The guard used here is also logically wrong, although in this patch series RESET_VECTOR_IN_RAM was the trigger.

Related to x86 bootblock, I am continuing CB:37895.