diff options
author | Sven Gothel <sgothel@jausoft.com> | 2012-09-20 19:03:26 +0200 |
---|---|---|
committer | Sven Gothel <sgothel@jausoft.com> | 2012-09-20 19:03:26 +0200 |
commit | 7da8654b49aa3ddd01fbc64a710b5efe04529381 (patch) | |
tree | 46f7f05d3accb5bc002ccfdf5090b035f5f4b3d9 | |
parent | 8be507ab52506f50804d0254aeacf45e171b919e (diff) |
GLFBODrawableImpl: Fix garbage @ swap, add [disabled] reset-quirk for testing
Fix garbage @ swap:
Reset the front buffer _after_ swap.
Add [disabled] reset-quirk for testing:
On some OSX GPUs (old NV 7xxx only ?), reconfig/resize of FBO fails,
recreate whole FBO w/ quirk enabled.
-rw-r--r-- | src/jogl/classes/jogamp/opengl/GLFBODrawableImpl.java | 65 |
1 files changed, 48 insertions, 17 deletions
diff --git a/src/jogl/classes/jogamp/opengl/GLFBODrawableImpl.java b/src/jogl/classes/jogamp/opengl/GLFBODrawableImpl.java index de45466f3..d7de7ca50 100644 --- a/src/jogl/classes/jogamp/opengl/GLFBODrawableImpl.java +++ b/src/jogl/classes/jogamp/opengl/GLFBODrawableImpl.java @@ -46,7 +46,7 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { private FBObject[] fbos; private int fboIBack; // points to GL_BACK buffer private int fboIFront; // points to GL_FRONT buffer - private FBObject pendingFBOReset = null; + private int pendingFBOReset = -1; private boolean fboBound; private static final int bufferCount = 2; // number of FBOs for double buffering. TODO: Possible to configure! @@ -132,7 +132,7 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { } fbos=null; fboBound = false; - pendingFBOReset = null; + pendingFBOReset = -1; } if(DEBUG) { System.err.println("GLFBODrawableImpl.initialize("+realize+"): "+this); @@ -144,13 +144,34 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { swapBufferContext = sbc; } - private static final void reset(GL gl, FBObject fbo, int fboIdx, int width, int height, int samples) { - fbo.reset(gl, width, height, samples); // implicit glClear(..) - if(fbo.getNumSamples() != samples) { - throw new InternalError("Sample number mismatch: "+samples+", fbos["+fboIdx+"] "+fbo); - } - } + static final boolean FBOResetQuirk = false; + private final void reset(GL gl, int idx, int width, int height, int samples, int alphaBits, int stencilBits) { + if( !FBOResetQuirk ) { + fbos[idx].reset(gl, width, height, samples); // implicit glClear(..) + if(fbos[idx].getNumSamples() != samples) { + throw new InternalError("Sample number mismatch: "+samples+", fbos["+idx+"] "+fbos[idx]); + } + } else { + fbos[idx].destroy(gl); + fbos[idx] = new FBObject(); + fbos[idx].reset(gl, getWidth(), getHeight(), samples); + if(fbos[idx].getNumSamples() != samples) { + throw new InternalError("Sample number mismatch: "+samples+", fbos["+idx+"] "+fbos[idx]); + } + if(samples > 0) { + fbos[idx].attachColorbuffer(gl, 0, alphaBits>0); + } else { + fbos[idx].attachTexture2D(gl, 0, alphaBits>0); + } + if( stencilBits > 0 ) { + fbos[idx].attachRenderbuffer(gl, Attachment.Type.DEPTH_STENCIL, 24); + } else { + fbos[idx].attachRenderbuffer(gl, Attachment.Type.DEPTH, 24); + } + } + } + private final void reset(GL gl, int newSamples) throws GLException { if(!initialized) { // NOP if not yet initializes @@ -187,10 +208,11 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { final int nWidth = getWidth(); final int nHeight = getHeight(); samples = newSamples; - pendingFBOReset = ( 1 < fbos.length ) ? fbos[fboIFront] : null; // pending-front reset only w/ double buffering (or zero samples) + pendingFBOReset = ( 1 < fbos.length ) ? fboIFront : -1; // pending-front reset only w/ double buffering (or zero samples) + final GLCapabilitiesImmutable caps = (GLCapabilitiesImmutable) surface.getGraphicsConfiguration().getChosenCapabilities(); for(int i=0; i<fbos.length; i++) { - if(1 == fbos.length || fboIFront != i) { - reset(gl, fbos[i], i, nWidth, nHeight, samples); + if( pendingFBOReset != i ) { + reset(gl, i, nWidth, nHeight, samples, caps.getAlphaBits(), caps.getStencilBits()); } } final GLCapabilities fboCapsNative = (GLCapabilities) surface.getGraphicsConfiguration().getChosenCapabilities(); @@ -264,6 +286,7 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { } else { if(fboBound) { swapFBOImpl(glc); + swapFBOImplPost(glc); fboBound=false; if(DEBUG) { System.err.println("Post FBO swap(@release): done"); @@ -275,9 +298,11 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { @Override protected void swapBuffersImpl(boolean doubleBuffered) { final GLContext ctx = GLContext.getCurrent(); + boolean doPostSwap = false; if(null!=ctx && ctx.getGLDrawable()==this) { if(fboBound) { swapFBOImpl(ctx); + doPostSwap = true; fboBound=false; if(DEBUG) { System.err.println("Post FBO swap(@swap): done"); @@ -287,17 +312,23 @@ public class GLFBODrawableImpl extends GLDrawableImpl implements GLFBODrawable { if(null != swapBufferContext) { swapBufferContext.swapBuffers(doubleBuffered); } + if(doPostSwap) { + swapFBOImplPost(ctx); + } + } + + private final void swapFBOImplPost(GLContext glc) { + // Safely reset the previous front FBO - after completing propagating swap + if(0 <= pendingFBOReset) { + final GLCapabilitiesImmutable caps = (GLCapabilitiesImmutable) surface.getGraphicsConfiguration().getChosenCapabilities(); + reset(glc.getGL(), pendingFBOReset, getWidth(), getHeight(), samples, caps.getAlphaBits(), caps.getStencilBits()); + pendingFBOReset = -1; + } } private final void swapFBOImpl(GLContext glc) { final GL gl = glc.getGL(); fbos[fboIBack].markUnbound(); // fast path, use(gl,..) is called below - - // Safely reset the previous front FBO - if(null != pendingFBOReset) { - reset(gl, pendingFBOReset, fboIFront, getWidth(), getHeight(), samples); - pendingFBOReset = null; - } if(DEBUG) { int _fboIFront = ( fboIFront + 1 ) % fbos.length; |