[coreboot-gerrit] New patch to review for coreboot: arch/x86: Reserve space for bootblock's stack in CAR layout

Andrey Petrov (andrey.petrov@intel.com) gerrit at coreboot.org
Tue Feb 9 03:26:18 CET 2016


Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13640

-gerrit

commit d5fb8bba753b42afdfabadb7628803b4dfde6063
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Mon Feb 8 17:17:05 2016 -0800

    arch/x86: Reserve space for bootblock's stack in CAR layout
    
    If bootblock uses CAR memory for stack it must be explicitly reserved
    in CAR memory layout. This can be used to track resource usage and
    enforce policy.
    
    Change-Id: I8dec768cf8763dbe235f0ba1339079ebc49cbd9a
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/arch/x86/car.ld | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/arch/x86/car.ld b/src/arch/x86/car.ld
index 191dcaf..6905cff 100644
--- a/src/arch/x86/car.ld
+++ b/src/arch/x86/car.ld
@@ -41,6 +41,11 @@
 	 * variables that are not shared like the timestamp and the pre-ram
 	 * cbmem console. This is useful for clearing this area on a per-stage
 	 * basis when more than one stage uses cache-as-ram for CAR_GLOBALs. */
+#if IS_ENABLED(CONFIG_C_ENVIRONMENT_BOOTBLOCK)
+	_boot_loader_stack_start = .;
+	. += CONFIG_DCACHE_RAM_BOOTBLOCK_STACK_SIZE;
+	_boot_loader_stack_end = .;
+#endif
 	_car_global_start = .;
 	*(.car.global_data);
 	. = ALIGN(ARCH_POINTER_ALIGN_SIZE);



More information about the coreboot-gerrit mailing list