Giovanni Mascellani (@giomasce) commented about dlls/dxgi/swapchain.c:
EnterCriticalSection(&swapchain->worker_cs); list_add_tail(&swapchain->worker_ops, &op->entry);
- WakeAllConditionVariable(&swapchain->worker_cv); LeaveCriticalSection(&swapchain->worker_cs);
- WakeAllConditionVariable(&swapchain->worker_cv);
I'm not against this. The opinions on whether that's good for performance varies across the internet and the answer ultimately depends on the underlying implementation, which Zeb probably knows better than me. But did you measure an actual performance gain with that change?