Re: [Piglit] [PATCH v2 00/17] ARB_gl_spirv: v2 ubo/ssbo tests

2018-10-22 Thread Alejandro Piñeiro
I have just resend the mesa series for ubo/ssbo support. I will not do
the same for the piglit series, as there are no changes even after
rebasing against master. If the mesa series gets reviewed, I plan to
push this piglit series. A review/ack would be welcome though.


BR


On 27/09/18 11:54, Alejandro Piñeiro wrote:
> Hi,
>
> this is the second version of the ubo/ssbo tests for
> ARB_gl_spirv. Differences compared with v1:
>
>   * Rebased against today master.
>
>   * Updated the pass checks on several tests. Instead of a silly
> addition/substraction of the ubo/ssbo content, and check a
> specific value, the value of the individual ubo/ssbo members are
> checked.
>
>   * array-complex became even more complex. Specifically thinking on
> testing some cases mentioned by Timothy feedback on the mesa-dev
> series.
>
> v1 announcement:
>https://lists.freedesktop.org/archives/piglit/2018-September/025065.html
>
> The series can be found on this branch:
>https://github.com/Igalia/piglit/tree/arb_gl_spirv-series5-ubo-ssbo-v2
>
>
> Alejandro Piñeiro (15):
>   shader_runner/spirv: log skip reason on ARB_gl_spirv extension check
>   shader_runner/spirv: check if test really includes spirv
>   shader_runner: add force_no_names mode
>   shader_runner: add support for glGetProgram queries
>   arb_gl_spirv: add GL_ACTIVE_UNIFORMS checks
>   arb_gl_spirv: add some simple ubo tests
>   arb_gl_spirv: add simple ubo tests with matrices
>   arb_gl_spirv: add complex ubo test with matrices
>   arb_gl_spirv: add a array of ubo test, with complex ubo content
>   arb_gl_spirv: add ubo matrix test with different matrix_stride
>   arb_gl_spirv: add ubo array test with different array_stride
>   arb_gl_spirv: add some simple ssbo tests
>   arb_gl_spirv: add simple ssbo tests with matrices
>   arb_gl_spirv: adding complex ssbo matrix test
>   arb_gl_spirv: add ssbo test using std140 and std430
>
> Neil Roberts (2):
>   arb_gl_spirv: Add tests for UBOs with explicit offsets
>   arb_gl_spirv: Add a test for SSBOs with an unsized array
>
>  tests/shaders/shader_runner.c  | 485 +++---
>  .../execution/ssbo/array-indirect.shader_test  | 121 
>  .../execution/ssbo/array-inside-ssbo.shader_test   | 189 ++
>  .../ssbo/array-of-arrays-inside-ssbo.shader_test   | 203 ++
>  .../arb_gl_spirv/execution/ssbo/array.shader_test  | 173 +
>  .../execution/ssbo/matrix/column-major.shader_test | 188 ++
>  .../ssbo/matrix/column-vs-row.shader_test  | 291 +
>  .../execution/ssbo/matrix/complex.shader_test  | 702 
> +
>  .../ssbo/matrix/indirect-column-major.shader_test  | 132 
>  .../ssbo/matrix/indirect-row-major.shader_test | 132 
>  .../execution/ssbo/matrix/row-major.shader_test| 194 ++
>  .../arb_gl_spirv/execution/ssbo/simple.shader_test | 138 
>  .../ssbo/two-ssbo-different-layouts.shader_test| 372 +++
>  .../execution/ssbo/two-ssbo.shader_test| 151 +
>  .../execution/ssbo/two-stages.shader_test  | 279 
>  .../execution/ssbo/unsized-array.shader_test   | 119 
>  .../execution/ubo/array-complex.shader_test| 360 +++
>  .../array-different-array-stride-ubo.shader_test   | 271 
>  .../execution/ubo/array-indirect.shader_test   | 108 
>  .../execution/ubo/array-inside-ubo.shader_test | 154 +
>  .../ubo/array-of-arrays-inside-ubo.shader_test | 184 ++
>  .../arb_gl_spirv/execution/ubo/array.shader_test   | 140 
>  .../ubo/explicit-offset-nested-struct.shader_test  |  81 +++
>  .../execution/ubo/explicit-offset.shader_test  |  80 +++
>  .../execution/ubo/location-0-crash.shader_test |  93 +++
>  .../execution/ubo/matrix/column-major.shader_test  | 176 ++
>  .../execution/ubo/matrix/column-vs-row.shader_test | 303 +
>  .../execution/ubo/matrix/complex.shader_test   | 624 ++
>  .../ubo/matrix/different-matrix-stride.shader_test | 260 
>  .../ubo/matrix/indirect-column-major.shader_test   | 120 
>  .../ubo/matrix/indirect-row-major.shader_test  | 120 
>  .../execution/ubo/matrix/row-major.shader_test | 175 +
>  .../arb_gl_spirv/execution/ubo/simple.shader_test  | 131 
>  .../execution/ubo/two-stages.shader_test   | 276 
>  .../execution/ubo/two-ubos.shader_test | 141 +
>  .../execution/uniform/array.shader_test|   2 +
>  .../execution/uniform/arrays-of-arrays.shader_test |   2 +
>  .../uniform/atomic-uint-aoa-cs.shader_test |   2 +
>  .../uniform/atomic-uint-aoa-fs.shader_test |   2 +
>  .../uniform/atomic-uint-array-cs.shader_test   |   2 +
>  .../uniform/atomic-uint-array-fs.shader_test   |   2 +
>  .../execution/uniform/atomic-uint-cs.shader_test   |   2 +
>  .../execution/uniform/atomic-uint-fs.shader_runner |   1 +
>  ...ic-uint-mixing-with-normal-uniforms.shader_test |

