diff options
author | Aaron O'Mullan <aaron.omullan@gmail.com> | 2023-05-09 22:31:52 -0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-10 03:31:52 +0200 |
commit | a7c6e6a8cf40a54b0b374cbd65078a8d9ed9a0f5 (patch) | |
tree | 3e460f37e1aa1368a4274879d8fe292ac72d6614 /src/Ryujinx.Ui.Common/Configuration/Ui | |
parent | 0bc8151c7ecdacc1506305a8d60e7b3c7b13622d (diff) |
fix(mvk): resumeLostDevice (#4800)1.1.775
Command buffer errors currently trigger an exception "DeviceLost" crashing the process.
Looking at [MKV's code](https://github.com/KhronosGroup/MoltenVK/blob/53a4eb26f2fbd7322eb087eb4af263fe466543b0/MoltenVK/MoltenVK/GPUObjects/MVKQueue.mm#L392-L408) we observe that:
- It hard fails if error is:
```
MTLCommandBufferErrorBlacklisted || MTLCommandBufferErrorNotPermitted || MTLCommandBufferErrorDeviceRemoved
```
- Otherwise fails conditionally if `config.resumeLostDevice == false` (current default)
For Ryujinx's use-case it's more graceful to resume on those errors rather than crashing the app, the error isn't totally silenced since `mvk` still logs it
Fixes #4704, #4575
Diffstat (limited to 'src/Ryujinx.Ui.Common/Configuration/Ui')
0 files changed, 0 insertions, 0 deletions