[Piglit] [PATCH] Fix expected behavior of glBlitFramebuffer() with conditional rendering

Ian Romanick idr at freedesktop.org
Fri Apr 25 20:10:31 PDT 2014


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 <anuj.phogat at gmail.com>

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77702
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

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();
>  



More information about the Piglit mailing list