ldr: Simplify memory copy within LoadNro()
We can use the dedicated memory function for performing copies instead of reading into a temporary buffer and then immediately writing it back out to memory. Eliminates a bit of heap memory churn.
This commit is contained in:
parent
860d73637e
commit
8f638e81e9
|
@ -387,11 +387,9 @@ public:
|
|||
const VAddr bss_end_addr{
|
||||
Common::AlignUp(bss_start + nro_header.bss_size, Kernel::PageSize)};
|
||||
|
||||
auto CopyCode{[&](VAddr src_addr, VAddr dst_addr, u64 size) {
|
||||
std::vector<u8> source_data(size);
|
||||
system.Memory().ReadBlock(src_addr, source_data.data(), source_data.size());
|
||||
system.Memory().WriteBlock(dst_addr, source_data.data(), source_data.size());
|
||||
}};
|
||||
const auto CopyCode = [this, process](VAddr src_addr, VAddr dst_addr, u64 size) {
|
||||
system.Memory().CopyBlock(*process, dst_addr, src_addr, size);
|
||||
};
|
||||
CopyCode(nro_addr + nro_header.segment_headers[TEXT_INDEX].memory_offset, text_start,
|
||||
nro_header.segment_headers[TEXT_INDEX].memory_size);
|
||||
CopyCode(nro_addr + nro_header.segment_headers[RO_INDEX].memory_offset, ro_start,
|
||||
|
|
Reference in New Issue