[Piglit] [PATCH] framework: Fix concurrency regression introduced by Test refactor

Marek Olšák maraeo at gmail.com
Fri Apr 11 12:45:31 PDT 2014


Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Fri, Apr 11, 2014 at 9:33 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> For whatever reason the concurrency flag was not passed to
> GLSLParserTest and ShaderTest after the refactor. This patch fixes that
> by always passing that keyword argument as was done before.
>
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>  framework/glsl_parser_test.py | 2 +-
>  framework/shader_test.py      | 3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/framework/glsl_parser_test.py b/framework/glsl_parser_test.py
> index e996643..4c1da49 100644
> --- a/framework/glsl_parser_test.py
> +++ b/framework/glsl_parser_test.py
> @@ -154,7 +154,7 @@ class GLSLParserTest(PiglitTest):
>                  command.append('--check-link')
>              command.extend(config.get('config', 'require_extensions').split())
>
> -            super(GLSLParserTest, self).__init__(command)
> +            super(GLSLParserTest, self).__init__(command, run_concurrent=True)
>
>
>  class GLSLParserException(Exception):
> diff --git a/framework/shader_test.py b/framework/shader_test.py
> index 5728100..fecd807 100644
> --- a/framework/shader_test.py
> +++ b/framework/shader_test.py
> @@ -83,7 +83,8 @@ class ShaderTest(PiglitTest):
>              else:
>                  raise ShaderTestParserException("No GL version set")
>
> -        super(ShaderTest, self).__init__([prog, arguments, '-auto'])
> +        super(ShaderTest, self).__init__([prog, arguments, '-auto'],
> +                                         run_concurrent=True)
>
>
>  class ShaderTestParserException(Exception):
> --
> 1.9.2
>


More information about the Piglit mailing list