[PATCH 8/9] wined3d: remove ENTER_GL/LEAVE_GL in texture and cube texture PreLoad.

Alexander Dorofeyev alexd4 at inbox.lv
Tue Mar 25 12:00:55 CDT 2008


Why, I checked what routines get called from that ENTER_GL/LEAVE_GL block in 
PreLoad, and they all seem to be pretty safe. LoadTexture doesn't do gl calls 
itself, but it calls LoadLocation. Other calls made in PreLoad like AddDirtyRect 
also don't do gl calls but may call LoadLocation. So it all comes down to 
LoadLocation (with location either INSYSMEM or INTEXTURE), and LoadLocation and 
all its callees that do gl calls appear to be handling this correctly.

Roderick Colenbrander wrote:
>> Not really needed for anything and can lead to ActivateContext being
>> called 
>> while holding lock.
> 
> Sure this is a problem but there needs to be a place which calls ENTER_GL/LEAVE_GL. I believe LoadTexture and some of the other functions aren't handling that properly yet, so I wouldn't add this yet.
> 
> It is very tricky.
> 
> Roderick




More information about the wine-devel mailing list