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

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Tue Mar 7 08:39:29 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 25a05015b34570360705976f60967e3404a6c913
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 087959d..32134e7 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