[1/4 try 3] wined3d: Activate GL context before making any GL calls in stretch_rect_fbo
Stefan Dösinger
stefandoesinger at gmx.at
Fri Dec 21 07:03:04 CST 2007
Hi,
I think your patches are on the right track, but there are a few issues to
look at. I'll also need Henri's comments, as he wrote the fbo code.
Did you try your patches with the Deferred Shading sample from codesampler?
http://www.codesampler.com/usersrc/usersrc_7.htm . It uses multiple render
targets, so it is a quite interesting test.
drawprimitive() in drawprim.c calls apply_fbo_state before calling
ActivateContext as well. This causes a lot of problems with multithreading,
so this should be fixed too.
The first patch is OK. Calling ActivateContext before activating the fbo is
required.
In patch 2, I was a bit confused by moving the "This->activeContext"
assignment inside Init3D, I first thought you would remove it. But otherwise
it looks good to me as well.
Patch 3 is ok too, keeping track of the currently bound fbo in the context is
good. Unrelated to this patch we should look if the glBindFramebufferEXT(...,
0) calls could be replaced with calling ActivateContext, and if that makes
sense(it's probably not a good idea everywhere).
In patch 4, you have to be aware that in CTXUSAGE_BLIT the fbo's depth and
color1+ attachments might be set to bad textures(e.g. different size). So
you'll have to set them to NULL or make sure that they are set properly.
I once discussed with Henri where we should have all the functions. I think we
could just move apply_fbo_state to context.c, make it static to make sure the
code that requires a drawable calls ActivateContext. I am not entirely sure
if that makes sense everywhere, e.g. the depth_blit code. Henri had some
arguments in that regard, but I have to check the irc logs and digg out the
discussion.
Other good testing applications are Half Life 2(demo is ok), it uses color
buffers and depth buffers with different sizes. Battlefield 2(not 2142, no
idea if there is a demo) depends on the depth blitting. Bioshock(demo is ok)
is a case of an application that crashes due to multithreading issues because
apply_fbo_state is applied before calling ActivateContext.
Thank you for all your excellent work,
Stefan
More information about the wine-patches
mailing list