Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33174 )
Change subject: nb/intel/sandybridge/mrc.bin: Increase the CAR size ......................................................................
Patch Set 5:
There is a check in the r6-binary (at offset 0x00002542) which checks if the signed difference between the (stack location - the heap location (0xff7e1000) ) is below 1000, for which there is seamingly no reason. This is always true as long as the stack is located above the heap and very likely untrue if the stack is located below the heap. A solution that works is just to skip bailing out (jg -> jmp below cmp). How are we on binary patching that binary? It's literally one byte that needs to be changed.