MacKwnoX/NX2navi
1
0
Fork 0
mirror of https://github.com/tangerineturtle1/NX2navi.git synced 2026-11-15 19:19:58 -04:00
Code Issues Packages Projects Releases Wiki Activity

Wait for the write-backs before returning to the game where pools are shadow copies.

Browse Source 
With shadow storage (the default on Linux and under Wine) what the GPU wrote reaches the game's memory in the completion thread's fence callback, after the executed timeline that the split wait returns at. The game could then rewrite a pool, and an older write-back overwrote what it wrote: in Kokiri Forest a sky coefficient the game had reset came back, and the world flickered and looked washed out. The wait now covers the write-backs whenever shadow storage is in use; with imported or mirrored pools, the Windows default, nothing changes. The contributor's RX 9070 XT under RADV no longer showed the flicker.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
This commit is contained in:
chrg6
2026-11-10 23:09:17 -05:00
parent 9dc5fa8b36
commit 4eea3d1c2f
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
runtime/vk_sync.c
View File

@@ -703,7 +703,11 @@ static int fence_wait_impl(uint64_t value, uint64_t timeout_ns) {
// CPU side effects of its fence may still be to come. 1 when reached.
int vk_sync_exec_wait(uint64_t value, uint64_t timeout_ns) {
vk_sync_start();
if (!vk_sync_wb_split()) return vk_sync_fence_wait(value, timeout_ns);
// With shadow storage (vk_mem_shadow_storage) what the GPU wrote reaches guest memory in the completion thread's
// fence callback, after the executed timeline: a wait that returned there let the game rewrite a pool that an
// older write-back then overwrote (Kokiri Forest: a sky coefficient the game had reset came back, and the world
// flickered washed out). So the wait covers the write-backs there, as without the split.
if (vk_mem_shadow_storage() || !vk_sync_wb_split()) return vk_sync_fence_wait(value, timeout_ns);
if (__atomic_load_n(&done, __ATOMIC_ACQUIRE) >= value) return 1;
uint64_t p0 = perf_begin();
hold_release(value, 0, 0); // a held submission the waiter needs goes to the queue now