[Piglit] [PATCH] Fix concurrency bug.

[Vinson Lee]

On Tue, Dec 24, 2013 at 9:59 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> Commit d525a0cededab9eff43d7597a3593d843428a68b introduces a bug into
> the -1/--no-concurrency option that causes only tests exculsively marked
> as non-concurrent to be run.
>
> This patch addresses that bug by causes all tests to be added to a
> serial pool when the -1/--no-concurrency option is used.
>
> bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73018
> CC: vlee at freedesktop.org
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>  framework/core.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/framework/core.py b/framework/core.py
> index c8dc4bc..559bce6 100644
> --- a/framework/core.py
> +++ b/framework/core.py
> @@ -575,8 +575,7 @@ class TestProfile:
>          elif env.concurrent == "none":
>              pool = ThreadPool(1)
>              for (path, test) in self.test_list.items():
> -                if not env.concurrent or not test.runConcurrent:
> -                    pool.add(test.execute, (env, path, json_writer))
> +                pool.add(test.execute, (env, path, json_writer))
>              pool.join()
>          else:
>              pool = ThreadPool(multiprocessing.cpu_count())
> --
> 1.8.5.2
>

Tested-by: Vinson Lee <vlee at freedesktop.org>