[Piglit] [Bug 101503] Piglit exits with "invalid literal for int() with base 10: '19225.json"

2017-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101503 --- Comment #7 from Dylan Baker --- This series fixes the issue: https://patchwork.freedesktop.org/series/26257/ -- You are receiving this mail because: You are the QA Contact for the bug.___ Piglit

[Piglit] [PATCH 4/4] framework: Fix the same bug for resume in the previous commit

2017-06-22 Thread Dylan Baker
The resume path duplicates the 2 lines of code, so the same fix is required there. Signed-off-by: Dylan Baker --- framework/backends/json.py| 5 +++-- unittests/framework/backends/test_json.py | 15 +++ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a

[Piglit] [PATCH 2/4] unittests: Add tests for file in tests dir not ending in .json

2017-06-22 Thread Dylan Baker
This is currently expected to fail. Signed-off-by: Dylan Baker --- unittests/framework/backends/test_json.py | 11 +++ 1 file changed, 11 insertions(+) diff --git a/unittests/framework/backends/test_json.py b/unittests/framework/backends/test_json.py index c9a02ffb8..a61425215 100644 -

[Piglit] [PATCH 1/4] unittests/framework: remove 'autouse' from fixture

2017-06-22 Thread Dylan Baker
This reworks the fixture to not use autouse, so that other tests can be added to the class without getting the setup it doesn't use. This also uses a class scope fixture instead of an instance scope one. Signed-off-by: Dylan Baker --- unittests/framework/backends/test_json.py | 29 ++

[Piglit] [PATCH 3/4] framework: Only sort files for json backend that end in .json

2017-06-22 Thread Dylan Baker
This fixes trying to load .tmp files that don't get moved during an interrupted run. bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101503 Signed-off-by: Dylan Baker --- framework/backends/json.py| 5 +++-- unittests/framework/backends/test_json.py | 1 - 2 files changed,

[Piglit] [Bug 101503] Piglit exits with "invalid literal for int() with base 10: '19225.json"

2017-06-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101503 --- Comment #6 from Dylan Baker --- Each of those json files is the result of a single test. Because piglit tries to make writes atomic when the test starts a .json file is created that contains the PID of the test and the status "incomplete", w

Re: [Piglit] [PATCH 1/1] cmake: bump gbm dependency to 17.1

2017-06-22 Thread Vinson Lee
On Thu, Jun 22, 2017 at 6:52 AM, Jan Vesely wrote: > c6e1dc5247 switched to use gbm_bo_get_offset and gbm_bo_get_stride_for_plane. > These symbols were added in mesa 17.1. Since there is no alternative to using > them, bump the gbm requirement to 17.1. > > Fixes: c6e1dc5247 "piglit_drm_dma_buf: f

Re: [Piglit] [PATCH v3 8/8] egl_khr_fence_sync: Enable android native fences extension test.

2017-06-22 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting Rafael Antognolli (2017-06-20 17:36:06) > Enable the test on the "all" test profile. > > Signed-off-by: Rafael Antognolli > Cc: Dylan Baker > --- > tests/all.py | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/tests/all.py b/tests/all.py > index

Re: [Piglit] [PATCH 3/3] framework: Exit if a filter removes all tests from a profile

2017-06-22 Thread Dylan Baker
Quoting Petri Latvala (2017-06-21 01:37:21) > On 06/20/2017 08:59 PM, Dylan Baker wrote: > > Quoting Petri Latvala (2017-06-20 05:41:11) > >> On 04/13/2017 09:46 PM, Dylan Baker wrote: > >>> Quoting Brian Paul (2017-04-12 13:04:59) > On 04/12/2017 11:55 AM, Dylan Baker wrote: > > It doesn'

Re: [Piglit] [PATCH] Revert "arb_get_texture_sub_image-errors: Fix expected error values"

2017-06-22 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset On 06/22/2017 05:50 PM, Juan A. Suarez Romero wrote: In commit aec149c we test that glGetTextureSubImage() and glGetCompressedTextureSubImage() returns INVALID_VALUE when the texture is not the name of an existing texture object. Nevertheless, this is a bug in the

[Piglit] [PATCH] Revert "arb_get_texture_sub_image-errors: Fix expected error values"

2017-06-22 Thread Juan A. Suarez Romero
In commit aec149c we test that glGetTextureSubImage() and glGetCompressedTextureSubImage() returns INVALID_VALUE when the texture is not the name of an existing texture object. Nevertheless, this is a bug in the OpenGL 4.5 spec itself, that has been fixed, but not released yet. In the fix the exp

[Piglit] [PATCH 1/1] cmake: bump gbm dependency to 17.1

2017-06-22 Thread Jan Vesely
c6e1dc5247 switched to use gbm_bo_get_offset and gbm_bo_get_stride_for_plane. These symbols were added in mesa 17.1. Since there is no alternative to using them, bump the gbm requirement to 17.1. Fixes: c6e1dc5247 "piglit_drm_dma_buf: fix GPU offsets and strides" Signed-off-by: Jan Vesely --- C

[Piglit] [PATCH v6] pbo: Change the value expected from GetPixelMap.

2017-06-22 Thread Sandra Koroniewska
This fixes tests/spec/gl-2.1/pbo on Windows Intel driver. --- tests/spec/gl-2.1/pbo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/spec/gl-2.1/pbo.c b/tests/spec/gl-2.1/pbo.c index bb4c0352c..5020eeb07 100644 --- a/tests/spec/gl-2.1/pbo.c +++ b/tests/spec/gl-2.1/pbo.c