Module: wine Branch: master Commit: 72597b170e814599ef2be4b19dc4b9de0932a30f URL: https://gitlab.winehq.org/wine/wine/-/commit/72597b170e814599ef2be4b19dc4b9d...
Author: Matteo Bruni mbruni@codeweavers.com Date: Wed Aug 5 20:09:38 2020 +0200
wined3d: Don't use separate GL contexts for different swapchains.
This avoids triggering a Mesa slowpath introduced by Mesa commit e7f3a8d6959c74f63c877dd8776fe519d54f946f.
Additionally, it helps applications using multiple swapchains and queries (e.g. the one in bug 43773) by not requiring extra context switches when polling queries. For the records, the actual issue from bug 43773 was probably fixed a long time ago, maybe already by 7b62a970e9ad3b4179394cf54f0232475fe2388a or thereabouts.
The downside of this change is that we'll now end up calling glFinish() when there are multiple threads using the same device when CSMT is disabled. That's a non-default setting and one that should only be used for troubleshooting purposes at this point, so it shouldn't be too bad of an issue. It might also make sense to just get rid of that glFinish() entirely, it's never going to be a complete fix.
---
dlls/wined3d/adapter_gl.c | 2 +- dlls/wined3d/context_gl.c | 10 ++++++ dlls/wined3d/device.c | 14 +++----- dlls/wined3d/swapchain.c | 81 ++++++------------------------------------ dlls/wined3d/wined3d_private.h | 7 ---- 5 files changed, 26 insertions(+), 88 deletions(-)
-
Alexandre Julliard