RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 04:47:35 +00:00
Code Issues Activity Actions

Profiler: Disassemble the containing function for non-function symbols

Browse source 
This can happen if the symbol is part of a switch-case, and not
a function, which would previously have made the disassembly view
appear empty.
Now we disassemble the containing function, starting at the given label
and continuing up until the last captured instruction.
This commit is contained in:
Ali Mohammad Pur 2021-08-08 04:32:28 +04:30 • committed by Ali Mohammad Pur
parent 4bef63fa6a
commit 2128ae4ab0
1 changed files with 20 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

24
Userland/DevTools/Profiler/DisassemblyModel.cpp
View file

@ -64,26 +64,42 @@ DisassemblyModel::DisassemblyModel(Profile& profile, ProfileNode& node)
VERIFY(elf != nullptr);
auto symbol = elf->find_symbol(node.address() - base_address);
FlatPtr function_address = node.address() - base_address;
Debug::DebugInfo debug_info { *elf, {}, base_address };
auto function = debug_info.get_containing_function(function_address);
if (function.has_value())
function_address = function->address_low;
else
dbgln("DisassemblyModel: Function containing {:p} ({}) not found", node.address() - base_address, node.symbol());
auto symbol = elf->find_symbol(function_address);
if (!symbol.has_value()) {
dbgln("DisassemblyModel: symbol not found");
return;
}
VERIFY(symbol.has_value());
auto view = symbol.value().raw_data();
auto symbol_offset_from_function_start = node.address() - base_address - symbol->value();
auto view = symbol.value().raw_data().substring_view(symbol_offset_from_function_start);
X86::ELFSymbolProvider symbol_provider(*elf);
X86::SimpleInstructionStream stream((const u8*)view.characters_without_null_termination(), view.length());
X86::Disassembler disassembler(stream);
Debug::DebugInfo debug_info { *elf, {}, base_address };
size_t offset_into_symbol = 0;
FlatPtr last_instruction_address = 0;
for (auto& event : node.events_per_address())
last_instruction_address = max(event.key, last_instruction_address);
auto last_instruction_offset = last_instruction_address - node.address();
for (;;) {
if (offset_into_symbol > last_instruction_offset)
break;
auto insn = disassembler.next();
if (!insn.has_value())
break;
FlatPtr address_in_profiled_program = base_address + symbol.value().value() + offset_into_symbol;
FlatPtr address_in_profiled_program = node.address() + offset_into_symbol;
auto disassembly = insn.value().to_string(address_in_profiled_program, &symbol_provider);