Re: [Mesa-dev] [PATCH 1/2] mesa: add GL_ARB_shader_atomic_counter_ops support

2016-03-06 Thread Ilia Mirkin
ping On Sat, Feb 20, 2016 at 12:13 AM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > src/compiler/glsl/builtin_functions.cpp | 110 > +++ > src/compiler/glsl/glcpp/glcpp-parse.y| 3 + > src/compiler/glsl/glsl_parser_extras.cpp | 1 + > src/compile

Re: [Mesa-dev] [PATCH v2] egl: support EGL_LARGEST_PBUFFER in eglCreatePbufferSurface(...)

2016-03-06 Thread Tapani Pälli
ping Emil On 02/22/2016 08:00 AM, Tapani Pälli wrote: From: Daniel Czarnowski Patch provides a default for a set pbuffer surface size when EGL_LARGEST_PBUFFER is used by the client. MIN2 macro is moved to egldefines so that it can be shared. Fixes following Piglit test: egl-create-largest

Re: [Mesa-dev] [android-x86-devel] Re: gralloc_drm_pipe

2016-03-06 Thread Rob Clark
On Sun, Mar 6, 2016 at 9:29 PM, Chih-Wei Huang wrote: > 2016-03-05 3:53 GMT+08:00 Rob Clark : >> On Fri, Mar 4, 2016 at 2:43 PM, Thomas Hellstrom >> wrote: >>> On 03/04/2016 07:07 PM, Rob Clark wrote: On Fri, Mar 4, 2016 at 12:59 PM, Rob Clark wrote: > So, I've been advocating that for

Re: [Mesa-dev] [PATCH] glsl: avoid stack smashing when there are too many attributes

2016-03-06 Thread Timothy Arceri
On Sun, 2016-03-06 at 12:21 -0500, Ilia Mirkin wrote: > This fixes a crash in > > dEQP- > GLES3.functional.transform_feedback.array_element.separate.points.low > p_mat3x2 > > and likely others. The vertex shader has > 16 input variables > (without > explicit locations), which causes us to index o

Re: [Mesa-dev] [android-x86-devel] Re: gralloc_drm_pipe

2016-03-06 Thread Chih-Wei Huang
2016-03-05 3:53 GMT+08:00 Rob Clark : > On Fri, Mar 4, 2016 at 2:43 PM, Thomas Hellstrom > wrote: >> On 03/04/2016 07:07 PM, Rob Clark wrote: >>> On Fri, Mar 4, 2016 at 12:59 PM, Rob Clark wrote: So, I've been advocating that for android, gallium drivers use gralloc_drm_pipe, since wit

[Mesa-dev] [Bug 94295] [swrast] piglit shader_runner fast_color_clear/all-colors regression

2016-03-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94295 Vinson Lee changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

Re: [Mesa-dev] [PATCH] gm107/ir: add emission for BAR

2016-03-06 Thread Ilia Mirkin
On Tue, Mar 1, 2016 at 12:44 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 52 > ++ > 1 file changed, 52 insertions(+) > > diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp > b/

Re: [Mesa-dev] [PATCH v2 3/3] gk110/ir: add missing src predicate emission for BAR.RED

2016-03-06 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Mon, Feb 29, 2016 at 3:54 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 8 > 1 file changed, 8 insertions(+) > > diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-06 Thread Kenneth Graunke
On Saturday, March 5, 2016 4:02:17 PM PST Jordan Justen wrote: > On 2016-03-04 16:07:08, Matt Turner wrote: > > Alternatively, if overwriting FFTID is bad, instead of emitting > > > >shr(1) sr0.1:UD g0.0:UD 16UD > > > > you could write only the low word: > > > >mov(1) sr0.2:UW g0.1:

Re: [Mesa-dev] [PATCH v2 2/3] gk110/ir: allow to emit immediates for BAR

2016-03-06 Thread Ilia Mirkin
On Mon, Feb 29, 2016 at 3:54 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 22 > -- > 1 file changed, 20 insertions(+), 2 deletions(-) > > diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit

Re: [Mesa-dev] [PATCH v2 1/3] gk110/ir: fix wrong emission of BAR.SYNC

2016-03-06 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Mon, Feb 29, 2016 at 3:54 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cp

Re: [Mesa-dev] [Patch v2] opencl: fix .gitignore for .install-gallium-links

2016-03-06 Thread Dieter Nützel
Can someone commit for me please, as I didn't have commit access, yet. Thanks, Dieter Am 06.03.2016 20:37, schrieb Dieter Nützel: Fixes: 0b6157e9713 "install-gallium-links: port changes from install-lib-links" v2: move this to the top level .gitignore and added Fixes: like Emil Velikov

