mirror of
				https://git.suyu.dev/suyu/suyu
				synced 2025-10-30 23:49:01 -05:00 
			
		
		
		
	video_core: Refactor command_processor.
Inline the WriteReg helper as it is called ~20k times per frame.
This commit is contained in:
		| @@ -28,51 +28,52 @@ enum class BufferMethods { | |||||||
|     CountBufferMethods = 0x40, |     CountBufferMethods = 0x40, | ||||||
| }; | }; | ||||||
|  |  | ||||||
| void GPU::WriteReg(u32 method, u32 subchannel, u32 value, u32 remaining_params) { |  | ||||||
|     LOG_TRACE(HW_GPU, |  | ||||||
|               "Processing method {:08X} on subchannel {} value " |  | ||||||
|               "{:08X} remaining params {}", |  | ||||||
|               method, subchannel, value, remaining_params); |  | ||||||
|  |  | ||||||
|     ASSERT(subchannel < bound_engines.size()); |  | ||||||
|  |  | ||||||
|     if (method == static_cast<u32>(BufferMethods::BindObject)) { |  | ||||||
|         // Bind the current subchannel to the desired engine id. |  | ||||||
|         LOG_DEBUG(HW_GPU, "Binding subchannel {} to engine {}", subchannel, value); |  | ||||||
|         bound_engines[subchannel] = static_cast<EngineID>(value); |  | ||||||
|         return; |  | ||||||
|     } |  | ||||||
|  |  | ||||||
|     if (method < static_cast<u32>(BufferMethods::CountBufferMethods)) { |  | ||||||
|         // TODO(Subv): Research and implement these methods. |  | ||||||
|         LOG_ERROR(HW_GPU, "Special buffer methods other than Bind are not implemented"); |  | ||||||
|         return; |  | ||||||
|     } |  | ||||||
|  |  | ||||||
|     const EngineID engine = bound_engines[subchannel]; |  | ||||||
|  |  | ||||||
|     switch (engine) { |  | ||||||
|     case EngineID::FERMI_TWOD_A: |  | ||||||
|         fermi_2d->WriteReg(method, value); |  | ||||||
|         break; |  | ||||||
|     case EngineID::MAXWELL_B: |  | ||||||
|         maxwell_3d->WriteReg(method, value, remaining_params); |  | ||||||
|         break; |  | ||||||
|     case EngineID::MAXWELL_COMPUTE_B: |  | ||||||
|         maxwell_compute->WriteReg(method, value); |  | ||||||
|         break; |  | ||||||
|     case EngineID::MAXWELL_DMA_COPY_A: |  | ||||||
|         maxwell_dma->WriteReg(method, value); |  | ||||||
|         break; |  | ||||||
|     default: |  | ||||||
|         UNIMPLEMENTED_MSG("Unimplemented engine"); |  | ||||||
|     } |  | ||||||
| } |  | ||||||
|  |  | ||||||
| MICROPROFILE_DEFINE(ProcessCommandLists, "GPU", "Execute command buffer", MP_RGB(128, 128, 192)); | MICROPROFILE_DEFINE(ProcessCommandLists, "GPU", "Execute command buffer", MP_RGB(128, 128, 192)); | ||||||
|  |  | ||||||
| void GPU::ProcessCommandLists(const std::vector<CommandListHeader>& commands) { | void GPU::ProcessCommandLists(const std::vector<CommandListHeader>& commands) { | ||||||
|     MICROPROFILE_SCOPE(ProcessCommandLists); |     MICROPROFILE_SCOPE(ProcessCommandLists); | ||||||
|  |  | ||||||
|  |     auto WriteReg = [this](u32 method, u32 subchannel, u32 value, u32 remaining_params) { | ||||||
|  |         LOG_TRACE(HW_GPU, | ||||||
|  |                   "Processing method {:08X} on subchannel {} value " | ||||||
|  |                   "{:08X} remaining params {}", | ||||||
|  |                   method, subchannel, value, remaining_params); | ||||||
|  |  | ||||||
|  |         ASSERT(subchannel < bound_engines.size()); | ||||||
|  |  | ||||||
|  |         if (method == static_cast<u32>(BufferMethods::BindObject)) { | ||||||
|  |             // Bind the current subchannel to the desired engine id. | ||||||
|  |             LOG_DEBUG(HW_GPU, "Binding subchannel {} to engine {}", subchannel, value); | ||||||
|  |             bound_engines[subchannel] = static_cast<EngineID>(value); | ||||||
|  |             return; | ||||||
|  |         } | ||||||
|  |  | ||||||
|  |         if (method < static_cast<u32>(BufferMethods::CountBufferMethods)) { | ||||||
|  |             // TODO(Subv): Research and implement these methods. | ||||||
|  |             LOG_ERROR(HW_GPU, "Special buffer methods other than Bind are not implemented"); | ||||||
|  |             return; | ||||||
|  |         } | ||||||
|  |  | ||||||
|  |         const EngineID engine = bound_engines[subchannel]; | ||||||
|  |  | ||||||
|  |         switch (engine) { | ||||||
|  |         case EngineID::FERMI_TWOD_A: | ||||||
|  |             fermi_2d->WriteReg(method, value); | ||||||
|  |             break; | ||||||
|  |         case EngineID::MAXWELL_B: | ||||||
|  |             maxwell_3d->WriteReg(method, value, remaining_params); | ||||||
|  |             break; | ||||||
|  |         case EngineID::MAXWELL_COMPUTE_B: | ||||||
|  |             maxwell_compute->WriteReg(method, value); | ||||||
|  |             break; | ||||||
|  |         case EngineID::MAXWELL_DMA_COPY_A: | ||||||
|  |             maxwell_dma->WriteReg(method, value); | ||||||
|  |             break; | ||||||
|  |         default: | ||||||
|  |             UNIMPLEMENTED_MSG("Unimplemented engine"); | ||||||
|  |         } | ||||||
|  |     }; | ||||||
|  |  | ||||||
|     for (auto entry : commands) { |     for (auto entry : commands) { | ||||||
|         Tegra::GPUVAddr address = entry.Address(); |         Tegra::GPUVAddr address = entry.Address(); | ||||||
|         u32 size = entry.sz; |         u32 size = entry.sz; | ||||||
|   | |||||||
| @@ -132,9 +132,6 @@ public: | |||||||
|     const Tegra::MemoryManager& MemoryManager() const; |     const Tegra::MemoryManager& MemoryManager() const; | ||||||
|  |  | ||||||
| private: | private: | ||||||
|     /// Writes a single register in the engine bound to the specified subchannel |  | ||||||
|     void WriteReg(u32 method, u32 subchannel, u32 value, u32 remaining_params); |  | ||||||
|  |  | ||||||
|     std::unique_ptr<Tegra::MemoryManager> memory_manager; |     std::unique_ptr<Tegra::MemoryManager> memory_manager; | ||||||
|  |  | ||||||
|     /// Mapping of command subchannels to their bound engine ids. |     /// Mapping of command subchannels to their bound engine ids. | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Markus Wick
					Markus Wick