History

scripts/faddr2line: Skip over mapping symbols in output from readelf

Mapping symbols emitted in the readelf output can confuse the 'faddr2line' symbol size calculation, resulting in the erroneous rejection of valid offsets. This is especially prevalent when building an arm64 kernel with CONFIG_CFI_CLANG=y, where most functions are prefixed with a 32-bit data value in a '$d.n' section. For example: 447538: ffff800080014b80 548 FUNC GLOBAL DEFAULT 2 do_one_initcall 104: ffff800080014c74 0 NOTYPE LOCAL DEFAULT 2 $x.73 106: ffff800080014d30 0 NOTYPE LOCAL DEFAULT 2 $x.75 111: ffff800080014da4 0 NOTYPE LOCAL DEFAULT 2 $d.78 112: ffff800080014da8 0 NOTYPE LOCAL DEFAULT 2 $x.79 36: ffff800080014de0 200 FUNC LOCAL DEFAULT 2 run_init_process Adding a warning to do_one_initcall() results in: | WARNING: CPU: 0 PID: 1 at init/main.c:1236 do_one_initcall+0xf4/0x260 Which 'faddr2line' refuses to accept: $ ./scripts/faddr2line vmlinux do_one_initcall+0xf4/0x260 skipping do_one_initcall address at 0xffff800080014c74 due to size mismatch (0x260 != 0x224) no match for do_one_initcall+0xf4/0x260 Filter out these entries from readelf using a shell reimplementation of is_mapping_symbol(), so that the size of a symbol is calculated as a delta to the next symbol present in ksymtab.

Will Deacon committed 3 years ago

Commit 60fd39a1 parent 86bf86e1 file · +5 −0Browse files

Showing the changes to scripts (1 of 1 file). Show all

Unified · Split

scripts/faddr2line+5−0
@@ -179,6 +179,11 @@ __faddr2line() { · 178 unmodified lines
179179 local cur_sym_elf_size=${fields[2]}
180180 local cur_sym_name=${fields[7]:-}
181181
182+ # is_mapping_symbol(cur_sym_name)
183+ if [[ ${cur_sym_name} =~ ^(\.L|L0|\$) ]]; then
184+ continue
185+ fi
186+
182187 if [[ $cur_sym_addr = $sym_addr ]] &&
183188 [[ $cur_sym_elf_size = $sym_elf_size ]] &&
184189 [[ $cur_sym_name = $sym_name ]]; then