[coreboot-gerrit] Patch set updated for coreboot: AGESA: Log heap initialisation

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Mar 8 06:42:49 CET 2017


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18535

-gerrit

commit 61e543e6f220820caa1c85fcff9dc145d27c0701
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Sun Nov 27 06:37:52 2016 +0200

    AGESA: Log heap initialisation
    
    This is useful for debugging S3 issues and in general
    to understand AGESA memory allocator behaviour.
    
    Change-Id: I422f2620ed0023f3920b8d2949ee1c33a6c227e0
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/cpu/amd/agesa/heapmanager.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/cpu/amd/agesa/heapmanager.c b/src/cpu/amd/agesa/heapmanager.c
index 9acd0bd..55f6b02 100644
--- a/src/cpu/amd/agesa/heapmanager.c
+++ b/src/cpu/amd/agesa/heapmanager.c
@@ -57,6 +57,9 @@ void EmptyHeap(void)
 {
 	void *base = GetHeapBase();
 	memset(base, 0, BIOS_HEAP_SIZE);
+
+	printk(BIOS_DEBUG, "Wiped HEAP at [%08x - %08x]\n",
+		(uint32_t)(uintptr_t) base, (uint32_t)(uintptr_t) base + BIOS_HEAP_SIZE - 1);
 }
 
 void ResumeHeap(void **heap, size_t *len)
@@ -64,6 +67,9 @@ void ResumeHeap(void **heap, size_t *len)
 	void *base = GetHeapBase();
 	*heap = base;
 	*len = BIOS_HEAP_SIZE;
+
+	printk(BIOS_DEBUG, "Using resume HEAP at [%08x - %08x]\n",
+		(uint32_t)(uintptr_t) base, (uint32_t)(uintptr_t) base + BIOS_HEAP_SIZE - 1);
 }
 
 #if (IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY15_TN) || \



More information about the coreboot-gerrit mailing list