[PATCH] scripts/gdb: De-reference per-CPU MCE interrupts

Florian Fainelli posted 1 patch 3 months, 2 weeks ago
There is a newer version of this series
scripts/gdb/linux/interrupts.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] scripts/gdb: De-reference per-CPU MCE interrupts
Posted by Florian Fainelli 3 months, 2 weeks ago
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
Re: [PATCH] scripts/gdb: De-reference per-CPU MCE interrupts
Posted by Andrew Morton 3 months, 2 weeks ago
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)
Re: [PATCH] scripts/gdb: De-reference per-CPU MCE interrupts
Posted by Florian Fainelli 3 months, 2 weeks ago

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