Hi Chad,
The patch breaks GetProcAddress for libGL, because libGL doesn't
export a lot of GL functions and glXGetProcAddress is the only way to
get them. Any idea for a fix?
Marek
On Fri, Apr 24, 2015 at 9:36 PM, Chad Versace chad.vers...@intel.com wrote:
On Mon 13 Apr 2015, Marek Olšák wrote
From: Marek Olšák marek.ol...@amd.com
I probably can't test more at this point, because it's a no-op in Mesa
just like glWaitSync.
---
tests/all.py | 6 +++
.../spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 59 --
2 files changed, 62
If the spec and the example in the spec don't agree, it's a spec bug, isn't it?
Marek
On Sat, Apr 25, 2015 at 6:59 AM, Timothy Arceri t_arc...@yahoo.com.au wrote:
On Mon, 2015-04-13 at 20:28 +0200, Marek Olšák wrote:
From: Daniel Kurtz djku...@chromium.org
AFAICT from [0], only PopDebugGroup
Sorry I forgot about your email.
You can find these and other patches in the tessellation branch here:
http://cgit.freedesktop.org/~mareko/piglit/
Marek
On Tue, Apr 7, 2015 at 3:53 AM, Ian Romanick i...@freedesktop.org wrote:
On 04/06/2015 04:39 PM, Marek Olšák wrote:
Most of the tessellation
-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/spec/ext_unpack_subimage/ext_unpack_subimage.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/spec/ext_unpack_subimage/ext_unpack_subimage.c
b/tests/spec/ext_unpack_subimage
From: Daniel Kurtz djku...@chromium.org
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/egl/spec/egl_khr_fence_sync
that the requested client extension is also
not supported.
This keeps piglit_is_extension_in_string() from crashing when it tries
to find a needle in a NULL haystack.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/util/piglit-util-egl.c | 10
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
b/tests/egl/spec
dispatch can ACK them. If not, I will drop them.
Please review.
Marek Olšák (1):
piglit-util-egl: fix a typo in piglit_egl_get_default_display
Daniel Kurtz (13):
piglit-util-egl: check if EGL_EXT_client_extensions is supported
egl_khr_fence_sync: fix eglGetSyncAttribKHR
From: Daniel Kurtz djku...@chromium.org
eglGetSyncAttribKHR() returns EGL_TRUE on success, thus ok will be true
on success.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 2 +-
1 file
PASS. For this test, we really want to
return SKIP if either thread returns SKIP.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 12 ++--
1 file changed, 10 insertions(+), 2
/egl/extensions/KHR/EGL_KHR_fence_sync.txt
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/tests/egl/spec/egl_khr_fence_sync
From: Daniel Kurtz djku...@chromium.org
Using ES1 dispatch works now, so let's use it for gles tests.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
tests/egl/spec/egl_khr_create_context/default-major-version-gles.c | 3 +--
tests/egl/spec
-by: Marek Olšák marek.ol...@amd.com
---
.../spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 51 --
1 file changed, 51 deletions(-)
diff --git a/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
b/tests/egl/spec/egl_khr_fence_sync/egl_khr_fence_sync.c
index e6447bc..e7b4b31
From: Marek Olšák marek.ol...@amd.com
---
tests/util/piglit-util-egl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/util/piglit-util-egl.c b/tests/util/piglit-util-egl.c
index 8d48f30..1754602 100644
--- a/tests/util/piglit-util-egl.c
+++ b/tests/util/piglit-util
From: Daniel Kurtz djku...@chromium.org
This is purely eye candy.
It makes eglCreateSyncKHR failures match their spec description.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
.../spec/egl_khr_fence_sync/egl_khr_fence_sync.c | 28
From: Marek Olšák marek.ol...@amd.com
---
.../glsl-1.10/execution/uniform-update.shader_test | 50 ++
1 file changed, 50 insertions(+)
create mode 100644 tests/spec/glsl-1.10/execution/uniform-update.shader_test
diff --git a/tests/spec/glsl-1.10/execution/uniform
From: Daniel Kurtz djku...@chromium.org
AFAICT from [0], only PopDebugGroup() adds a message to the log, not
PushDebugGroup().
[0] https://www.opengl.org/registry/specs/KHR/debug.txt
Thus, there should only be three messages in test_push_pop_debug_group:
(1) DebugMessageInsert() -
From: Daniel Kurtz djku...@chromium.org
eglGetProcAddress() only supports extension functions.
Therefore, we must use dlsym() directly on the GL client library to look
up core functions.
The implementation here is a much simplified version of the one in
libepoxy. Most of the simplification is
/OES_compressed_paletted_texture.txt
It is totally reasonable that some implementations may return that error.
So, allow the test to pass in that case.
Signed-off-by: Daniel Kurtz djku...@chromium.org
Reviewed-by: Marek Olšák marek.ol...@amd.com
---
.../oes_compressed_paletted_texture-api.c | 14
Cc Daniel and Chad.
On Mon, Apr 13, 2015 at 8:28 PM, Marek Olšák mar...@gmail.com wrote:
Hi everyone,
These are mostly fixes that I took from Daniel Kurtz's github repository
because I think they are good and he said he had no time to upstream them.
I've been working on the fence
From: Marek Olšák marek.ol...@amd.com
---
...es-patch-input-array-float-index-rd.shader_test | 233 +
...tes-patch-input-array-vec2-index-rd.shader_test | 233 +
...tes-patch-input-array-vec3-index-rd.shader_test | 233 +
...tes-patch
From: Marek Olšák marek.ol...@amd.com
i.e. both patch and non-patch types.
This tests 64 inputs and outputs - more than any other shader can have.
---
...tes-both-input-array-float-index-rd.shader_test | 252 +
.../tes-both-input-array-vec2-index-rd.shader_test | 252
From: Marek Olšák marek.ol...@amd.com
The only allowed primitive type with tessellation is GL_PATCHES, so this is
the only way to draw rectangles.
---
tests/shaders/shader_runner.c | 4
tests/util/piglit-util-gl.c | 22 --
tests/util/piglit-util-gl.h | 5 -
3
From: Marek Olšák marek.ol...@amd.com
---
.../tcs-input-array-float-index-rd.shader_test | 238 +
.../tcs-input-array-vec2-index-rd.shader_test | 238 +
.../tcs-input-array-vec3-index-rd.shader_test | 238 +
.../tcs-input
From: Marek Olšák marek.ol...@amd.com
---
...s-patch-output-array-float-index-wr.shader_test | 218 +
...cs-patch-output-array-vec2-index-wr.shader_test | 218 +
...cs-patch-output-array-vec3-index-wr.shader_test | 218 +
...cs-patch
From: Marek Olšák marek.ol...@amd.com
---
.../tes-input-array-float-index-rd.shader_test | 237 +
.../tes-input-array-vec2-index-rd.shader_test | 237 +
.../tes-input-array-vec3-index-rd.shader_test | 237 +
.../tes-input
From: Marek Olšák marek.ol...@amd.com
---
...put-array-float-index-wr-before-tcs.shader_test | 224 +
...tput-array-vec2-index-wr-before-tcs.shader_test | 224 +
...tput-array-vec3-index-wr-before-tcs.shader_test | 224 +
...tput-array
From: Marek Olšák marek.ol...@amd.com
Each test has 2 arrays and one of them uses a negative relative index.
---
.../fs-input-array-float-index-rd.shader_test | 77 ++
.../fs-input-array-vec2-index-rd.shader_test | 77 ++
.../fs-input-array-vec3
From: Marek Olšák marek.ol...@amd.com
---
.../gs-output-array-float-index-wr.shader_test | 198 +
.../gs-output-array-vec2-index-wr.shader_test | 198 +
.../gs-output-array-vec3-index-wr.shader_test | 198 +
.../gs-output
From: Marek Olšák marek.ol...@amd.com
---
.../gs-input-array-float-index-rd.shader_test | 217 +
.../gs-input-array-vec2-index-rd.shader_test | 217 +
.../gs-input-array-vec3-index-rd.shader_test | 217 +
.../gs-input
From: Marek Olšák marek.ol...@amd.com
---
...tput-array-float-index-wr-before-gs.shader_test | 201 +
...utput-array-vec2-index-wr-before-gs.shader_test | 201 +
...utput-array-vec3-index-wr-before-gs.shader_test | 201 +
...utput-array
Most of the tessellation tests haven't been fully tested because of bugs in the
GLSL compiler or incomplete drivers. Sometimes I could only check if the
produced TGSI looks good, but in other cases even the GLSL compiler can't
produce an IR which makes sense.
The tests that can't be run with
From: Marek Olšák marek.ol...@amd.com
---
.../vs-output-array-float-index-wr.shader_test | 81 +
.../vs-output-array-vec2-index-wr.shader_test | 81 +
.../vs-output-array-vec3-index-wr.shader_test | 81 +
.../vs-output
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Thu, Mar 26, 2015 at 6:10 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
This is a rip off of:
https://www.khronos.org/registry/webgl/conformance-suites/1.0.2/conformance/attribs/gl-vertex-attrib-render.html
Which presently fails
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Mar 20, 2015 at 2:41 AM, Tom Stellard thomas.stell...@amd.com wrote:
From: Marek Olšák marek.ol...@amd.com
This tests for a bug causing either a crash or a mis-compile in the
radeonsi compiler.
Tom Stellard:
- Decrease
Well, technically speaking, this should be in
spec/glsl-1.10/execution/variable-indexing.
Marek
On Fri, Mar 20, 2015 at 10:34 AM, Marek Olšák mar...@gmail.com wrote:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Mar 20, 2015 at 2:41 AM, Tom Stellard thomas.stell...@amd.com
For the series:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Sun, Feb 1, 2015 at 11:22 PM, Eric Anholt e...@anholt.net wrote:
---
tests/spec/arb_color_buffer_float/common.h | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/tests/spec
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Feb 20, 2015 at 11:01 PM, Jose Fonseca jfons...@vmware.com wrote:
We might want to have some OS abstractions for aligned memory if more
tests need it, but go for a simple solution for now, as it seems a
special case.
---
tests/spec
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 9 ++
tests/spec/CMakeLists.txt | 1 +
tests/spec/amd_pinned_memory/CMakeLists.gl.txt | 14 ++
tests/spec/amd_pinned_memory/CMakeLists.txt| 1 +
tests/spec/amd_pinned_memory
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 1 +
tests/spec/arb_es3_compatibility/CMakeLists.gl.txt | 1 +
.../es3-primrestart-fixedindex.c | 110 +
3 files changed, 112 insertions(+)
create mode 100644
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 1 +
tests/spec/arb_draw_indirect/CMakeLists.gl.txt | 1 +
.../arb_draw_indirect/draw-arrays-prim-restart.c | 115 +
3 files changed, 117 insertions(+)
create mode 100644
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 1 +
tests/spec/CMakeLists.txt | 1 +
tests/spec/arb_es3_compatibility/CMakeLists.gl.txt | 12 +++
tests/spec/arb_es3_compatibility/CMakeLists.txt| 1 +
.../es3
Actually, if you want, you can remove these profiles. I don't use them
anymore and I don't know of anyone who does.
Marek
On Fri, Jan 2, 2015 at 3:15 AM, Dylan Baker baker.dyla...@gmail.com wrote:
You're right. Copy and paste error
On Dec 31, 2014 6:15 PM, Ilia Mirkin imir...@alum.mit.edu
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 1 +
.../spec/ext_transform_feedback/CMakeLists.gl.txt | 1 +
.../immediate-reuse-index-buffer.c | 141 +
3 files changed, 143 insertions(+)
create mode 100644
From: Marek Olšák marek.ol...@amd.com
---
.../2-buffers-bug.shader_test | 58 ++
1 file changed, 58 insertions(+)
create mode 100644
tests/spec/arb_uniform_buffer_object/2-buffers-bug.shader_test
diff --git a/tests/spec/arb_uniform_buffer_object/2
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 1 +
.../spec/ext_transform_feedback/CMakeLists.gl.txt | 1 +
.../immediate-reuse-uniform-buffer.c | 144 +
3 files changed, 146 insertions(+)
create mode 100644
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Thu, Jan 1, 2015 at 2:46 AM, Dylan Baker baker.dyla...@gmail.com wrote:
The above commit changed r300.py and r500.py to no longer import
quick.profile and modify that, but instead made them create a new
profile with a single test
From: Marek Olšák marek.ol...@amd.com
The back buffer may already be green after allocation.
---
tests/spec/arb_tessellation_shader/execution/nop.shader_test| 2 ++
tests/spec/arb_tessellation_shader/execution/sanity.shader_test | 2 ++
.../arb_tessellation_shader/execution
From: Marek Olšák marek.ol...@amd.com
4, 8, 16, 32 pass on radeonsi.
48 fails in a weird way.
64, 128, 256 crash on an assertion failure.
softpipe passes all of them.
---
...-array-indirect-access-004-elements.shader_test | 35 +++
...-array-indirect-access-008-elements.shader_test | 39
From: Marek Olšák marek.ol...@amd.com
This verifies that assigned locations are correct for arrays.
---
BTW, the second patch depends on chrisf's patches.
.../execution/tcs-tes-patch-array.shader_test | 59 ++
1 file changed, 59 insertions(+)
create mode 100644
tests
From: Marek Olšák marek.ol...@amd.com
---
.../execution/quads.shader_test| 125 +
1 file changed, 125 insertions(+)
create mode 100644
tests/spec/arb_tessellation_shader/execution/quads.shader_test
diff --git a/tests/spec/arb_tessellation_shader
From: Marek Olšák marek.ol...@amd.com
Mesa things Name is a type in gl_Position = Name.
---
.../compiler/scoping-struct-vs-variable.vert | 24 ++
1 file changed, 24 insertions(+)
create mode 100644
tests/spec/glsl-1.20/compiler/scoping-struct-vs-variable.vert
diff
From: Marek Olšák marek.ol...@amd.com
---
tests/bugs/fdo10370.c| 1 +
tests/util/piglit-util.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/tests/bugs/fdo10370.c b/tests/bugs/fdo10370.c
index 95e3d56..7826543 100644
--- a/tests/bugs/fdo10370.c
+++ b/tests/bugs/fdo10370.c
@@ -15,6
From: Marek Olšák marek.ol...@amd.com
It tests:
- clearing
- layered clearing
- drawing
- drawing with a depth or stencil shader output
- various sizes of TEXTURE_2D_ARRAY
- texturing
with both depth and stencil.
---
tests/all.py| 8 +-
tests/fbo/fbo-depth-array.c | 513
From: Marek Olšák marek.ol...@amd.com
This is a lot faster than RGBA FLOAT.
There is a potential to speed up piglit by switching tests to ubyte
r, rg, or rgba probing.
---
tests/util/piglit-util-gl.c | 32
tests/util/piglit-util-gl.h | 1 +
2 files changed, 33
Hi Michel,
I recommend using the -c option. It will force concurrency for all
tests. With that, I don't care which tests aren't concurrent by
default.
Marek
On Wed, Aug 20, 2014 at 5:13 AM, Michel Dänzer mic...@daenzer.net wrote:
From: Michel Dänzer michel.daen...@amd.com
This reverts commit
I actually agree with the patch. I was just making a suggestion. It
seems nobody has NAK'd this patch yet. :)
Marek
On Thu, Aug 21, 2014 at 11:24 AM, Michel Dänzer mic...@daenzer.net wrote:
On 21.08.2014 17:57, Marek Olšák wrote:
I recommend using the -c option. It will force concurrency
From: Marek Olšák marek.ol...@amd.com
llvmpipe passes, which is probably the only driver which implements it
correctly.
---
tests/all.py | 8
tests/texturing/tex-miplevel-selection.c | 20 +++-
2 files changed, 23 insertions(+), 5 deletions
/nv_texture_barrier/blending-in-shader-arb.c
new file mode 100644
index 000..c0c3db8
--- /dev/null
+++ b/tests/spec/nv_texture_barrier/blending-in-shader-arb.c
@@ -0,0 +1,118 @@
+/*
+ * Copyright © 2011 Marek Olšák
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Thu, Aug 7, 2014 at 4:51 AM, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
GL_REPEAT is not legal for the rectangle targets, and because the test uses
sampler objects the default GL_REPEAT value will be used even
I have the same failure on radeonsi.
Marek
On Thu, Aug 7, 2014 at 7:43 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
Can I assume that you tested this on llvmpipe and it worked? I'm
getting failures on render-1darray with nvc0, whereby layers 1+ are
random data, but we pass all the regular
FYI, you forgot to add the test to all.py.
Marek
On Wed, Aug 6, 2014 at 4:07 AM, Anuj Phogat anuj.pho...@gmail.com wrote:
This test verifies the bug fix in i965 drivers in mesa commit 984a02b.
Reproduces another bug in the driver with GL_DEPTH_COMPONENT16
internal format.
Signed-off-by:
From: Marek Olšák marek.ol...@amd.com
---
tests/general/attribs.c | 31 +--
1 file changed, 25 insertions(+), 6 deletions(-)
diff --git a/tests/general/attribs.c b/tests/general/attribs.c
index 3f02142..f48d8b9 100644
--- a/tests/general/attribs.c
+++ b/tests/general
From: Marek Olšák marek.ol...@amd.com
---
tests/spec/arb_vertex_type_2_10_10_10_rev/draw-vertices-2101010.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tests/spec/arb_vertex_type_2_10_10_10_rev/draw-vertices-2101010.c
b/tests/spec/arb_vertex_type_2_10_10_10_rev/draw-vertices
From: Marek Olšák marek.ol...@amd.com
Clearing a buffer with masked channels results in not clearing anything
in those channels, so there is always random garbage. No wonder that
the test was always failing. Apparently, i965 ignores the colormask,
otherwise it wouldn't have been unnoticed
.
Marek
On Mon, Jul 7, 2014 at 11:09 PM, Matt Turner matts...@gmail.com wrote:
On Fri, Jul 4, 2014 at 7:09 AM, Marek Olšák mar...@gmail.com wrote:
This adds tests for all GLSL texture function. The tex-miplevel-selection
test is used, since it's a good foundation for testing texturing
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, Jul 29, 2014 at 5:27 PM, Neil Roberts n...@linux.intel.com wrote:
The unsigned half float format needs special handling because the
texture that the test uploads tries to use negative values.
---
tests/all.py
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, Jul 29, 2014 at 5:27 PM, Neil Roberts n...@linux.intel.com wrote:
---
tests/all.py | 1 +
tests/texturing/texwrap.c | 13 +
2 files changed, 14 insertions(+)
diff --git a/tests/all.py b/tests/all.py
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, Jul 29, 2014 at 5:27 PM, Neil Roberts n...@linux.intel.com wrote:
---
tests/all.py | 4
tests/texturing/compressedteximage.c | 10 ++
2 files changed, 14 insertions(+)
diff --git a/tests
Well, the meta code is pretty much an Intel-specific thing, because
other drivers which are being actively worked on do not use it. One
way to test compression and decompression is to add support to the
fbo-generatemipmap-formats test, because mipmap generation is done
entirely in software for
Adding a new test for BPTC might be useful if want to test a feature
that isn't covered by other tests, because there already is a test
infrastructure for testing all OpenGL formats and it's just a matter
of adding BPTC support to existing tests. Mostly, just adding the new
formats to appropriate
By removing offset++, you left some empty if statements in there.
Other than that:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Mon, Jul 14, 2014 at 4:37 PM, Brian Paul bri...@vmware.com wrote:
Always draw a (mode, filter) test pattern at the same window position,
regardless
SNB turned up a bunch of failures, followed by a few
GPU hangs it recovered from, followed by a GPU hang that killed the
system (not even SysRq could reboot it, had to use the dreaded power
button).
On Fri, Jul 4, 2014 at 2:26 PM, Marek Olšák mar...@gmail.com wrote:
Well, I think piglit should
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 10 ++
tests/texturing/tex-miplevel-selection.c | 54 +---
2 files changed, 46 insertions(+), 18 deletions(-)
diff --git a/tests/all.py b/tests/all.py
index bb13c05..228bd25
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 8
tests/texturing/tex-miplevel-selection.c | 17 ++---
2 files changed, 22 insertions(+), 3 deletions(-)
diff --git a/tests/all.py b/tests/all.py
index 154891a..bc8f31c 100644
--- a/tests
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 8 +++
tests/texturing/tex-miplevel-selection.c | 37 ++--
2 files changed, 43 insertions(+), 2 deletions(-)
diff --git a/tests/all.py b/tests/all.py
index 3a18ebf..a1969e8
Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
This is required for radeonsi to pass.
---
.../execution/fs-textureQueryLOD-linear.shader_test | 9
+++--
.../execution/fs-textureQueryLOD-nearest.shader_test | 6 +-
.../execution/fs
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Wed, Jun 11, 2014 at 2:11 AM, Ilia Mirkin imir...@alum.mit.edu wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
.../arb_sample_shading/execution/builtin-gl-sample-mask-simple.cpp | 5
-
tests/spec/arb_sample_shading
From: Marek Olšák marek.ol...@amd.com
This one is simpler and easier to debug. It passes on my hardware, while
the other one fails (due to a shader bug probably).
I don't like tests that return the pass or fail status from the shader,
because they are impossible to debug. Therefore, this one
First, I'm sorry for complaining and not reviewing patches, because I
don't know Python very well and I'm also quite busy. I'm grateful for
all the work on piglit that's happening.
I always keep old piglit results from old Mesa around, that includes
R580, RV670, RV730, EG/Redwood, NI/Cayman,
workflow.
- Dylan
On Tuesday, March 18, 2014 02:34:57 Marek Olšák wrote:
I cannot review this, because I'm not familiar with the code, but the
last patch looks good to me and I like the overall idea.
Marek
On Mon, Mar 17, 2014 at 8:06 PM, Dylan Baker baker.dyla...@gmail.com
wrote:
This series
Also, the changes page doesn't contain disabled tests. It only
contains enabled tests and changed results.
Marek
On Wed, May 28, 2014 at 3:55 PM, Marek Olšák mar...@gmail.com wrote:
I think there is a bug. It looks like subtest results of one of the
tests are also added to all the following
Yeah, it looks like it's the same bug.
Marek
On Wed, May 28, 2014 at 4:18 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Wed, May 28, 2014 at 9:55 AM, Marek Olšák mar...@gmail.com wrote:
I think there is a bug. It looks like subtest results of one of the
tests are also added to all
If you fix the typo that Ilia pointed out, the series is:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Mon, May 19, 2014 at 10:51 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
.../arb_framebuffer_object/fbo-drawbuffers-none.c
I'm not sure I undertand this. I think the limitation for
non-concurrent tests is that they cannot be concurrently, because they
do front buffer rendering and other things. That doesn't mean all the
off-screen tests cannot be run with the non-concurrent tests
simultaneously.
Marek
On Thu, Apr
:47 Marek Olšák wrote:
I'm not sure I undertand this. I think the limitation for
non-concurrent tests is that they cannot be concurrently, because they
do front buffer rendering and other things. That doesn't mean all the
off-screen tests cannot be run with the non-concurrent tests
Hi,
I have updated piglit today (c5060a8..3456230) and found out that
concurrency is broken. When I run piglit, about 5000 tests run
concurrently with all 4 CPU cores utilized and then it drops to 1 CPU
core for the rest of the run (there are 16000 tests or so) and takes a
very long time to
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Apr 11, 2014 at 9:33 PM, Dylan Baker baker.dyla...@gmail.com wrote:
For whatever reason the concurrency flag was not passed to
GLSLParserTest and ShaderTest after the refactor. This patch fixes that
by always passing that keyword
Hi,
There is a serious issue with piglit that crashing tests just
disappear from the HTML summary. Let's say I apply this:
diff --git a/tests/fbo/fbo-generatemipmap-formats.c
b/tests/fbo/fbo-generatemipmap-formats.c
index 666030f..0d4a06b 100644
--- a/tests/fbo/fbo-generatemipmap-formats.c
+++
From: Marek Olšák marek.ol...@amd.com
Instead of offsetting the vertex position for the second half of the screen,
setup both halves as separate viewports.
---
tests/general/primitive-restart-draw-mode.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/general
Paul bri...@vmware.com wrote:
Two very minor nits (applies to other tests too).
On 03/15/2014 10:53 AM, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
---
tests/all.py | 2 +
tests/fbo/CMakeLists.gl.txt | 1 +
tests/fbo/fbo-generatemipmap
Hi Johney,
The development takes place in this repository:
http://cgit.freedesktop.org/piglit/
If you have patches for piglit, you can send them to this mailing list.
Marek
On Mon, Mar 17, 2014 at 7:29 AM, Guo, Johney weijun@amd.com wrote:
Hi,
I found piglit is a so amazing software.
I cannot review this, because I'm not familiar with the code, but the
last patch looks good to me and I like the overall idea.
Marek
On Mon, Mar 17, 2014 at 8:06 PM, Dylan Baker baker.dyla...@gmail.com wrote:
This series separates the SKIP and NOTRUN status changes from fixes and
regressions
This patch looks good to me.
Marek
On Thu, Mar 13, 2014 at 5:07 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
There's no reason to attach the teximages to the framebuffer. Instead
use renderbuffers so that we don't need to depend on ARB_depth_texture.
Signed-off-by: Ilia Mirkin
Did you forget GL_DEPTH_COMPONENT32 or is it not tested by this test?
Marek
On Thu, Mar 13, 2014 at 5:07 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
tests/texturing/copyteximage.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
This series is:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Thu, Mar 13, 2014 at 5:07 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
This removes an implicit dependency on ARB_depth_texture (and is more
efficient).
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
] },
{ GL_DEPTH_COMPONENT32F, fcolor[7] },
{ GL_DEPTH24_STENCIL8, fcolor[7] },
{ GL_DEPTH32F_STENCIL8, fcolor[7] },
On Sun, Mar 16, 2014 at 10:03 AM, Marek Olšák mar...@gmail.com wrote:
Did you forget GL_DEPTH_COMPONENT32 or is it not tested by this test?
Marek
On Thu, Mar 13, 2014 at 5:07 PM, Ilia
at 1:36 PM, Daniel Vetter dan...@ffwll.ch wrote:
On Fri, Nov 22, 2013 at 12:10:43AM +0100, Marek Olšák wrote:
It looks like people have different expectations from notrun and
skip, so I have a proposal:
- add a notrun page similar to the skipped page
- don't include the notrun and skip statuses
From: Marek Olšák marek.ol...@amd.com
---
tests/spec/gl-3.2/minmax.c | 4 +++-
tests/spec/gl-3.3/minmax.c | 4 +++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/spec/gl-3.2/minmax.c b/tests/spec/gl-3.2/minmax.c
index e4d135d..448b02e 100644
--- a/tests/spec/gl-3.2
301 - 400 of 527 matches
Mail list logo