I put it as radeonsi bug, but it is probably r600 bug given the implementation seems shared there.
Some d3d9 games do manual throttling as advised at the end of:
"A second solution is to use DirectX 9's Asynchronous Query functionality (analogous to using fences in OpenGL). At the end of your frame, insert a D3DQUERYTYPE_EVENT query into your rendering stream. You can then poll whether the GPU has reached this event yet by using GetData."
Games like Heroes V of Might and Magic uses two d3d9 event queries (mapped to PIPE_QUERY_GPU_FINISHED) to do manual throttling:
end query A
loop until query B is OK (d3d9: loop on GetData /pipe: loop on pipe->get_query_result)
end query B
loop until query A is OK
Only old apps seems to do this manual throttling, as likely recent drivers do it automatically, like Mesa.
Both Gallium Nine and Wine get poor performance with this scheme and get same performance (and is the same performance than by forcing a glfinish)
Not advertising the query under Gallium Nine gives a enormous performance boost to the app. Similarly advertising the query, but not using PIPE_QUERY_GPU_FINISHED but rather a custom implementation with pipe fences, gives the correct performance.
In both cases, forcing glFinish gives the same bad performance than before.
Thus PIPE_QUERY_GPU_FINISHED implementation seems to have a bug that makes it acts as glFinish instead of just waiting what was before the end query is rendered.
What seems strange is that Wine uses ARB_sync to implement the query, and it doesn't seem to be implemented in Mesa with PIPE_QUERY_GPU_FINISHED.
Created attachment 113232 [details]
Hack used to use pipe fences instead of PIPE_QUERY_GPU_FINISHED
Created attachment 113242 [details] [review]
Created attachment 113243 [details] [review]
Can you try these patches? Patch 1 is there only to avoid merge conflicts.
Yes, I confirm the patch does the trick.
Also for the comment that wine was too getting same performance than glFinish, I checked twice, and I think this is just mere coincidence that it gets performance around that. In specific scenes, wine got better than that.
Fixed by 5f1cef76f9bbaae772120dcb38e0b98d68a93f26. Closing.