|
 |
ed1787d |
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
ed1787d |
From: Daniel Axtens <dja@axtens.net>
|
|
|
ed1787d |
Date: Thu, 21 Apr 2022 15:24:16 +1000
|
|
|
ed1787d |
Subject: [PATCH] mm: Debug support for region operations
|
|
|
ed1787d |
|
|
|
ed1787d |
This is handy for debugging. Enable with "set debug=regions".
|
|
|
ed1787d |
|
|
|
ed1787d |
Signed-off-by: Daniel Axtens <dja@axtens.net>
|
|
|
ed1787d |
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
|
|
|
ed1787d |
Tested-by: Patrick Steinhardt <ps@pks.im>
|
|
|
ed1787d |
(cherry picked from commit 8afa5ef45b797ba5d8147ceee85ac2c59dcc7f09)
|
|
|
ed1787d |
---
|
|
|
ed1787d |
grub-core/kern/mm.c | 19 ++++++++++++++++---
|
|
|
ed1787d |
1 file changed, 16 insertions(+), 3 deletions(-)
|
|
|
ed1787d |
|
|
|
ed1787d |
diff --git a/grub-core/kern/mm.c b/grub-core/kern/mm.c
|
|
|
ed1787d |
index 7be33e23bf..38bfb01df9 100644
|
|
|
ed1787d |
--- a/grub-core/kern/mm.c
|
|
|
ed1787d |
+++ b/grub-core/kern/mm.c
|
|
|
ed1787d |
@@ -115,9 +115,8 @@ grub_mm_init_region (void *addr, grub_size_t size)
|
|
|
ed1787d |
grub_mm_header_t h;
|
|
|
ed1787d |
grub_mm_region_t r, *p, q;
|
|
|
ed1787d |
|
|
|
ed1787d |
-#if 0
|
|
|
ed1787d |
- grub_printf ("Using memory for heap: start=%p, end=%p\n", addr, addr + (unsigned int) size);
|
|
|
ed1787d |
-#endif
|
|
|
ed1787d |
+ grub_dprintf ("regions", "Using memory for heap: start=%p, end=%p\n",
|
|
|
ed1787d |
+ addr, (char *) addr + (unsigned int) size);
|
|
|
ed1787d |
|
|
|
ed1787d |
/* Exclude last 4K to avoid overflows. */
|
|
|
ed1787d |
/* If addr + 0x1000 overflows then whole region is in excluded zone. */
|
|
|
ed1787d |
@@ -142,8 +141,14 @@ grub_mm_init_region (void *addr, grub_size_t size)
|
|
|
ed1787d |
* addr q
|
|
|
ed1787d |
* |----size-----|-q->pre_size-|<q region>|
|
|
|
ed1787d |
*/
|
|
|
ed1787d |
+ grub_dprintf ("regions", "Can we extend into region above?"
|
|
|
ed1787d |
+ " %p + %" PRIxGRUB_SIZE " + %" PRIxGRUB_SIZE " ?=? %p\n",
|
|
|
ed1787d |
+ (grub_uint8_t *) addr, size, q->pre_size, (grub_uint8_t *) q);
|
|
|
ed1787d |
if ((grub_uint8_t *) addr + size + q->pre_size == (grub_uint8_t *) q)
|
|
|
ed1787d |
{
|
|
|
ed1787d |
+ grub_dprintf ("regions", "Yes: extending a region: (%p -> %p) -> (%p -> %p)\n",
|
|
|
ed1787d |
+ q, (grub_uint8_t *) q + sizeof (*q) + q->size,
|
|
|
ed1787d |
+ addr, (grub_uint8_t *) q + sizeof (*q) + q->size);
|
|
|
ed1787d |
/*
|
|
|
ed1787d |
* Yes, we can merge the memory starting at addr into the
|
|
|
ed1787d |
* existing region from below. Align up addr to GRUB_MM_ALIGN
|
|
|
ed1787d |
@@ -185,9 +190,15 @@ grub_mm_init_region (void *addr, grub_size_t size)
|
|
|
ed1787d |
* q addr
|
|
|
ed1787d |
* |<q region>|-q->post_size-|----size-----|
|
|
|
ed1787d |
*/
|
|
|
ed1787d |
+ grub_dprintf ("regions", "Can we extend into region below?"
|
|
|
ed1787d |
+ " %p + %" PRIxGRUB_SIZE " + %" PRIxGRUB_SIZE " + %" PRIxGRUB_SIZE " ?=? %p\n",
|
|
|
ed1787d |
+ (grub_uint8_t *) q, sizeof(*q), q->size, q->post_size, (grub_uint8_t *) addr);
|
|
|
ed1787d |
if ((grub_uint8_t *) q + sizeof (*q) + q->size + q->post_size ==
|
|
|
ed1787d |
(grub_uint8_t *) addr)
|
|
|
ed1787d |
{
|
|
|
ed1787d |
+ grub_dprintf ("regions", "Yes: extending a region: (%p -> %p) -> (%p -> %p)\n",
|
|
|
ed1787d |
+ q, (grub_uint8_t *) q + sizeof (*q) + q->size,
|
|
|
ed1787d |
+ q, (grub_uint8_t *) addr + size);
|
|
|
ed1787d |
/*
|
|
|
ed1787d |
* Yes! Follow a similar pattern to above, but simpler.
|
|
|
ed1787d |
* Our header starts at address - post_size, which should align us
|
|
|
ed1787d |
@@ -213,6 +224,8 @@ grub_mm_init_region (void *addr, grub_size_t size)
|
|
|
ed1787d |
}
|
|
|
ed1787d |
}
|
|
|
ed1787d |
|
|
|
ed1787d |
+ grub_dprintf ("regions", "No: considering a new region at %p of size %" PRIxGRUB_SIZE "\n",
|
|
|
ed1787d |
+ addr, size);
|
|
|
ed1787d |
/* Allocate a region from the head. */
|
|
|
ed1787d |
r = (grub_mm_region_t) ALIGN_UP ((grub_addr_t) addr, GRUB_MM_ALIGN);
|
|
|
ed1787d |
|