scripts/gdb/linux/interrupts.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
The per-CPU MCE interrupts are looked up by reference and need to be
de-referenced before printing.
Fixes: b0969d7687a7 ("scripts/gdb: print interrupts")
Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>
---
scripts/gdb/linux/interrupts.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/gdb/linux/interrupts.py b/scripts/gdb/linux/interrupts.py
index 199d9e8193f4..8bd0d7108ae0 100644
--- a/scripts/gdb/linux/interrupts.py
+++ b/scripts/gdb/linux/interrupts.py
@@ -110,7 +110,7 @@ def x86_show_mce(prec, var, pfx, desc):
pvar = gdb.parse_and_eval(var)
text = "%*s: " % (prec, pfx)
for cpu in cpus.each_online_cpu():
- text += "%10u " % (cpus.per_cpu(pvar, cpu))
+ text += "%10u " % (cpus.per_cpu(pvar, cpu).dereference())
text += " %s\n" % (desc)
return text
--
2.43.0
On Mon, 23 Jun 2025 20:00:19 -0700 Florian Fainelli <florian.fainelli@broadcom.com> wrote: > The per-CPU MCE interrupts are looked up by reference and need to be > de-referenced before printing. Again, please fully describe the problem which is being addressed. (OK, I can see it's "prints wrong stuff", but something like example before-and-after output would be very helpful)
On 6/24/2025 4:53 PM, Andrew Morton wrote: > On Mon, 23 Jun 2025 20:00:19 -0700 Florian Fainelli <florian.fainelli@broadcom.com> wrote: > >> The per-CPU MCE interrupts are looked up by reference and need to be >> de-referenced before printing. > > Again, please fully describe the problem which is being addressed. > > (OK, I can see it's "prints wrong stuff", but something like example > before-and-after output would be very helpful) Sure, I thought it was clear enough we were printing the variable address rather than its content, but I will make that clear in a v2. Thanks! -- Florian
© 2016 - 2025 Red Hat, Inc.