On 04/25/2014 04:53 PM, Anuj Phogat wrote: > It is clarified on page 679 of OpenGL 4.4 core profile spec: > "Added BlitFramebuffer to commands affected by conditional > rendering in section 10.10 (Bug 9562)." > > This addition is also made to OpenGL 4.4 compatibility profile spec. > > Tests the bug fix made in mesa commits 1d350b9 and 8ed42dd: > "i965: Add glBlitFramebuffer to commands affected by conditional > rendering"
Samuel Iglesias also mentioned in bug #77702 that this test current fails on NVIDIA. We should add something like the following to the commit message: Previous to this patch, this test fail on NVIDIA 4200M (driver version 331.49), as reported by Samuel Iglesias. > Signed-off-by: Anuj Phogat <[email protected]> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77702 Reviewed-by: Ian Romanick <[email protected]> Maybe wait a day or two for Samuel to send a 'Tested-by'. > --- > tests/spec/nv_conditional_render/blitframebuffer.c | 15 ++++++++------- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/tests/spec/nv_conditional_render/blitframebuffer.c > b/tests/spec/nv_conditional_render/blitframebuffer.c > index 6d83316..c410973 100644 > --- a/tests/spec/nv_conditional_render/blitframebuffer.c > +++ b/tests/spec/nv_conditional_render/blitframebuffer.c > @@ -27,11 +27,11 @@ > /** > * @file blitframebuffer.c > * > - * Tests that conditional rendering does not affect glBlitFramebuffer(). > + * Tests that conditional rendering also affects glBlitFramebuffer(). > * > - * It's something that would be likely to be implemented through > - * normal rendering inside of the driver, and thus easy to > - * accidentally disable during conditional rendering. > + * It is clarified on page 679 of OpenGL 4.4 spec: > + * "Added BlitFramebuffer to commands affected by conditional rendering in > + * section 10.10 (Bug 9562)." > */ > > PIGLIT_GL_TEST_CONFIG_BEGIN > @@ -112,7 +112,7 @@ piglit_display(void) > glBeginQuery(GL_SAMPLES_PASSED, q); > glEndQuery(GL_SAMPLES_PASSED); > > - /* This should not be affected by conditional rendering. */ > + /* This should be affected by conditional rendering. */ > glBeginConditionalRenderNV(q, GL_QUERY_WAIT_NV); > blit_window_to_tex(texture, piglit_width, piglit_height / 2); > glEndConditionalRenderNV(); > @@ -123,8 +123,9 @@ piglit_display(void) > 0, 0, 1, 1); > glDisable(GL_TEXTURE_2D); > > - pass = piglit_probe_rect_rgba(0, 0, piglit_width, piglit_height, > - green); > + pass = piglit_probe_rect_rgba(0, piglit_width / 2, > + piglit_width, piglit_height / 2, > + red); > > piglit_present_results(); > _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
