Threaded Renderer Crash, Threaded Renderer Index Desync, Logger Disposal #174

Merged
LotP merged 4 commits from MabelAmber/Ryubing:lotps-fixes into master 2026-07-27 20:03:46 +00:00
Contributor

These 3 are all fixes made by LotP, who said they just didn't feel like making the PR atm and that someone else could do it if they wanted to (https://discord.com/channels/1294443224030511104/1295891559056674816/1531034369232211968)

Addresses these 3 specific issues:

  1. Toggling the hidden console tries to assign a new console logger, which is also done on setting initial state, but at that point a logger already exists so it just returns, except previously it did not dispose of the new logger.
  2. The threaded renderer would sometimes just crash when emulation is stopped, the fix being to make sure the backend render thread is joined before touching common objects
  3. The disposing workflow of other threads than the main GPU adds commands to the command queue, causing a problem where the index can get desynced because of race conditions
These 3 are all fixes made by LotP, who said they just didn't feel like making the PR atm and that someone else could do it if they wanted to (https://discord.com/channels/1294443224030511104/1295891559056674816/1531034369232211968) Addresses these 3 specific issues: 1. Toggling the hidden console tries to assign a new console logger, which is also done on setting initial state, but at that point a logger already exists so it just returns, except previously it did not dispose of the new logger. 2. The threaded renderer would sometimes just crash when emulation is stopped, the fix being to make sure the backend render thread is joined before touching common objects 3. The disposing workflow of other threads than the main GPU adds commands to the command queue, causing a problem where the index can get desynced because of race conditions
These are all fixes by LotP who did not feel like creating the PR themself atm so they asked someone else to, so I am
These are all fixes by LotP who did not feel like creating the PR themself atm so they asked someone else to, so I am
Merge remote-tracking branch 'origin/lotps-fixes' into lotps-fixes
Some checks failed
Pull Request Triage / triage (pull_request_target) Successful in 1m5s
Build PR / macOS Universal (Release) (pull_request) Has been cancelled
Build PR / linux-arm64 (Release) (pull_request) Has been cancelled
Build PR / linux-x64 (Release) (pull_request) Has been cancelled
Build PR / win-arm64 (Release) (pull_request) Has been cancelled
Build PR / win-x64 (Release) (pull_request) Has been cancelled
1d3ff79982
Change Thread Joins Pattern Syntax
All checks were successful
Build PR / linux-arm64 (Release) (pull_request) Successful in 5m10s
Build PR / win-arm64 (Release) (pull_request) Successful in 9m21s
Build PR / win-x64 (Release) (pull_request) Successful in 9m21s
Build PR / linux-x64 (Release) (pull_request) Successful in 9m24s
Build PR / macOS Universal (Release) (pull_request) Successful in 7m12s
7c3357f920
LotP merged commit e3940abe01 into master 2026-07-27 20:03:46 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
projects/Ryubing!174
No description provided.