[Piglit] [PATCH] all.py: add run_concurrent=False where needed

arthur.huillet at free.fr arthur.huillet at free.fr
Wed Nov 14 12:41:04 UTC 2018


From: Arthur Huillet <ahuillet at nvidia.com>

Commit 57537d45b75218438716506594e16b91dade968f removed run_concurrent=False
from a bunch of tests, saying there was no reason for them to be marked as
such. This is wrong for at least two tests, which require a displayed window
(cannot render to an FBO).  As such, when run concurrently they are liable to
write over each other's pixels if the windows overlap, which can happen on some
setups.

This change fixes the problem by marking clear-accum and masked-clear as
run_concurrent=False. It is likely to be the correct thing to do for all tests
that require a displayed window for the same reason, but these two are the only
one that were so far identified by NVIDIA.
---
 tests/all.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/all.py b/tests/all.py
index 4cd911fab..e1b182f15 100644
--- a/tests/all.py
+++ b/tests/all.py
@@ -779,7 +779,7 @@ with profile.test_list.group_manager(
     g(['gl-1.1-read-pixels-after-display-list'])
     g(['gl-1.1-set-vertex-color-after-draw'])
     g(['array-stride'])
-    g(['clear-accum'])
+    g(['clear-accum'], run_concurrent=False)
     g(['clipflat'])
     g(['copypixels-draw-sync'])
     g(['copypixels-sync'])
@@ -813,7 +813,7 @@ with profile.test_list.group_manager(
     g(['lineloop', '-dlist'], 'lineloop-dlist')
     g(['linestipple'], run_concurrent=False)
     g(['longprim'])
-    g(['masked-clear'])
+    g(['masked-clear'], run_concurrent=False)
     g(['point-line-no-cull'])
     g(['polygon-mode'])
     g(['polygon-mode-facing'])
-- 
2.19.0



More information about the Piglit mailing list