[coreboot-gerrit] [S] Change in coreboot[main]: libpayload: Add dma_allocator_range()

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/78407?usp=email )

Change subject: libpayload: Add dma_allocator_range()
......................................................................

libpayload: Add dma_allocator_range()

Some sensitive data may remain DMA buffer, we will want to zero out
everything on the DMA buffer before we jump into the kernel to
prevent leaking sensitive data into the kernel.

To accomplish that, we will need this function to get the range of
memory that can be allocated by the dma allocator.

BUG=b:248610274
TEST=emerge-cherry libpayload
BRANCH=none

Signed-off-by: Yi Chou 
Change-Id: I8f3058dfd861ed44f716623967201b8cabe8d166
Reviewed-on: https://review.coreboot.org/c/coreboot/+/78407
Tested-by: build bot (Jenkins) 
Reviewed-by: Yu-Ping Wu 
---
M payloads/libpayload/include/stdlib.h
M payloads/libpayload/libc/malloc.c
2 files changed, 13 insertions(+), 0 deletions(-)

Approvals:
  Yu-Ping Wu: Looks good to me, approved
  build bot (Jenkins): Verified

diff --git a/payloads/libpayload/include/stdlib.h b/payloads/libpayload/include/stdlib.h
index e129cd1..cb9addf 100644
--- a/payloads/libpayload/include/stdlib.h
+++ b/payloads/libpayload/include/stdlib.h
@@ -137,6 +137,7 @@
 void init_dma_memory(void *start, u32 size);
 int dma_initialized(void);
 int dma_coherent(const void *ptr);
+void dma_allocator_range(void **start_out, size_t *size_out);

static inline void *xmalloc_work(size_t size, const char *file,
    			 const char *func, int line)
diff --git a/payloads/libpayload/libc/malloc.c b/payloads/libpayload/libc/malloc.c
index 1fc2ef1..f8a9ddf 100644
--- a/payloads/libpayload/libc/malloc.c
+++ b/payloads/libpayload/libc/malloc.c
@@ -123,6 +123,18 @@
    return !dma_initialized() || (dma->start <= ptr && dma->end > ptr);
 }

+/* Get the range of memory that can be allocated by the dma allocator. */
+void dma_allocator_range(void **start_out, size_t *size_out)
+{
+	if (dma_initialized()) {
+		*start_out = dma->start;
+		*size_out = dma->end - dma->start;
+	} else {
+		*start_out = NULL;
+		*size_out = 0;
+	}
+}
+
 /* Find free block of size >= len */
 static hdrtype_t volatile *find_free_block(int len, struct memory_type *type)
 {