[Mesa-dev] [Patch v2] opencl: fix .gitignore for .install-gallium-links

2016-03-06 Thread Dieter Nützel
Fixes: 0b6157e9713 "install-gallium-links: port changes from install-lib-links" v2: move this to the top level .gitignore and added Fixes: like Emil Velikov suggested Signed-off-by: Dieter Nützel Reviewed-by: Eduardo Lima Mitev --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --

Re: [Mesa-dev] [PATCH] mesa: flip current tf object back to default if current is being deleted

2016-03-06 Thread Matt Turner
Yes, because transform feedback objects are not shared between contexts, their deletion takes effect immediately. Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] mesa: flip current tf object back to default if current is being deleted

2016-03-06 Thread Ilia Mirkin
In the rather unusual case of Bind + Delete, we need to make sure that we unbind the current tf object. Fixes dEQP-GLES3.functional.lifetime.delete_bound.transform_feedback Signed-off-by: Ilia Mirkin --- src/mesa/main/transformfeedback.c | 5 + 1 file changed, 5 insertions(+) diff --git a/

[Mesa-dev] [PATCH] glsl: avoid stack smashing when there are too many attributes

2016-03-06 Thread Ilia Mirkin
This fixes a crash in dEQP-GLES3.functional.transform_feedback.array_element.separate.points.lowp_mat3x2 and likely others. The vertex shader has > 16 input variables (without explicit locations), which causes us to index outside of the to_assign array. Signed-off-by: Ilia Mirkin Cc: "11.1 11.2

Re: [Mesa-dev] Removing R600_BIG_ENDIAN and using #ifdef instead

2016-03-06 Thread Marek Olšák
Hi Oded, I prefer "if" over #ifdef. The idea is that everybody should be able to test if the compilation succeeds without a BE machine. #ifdef disallows that. R600_BIG_ENDIAN can be moved to r600_pipe_common.h. Marek On Sun, Mar 6, 2016 at 9:01 AM, Oded Gabbay wrote: > Hi, > > Do you mind if

Re: [Mesa-dev] [PATCH v2] st/mesa: don't force per-sample interp if only sampleid/pos are used

2016-03-06 Thread Marek Olšák
On Fri, Mar 4, 2016 at 7:13 PM, Ilia Mirkin wrote: > On Fri, Mar 4, 2016 at 12:26 PM, Ilia Mirkin wrote: >> The OES extensions clarify this behaviour to differentiate between >> per-sample invocation and per-sample interpolation. Using sampleid/pos >> will force per-sample invocation but not per-

Re: [Mesa-dev] [PATCH] r600g: Adjust pipe format when decompressing depth in BE

2016-03-06 Thread Oded Gabbay
On Fri, Mar 4, 2016 at 10:08 PM, Marek Olšák wrote: > On Fri, Mar 4, 2016 at 6:53 PM, Oded Gabbay wrote: >> On Fri, Mar 4, 2016 at 6:59 PM, Marek Olšák wrote: >>> On Fri, Mar 4, 2016 at 4:56 PM, Oded Gabbay wrote: On Fri, Mar 4, 2016 at 2:19 PM, Marek Olšák wrote: > Note that the DB o

Re: [Mesa-dev] [PATCH] st/va: avoid dereference after free

2016-03-06 Thread Thomas H.P. Andersen
On Sat, Mar 5, 2016 at 1:30 PM, Emil Velikov wrote: > Hi Thomas, > > On 5 March 2016 at 12:07, Thomas Hindoe Paaboel Andersen > wrote: > > --- > > src/gallium/state_trackers/va/image.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/src/gallium/state_trackers/v

Re: [Mesa-dev] [PATCH] egl/x11: check the return value of xcb_dri2_get_buffers_reply()

2016-03-06 Thread Eduardo Lima Mitev
On 03/05/2016 10:25 PM, Emil Velikov wrote: ... before using it. The function can return NULL, which we should check prior to refererencing it in the next function(s). Cc: Fabian Vogt Cc: "11.1 11.2" Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93667 Signed-off-by: Emil Velikov ---

[Mesa-dev] Removing R600_BIG_ENDIAN and using #ifdef instead

2016-03-06 Thread Oded Gabbay
Hi, Do you mind if I totally remove R600_BIG_ENDIAN global variable and instead use in all places #ifdef PIPE_ARCH_BIG_ENDIAN ? It's just that: 1. Checking for R600_BIG_ENDIAN is an extra check which can be eliminated using #ifdef 2. Some files, e.g r600_texture.c, don't know R600_BIG_ENDIAN so