mirror of
				https://github.com/PabloMK7/citra.git
				synced 2025-10-31 05:40:04 +00:00 
			
		
		
		
	gsp::gpu: Reset g_thread_id in UnregisterInterruptRelayQueue
This commit is contained in:
		
							parent
							
								
									552018c50a
								
							
						
					
					
						commit
						07cc781163
					
				
					 3 changed files with 23 additions and 16 deletions
				
			
		|  | @ -26,6 +26,7 @@ enum class ErrorDescription : u32 { | ||||||
|     FS_NotAFile = 250, |     FS_NotAFile = 250, | ||||||
|     FS_NotFormatted = 340, ///< This is used by the FS service when creating a SaveData archive
 |     FS_NotFormatted = 340, ///< This is used by the FS service when creating a SaveData archive
 | ||||||
|     OutofRangeOrMisalignedAddress = 513, // TODO(purpasmart): Check if this name fits its actual usage
 |     OutofRangeOrMisalignedAddress = 513, // TODO(purpasmart): Check if this name fits its actual usage
 | ||||||
|  |     GPU_FirstInitialization = 519, | ||||||
|     FS_InvalidPath = 702, |     FS_InvalidPath = 702, | ||||||
|     InvalidSection = 1000, |     InvalidSection = 1000, | ||||||
|     TooLarge = 1001, |     TooLarge = 1001, | ||||||
|  |  | ||||||
|  | @ -44,7 +44,7 @@ Kernel::SharedPtr<Kernel::SharedMemory> g_shared_memory; | ||||||
| u32 g_thread_id = 0; | u32 g_thread_id = 0; | ||||||
| 
 | 
 | ||||||
| static bool gpu_right_acquired = false; | static bool gpu_right_acquired = false; | ||||||
| 
 | static bool first_initialization = true; | ||||||
| /// Gets a pointer to a thread command buffer in GSP shared memory
 | /// Gets a pointer to a thread command buffer in GSP shared memory
 | ||||||
| static inline u8* GetCommandBuffer(u32 thread_id) { | static inline u8* GetCommandBuffer(u32 thread_id) { | ||||||
|     return g_shared_memory->GetPointer(0x800 + (thread_id * sizeof(CommandBuffer))); |     return g_shared_memory->GetPointer(0x800 + (thread_id * sizeof(CommandBuffer))); | ||||||
|  | @ -347,24 +347,25 @@ static void RegisterInterruptRelayQueue(Service::Interface* self) { | ||||||
|     u32 flags = cmd_buff[1]; |     u32 flags = cmd_buff[1]; | ||||||
| 
 | 
 | ||||||
|     g_interrupt_event = Kernel::g_handle_table.Get<Kernel::Event>(cmd_buff[3]); |     g_interrupt_event = Kernel::g_handle_table.Get<Kernel::Event>(cmd_buff[3]); | ||||||
|  |     // TODO(mailwl): return right error code instead assert
 | ||||||
|     ASSERT_MSG((g_interrupt_event != nullptr), "handle is not valid!"); |     ASSERT_MSG((g_interrupt_event != nullptr), "handle is not valid!"); | ||||||
| 
 | 
 | ||||||
|     g_interrupt_event->name = "GSP_GPU::interrupt_event"; |     g_interrupt_event->name = "GSP_GPU::interrupt_event"; | ||||||
| 
 | 
 | ||||||
|     using Kernel::MemoryPermission; |     if (first_initialization) { | ||||||
|     g_shared_memory = Kernel::SharedMemory::Create(nullptr, 0x1000, |         // This specific code is required for a successful initialization, rather than 0
 | ||||||
|                                                    MemoryPermission::ReadWrite, MemoryPermission::ReadWrite, |         first_initialization = false; | ||||||
|                                                    0, Kernel::MemoryRegion::BASE, "GSP:SharedMemory"); |         cmd_buff[1] = ResultCode(ErrorDescription::GPU_FirstInitialization, ErrorModule::GX, | ||||||
| 
 |                                  ErrorSummary::Success, ErrorLevel::Success).raw; | ||||||
|     Handle shmem_handle = Kernel::g_handle_table.Create(g_shared_memory).MoveFrom(); |     } else { | ||||||
| 
 |         cmd_buff[1] = RESULT_SUCCESS.raw; | ||||||
|     // This specific code is required for a successful initialization, rather than 0
 |     } | ||||||
|     cmd_buff[1] = ResultCode((ErrorDescription)519, ErrorModule::GX, |  | ||||||
|                              ErrorSummary::Success, ErrorLevel::Success).raw; |  | ||||||
|     cmd_buff[2] = g_thread_id++; // Thread ID
 |     cmd_buff[2] = g_thread_id++; // Thread ID
 | ||||||
|     cmd_buff[4] = shmem_handle; // GSP shared memory
 |     cmd_buff[4] = Kernel::g_handle_table.Create(g_shared_memory).MoveFrom(); // GSP shared memory
 | ||||||
| 
 | 
 | ||||||
|     g_interrupt_event->Signal(); // TODO(bunnei): Is this correct?
 |     g_interrupt_event->Signal(); // TODO(bunnei): Is this correct?
 | ||||||
|  | 
 | ||||||
|  |     LOG_WARNING(Service_GSP, "called, flags=0x%08X", flags); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| /**
 | /**
 | ||||||
|  | @ -375,12 +376,12 @@ static void RegisterInterruptRelayQueue(Service::Interface* self) { | ||||||
| static void UnregisterInterruptRelayQueue(Service::Interface* self) { | static void UnregisterInterruptRelayQueue(Service::Interface* self) { | ||||||
|     u32* cmd_buff = Kernel::GetCommandBuffer(); |     u32* cmd_buff = Kernel::GetCommandBuffer(); | ||||||
| 
 | 
 | ||||||
|     g_shared_memory = nullptr; |     g_thread_id = 0; | ||||||
|     g_interrupt_event = nullptr; |     g_interrupt_event = nullptr; | ||||||
| 
 | 
 | ||||||
|     cmd_buff[1] = RESULT_SUCCESS.raw; |     cmd_buff[1] = RESULT_SUCCESS.raw; | ||||||
| 
 | 
 | ||||||
|     LOG_WARNING(Service_GSP, "called"); |     LOG_WARNING(Service_GSP, "(STUBBED) called"); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| /**
 | /**
 | ||||||
|  | @ -718,10 +719,15 @@ Interface::Interface() { | ||||||
|     Register(FunctionTable); |     Register(FunctionTable); | ||||||
| 
 | 
 | ||||||
|     g_interrupt_event = nullptr; |     g_interrupt_event = nullptr; | ||||||
|     g_shared_memory = nullptr; | 
 | ||||||
|  |     using Kernel::MemoryPermission; | ||||||
|  |     g_shared_memory = Kernel::SharedMemory::Create(nullptr, 0x1000, | ||||||
|  |                                                    MemoryPermission::ReadWrite, MemoryPermission::ReadWrite, | ||||||
|  |                                                    0, Kernel::MemoryRegion::BASE, "GSP:SharedMemory"); | ||||||
| 
 | 
 | ||||||
|     g_thread_id = 0; |     g_thread_id = 0; | ||||||
|     gpu_right_acquired = false; |     gpu_right_acquired = false; | ||||||
|  |     first_initialization = true; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| Interface::~Interface() { | Interface::~Interface() { | ||||||
|  |  | ||||||
|  | @ -450,7 +450,7 @@ static const char* GetType(GLenum type) { | ||||||
| #undef RET | #undef RET | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static void DebugHandler(GLenum source, GLenum type, GLuint id, GLenum severity, GLsizei length, | static void APIENTRY DebugHandler(GLenum source, GLenum type, GLuint id, GLenum severity, GLsizei length, | ||||||
|                          const GLchar* message, const void* user_param) { |                          const GLchar* message, const void* user_param) { | ||||||
|     Log::Level level; |     Log::Level level; | ||||||
|     switch (severity) { |     switch (severity) { | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue