Re: [Piglit] [PATCH 0/3] tests: intel blackhole render fixes

2018-04-06 Thread Dylan Baker
for the series: Reviewed-by: Dylan Baker Quoting Lionel Landwerlin (2018-04-06 03:25:13) > Hi, > > I've made a few mistake when sending the initial patch. I'm not quite > sure how I missed them :( > > Thanks, > > Lionel Landwerlin (3): > tests: fix INTEL_blackhole_render test names > tests

Re: [Piglit] [PATCH] vkrunner: Add the script used to generate vr-format-table.h

2018-04-06 Thread Dylan Baker
Thanks for making the changes I requested! Reviewed-by: Dylan Baker Quoting Neil Roberts (2018-04-06 05:14:20) > Note that the script is not run automatically as part of the build > process and instead vr-format-table.h is checked into git. > --- > > I’ve split the script out to make it easier

Re: [Piglit] [PATCH v2] framework: Add a vulkan tests profile

2018-04-06 Thread Dylan Baker
Looks good, thank you! Quoting Neil Roberts (2018-04-06 05:19:57) > This searches for files named *.vk_shader_test in the tests/vulkan > directory and runs them with VkRunner. > > Reviewed-by: Dylan Baker > > v2: Move VkShaderTest to piglit_test.py and rename to VkRunnerTest. > Add future i

[Piglit] [PATCH v2] framework: Add a vulkan tests profile

2018-04-06 Thread Neil Roberts
This searches for files named *.vk_shader_test in the tests/vulkan directory and runs them with VkRunner. Reviewed-by: Dylan Baker v2: Move VkShaderTest to piglit_test.py and rename to VkRunnerTest. Add future imports. Remove unused import. --- framework/test/piglit_test.py | 10 ++

[Piglit] [PATCH] vkrunner: Add the script used to generate vr-format-table.h

2018-04-06 Thread Neil Roberts
Note that the script is not run automatically as part of the build process and instead vr-format-table.h is checked into git. --- I’ve split the script out to make it easier to review. Thanks for the feedback Dylan. tests/vulkan/vkrunner/make-formats.py | 137 ++

[Piglit] [PATCH 2/3] tests: intel_blackhole_render: require extension in dispatch test

2018-04-06 Thread Lionel Landwerlin
Fixes: c8d89e733c52a6 ("tests: add INTEL_blackhole_render") Signed-off-by: Lionel Landwerlin --- tests/spec/intel_blackhole_render/blackhole_dispatch.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/spec/intel_blackhole_render/blackhole_dispatch.c b/tests/spec/intel_blackhole_render/b

[Piglit] [PATCH 0/3] tests: intel blackhole render fixes

2018-04-06 Thread Lionel Landwerlin
Hi, I've made a few mistake when sending the initial patch. I'm not quite sure how I missed them :( Thanks, Lionel Landwerlin (3): tests: fix INTEL_blackhole_render test names tests: intel_blackhole_render: require extension in dispatch test tests: nv_image_formats: remove unintended new f

[Piglit] [PATCH 1/3] tests: fix INTEL_blackhole_render test names

2018-04-06 Thread Lionel Landwerlin
Fixes: c8d89e733c52a6 ("tests: add INTEL_blackhole_render") Signed-off-by: Lionel Landwerlin --- tests/all.py | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/all.py b/tests/all.py index 39ad46926..26638cd82 100644 --- a/tests/all.py +++ b/tests/all.py @@ -4978,10

[Piglit] [PATCH 3/3] tests: nv_image_formats: remove unintended new files

2018-04-06 Thread Lionel Landwerlin
This file was added by mistake. Fixes: c8d89e733c52a6 ("tests: add INTEL_blackhole_render") Signed-off-by: Lionel Landwerlin --- tests/spec/nv_image_formats/CMakeLists.gl.txt | 8 1 file changed, 8 deletions(-) delete mode 100644 tests/spec/nv_image_formats/CMakeLists.gl.txt diff --gi