[Piglit] [PATCH 1/2] cl: Add test for call stack realignment

2018-04-05 Thread Matt Arsenault
v2: Use uintptr_t --- tests/cl/program/execute/realign-stack.cl | 96 +++ 1 file changed, 96 insertions(+) create mode 100644 tests/cl/program/execute/realign-stack.cl diff --git a/tests/cl/program/execute/realign-stack.cl b/tests/cl/program/execute/realign-stack.cl

Re: [Piglit] [PATCH] cl: Add test for CSR VGPRs caused by SGPR spilling

2018-04-05 Thread Matt Arsenault
ping > On Mar 29, 2018, at 11:29, Matt Arsenault wrote: > > Make sure if a CSR VGPR is needed for SGPR spilling, it is > properly saved and restored. > --- > .../execute/amdgcn-callee-saved-registers.cl | 52 ++ > 1 file changed, 52 insertions(+) >

Re: [Piglit] [PATCH] cl: Add test for call stack realignment

2018-04-05 Thread Matt Arsenault
> On Apr 4, 2018, at 15:52, Jan Vesely wrote: > > redundant newline Not sure what you mean by this. Do you mean the newline to put the single array element on its own line? I was trying to be consistent with buffer formatting as most tests

Re: [Piglit] [PATCH] cl: Add test for call stack realignment

2018-04-05 Thread Matt Arsenault
> On Apr 4, 2018, at 15:52, Jan Vesely wrote: > > On Tue, 2018-04-03 at 18:03 -0400, Matt Arsenault wrote: >> ping >> >>> On Mar 29, 2018, at 10:35, Matt Arsenault wrote: >>> >>> --- >>> tests/cl/program/execute/realign-stack.cl | 96 >>>

Re: [Piglit] [PATCH 00/35] Serialize profiles into XML at build time

2018-04-05 Thread Marek Olšák
On Thu, Apr 5, 2018 at 2:05 PM, Dylan Baker wrote: > Quoting Marek Olšák (2018-04-04 16:37:47) > > Is this use case affected? > > > > piglit run --deqp-mustpass-list --process-isolation 0 -p gbm -c quick > cts_gl45 > > deqp_gles2 deqp_gles3 deqp_gles31 > > You could either

[Piglit] [PATCH v2 34/35] Generate xml for builtin profiles

2018-04-05 Thread Dylan Baker
This creates and installs xml for all builtin profiles. Using these profiles I see startup times of ~1 second as opposed to more than 10 with current master, and runtimes that drop by ~1 minute. v2: - Fix compilation with make. v1 only worked with ninja. --- CMakeLists.txt | 8 +++-

Re: [Piglit] [PATCH 7/8] framework: Add a vulkan tests profile

2018-04-05 Thread Dylan Baker
Quoting Neil Roberts (2018-04-04 15:56:54) > This searches for files named *.vk_shader_test in the tests/vulkan > directory and runs them with VkRunner. > --- > framework/test/vk_shader_test.py | 39 +++ > tests/vulkan.py | 30

Re: [Piglit] [PATCH 6/8] Import VkRunner

2018-04-05 Thread Dylan Baker
Quoting Neil Roberts (2018-04-04 15:56:53) [snip] > diff --git a/tests/vulkan/vkrunner/make-formats.py > b/tests/vulkan/vkrunner/make-formats.py > new file mode 100755 > index 0..6bb1030c0 > --- /dev/null > +++ b/tests/vulkan/vkrunner/make-formats.py > @@ -0,0 +1,93 @@ >

Re: [Piglit] [PATCH 00/35] Serialize profiles into XML at build time

2018-04-05 Thread Dylan Baker
Quoting Marek Olšák (2018-04-04 16:37:47) > Is this use case affected? > > piglit run --deqp-mustpass-list --process-isolation 0 -p gbm -c quick cts_gl45 > deqp_gles2 deqp_gles3 deqp_gles31 You could either run that as: piglit run --deqp-mustpass-list --process-isolation 0 -p gbm -c quick_gl

Re: [Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map

2018-04-05 Thread Juan A. Suarez Romero
On Thu, 2018-04-05 at 10:19 +0200, Juan A. Suarez Romero wrote: > On Wed, 2018-03-28 at 11:15 -0400, Anthony Pesch wrote: > > From: Anthony Pesch > > > > Update cube map tests to complete the cube map before performing the final > > query. This final query is expected to

Re: [Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map

2018-04-05 Thread Juan A. Suarez Romero
On Wed, 2018-03-28 at 11:15 -0400, Anthony Pesch wrote: > From: Anthony Pesch > > Update cube map tests to complete the cube map before performing the final > query. This final query is expected to succeed, however, querying a cube map > which is not cube complete should set

Re: [Piglit] [PATCH 0/8] Vulkan testing with VkRunner

2018-04-05 Thread Neil Roberts
The sixth patch seems to have gotten stuck in the moderator queue for being too big. In the meantime I’ve also pushed a branch here: https://github.com/Igalia/piglit/commits/nroberts/vkrunner - Neil Neil Roberts writes: > This series sets up a framework for testing Vulkan