mirror of
				https://github.com/PabloMK7/citra.git
				synced 2025-10-31 05:40:04 +00:00 
			
		
		
		
	memory: add overload with process parameter for ZeroBlock and CopyBlock
This commit is contained in:
		
							parent
							
								
									de8dbbf0b6
								
							
						
					
					
						commit
						8e4e663113
					
				
					 2 changed files with 38 additions and 24 deletions
				
			
		|  | @ -615,7 +615,8 @@ void WriteBlock(const VAddr dest_addr, const void* src_buffer, const size_t size | ||||||
|     WriteBlock(*Kernel::g_current_process, dest_addr, src_buffer, size); |     WriteBlock(*Kernel::g_current_process, dest_addr, src_buffer, size); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| void ZeroBlock(const VAddr dest_addr, const size_t size) { | void ZeroBlock(const Kernel::Process& process, const VAddr dest_addr, const size_t size) { | ||||||
|  |     auto& page_table = process.vm_manager.page_table; | ||||||
|     size_t remaining_size = size; |     size_t remaining_size = size; | ||||||
|     size_t page_index = dest_addr >> PAGE_BITS; |     size_t page_index = dest_addr >> PAGE_BITS; | ||||||
|     size_t page_offset = dest_addr & PAGE_MASK; |     size_t page_offset = dest_addr & PAGE_MASK; | ||||||
|  | @ -626,36 +627,37 @@ void ZeroBlock(const VAddr dest_addr, const size_t size) { | ||||||
|         const size_t copy_amount = std::min(PAGE_SIZE - page_offset, remaining_size); |         const size_t copy_amount = std::min(PAGE_SIZE - page_offset, remaining_size); | ||||||
|         const VAddr current_vaddr = static_cast<VAddr>((page_index << PAGE_BITS) + page_offset); |         const VAddr current_vaddr = static_cast<VAddr>((page_index << PAGE_BITS) + page_offset); | ||||||
| 
 | 
 | ||||||
|         switch (current_page_table->attributes[page_index]) { |         switch (page_table.attributes[page_index]) { | ||||||
|         case PageType::Unmapped: { |         case PageType::Unmapped: { | ||||||
|             LOG_ERROR(HW_Memory, "unmapped ZeroBlock @ 0x%08X (start address = 0x%08X, size = %zu)", |             LOG_ERROR(HW_Memory, "unmapped ZeroBlock @ 0x%08X (start address = 0x%08X, size = %zu)", | ||||||
|                       current_vaddr, dest_addr, size); |                       current_vaddr, dest_addr, size); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::Memory: { |         case PageType::Memory: { | ||||||
|             DEBUG_ASSERT(current_page_table->pointers[page_index]); |             DEBUG_ASSERT(page_table.pointers[page_index]); | ||||||
| 
 | 
 | ||||||
|             u8* dest_ptr = current_page_table->pointers[page_index] + page_offset; |             u8* dest_ptr = page_table.pointers[page_index] + page_offset; | ||||||
|             std::memset(dest_ptr, 0, copy_amount); |             std::memset(dest_ptr, 0, copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::Special: { |         case PageType::Special: { | ||||||
|             DEBUG_ASSERT(GetMMIOHandler(current_vaddr)); |             MMIORegionPointer handler = GetMMIOHandler(page_table, current_vaddr); | ||||||
| 
 |             DEBUG_ASSERT(handler); | ||||||
|             GetMMIOHandler(current_vaddr)->WriteBlock(current_vaddr, zeros.data(), copy_amount); |             handler->WriteBlock(current_vaddr, zeros.data(), copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::RasterizerCachedMemory: { |         case PageType::RasterizerCachedMemory: { | ||||||
|             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), |             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), | ||||||
|                                          FlushMode::FlushAndInvalidate); |                                          FlushMode::FlushAndInvalidate); | ||||||
|             std::memset(GetPointerFromVMA(current_vaddr), 0, copy_amount); |             std::memset(GetPointerFromVMA(process, current_vaddr), 0, copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::RasterizerCachedSpecial: { |         case PageType::RasterizerCachedSpecial: { | ||||||
|             DEBUG_ASSERT(GetMMIOHandler(current_vaddr)); |             MMIORegionPointer handler = GetMMIOHandler(page_table, current_vaddr); | ||||||
|  |             DEBUG_ASSERT(handler); | ||||||
|             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), |             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), | ||||||
|                                          FlushMode::FlushAndInvalidate); |                                          FlushMode::FlushAndInvalidate); | ||||||
|             GetMMIOHandler(current_vaddr)->WriteBlock(current_vaddr, zeros.data(), copy_amount); |             handler->WriteBlock(current_vaddr, zeros.data(), copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         default: |         default: | ||||||
|  | @ -668,7 +670,12 @@ void ZeroBlock(const VAddr dest_addr, const size_t size) { | ||||||
|     } |     } | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| void CopyBlock(VAddr dest_addr, VAddr src_addr, const size_t size) { | void ZeroBlock(const VAddr dest_addr, const size_t size) { | ||||||
|  |     ZeroBlock(*Kernel::g_current_process, dest_addr, size); | ||||||
|  | } | ||||||
|  | 
 | ||||||
|  | void CopyBlock(const Kernel::Process& process, VAddr dest_addr, VAddr src_addr, const size_t size) { | ||||||
|  |     auto& page_table = process.vm_manager.page_table; | ||||||
|     size_t remaining_size = size; |     size_t remaining_size = size; | ||||||
|     size_t page_index = src_addr >> PAGE_BITS; |     size_t page_index = src_addr >> PAGE_BITS; | ||||||
|     size_t page_offset = src_addr & PAGE_MASK; |     size_t page_offset = src_addr & PAGE_MASK; | ||||||
|  | @ -677,41 +684,42 @@ void CopyBlock(VAddr dest_addr, VAddr src_addr, const size_t size) { | ||||||
|         const size_t copy_amount = std::min(PAGE_SIZE - page_offset, remaining_size); |         const size_t copy_amount = std::min(PAGE_SIZE - page_offset, remaining_size); | ||||||
|         const VAddr current_vaddr = static_cast<VAddr>((page_index << PAGE_BITS) + page_offset); |         const VAddr current_vaddr = static_cast<VAddr>((page_index << PAGE_BITS) + page_offset); | ||||||
| 
 | 
 | ||||||
|         switch (current_page_table->attributes[page_index]) { |         switch (page_table.attributes[page_index]) { | ||||||
|         case PageType::Unmapped: { |         case PageType::Unmapped: { | ||||||
|             LOG_ERROR(HW_Memory, "unmapped CopyBlock @ 0x%08X (start address = 0x%08X, size = %zu)", |             LOG_ERROR(HW_Memory, "unmapped CopyBlock @ 0x%08X (start address = 0x%08X, size = %zu)", | ||||||
|                       current_vaddr, src_addr, size); |                       current_vaddr, src_addr, size); | ||||||
|             ZeroBlock(dest_addr, copy_amount); |             ZeroBlock(process, dest_addr, copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::Memory: { |         case PageType::Memory: { | ||||||
|             DEBUG_ASSERT(current_page_table->pointers[page_index]); |             DEBUG_ASSERT(page_table.pointers[page_index]); | ||||||
|             const u8* src_ptr = current_page_table->pointers[page_index] + page_offset; |             const u8* src_ptr = page_table.pointers[page_index] + page_offset; | ||||||
|             WriteBlock(dest_addr, src_ptr, copy_amount); |             WriteBlock(process, dest_addr, src_ptr, copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::Special: { |         case PageType::Special: { | ||||||
|             DEBUG_ASSERT(GetMMIOHandler(current_vaddr)); |             MMIORegionPointer handler = GetMMIOHandler(page_table, current_vaddr); | ||||||
| 
 |             DEBUG_ASSERT(handler); | ||||||
|             std::vector<u8> buffer(copy_amount); |             std::vector<u8> buffer(copy_amount); | ||||||
|             GetMMIOHandler(current_vaddr)->ReadBlock(current_vaddr, buffer.data(), buffer.size()); |             handler->ReadBlock(current_vaddr, buffer.data(), buffer.size()); | ||||||
|             WriteBlock(dest_addr, buffer.data(), buffer.size()); |             WriteBlock(process, dest_addr, buffer.data(), buffer.size()); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::RasterizerCachedMemory: { |         case PageType::RasterizerCachedMemory: { | ||||||
|             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), |             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), | ||||||
|                                          FlushMode::Flush); |                                          FlushMode::Flush); | ||||||
|             WriteBlock(dest_addr, GetPointerFromVMA(current_vaddr), copy_amount); |             WriteBlock(process, dest_addr, GetPointerFromVMA(process, current_vaddr), copy_amount); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         case PageType::RasterizerCachedSpecial: { |         case PageType::RasterizerCachedSpecial: { | ||||||
|             DEBUG_ASSERT(GetMMIOHandler(current_vaddr)); |             MMIORegionPointer handler = GetMMIOHandler(page_table, current_vaddr); | ||||||
|  |             DEBUG_ASSERT(handler); | ||||||
|             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), |             RasterizerFlushVirtualRegion(current_vaddr, static_cast<u32>(copy_amount), | ||||||
|                                          FlushMode::Flush); |                                          FlushMode::Flush); | ||||||
| 
 | 
 | ||||||
|             std::vector<u8> buffer(copy_amount); |             std::vector<u8> buffer(copy_amount); | ||||||
|             GetMMIOHandler(current_vaddr)->ReadBlock(current_vaddr, buffer.data(), buffer.size()); |             handler->ReadBlock(current_vaddr, buffer.data(), buffer.size()); | ||||||
|             WriteBlock(dest_addr, buffer.data(), buffer.size()); |             WriteBlock(process, dest_addr, buffer.data(), buffer.size()); | ||||||
|             break; |             break; | ||||||
|         } |         } | ||||||
|         default: |         default: | ||||||
|  | @ -726,6 +734,10 @@ void CopyBlock(VAddr dest_addr, VAddr src_addr, const size_t size) { | ||||||
|     } |     } | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
|  | void CopyBlock(VAddr dest_addr, VAddr src_addr, const size_t size) { | ||||||
|  |     CopyBlock(*Kernel::g_current_process, dest_addr, src_addr, size); | ||||||
|  | } | ||||||
|  | 
 | ||||||
| template <> | template <> | ||||||
| u8 ReadMMIO<u8>(MMIORegionPointer mmio_handler, VAddr addr) { | u8 ReadMMIO<u8>(MMIORegionPointer mmio_handler, VAddr addr) { | ||||||
|     return mmio_handler->Read8(addr); |     return mmio_handler->Read8(addr); | ||||||
|  |  | ||||||
|  | @ -211,7 +211,9 @@ void ReadBlock(const VAddr src_addr, void* dest_buffer, size_t size); | ||||||
| void WriteBlock(const Kernel::Process& process, const VAddr dest_addr, const void* src_buffer, | void WriteBlock(const Kernel::Process& process, const VAddr dest_addr, const void* src_buffer, | ||||||
|                 size_t size); |                 size_t size); | ||||||
| void WriteBlock(const VAddr dest_addr, const void* src_buffer, size_t size); | void WriteBlock(const VAddr dest_addr, const void* src_buffer, size_t size); | ||||||
|  | void ZeroBlock(const Kernel::Process& process, const VAddr dest_addr, const size_t size); | ||||||
| void ZeroBlock(const VAddr dest_addr, const size_t size); | void ZeroBlock(const VAddr dest_addr, const size_t size); | ||||||
|  | void CopyBlock(const Kernel::Process& process, VAddr dest_addr, VAddr src_addr, size_t size); | ||||||
| void CopyBlock(VAddr dest_addr, VAddr src_addr, size_t size); | void CopyBlock(VAddr dest_addr, VAddr src_addr, size_t size); | ||||||
| 
 | 
 | ||||||
| u8* GetPointer(VAddr virtual_address); | u8* GetPointer(VAddr virtual_address); | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue