Kyösti Mälkki has uploaded this change for review.

View Change

[WIP] nb/intel/sandybridge: Reserve CAR region with !NATIVE_RAMINIT

Fail builds if MRC blobs pool heap would get corrupted
by CAR relocatable data from coreboot proper.

Patch the blob instead:
https://review.coreboot.org/c/blobs/+/31149

Change-Id: Ibc771b592b35d77be81fce87769314fe6bb84c87
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
---
M src/arch/x86/car.ld
1 file changed, 7 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/31150/1
diff --git a/src/arch/x86/car.ld b/src/arch/x86/car.ld
index 7b10f43..fe55071 100644
--- a/src/arch/x86/car.ld
+++ b/src/arch/x86/car.ld
@@ -89,6 +89,13 @@
_car_global_end = .;
_car_relocatable_data_end = .;

+#if IS_ENABLED(CONFIG_NORTHBRIDGE_INTEL_SANDYBRIDGE) && !IS_ENABLED(CONFIG_USE_NATIVE_RAMINIT)
+ . = ABSOLUTE(0xff7e1000);
+ _mrc_var_heap_start = .;
+ . += 0x3000;
+ _mrc_var_heap_end = .;
+#endif
+
#if !IS_ENABLED(CONFIG_C_ENVIRONMENT_BOOTBLOCK)
_car_stack_start = .;
_car_stack_end = _car_region_end;

To view, visit change 31150. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibc771b592b35d77be81fce87769314fe6bb84c87
Gerrit-Change-Number: 31150
Gerrit-PatchSet: 1
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-MessageType: newchange