[Piglit] [PATCH] ssbo: fix rendering test to check for vertex ssbo support.

Ilia Mirkin imirkin at alum.mit.edu
Mon Feb 15 05:48:15 UTC 2016


Sounds right, I think it's only required in frag shader.

Acked-by: Ilia Mirkin <imirkin at alum.mit.edu>
On Feb 14, 2016 11:28 PM, "Dave Airlie" <airlied at gmail.com> wrote:

> From: Dave Airlie <airlied at redhat.com>
>
> This tests uses vertex ssbo's which fglrx on evergreen doesn't do,
> so refuse to run in this case.
>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
>  tests/spec/arb_shader_storage_buffer_object/rendering.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/tests/spec/arb_shader_storage_buffer_object/rendering.c
> b/tests/spec/arb_shader_storage_buffer_object/rendering.c
> index 52d5882..cfb7fb7 100644
> --- a/tests/spec/arb_shader_storage_buffer_object/rendering.c
> +++ b/tests/spec/arb_shader_storage_buffer_object/rendering.c
> @@ -161,6 +161,7 @@ setup_ubos(void)
>  void
>  piglit_init(int argc, char **argv)
>  {
> +       GLint num_vs_blocks;
>         piglit_require_extension("GL_ARB_shader_storage_buffer_object");
>         piglit_require_extension("GL_ARB_program_interface_query");
>
> @@ -168,6 +169,11 @@ piglit_init(int argc, char **argv)
>                 test_buffer_offset = true;
>         }
>
> +       glGetIntegerv(GL_MAX_VERTEX_SHADER_STORAGE_BLOCKS, &num_vs_blocks);
> +       if (num_vs_blocks == 0) {
> +               printf("Test requires SSBO vertex shader support.\n");
> +               piglit_report_result(PIGLIT_SKIP);
> +       }
>         prog = piglit_build_simple_program(vert_shader_text,
> frag_shader_text);
>         assert(prog);
>         glUseProgram(prog);
> --
> 2.1.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20160215/437d5706/attachment.html>


More information about the Piglit mailing list