| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20260117052849.2205545-12-irogers@google.com>
Date: Fri, 16 Jan 2026 21:28:37 -0800
From: Ian Rogers <irogers@...gle.com>
To: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>, Namhyung Kim <namhyung@...nel.org>, Jiri Olsa <jolsa@...nel.org>,
Ian Rogers <irogers@...gle.com>, Adrian Hunter <adrian.hunter@...el.com>,
James Clark <james.clark@...aro.org>, John Garry <john.g.garry@...cle.com>,
Will Deacon <will@...nel.org>, Leo Yan <leo.yan@...ux.dev>, Guo Ren <guoren@...nel.org>,
Paul Walmsley <pjw@...nel.org>, Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>,
Alexandre Ghiti <alex@...ti.fr>, Shimin Guo <shimin.guo@...dio.com>,
Athira Rajeev <atrajeev@...ux.ibm.com>, Stephen Brennan <stephen.s.brennan@...cle.com>,
Howard Chu <howardchu95@...il.com>, Thomas Falcon <thomas.falcon@...el.com>,
Andi Kleen <ak@...ux.intel.com>, "Dr. David Alan Gilbert" <linux@...blig.org>,
Dmitry Vyukov <dvyukov@...gle.com>,
"Krzysztof Łopatowski" <krzysztof.m.lopatowski@...il.com>, Chun-Tse Shao <ctshao@...gle.com>,
Aditya Bodkhe <aditya.b1@...ux.ibm.com>, Haibo Xu <haibo1.xu@...el.com>,
Sergei Trofimovich <slyich@...il.com>, linux-kernel@...r.kernel.org,
linux-perf-users@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-csky@...r.kernel.org, linux-riscv@...ts.infradead.org,
Mark Wielaard <mark@...mp.org>
Subject: [PATCH v1 11/23] perf dwarf-regs: Clean up x86 dwarf_regnum code
The i386 and x86-64 register numbers differ on x86, but previously x86
was a single arch string and so this couldn't be handled. The
transition to using ELF EM_* values means we can translate x86
registers correctly for either the x86-64 dwarf register mappings
(from the System V ABI) or i386 register mappings. Correct the mappings.
Signed-off-by: Ian Rogers <irogers@...gle.com>
---
.../util/dwarf-regs-arch/dwarf-regs-x86.c | 133 ++++++++++++++++--
tools/perf/util/dwarf-regs.c | 5 +-
tools/perf/util/include/dwarf-regs.h | 3 +-
3 files changed, 129 insertions(+), 12 deletions(-)
diff --git a/tools/perf/util/dwarf-regs-arch/dwarf-regs-x86.c b/tools/perf/util/dwarf-regs-arch/dwarf-regs-x86.c
index 7a55c65e8da6..f0c42e4d7423 100644
--- a/tools/perf/util/dwarf-regs-arch/dwarf-regs-x86.c
+++ b/tools/perf/util/dwarf-regs-arch/dwarf-regs-x86.c
@@ -13,10 +13,65 @@
struct dwarf_regs_idx {
const char *name;
- int idx;
+ int dwarf_regnum;
};
-static const struct dwarf_regs_idx x86_regidx_table[] = {
+static const struct dwarf_regs_idx i386_regidx_table[] = {
+ { "eax", 0 }, { "ax", 0 }, { "al", 0 },
+ { "ecx", 1 }, { "cx", 1 }, { "cl", 1 },
+ { "edx", 2 }, { "dx", 2 }, { "dl", 2 },
+ { "ebx", 3 }, { "bx", 3 }, { "bl", 3 },
+ { "esp", 4 }, { "sp", 4 }, { "$stack", 4},
+ { "ebp", 5 }, { "bp", 5 },
+ { "esi", 6 }, { "si", 6 },
+ { "edi", 7 }, { "di", 7 },
+ // 8 - Return Address RA
+ { "eflags", 9}, { "flags", 9},
+ // 10 - reserved
+ { "st0", 11},
+ { "st1", 12},
+ { "st2", 13},
+ { "st3", 14},
+ { "st4", 15},
+ { "st5", 16},
+ { "st6", 17},
+ { "st7", 18},
+ // 19-20 - reserved
+ { "xmm0", 21},
+ { "xmm1", 22},
+ { "xmm2", 23},
+ { "xmm3", 24},
+ { "xmm4", 25},
+ { "xmm5", 26},
+ { "xmm6", 27},
+ { "xmm7", 28},
+ { "mm0", 29},
+ { "mm1", 30},
+ { "mm2", 31},
+ { "mm3", 32},
+ { "mm4", 33},
+ { "mm5", 34},
+ { "mm6", 35},
+ { "mm7", 36},
+ // 37-38 - unknown
+ { "mxcsr", 39}, // 128-bit Media Control and Status
+ { "es", 40},
+ { "cs", 41},
+ { "ss", 42},
+ { "ds", 43},
+ { "fs", 44},
+ { "gs", 45},
+ // 46-47 - reserved
+ { "tr", 48}, // Task Register
+ { "ldtr", 49}, // LDT Register
+ // 50-92 - reserved
+ { "fs.base", 92},
+ { "gs.base", 93},
+ // End of regular dwarf registers.
+ { "eip", DWARF_REG_PC }, { "ip", DWARF_REG_PC },
+};
+
+static const struct dwarf_regs_idx x86_64_regidx_table[] = {
{ "rax", 0 }, { "eax", 0 }, { "ax", 0 }, { "al", 0 },
{ "rdx", 1 }, { "edx", 1 }, { "dx", 1 }, { "dl", 1 },
{ "rcx", 2 }, { "ecx", 2 }, { "cx", 2 }, { "cl", 2 },
@@ -33,18 +88,78 @@ static const struct dwarf_regs_idx x86_regidx_table[] = {
{ "r13", 13 }, { "r13d", 13 }, { "r13w", 13 }, { "r13b", 13 },
{ "r14", 14 }, { "r14d", 14 }, { "r14w", 14 }, { "r14b", 14 },
{ "r15", 15 }, { "r15d", 15 }, { "r15w", 15 }, { "r15b", 15 },
- { "rip", DWARF_REG_PC },
+ // 16 - Return Address RA
+ { "xmm0", 17},
+ { "xmm1", 18},
+ { "xmm2", 19},
+ { "xmm3", 20},
+ { "xmm4", 21},
+ { "xmm5", 22},
+ { "xmm6", 23},
+ { "xmm7", 24},
+ { "xmm8", 25},
+ { "xmm9", 26},
+ { "xmm10", 27},
+ { "xmm11", 28},
+ { "xmm12", 29},
+ { "xmm13", 30},
+ { "xmm14", 31},
+ { "xmm15", 32},
+ { "st0", 33},
+ { "st1", 34},
+ { "st2", 35},
+ { "st3", 36},
+ { "st4", 37},
+ { "st5", 38},
+ { "st6", 39},
+ { "st7", 40},
+ { "mm0", 41},
+ { "mm1", 42},
+ { "mm2", 43},
+ { "mm3", 44},
+ { "mm4", 45},
+ { "mm5", 46},
+ { "mm6", 47},
+ { "mm7", 48},
+ { "rflags", 49}, { "eflags", 49}, { "flags", 49},
+ { "es", 50},
+ { "cs", 51},
+ { "ss", 52},
+ { "ds", 53},
+ { "fs", 54},
+ { "gs", 55},
+ // 56-47 - reserved
+ { "fs.base", 58},
+ { "gs.base", 59},
+ // 60-61 - reserved
+ { "tr", 62}, // Task Register
+ { "ldtr", 63}, // LDT Register
+ { "mxcsr", 64}, // 128-bit Media Control and Status
+ { "fcw", 65}, // x87 Control Word
+ { "fsw", 66}, // x87 Status Word
+ // End of regular dwarf registers.
+ { "rip", DWARF_REG_PC }, { "eip", DWARF_REG_PC }, { "ip", DWARF_REG_PC },
};
-int get_x86_regnum(const char *name)
+static int get_regnum(const struct dwarf_regs_idx *entries, size_t num_entries, const char *name)
{
- unsigned int i;
-
if (*name != '%')
return -EINVAL;
- for (i = 0; i < ARRAY_SIZE(x86_regidx_table); i++)
- if (!strcmp(x86_regidx_table[i].name, name + 1))
- return x86_regidx_table[i].idx;
+ name++;
+ for (size_t i = 0; i < num_entries; i++) {
+ if (!strcmp(entries[i].name, name))
+ return entries[i].dwarf_regnum;
+ }
return -ENOENT;
}
+
+int __get_dwarf_regnum_i386(const char *name)
+{
+ return get_regnum(i386_regidx_table, ARRAY_SIZE(i386_regidx_table), name);
+}
+
+int __get_dwarf_regnum_x86_64(const char *name)
+{
+ return get_regnum(x86_64_regidx_table, ARRAY_SIZE(x86_64_regidx_table), name);
+}
diff --git a/tools/perf/util/dwarf-regs.c b/tools/perf/util/dwarf-regs.c
index b2f37299147e..ef249dd589e3 100644
--- a/tools/perf/util/dwarf-regs.c
+++ b/tools/perf/util/dwarf-regs.c
@@ -92,9 +92,10 @@ int get_dwarf_regnum(const char *name, unsigned int machine, unsigned int flags
}
switch (machine) {
case EM_X86_64:
- fallthrough;
+ reg = __get_dwarf_regnum_x86_64(name);
+ break;
case EM_386:
- reg = get_x86_regnum(regname);
+ reg = __get_dwarf_regnum_i386(name);
break;
default:
pr_err("ELF MACHINE %x is not supported.\n", machine);
diff --git a/tools/perf/util/include/dwarf-regs.h b/tools/perf/util/include/dwarf-regs.h
index 015d1ade645f..bb5413b0fee4 100644
--- a/tools/perf/util/include/dwarf-regs.h
+++ b/tools/perf/util/include/dwarf-regs.h
@@ -99,7 +99,8 @@ const char *get_csky_regstr(unsigned int n, unsigned int flags);
*/
const char *get_dwarf_regstr(unsigned int n, unsigned int machine, unsigned int flags);
-int get_x86_regnum(const char *name);
+int __get_dwarf_regnum_i386(const char *name);
+int __get_dwarf_regnum_x86_64(const char *name);
/*
* get_dwarf_regnum - Returns DWARF regnum from register name
--
2.52.0.457.g6b5491de43-goog
Powered by blists - more mailing lists