vkrunner recently added a new option (--device-id=) for selecting
the physical device. This is quite useful in a multi-GPU scenario.
This allows to set some vkrunner options via vkrunner:extra_args
in the configuration file or with PIGLIT_VKRUNNER_EXTRA_ARGS.
Signed-off-by: Samuel Pitoiset
Reviewed-by: Samuel Pitoiset
Thanks Karol.
On 04/25/2018 12:09 PM, Karol Herbst wrote:
currently fails in mesa with:
ir_dereference_variable @ 0x1446830 specifies undeclared variable `wrongHandle'
@ 0xffd1b0
v2: add image test
convert to compiler test
quote the spec
v3: use
Patches 1,2,3,5 are:
Reviewed-by: Samuel Pitoiset
Sorry for the delay.
On 04/12/2018 12:58 PM, Karol Herbst wrote:
conversion of bounded sampler2D to uvec2 is legal with
ARB_bindless_texture. Currently Mesa fails with an assert:
../src/compiler/glsl/opt_function_inlining.cpp:248:
void
On 04/12/2018 12:58 PM, Karol Herbst wrote:
currently fails in mesa with:
ir_dereference_variable @ 0x1446830 specifies undeclared variable `wrongHandle'
@ 0xffd1b0
v2: add image test
convert to compiler test
quote the spec
Signed-off-by: Karol Herbst
---
.../compiler/images/fun
Same comment as the previous one. Images need to be tested.
On 04/08/2018 10:56 PM, Karol Herbst wrote:
Signed-off-by: Karol Herbst
---
.../sampler-vertex-attrib-input-output.shader_test | 41 ++
...ampler-vertex-uniform-input-output.shader_test} | 0
2 files changed, 4
Can you please write a similar test with bindless images? We usually
have both tests.
Reviewed-by: Samuel Pitoiset
On 04/08/2018 10:56 PM, Karol Herbst wrote:
Signed-off-by: Karol Herbst
---
.../basic-arithmetic-uvec2-texture2D.shader_test | 37 ++
1 file changed
support();
+ parse_uint64s(line, uint64s, 1, NULL);
parse_uints() is enough because it's an index in the resident_handles
array (where the max is 31).
With that changed, patch is:
Reviewed-by: Samuel Pitoiset
+ glVertexAttribL1ui64ARB(lo
Can you please reference the spec as much as possible? See other
ARB_bindless_texture if you need an example.
More comments below.
On 04/10/2018 12:32 PM, Karol Herbst wrote:
currently fails in mesa with:
ir_dereference_variable @ 0x1446830 specifies undeclared variable `wrongHandle'
@ 0xffd1
Reviewed-by: Samuel Pitoiset
On 11/01/2017 11:25 PM, Timothy Arceri wrote:
---
tests/spec/arb_shader_atomic_counter_ops/compiler/functions.frag | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/spec/arb_shader_atomic_counter_ops/compiler/functions.frag
b/tests/spec
Reviewed-by: Samuel Pitoiset
On 10/23/2017 02:20 AM, Ilia Mirkin wrote:
In this case, the derived values are used in a shader that doesn't
contain the main function.
Signed-off-by: Ilia Mirkin
---
.../execution/separate-global-id-2.shader_test
Reviewed-by: Samuel Pitoiset
On 10/23/2017 12:12 AM, Ilia Mirkin wrote:
The new gl_LocalGroupSizeARB variable is meant to work with fixed sizes
as well. Test it.
Signed-off-by: Ilia Mirkin
---
.../execution/fixed-local-size.shader_test | 32 ++
1 file changed
Reviewed-by: Samuel Pitoiset
On 10/22/2017 11:58 PM, Ilia Mirkin wrote:
Only one shader is required to have a local_size layout, and it's not
required to be the one with the main function. Add tests which verify
that these derived values are properly computed in such a circumstance.
S
On 08/25/2017 06:08 AM, Timothy Arceri wrote:
On 23/08/17 17:23, Samuel Pitoiset wrote:
On 08/23/2017 06:11 AM, Timothy Arceri wrote:
---
This was my fault. The flaw was in my suggestion from the code
review.
You probably need to use PIGLIT_HAS_ERRORS as well.
I don't thi
On 08/23/2017 06:11 AM, Timothy Arceri wrote:
---
This was my fault. The flaw was in my suggestion from the code
review.
You probably need to use PIGLIT_HAS_ERRORS as well.
tests/general/draw-pixels.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/g
On 08/22/2017 06:45 AM, Timothy Arceri wrote:
On 11/08/17 01:22, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
tests/general/draw-pixels.c | 2 ++
tests/general/geterror-inside-begin.c | 2 ++
tests/general/geterror-invalid-enum.c | 2 ++
tests/general/masked
On 08/22/2017 06:06 AM, Timothy Arceri wrote:
You seem to have skipped over 2.1
I have a patch locally for that.
Reviewed-by: Timothy Arceri
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pig
On 08/22/2017 02:15 AM, Timothy Arceri wrote:
On 11/08/17 01:22, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-2.0/api/active-sampler-conflict.c | 1 +
tests/spec/gl-2.0/api/clip-flag-behavior.c | 33
++---
You missed
On 08/22/2017 02:06 AM, Timothy Arceri wrote:
On 11/08/17 01:22, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_view/clear-into-view-2d-array.c | 1 +
tests/spec/arb_texture_view/clear-into-view-2d.c | 1 +
tests/spec
On 08/21/2017 09:38 AM, Timothy Arceri wrote:
On 11/08/17 01:22, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_occlusion_query2/api.c| 34
+---
tests/spec/arb_occlusion_query2/render.c | 1 +
2 files changed, 24 insertions
Only allowed with GLSL 4.60.
Cc: Ilia Mirkin
Signed-off-by: Samuel Pitoiset
---
.../glsl-4.50/compiler/extra-semilons-at-global-scope.frag | 12
1 file changed, 12 insertions(+)
create mode 100644
tests/spec/glsl-4.50/compiler/extra-semilons-at-global-scope.frag
diff --git a
On 08/10/2017 07:04 PM, Ilia Mirkin wrote:
And an identical test to make sure it's disallowed at earlier GLSL
versions? I see no such test already.
I can do that yeah.
On Thu, Aug 10, 2017 at 12:44 PM, Samuel Pitoiset
wrote:
Cc: Ian Romanick
Signed-off-by: Samuel Pit
Cc: Ian Romanick
Signed-off-by: Samuel Pitoiset
---
.../compiler/extra-semilons-at-global-scope.frag| 17 +
1 file changed, 17 insertions(+)
create mode 100644
tests/spec/glsl-4.60/compiler/extra-semilons-at-global-scope.frag
diff --git a/tests/spec/glsl-4.60/compiler
Signed-off-by: Samuel Pitoiset
---
tests/util/piglit-util-gl.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/util/piglit-util-gl.c b/tests/util/piglit-util-gl.c
index 5dc1bbc7d..d8b8f8559 100644
--- a/tests/util/piglit-util-gl.c
+++ b/tests/util/piglit-util-gl.c
@@ -646,6 +646,7
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-3.2/adj-prims.c | 1 +
tests/spec/gl-3.2/basevertex-vertexid.c | 1 +
tests/spec/gl-3.2/clear-no-buffers.c | 1 +
tests/spec/gl-3.2/depth-tex-sampling.c
Signed-off-by: Samuel Pitoiset
---
tests/general/draw-pixels.c | 2 ++
tests/general/geterror-inside-begin.c | 2 ++
tests/general/geterror-invalid-enum.c | 2 ++
tests/general/masked-clear.c | 1 +
tests/general/roundmode-pixelstore.c | 10 +++---
tests/general
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c | 2 ++
tests/spec/gl-4.5/named-framebuffer-draw-buffers-errors.c | 2 ++
tests/spec/gl-4.5/named-framebuffer-read-buffer-errors.c | 2 ++
3 files changed, 6 insertions(+)
diff --git a/tests/spec/gl
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-3.1/default-vao.c | 1 +
tests/spec/gl-3.1/draw-buffers-errors.c | 1 +
tests/spec/gl-3.1/enable-vertex-array.c | 16 ++--
tests/spec/gl-3.1/genned-names.c | 1 +
tests/spec/gl-3.1/glsl-link-empty-prog.c | 2
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-3.3/minmax.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/gl-3.3/minmax.c b/tests/spec/gl-3.3/minmax.c
index 964e49964..2eae411b5 100644
--- a/tests/spec/gl-3.3/minmax.c
+++ b/tests/spec/gl-3.3/minmax.c
@@ -35,6 +35,7
Signed-off-by: Samuel Pitoiset
---
.../execution/check-explicit-location.c| 2 ++
.../execution/double_attribs.c | 22 +-
.../execution/getactiveattrib.c| 1 +
.../execution/max-vertex-attrib.c | 1
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_sync/ClientWaitSync-errors.c | 1 +
tests/spec/arb_sync/ClientWaitSync-timeout.c | 1 +
tests/spec/arb_sync/DeleteSync.c | 9 ++---
tests/spec/arb_sync/FenceSync-errors.c | 1 +
tests/spec/arb_sync/GetSynciv-errors.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_provoking_vertex/provoking-vertex-control.c | 1 +
tests/spec/arb_provoking_vertex/provoking-vertex-initial.c | 1 +
tests/spec/arb_provoking_vertex/quads-follow-provoking-vertex.c | 1 +
tests/spec/arb_provoking_vertex/render.c
Signed-off-by: Samuel Pitoiset
---
.../gl-3.0/api/bindfragdata-invalid-parameters.c | 1 +
tests/spec/gl-3.0/api/bindfragdata-link-error.c| 1 +
.../gl-3.0/api/bindfragdata-nonexistent-variable.c | 1 +
tests/spec/gl-3.0/api/clearbuffer-common.c | 1 +
tests/spec/gl-3.0/api
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_shader_objects/bindattriblocation-scratch-name.c | 2 ++
tests/spec/arb_shader_objects/clear-with-deleted.c| 8 ++--
tests/spec/arb_shader_objects/delete-repeat.c | 8 ++--
tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/gl-2.0/api/active-sampler-conflict.c | 1 +
tests/spec/gl-2.0/api/clip-flag-behavior.c | 33 ++---
2 files changed, 19 insertions(+), 15 deletions(-)
diff --git a/tests/spec/gl-2.0/api/active-sampler-conflict.c
b/tests
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_occlusion_query2/api.c| 34 +---
tests/spec/arb_occlusion_query2/render.c | 1 +
2 files changed, 24 insertions(+), 11 deletions(-)
diff --git a/tests/spec/arb_occlusion_query2/api.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_framebuffer_no_attachments/atomic.c | 1 +
tests/spec/arb_framebuffer_no_attachments/minmax.c | 1 +
tests/spec/arb_framebuffer_no_attachments/params.c | 1 +
tests/spec/arb_framebuffer_no_attachments/query.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_debug_output/api_error.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/spec/arb_debug_output/api_error.c
b/tests/spec/arb_debug_output/api_error.c
index 596092f45..43294fbfb 100644
--- a/tests/spec/arb_debug_output/api_error.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_cube_map_array/cubemap-lod.c | 1 +
tests/spec/arb_texture_cube_map_array/cubemap.c | 1 +
tests/spec/arb_texture_cube_map_array/fbo-cubemap-array.c | 1 +
tests/spec/arb_texture_cube_map_array
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_storage_multisample/tex-param.c | 2 ++
tests/spec/arb_texture_storage_multisample/tex-storage.c | 2 ++
2 files changed, 4 insertions(+)
diff --git a/tests/spec/arb_texture_storage_multisample/tex-param.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_rectangle/texrect-simple.c | 1 +
tests/spec/arb_texture_rectangle/texture-base-level-error.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/arb_texture_rectangle/texrect-simple.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_view/clear-into-view-2d-array.c | 1 +
tests/spec/arb_texture_view/clear-into-view-2d.c | 1 +
tests/spec/arb_texture_view/clear-into-view-layered.c| 1 +
tests/spec/arb_texture_view/copytexsubimage-layers.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_explicit_uniform_location/array-elements.c| 1 +
tests/spec/arb_explicit_uniform_location/inactive-uniform.c | 1 +
tests/spec/arb_explicit_uniform_location/loc-boundaries.c| 1 +
tests/spec/arb_explicit_uniform_location/minmax.c
Signed-off-by: Samuel Pitoiset
---
.../spec/arb_occlusion_query/gen_delete_while_active.c | 18 +++---
tests/spec/arb_occlusion_query/occlusion_query.c | 1 +
.../spec/arb_occlusion_query/occlusion_query_conform.c | 2 ++
.../occlusion_query_meta_no_fragments.c
Signed-off-by: Samuel Pitoiset
---
.../arb_shading_language_420pack/execution/active-sampler-conflict.c| 2 ++
tests/spec/arb_shading_language_420pack/execution/binding-layout.c | 1 +
.../arb_shading_language_420pack/execution/multiple-layout-qualifiers.c | 1 +
3 files changed, 4
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_stencil8/draw.c| 1 +
tests/spec/arb_texture_stencil8/fbo-stencil8.c| 1 +
tests/spec/arb_texture_stencil8/getteximage.c | 1 +
tests/spec/arb_texture_stencil8/stencil-texture.c | 23 ++-
4 files
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_gpu_shader_fp64/double_in_bool_uniform.c | 1 +
.../arb_gpu_shader_fp64/execution/double-gettransformfeedbackvarying.c | 1 +
tests/spec/arb_gpu_shader_fp64/execution/fs-getuniformdv.c | 1 +
tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_get_program_binary/api-errors.c | 1 +
tests/spec/arb_get_program_binary/overrun.c | 1 +
tests/spec/arb_get_program_binary/retrievable_hint.c | 1 +
3 files changed, 3 insertions(+)
diff --git a/tests/spec/arb_get_program_binary
Signed-off-by: Samuel Pitoiset
---
.../spec/arb_program_interface_query/compare-with-shader-subroutine.c | 1 +
tests/spec/arb_program_interface_query/getprograminterfaceiv.c| 1 +
tests/spec/arb_program_interface_query/getprogramresourceindex.c | 1 +
tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/khr_debug/debug-object-label.c | 1 +
tests/spec/khr_debug/debug-push-pop-group.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/khr_debug/debug-object-label.c
b/tests/spec/khr_debug/debug-object-label.c
index 7e89bd50f
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_internalformat_query2/api-errors.c | 1 +
tests/spec/arb_internalformat_query2/color-encoding.c | 1 +
tests/spec/arb_internalformat_query2/filter.c | 1 +
tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_clear_buffer_object/formats.c | 1 +
tests/spec/arb_clear_buffer_object/invalid-internal-format.c | 1 +
tests/spec/arb_clear_buffer_object/invalid-size.c| 1 +
tests/spec/arb_clear_buffer_object/mapped.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_tessellation_shader/get-tcs-params.c | 1 +
tests/spec/arb_tessellation_shader/get-tes-params.c | 1 +
tests/spec/arb_tessellation_shader/invalid-get-program-params.c | 1 +
tests/spec/arb_tessellation_shader/invalid
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_clear_texture/3d.c | 2 ++
tests/spec/arb_clear_texture/base-formats.c| 2 ++
tests/spec/arb_clear_texture/clear-max-level.c | 1 +
tests/spec/arb_clear_texture/cube.c| 1 +
tests/spec/arb_clear_texture/depth
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_texture_array/errors.c | 1 +
tests/spec/ext_texture_array/gen-mipmap.c | 1 +
tests/spec/ext_texture_array/maxlayers.c | 1 +
3 files changed, 3 insertions(+)
diff --git a/tests/spec/ext_texture_array/errors.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_transform_feedback3/begin_end.c | 1 +
.../arb_transform_feedback3/bind_buffer_invalid_index.c | 1 +
.../draw_using_invalid_stream_index.c| 1 +
.../arb_transform_feedback3/end_query_with_name_zero.c | 1
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_transform_feedback_overflow_query/basic.c | 1 +
tests/spec/arb_transform_feedback_overflow_query/errors.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/arb_transform_feedback_overflow_query/basic.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/nv_conditional_render/begin-while-active.c | 1 +
tests/spec/nv_conditional_render/begin-zero.c | 1 +
tests/spec/nv_conditional_render/bitmap.c | 1 +
tests/spec/nv_conditional_render/blitframebuffer.c| 1 +
tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_timer_query/timestamp-get.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/arb_timer_query/timestamp-get.c
b/tests/spec/arb_timer_query/timestamp-get.c
index 9ff6ee7c3..39676c692 100644
--- a/tests/spec/arb_timer_query/timestamp
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_texture_format_bgra/api-errors.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/ext_texture_format_bgra/api-errors.c
b/tests/spec/ext_texture_format_bgra/api-errors.c
index e0fe53285..1a59e63b1 100644
--- a/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_packed_float/getteximage-invalid-format-for-packed-type.c | 1 +
tests/spec/ext_packed_float/pack.c | 1 +
2 files changed, 2 insertions(+)
diff --git
a/tests/spec/ext_packed_float/getteximage-invalid
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_buffer_range/dlist.c| 1 +
tests/spec/arb_texture_buffer_range/errors.c | 1 +
tests/spec/arb_texture_buffer_range/ranges-2.c | 1 +
tests/spec/arb_texture_buffer_range/ranges.c | 1 +
4 files changed, 4 insertions(+)
diff --git
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_texture_storage/attach-before-storage.c | 1 +
tests/spec/arb_texture_storage/texture-storage.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/arb_texture_storage/attach-before-storage.c
b/tests/spec/arb_texture_storage
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_texture_integer/api-drawpixels.c | 1 +
tests/spec/ext_texture_integer/api-readpixels.c | 1 +
tests/spec/ext_texture_integer/api-teximage.c | 1 +
tests/spec/ext_texture_integer/fbo
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_multisample/beginend.c | 2 ++
tests/spec/arb_multisample/pushpop.c | 2 ++
2 files changed, 4 insertions(+)
diff --git a/tests/spec/arb_multisample/beginend.c
b/tests/spec/arb_multisample/beginend.c
index 5d2a5b40b..015a3fb9a 100644
--- a
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_vertex_array_bgra/api-errors.c | 1 +
tests/spec/arb_vertex_array_bgra/get.c| 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/arb_vertex_array_bgra/api-errors.c
b/tests/spec/arb_vertex_array_bgra/api-errors.c
index
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c | 1 +
tests/spec/ext_packed_depth_stencil/errors.c| 1 +
tests/spec/ext_packed_depth_stencil/getteximage.c | 1 +
tests/spec/ext_packed_depth_stencil/readdrawpixels.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_draw_instanced/execution/dlist.c | 1 +
tests/spec/arb_draw_instanced/execution/negative-arrays-first-negative.c | 1 +
tests/spec/arb_draw_instanced/execution/negative-elements-type.c | 1 +
3 files changed, 3
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_map_buffer_range/map_buffer_range_error_check.c | 2 ++
tests/spec/arb_map_buffer_range/map_buffer_range_test.c| 2 ++
2 files changed, 4 insertions(+)
diff --git a/tests/spec/arb_map_buffer_range/map_buffer_range_error_check.c
b/tests
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_draw_elements_base_vertex/dlist.c | 1 +
tests/spec/arb_draw_elements_base_vertex/multidrawelements.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/spec/arb_draw_elements_base_vertex/dlist.c
b/tests/spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_texture_swizzle/api.c | 12 +++-
.../ext_texture_swizzle/depth_texture_mode_and_swizzle.c | 1 +
tests/spec/ext_texture_swizzle/swizzle.c | 1 +
3 files changed, 9 insertions(+), 5
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_transform_feedback2/api-queries.c | 1 +
.../spec/arb_transform_feedback2/cannot-bind-when-active.c | 13 +
.../arb_transform_feedback2/change-objects-while-paused.c | 1 +
tests/spec/arb_transform_feedback2/gen-names
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_instanced_arrays/vertex-attrib-divisor-index-error.c | 1 +
1 file changed, 1 insertion(+)
diff --git
a/tests/spec/arb_instanced_arrays/vertex-attrib-divisor-index-error.c
b/tests/spec/arb_instanced_arrays/vertex-attrib-divisor-index-error.c
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_robustness/client-mem-bounds.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/arb_robustness/client-mem-bounds.c
b/tests/spec/arb_robustness/client-mem-bounds.c
index cbc0923e7..b67907cd7 100644
--- a/tests/spec/arb_robustness
Reviewed-by: Samuel Pitoiset
On 08/09/2017 09:27 AM, Iago Toral Quiroga wrote:
TextureStorage* functions should produce INVALID_OPERATION instead
of INVALID_ENUM when the target is not valid.
---
tests/spec/arb_direct_state_access/getcompressedtextureimage.c | 2 +-
tests/spec
Reviewed-by: Samuel Pitoiset
On 08/06/2017 06:48 AM, Timothy Arceri wrote:
This only tests for errors when is 0, but its a start.
v2:
- make use of piglit_check_gl_error()
- shorten spec quote
Cc: Samuel Pitoiset
---
Please note the patch to update gl.xml was too big for the list
On 08/04/2017 03:43 AM, Timothy Arceri wrote:
This only tests for errors when is 0, but its a start.
---
tests/all.py | 6 +
tests/spec/CMakeLists.txt | 1 +
tests/spec/ext_memory_object/CMakeLists.gl.txt | 14 ++
tests/spec/ext_
1/2 got lost?
On 08/04/2017 03:43 AM, Timothy Arceri wrote:
This only tests for errors when is 0, but its a start.
---
tests/all.py | 6 +
tests/spec/CMakeLists.txt | 1 +
tests/spec/ext_memory_object/CMakeLists.gl.txt | 14 ++
te
This test currently fails to return the correct 64-bit handle value
with Mesa. This is because it ends up by performing the slower
convert-and-copy process instead of just memcpy'ing the value.
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_bindless_texture/uniform.c
Thanks!
For the series:
Reviewed-by: Samuel Pitoiset
On 07/06/2017 05:53 AM, Timothy Arceri wrote:
---
.../spec/ext_framebuffer_multisample/alpha-blending-after-rendering.c | 1 +
.../ext_framebuffer_multisample/alpha-to-coverage-dual-src-blend.cpp | 1 +
.../alpha-to-coverage-no-draw
Reviewed-by: Samuel Pitoiset
On 07/06/2017 06:41 AM, Timothy Arceri wrote:
Here we also move some of the config setup to the top of the
file as this is the layout of piglit tests everywhere else in
piglit.
---
tests/spec/arb_texture_buffer_object/bufferstorage.c| 11
For the series:
Reviewed-by: Samuel Pitoiset
On 07/05/2017 07:23 AM, Timothy Arceri wrote:
---
tests/spec/arb_shader_atomic_counters/buffer-binding.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/tests/spec/arb_shader_atomic_counters/buffer-binding.c
b
Reviewed-by: Samuel Pitoiset
On 06/29/2017 07:39 AM, Timothy Arceri wrote:
---
tests/bugs/crash-cubemap-order.c| 1 +
tests/bugs/crash-texparameter-before-teximage.c | 1 +
tests/bugs/drawbuffer-modes.c | 1 +
tests/bugs/fdo10370.c
Reviewed-by: Samuel Pitoiset
On 06/29/2017 08:07 AM, Timothy Arceri wrote:
---
tests/spec/gl-1.0/blend.c | 1 +
tests/spec/gl-1.0/dlist-bitmap.c| 1 +
tests/spec/gl-1.0/dlist-shademodel.c| 1 +
tests/spec/gl-1.0/drawpixels
Reviewed-by: Samuel Pitoiset
On 06/29/2017 08:24 AM, Timothy Arceri wrote:
---
.../api/bindfragdataindexed-invalid-parameters.c | 1 +
tests/spec/arb_blend_func_extended/api/blend-api.c| 1 +
tests/spec/arb_blend_func_extended/api/builtins.c | 1 +
.../spec
Signed-off-by: Samuel Pitoiset
---
tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c | 1 +
tests/spec/ext_packed_depth_stencil/errors.c| 8 ++--
tests/spec/ext_packed_depth_stencil/getteximage.c | 1 +
tests/spec/ext_packed_depth_stencil/readdrawpixels.c
On 06/27/2017 02:19 AM, Timothy Arceri wrote:
On 26/06/17 19:04, Samuel Pitoiset wrote:
Looks like you forgot to update fbo-storage-formats.c.
It's updated. Did you mean something else?
I meant, there is a GL_INVALID_ENUM which is not handled correctly, no?
With that fixed, pat
Looks like you forgot to update fbo-storage-formats.c.
With that fixed, patch is:
Reviewed-by: Samuel Pitoiset
Thanks!
On 06/26/2017 08:00 AM, Timothy Arceri wrote:
---
tests/fbo/fbo-1d.c | 1 +
tests/fbo/fbo-3d.c | 1
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_viewport_array/bounds.c | 53 --
tests/spec/arb_viewport_array/clear.c | 1 +
.../spec/arb_viewport_array/depth_range_indices.c | 45 +-
tests/spec/arb_viewport_array/minmax.c
---
tests/spec/amd_vertex_shader_viewport_index/render.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/amd_vertex_shader_viewport_index/render.c
b/tests/spec/amd_vertex_shader_viewport_index/render.c
index a5c59c96b..a08cbe931 100644
--- a/tests/spec/amd_vertex_shader_viewport_in
Signed-off-by: Samuel Pitoiset
---
.../clip-control-depth-precision.c | 1 +
tests/spec/arb_clip_control/clip-control.c | 24 --
tests/spec/arb_clip_control/viewport.c | 1 +
3 files changed, 15 insertions(+), 11 deletions(-)
diff --git
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
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_bindless_texture/border-color.c | 7 +++
tests/spec/arb_bindless_texture/conversions.c| 1 +
tests/spec/arb_bindless_texture/errors.c | 1 +
tests/spec/arb_bindless_texture/handles.c| 16
tests/spec
ping?
On 05/25/2017 08:02 PM, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
.../nested-struct-arrays-mismatch-format.frag | 27
.../nested-struct-with-arrays-no-qualifiers.frag | 29 ++
.../images/struct-with-arrays-mismatch
Thanks for this patchset Tim.
Reviewed-by: Samuel Pitoiset
On 06/05/2017 02:19 AM, Timothy Arceri wrote:
---
tests/fbo/fbo-generatemipmap-1d.c | 1 +
tests/spec/arb_texture_view/lifetime_format.c | 1 +
tests/spec/arb_texture_view/rendering_target.c | 1 +
tests/texturing
UAL_DOUBLE;
+config.khr_no_error_support = PIGLIT_NO_ERRORS
There is a missing ; here. Fixed locally.
Reviewed-by: Samuel Pitoiset
PIGLIT_GL_TEST_CONFIG_END
static GLint MaxSize;
/**
* Compute size (in megabytes) of a texture of the given dime
Reviewed-by: Samuel Pitoiset
On 06/05/2017 02:19 AM, Timothy Arceri wrote:
---
tests/spec/gl-3.2/layered-rendering/blit.c | 1 +
tests/spec/gl-3.2/layered-rendering/clear-color-all-types.c| 1 +
.../layered-rendering/clear-color-mismatched-layer-count.c | 1
Reviewed-by: Samuel Pitoiset
On 06/05/2017 02:19 AM, Timothy Arceri wrote:
---
tests/spec/arb_invalidate_subdata/buffer.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/spec/arb_invalidate_subdata/buffer.c
b/tests/spec/arb_invalidate_subdata/buffer.c
index 637596a..dedb937
Signed-off-by: Samuel Pitoiset
---
tests/spec/arb_shader_image_load_store/atomicity.c | 1 +
tests/spec/arb_shader_image_load_store/bitcast.c| 1 +
tests/spec/arb_shader_image_load_store/coherency.c | 1 +
tests/spec/arb_shader_image_load_store/dead-fragments.c
Signed-off-by: Samuel Pitoiset
---
.../nested-struct-arrays-mismatch-format.frag | 27
.../nested-struct-with-arrays-no-qualifiers.frag | 29 ++
.../images/struct-with-arrays-mismatch-format.frag | 22
.../images/struct-with-arrays
Reviewed-by: Samuel Pitoiset
On 05/18/2017 04:00 AM, Timothy Arceri wrote:
---
.../array-ssbo-binding.c | 1 +
.../deletebuffers.c| 1 +
.../getintegeri_v.c| 13 +---
.../layout-std140-write
1 - 100 of 213 matches
Mail list logo
