[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 8/8] gdb: Get correct mod variable value
From: |
Glenn Washburn |
Subject: |
[PATCH 8/8] gdb: Get correct mod variable value |
Date: |
Sun, 13 Feb 2022 21:42:45 -0600 |
For some reason, GDB is breaking on grub_dl_add() before the function has
setup its stack frame, but GDB thinks it has. So the value of mod is bogus.
To get the correct value, create a one-time break on grub_dl_get(), which is
the first line of grub_dl_add(). When this break point hits, grub_dl_add()
will have finished setting up it stack frame. But at this point we will be
in grub_dl_get()'s stack frame. So go one frame up, which will be
grub_dl_add(), to get mod's value.
Signed-off-by: Glenn Washburn <development@efficientek.com>
---
grub-core/gdb_grub.in | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/grub-core/gdb_grub.in b/grub-core/gdb_grub.in
index 4c2b8bf52..a1031e58d 100644
--- a/grub-core/gdb_grub.in
+++ b/grub-core/gdb_grub.in
@@ -134,7 +134,16 @@ define runtime_load_module
break grub_dl_add
commands
silent
- load_module mod
+ # GDB has stopped before the call frame is setup, so mod does
+ # not have the correct value. Create a one-time break on the
+ # next function call and then go one frame up, back to the
+ # grub_dl_add frame, to get the correct value for mod.
+ tbreak grub_dl_get
+ commands
+ fr 1
+ load_module mod
+ cont
+ end
cont
end
end
--
2.27.0
- [PATCH 0/8] GDB script fixes and improvements, Glenn Washburn, 2022/02/13
- [PATCH 1/8] gdb: Move runtime module loading into runtime_load_module, Glenn Washburn, 2022/02/13
- [PATCH 2/8] gdb: If no modules have been loaded, do not try to load module symbols, Glenn Washburn, 2022/02/13
- [PATCH 3/8] gdb: Do not lazy load module symbols, Glenn Washburn, 2022/02/13
- [PATCH 4/8] gdb: Prevent wrapping when writing to .segments.tmp, Glenn Washburn, 2022/02/13
- [PATCH 5/8] gdb: Add functions to make loading from dynamically positioned targets easier, Glenn Washburn, 2022/02/13
- [PATCH 6/8] gdb: If enabled, print line used to load EFI kernel symbols when using gdb_grub script, Glenn Washburn, 2022/02/13
- [PATCH 7/8] gdb: Conditionally run GDB script logic for dynamically or statically positioned GRUB, Glenn Washburn, 2022/02/13
- [PATCH 8/8] gdb: Get correct mod variable value,
Glenn Washburn <=