[Piglit] [PATCH v2 00/17] ARB_gl_spirv: v2 ubo/ssbo tests

2018-09-27 Thread Alejandro Piñeiro

Hi,

this is the second version of the ubo/ssbo tests for
ARB_gl_spirv. Differences compared with v1:

  * Rebased against today master.

  * Updated the pass checks on several tests. Instead of a silly
addition/substraction of the ubo/ssbo content, and check a
specific value, the value of the individual ubo/ssbo members are
checked.

  * array-complex became even more complex. Specifically thinking on
testing some cases mentioned by Timothy feedback on the mesa-dev
series.

v1 announcement:
   https://lists.freedesktop.org/archives/piglit/2018-September/025065.html

The series can be found on this branch:
   https://github.com/Igalia/piglit/tree/arb_gl_spirv-series5-ubo-ssbo-v2


Alejandro Piñeiro (15):
  shader_runner/spirv: log skip reason on ARB_gl_spirv extension check
  shader_runner/spirv: check if test really includes spirv
  shader_runner: add force_no_names mode
  shader_runner: add support for glGetProgram queries
  arb_gl_spirv: add GL_ACTIVE_UNIFORMS checks
  arb_gl_spirv: add some simple ubo tests
  arb_gl_spirv: add simple ubo tests with matrices
  arb_gl_spirv: add complex ubo test with matrices
  arb_gl_spirv: add a array of ubo test, with complex ubo content
  arb_gl_spirv: add ubo matrix test with different matrix_stride
  arb_gl_spirv: add ubo array test with different array_stride
  arb_gl_spirv: add some simple ssbo tests
  arb_gl_spirv: add simple ssbo tests with matrices
  arb_gl_spirv: adding complex ssbo matrix test
  arb_gl_spirv: add ssbo test using std140 and std430

Neil Roberts (2):
  arb_gl_spirv: Add tests for UBOs with explicit offsets
  arb_gl_spirv: Add a test for SSBOs with an unsized array

 tests/shaders/shader_runner.c  | 485 +++---
 .../execution/ssbo/array-indirect.shader_test  | 121 
 .../execution/ssbo/array-inside-ssbo.shader_test   | 189 ++
 .../ssbo/array-of-arrays-inside-ssbo.shader_test   | 203 ++
 .../arb_gl_spirv/execution/ssbo/array.shader_test  | 173 +
 .../execution/ssbo/matrix/column-major.shader_test | 188 ++
 .../ssbo/matrix/column-vs-row.shader_test  | 291 +
 .../execution/ssbo/matrix/complex.shader_test  | 702 +
 .../ssbo/matrix/indirect-column-major.shader_test  | 132 
 .../ssbo/matrix/indirect-row-major.shader_test | 132 
 .../execution/ssbo/matrix/row-major.shader_test| 194 ++
 .../arb_gl_spirv/execution/ssbo/simple.shader_test | 138 
 .../ssbo/two-ssbo-different-layouts.shader_test| 372 +++
 .../execution/ssbo/two-ssbo.shader_test| 151 +
 .../execution/ssbo/two-stages.shader_test  | 279 
 .../execution/ssbo/unsized-array.shader_test   | 119 
 .../execution/ubo/array-complex.shader_test| 360 +++
 .../array-different-array-stride-ubo.shader_test   | 271 
 .../execution/ubo/array-indirect.shader_test   | 108 
 .../execution/ubo/array-inside-ubo.shader_test | 154 +
 .../ubo/array-of-arrays-inside-ubo.shader_test | 184 ++
 .../arb_gl_spirv/execution/ubo/array.shader_test   | 140 
 .../ubo/explicit-offset-nested-struct.shader_test  |  81 +++
 .../execution/ubo/explicit-offset.shader_test  |  80 +++
 .../execution/ubo/location-0-crash.shader_test |  93 +++
 .../execution/ubo/matrix/column-major.shader_test  | 176 ++
 .../execution/ubo/matrix/column-vs-row.shader_test | 303 +
 .../execution/ubo/matrix/complex.shader_test   | 624 ++
 .../ubo/matrix/different-matrix-stride.shader_test | 260 
 .../ubo/matrix/indirect-column-major.shader_test   | 120 
 .../ubo/matrix/indirect-row-major.shader_test  | 120 
 .../execution/ubo/matrix/row-major.shader_test | 175 +
 .../arb_gl_spirv/execution/ubo/simple.shader_test  | 131 
 .../execution/ubo/two-stages.shader_test   | 276 
 .../execution/ubo/two-ubos.shader_test | 141 +
 .../execution/uniform/array.shader_test|   2 +
 .../execution/uniform/arrays-of-arrays.shader_test |   2 +
 .../uniform/atomic-uint-aoa-cs.shader_test |   2 +
 .../uniform/atomic-uint-aoa-fs.shader_test |   2 +
 .../uniform/atomic-uint-array-cs.shader_test   |   2 +
 .../uniform/atomic-uint-array-fs.shader_test   |   2 +
 .../execution/uniform/atomic-uint-cs.shader_test   |   2 +
 .../execution/uniform/atomic-uint-fs.shader_runner |   1 +
 ...ic-uint-mixing-with-normal-uniforms.shader_test |   2 +
 .../uniform/atomic-uint-several-slots.shader_test  |   2 +
 .../execution/uniform/embedded-structs.shader_test |   2 +
 .../uniform/index-matches-location.shader_test |   2 +
 .../uniform/initializer-complex.shader_test|   2 +
 .../uniform/initializer-dvec4.shader_test  |   3 +
 .../uniform/initializer-mat4x3.shader_test |   4 +
 .../execution/uniform/initializer.shader_test  |   2 +
 .../uniform/nonsequential-locations.shader_test|   2 +
 .