[Mesa-dev] [PATCH demos 1/4] eglinfo: Note when an EGLConfig is streams-compatible

2016-04-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/egl/opengl/eglinfo.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index 1f79fef..ca22df2 100644 --- a/src/egl/opengl/eglinfo.c +++ b/src/egl/opengl/eglinfo.c @@ -93,6 +93,8 @@

Re: [Mesa-dev] [PATCH 2/4] st/mesa: check return value of begin/end_query

2016-04-20 Thread Samuel Pitoiset
On 04/20/2016 09:37 PM, Nicolai Hähnle wrote: On 20.04.2016 11:13, Samuel Pitoiset wrote: On 04/20/2016 05:43 PM, Nicolai Hähnle wrote: From: Nicolai Hähnle They can only indicate out of memory conditions, since the other error conditions are caught earlier. ---

[Mesa-dev] [PATCH] winsys/amdgpu: fix preamble IB size

2016-04-20 Thread Thomas Hindoe Paaboel Andersen
The missing break caused the IB size to be overwritten with the size of IB_CONST. This was introduced in: 7201230582e060aa2eb79c825d3188b437ef7bb8 --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c

[Mesa-dev] [Bug 95038] atomic_add and atomic_or cause a OpenCL crashes

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95038 --- Comment #1 from Aaron Watry --- Are the atomic_add/atomic_or operations causing the crash operating against local or global memory? Last I checked, I think that atomics for r600g-based cards were only implemented in LLVM

Re: [Mesa-dev] [PATCH v2 4/4] i965: Properly handle integer types in opt_vector_float().

2016-04-20 Thread Matt Turner
On Tue, Apr 19, 2016 at 12:44 PM, Matt Turner wrote: > On Mon, Apr 18, 2016 at 11:52 PM, Kenneth Graunke > wrote: >> Previously, opt_vector_float() always interpreted MOV sources as >> floating point, and always created a MOV with a F-type destination.

Re: [Mesa-dev] [PATCH demos 4/4] eglinfo: Add EXT_platform_* awareness

2016-04-20 Thread Alex Deucher
On Wed, Apr 20, 2016 at 4:10 PM, Adam Jackson wrote: > Signed-off-by: Adam Jackson Series is: Reviewed-by: Alex Deucher > --- > src/egl/opengl/eglinfo.c | 41 +++-- > 1 file changed, 35

Re: [Mesa-dev] [PATCH 2/7] gk110/ir: add emission for OP_MADSP

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 23 > ++ > 1 file

[Mesa-dev] [Bug 95038] atomic_add and atomic_or cause a OpenCL crashes

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95038 Bug ID: 95038 Summary: atomic_add and atomic_or cause a OpenCL crashes Product: Mesa Version: 11.2 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH 2/2] gk110/ir: use separate signed expressions for dst/src with IMUL32I

2016-04-20 Thread Samuel Pitoiset
On 04/20/2016 07:47 PM, Ilia Mirkin wrote: Presumably you'd want to touch up the non-limm side of this as well? Although TBH, I can't really think of a time when it'd matter. I think you're pretty much guaranteed to have stype == dtype for mul's. Maybe I'm wrong? Well, the non-limm part is

[Mesa-dev] [Bug 95036] make check egl-symbols-check regression

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95036 Vinson Lee changed: What|Removed |Added Keywords||bisected

Re: [Mesa-dev] [PATCH 1/2] winsys/amdgpu: adjust IB size based on buffer wait time

2016-04-20 Thread Marek Olšák
On Wed, Apr 20, 2016 at 8:18 PM, Grigori Goronzy wrote: > On 2016-04-20 02:20, Nicolai Hähnle wrote: >> >> This is just a slight massaging of the patch you sent previously. What >> happened to the discussion we had about how to do this properly? >> > > This already provides

Re: [Mesa-dev] [PATCH] glsl: Relax GLSL 1.10 float suffix error to a warning.

2016-04-20 Thread Roland Scheidegger
Am 20.04.2016 um 21:29 schrieb Matt Turner: > Float suffixes are allowed in all subsequent GLSL specifications, and > it's obvious what the user meant if they specify one. Accept it with a > warning to avoid breaking applications, like Planeshift. > --- > src/compiler/glsl/glsl_lexer.ll | 4 ++--

[Mesa-dev] [PATCH demos 4/4] eglinfo: Add EXT_platform_* awareness

2016-04-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/egl/opengl/eglinfo.c | 41 +++-- 1 file changed, 35 insertions(+), 6 deletions(-) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index 875e407..75d9fe5 100644 ---

[Mesa-dev] [PATCH demos 3/4] eglinfo: Factor out a "probe one display" function

2016-04-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/egl/opengl/eglinfo.c | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index b044eaa..875e407 100644 --- a/src/egl/opengl/eglinfo.c +++

[Mesa-dev] [PATCH demos 2/4] eglinfo: Print client extensions first

2016-04-20 Thread Adam Jackson
These are independent of the display, and I want to iterate over those. Signed-off-by: Adam Jackson --- src/egl/opengl/eglinfo.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index ca22df2..b044eaa

Re: [Mesa-dev] [PATCH] gk110/ir: do not overwrite def value with zero for EXCH ops

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Wed, Apr 20, 2016 at 1:47 PM, Samuel Pitoiset wrote: > This is only valid for other atomic operations (including CAS). This > fixes an invalid opcode error from dmesg. While we are it, make sure > to initialize global

Re: [Mesa-dev] [PATCH 4/7] gk110/ir: add emission for OP_SUEAU, OP_SUBFM and OP_SUCLAMP

2016-04-20 Thread Ilia Mirkin
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87 > ++ > 1 file changed, 87 insertions(+) > > diff --git

[Mesa-dev] [Bug 95005] Unreal engine demos segfault after shader compilation error with OpenGL 4.3

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95005 --- Comment #6 from Christoph Haag --- Just FYI: the patch from comment 1 helps for the (very) old unreal demos that can be downloaded from the unreal wiki, but the current unreal engine has a similar (something with

Re: [Mesa-dev] [PATCH] gk110/ir: do not overwrite def value with zero for EXCH ops

2016-04-20 Thread Samuel Pitoiset
On 04/20/2016 07:49 PM, Ilia Mirkin wrote: The SM20/SM30 logic does this for exch || cas. Are you *sure* this shouldn't be the same? It's pretty silly to do a CAS without a dst, but it's definitely possible (through DCE). Yeah, I'm sure. It's definitely not the same logic between gk104 and

Re: [Mesa-dev] [PATCH 6/7] nvc0: enable ARB_shader_image_load_store on GK110

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin Might also want to see what happens on GK20A. Should Just Work (tm). On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > This exposes 8 images for all shader types. > > Signed-off-by: Samuel Pitoiset

Re: [Mesa-dev] [PATCH 7/7] nvc0: expose GLSL version 420 on GK110

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin Same comment re GK20A. On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +- > 1 file

Re: [Mesa-dev] [PATCH 2/2] gk110/ir: use separate signed expressions for dst/src with IMUL32I

2016-04-20 Thread Ilia Mirkin
On Wed, Apr 20, 2016 at 3:14 PM, Samuel Pitoiset wrote: > > > On 04/20/2016 07:47 PM, Ilia Mirkin wrote: >> >> Presumably you'd want to touch up the non-limm side of this as well? >> Although TBH, I can't really think of a time when it'd matter. I think >> you're pretty

[Mesa-dev] [PATCH 3/7] gk110/ir: add emission for OP_SULDB and OP_SUSTx

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 153 + 1 file changed, 153 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp

[Mesa-dev] [PATCH 6/7] nvc0: enable ARB_shader_image_load_store on GK110

2016-04-20 Thread Samuel Pitoiset
This exposes 8 images for all shader types. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c

[Mesa-dev] [PATCH 1/7] gk110/ir: add emission for OP_PERMT

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 12 1 file changed, 12 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp

[Mesa-dev] [PATCH 7/7] nvc0: expose GLSL version 420 on GK110

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c index f8d74d5..c017f4f

[Mesa-dev] [PATCH 4/7] gk110/ir: add emission for OP_SUEAU, OP_SUBFM and OP_SUCLAMP

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87 ++ 1 file changed, 87 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp

[Mesa-dev] [PATCH 0/7] nvc0: ARB_shader_image_load_store/size on GK110

2016-04-20 Thread Samuel Pitoiset
Hi, This series is based on the previous one for GK104. Between GK104 and GK110 only the ISA has changed. This adds code emission for all instructions that are needed to support surfaces/images. I have exactly the same passrate for both deqp and piglit as GK104. Please review, Thanks! Samuel

[Mesa-dev] [PATCH 2/7] gk110/ir: add emission for OP_MADSP

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 23 ++ 1 file changed, 23 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp

[Mesa-dev] [PATCH 5/7] gk110/ir: add emission for VSHL

2016-04-20 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 58 ++ 1 file changed, 58 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp

Re: [Mesa-dev] [PATCH 3/7] gk110/ir: add emission for OP_SULDB and OP_SUSTx

2016-04-20 Thread Ilia Mirkin
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 153 > + > 1 file changed, 153 insertions(+) > > diff --git

[Mesa-dev] [PATCH 5/4] eglinfo: Restore eglTerminate

2016-04-20 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/egl/opengl/eglinfo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c index 75d9fe5..f7da372 100644 --- a/src/egl/opengl/eglinfo.c +++ b/src/egl/opengl/eglinfo.c @@ -178,6 +178,7 @@

Re: [Mesa-dev] [PATCH 2/2] gk110/ir: use separate signed expressions for dst/src with IMUL32I

2016-04-20 Thread Samuel Pitoiset
On 04/20/2016 10:13 PM, Ilia Mirkin wrote: On Wed, Apr 20, 2016 at 3:14 PM, Samuel Pitoiset wrote: On 04/20/2016 07:47 PM, Ilia Mirkin wrote: Presumably you'd want to touch up the non-limm side of this as well? Although TBH, I can't really think of a time when

Re: [Mesa-dev] [PATCH] glsl: Relax GLSL 1.10 float suffix error to a warning.

2016-04-20 Thread Lars Hamre
I am fine with this, it would be nice if we could modify the following piglit tests to pass when a warning is emitted: https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/compiler/literals/invalid-float-suffix-capital-f.vert

Re: [Mesa-dev] [PATCH demos 3/4] eglinfo: Factor out a "probe one display" function

2016-04-20 Thread Eric Anholt
Adam Jackson writes: > Signed-off-by: Adam Jackson > --- > src/egl/opengl/eglinfo.c | 29 ++--- > 1 file changed, 18 insertions(+), 11 deletions(-) > > diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c > index

Re: [Mesa-dev] [PATCH 4/7] gk110/ir: add emission for OP_SUEAU, OP_SUBFM and OP_SUCLAMP

2016-04-20 Thread Ilia Mirkin
On Wed, Apr 20, 2016 at 4:41 PM, Samuel Pitoiset wrote: > > > On 04/20/2016 10:35 PM, Ilia Mirkin wrote: >> >> On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset >> wrote: >>> >>> Signed-off-by: Samuel Pitoiset >>>

[Mesa-dev] [PATCH] i965/fs: Readd opt_drop_redundant_mov_to_flags().

2016-04-20 Thread Matt Turner
This reverts commit b449366587b5f3f64c6fb45fe22c39e4bc8a4309. I removed the pass thinking that it was now not useful, but that was not true. I believe I ran shader-db on HSW and saw no results, but HSW does not use the unlit centroid workaround code and as a result does not emit redundant

Re: [Mesa-dev] [PATCH 5/4] eglinfo: Restore eglTerminate

2016-04-20 Thread Alex Deucher
On Wed, Apr 20, 2016 at 4:55 PM, Adam Jackson wrote: > Signed-off-by: Adam Jackson Reviewed-by: Alex Deucher > --- > src/egl/opengl/eglinfo.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/egl/opengl/eglinfo.c

[Mesa-dev] [PATCH] glsl: Relax GLSL 1.10 float suffix error to a warning.

2016-04-20 Thread Matt Turner
Float suffixes are allowed in all subsequent GLSL specifications, and it's obvious what the user meant if they specify one. Accept it with a warning to avoid breaking applications, like Planeshift. --- src/compiler/glsl/glsl_lexer.ll | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[Mesa-dev] [PATCH] gallium/radeon: implement randomized SDMA testing

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/r600_pipe.c | 3 + src/gallium/drivers/radeon/Makefile.sources | 1 + src/gallium/drivers/radeon/r600_pipe_common.c | 2 + src/gallium/drivers/radeon/r600_pipe_common.h | 5 +

Re: [Mesa-dev] [PATCH 2/4] st/mesa: check return value of begin/end_query

2016-04-20 Thread Nicolai Hähnle
On 20.04.2016 11:13, Samuel Pitoiset wrote: On 04/20/2016 05:43 PM, Nicolai Hähnle wrote: From: Nicolai Hähnle They can only indicate out of memory conditions, since the other error conditions are caught earlier. --- src/mesa/state_tracker/st_cb_queryobj.c | 55

Re: [Mesa-dev] [PATCH] glsl: Relax GLSL 1.10 float suffix error to a warning.

2016-04-20 Thread Matt Turner
On Wed, Apr 20, 2016 at 12:48 PM, Lars Hamre wrote: > I am fine with this, it would be nice if we could modify the following > piglit tests to pass when a warning is emitted: >

Re: [Mesa-dev] [PATCH 1/7] gk110/ir: add emission for OP_PERMT

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 12 > 1 file changed,

Re: [Mesa-dev] [PATCH 5/7] gk110/ir: add emission for VSHL

2016-04-20 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 58 > ++ > 1 file

Re: [Mesa-dev] [PATCH 4/7] gk110/ir: add emission for OP_SUEAU, OP_SUBFM and OP_SUCLAMP

2016-04-20 Thread Samuel Pitoiset
On 04/20/2016 10:35 PM, Ilia Mirkin wrote: On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87 ++ 1 file changed, 87

[Mesa-dev] [Bug 95038] atomic_add and atomic_or cause a OpenCL crashes

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95038 ros...@gmail.com changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH 4/4] radeonsi: Print a message when scratch allocation fails.

2016-04-20 Thread eocallaghan
On 2016-04-20 11:46, Nicolai Hähnle wrote: On 19.04.2016 17:50, Bas Nieuwenhuizen wrote: Signed-off-by: Bas Nieuwenhuizen --- src/gallium/drivers/radeonsi/si_compute.c | 5 - src/gallium/drivers/radeonsi/si_state_shaders.c | 5 - 2 files changed, 8

Re: [Mesa-dev] Mesa (master): 34 new commits

2016-04-20 Thread Michel Dänzer
On 20.04.2016 01:32, Bas Nieuwenhuizen wrote: > URL: > http://cgit.freedesktop.org/mesa/mesa/commit/?id=464cef5b06e65aa740704e4adac68b7f5fee1b88 > Author: Bas Nieuwenhuizen > Date: Sat Mar 19 15:16:50 2016 +0100 > > radeonsi: enable TGSI support cap for

Re: [Mesa-dev] [PATCH 04/13] nir: add a pass to lower some double operations

2016-04-20 Thread Samuel Iglesias Gonsálvez
On 19/04/16 23:52, Jason Ekstrand wrote: > On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez < > sigles...@igalia.com> wrote: > >> From: Connor Abbott >> >> v2: Move to compiler/nir (Iago) >> >> Signed-off-by: Iago Toral Quiroga >> --- >>

Re: [Mesa-dev] [PATCH 05/13] nir/lower_double_ops: lower trunc()

2016-04-20 Thread Iago Toral
On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote: > > > On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez > wrote: > From: Iago Toral Quiroga > > At least i965 hardware does not have native support for >

Re: [Mesa-dev] [PATCH v2 0/2] Fix Gallium RGB565 image support

2016-04-20 Thread Michel Dänzer
On 19.04.2016 00:55, Nicolas Dufresne wrote: > Le lundi 18 avril 2016 à 11:40 +0900, Michel Dänzer a écrit : >> On 17.04.2016 09:49, nico...@ndufresne.ca wrote: >>> >>> From: Nicolas Dufresne >>> >>> Sorry for the long delay breaking down this patch. I have now >>>

Re: [Mesa-dev] [PATCH v2 04/11] gallium: add endian_format field to struct pipe_resource

2016-04-20 Thread Michel Dänzer
On 20.04.2016 17:48, Oded Gabbay wrote: > On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote: >> On 20.04.2016 03:13, Oded Gabbay wrote: >>> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote: On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay

Re: [Mesa-dev] [PATCH v2 04/11] gallium: add endian_format field to struct pipe_resource

2016-04-20 Thread Marek Olšák
On Wed, Apr 20, 2016 at 11:14 AM, Oded Gabbay wrote: > On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote: >> On 20.04.2016 17:48, Oded Gabbay wrote: >>> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote: On 20.04.2016

Re: [Mesa-dev] [PATCH 05/13] nir/lower_double_ops: lower trunc()

2016-04-20 Thread Iago Toral
On Wed, 2016-04-20 at 08:37 +0200, Iago Toral wrote: > On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote: > > > > > > On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez > > wrote: > > From: Iago Toral Quiroga > > > >

[Mesa-dev] [Bug 95003] [Clover / OpenCL] CL_DEVICE_WAVEFRONT_WIDTH_AMD - 0x4043 unimplemented

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95003 --- Comment #3 from Serge Martin --- That would be CL_KERNEL_PREFERRED_WORK_GROUP_SIZE_MULTIPLE from clGetKernelWorkGroupInfo -- You are receiving this mail because: You are the assignee for the bug. You are the QA

Re: [Mesa-dev] [PATCH v2 04/11] gallium: add endian_format field to struct pipe_resource

2016-04-20 Thread Oded Gabbay
On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote: > On 20.04.2016 17:48, Oded Gabbay wrote: >> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote: >>> On 20.04.2016 03:13, Oded Gabbay wrote: On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák

Re: [Mesa-dev] [PATCH] configure.ac: fix the --disable-llvm-shared-libs build

2016-04-20 Thread Michel Dänzer
On 20.04.2016 03:39, Emil Velikov wrote: > On 19 April 2016 at 15:47, Chuck Atkins wrote: >> This still doesn't quite give what you want. One can also have an llvm with >> component shared libs. So there's three different options for llvm library >> configurations: a

Re: [Mesa-dev] [PATCH v2 04/11] gallium: add endian_format field to struct pipe_resource

2016-04-20 Thread Michel Dänzer
On 20.04.2016 03:13, Oded Gabbay wrote: > On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote: >> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote: >>> On Mon, Apr 18, 2016 at 6:03 PM, >>> Ilia Mirkin wrote: On Mon, Apr 18,

[Mesa-dev] [PATCH 0/3] glsl: tests for glsl warnings

2016-04-20 Thread Alejandro Piñeiro
In order to implement the "uninitialized variable" warning it was needed to hande several corner cases manually. Recently I sent a series to fix the last false positive: https://lists.freedesktop.org/archives/mesa-dev/2016-April/113614.html At this point I think that it would be better to include

Re: [Mesa-dev] [PATCH v2 04/11] gallium: add endian_format field to struct pipe_resource

2016-04-20 Thread Oded Gabbay
On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote: > On 20.04.2016 03:13, Oded Gabbay wrote: >> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote: >>> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote: On Mon, Apr 18, 2016

[Mesa-dev] [PATCH 3/3] glsl: add unit tests data vertex/expected outcome for unitialized warning

2016-04-20 Thread Alejandro Piñeiro
--- Those are basically the shaders I used while developing the warning, plus some cleaning and some extra ones. The only reason they are vertex shaders is that the standalone compiler needs to receive a shader with a valid shader extension. .../glsl/tests/warnings/000-basic-test.vert

[Mesa-dev] [PATCH 1/3] glsl: add just-log option for the standalone compiler.

2016-04-20 Thread Alejandro Piñeiro
Add an option in order to ask to just print the InfoLog, without any header or separator. Useful if we want to use the standalone compiler to track only the warning/error messages. --- src/compiler/glsl/main.cpp | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff

[Mesa-dev] [PATCH 2/3] glsl: add warning-test

2016-04-20 Thread Alejandro Piñeiro
It executes compiler-glsl on all the available shaders, and it checks that the outcome is the expected. Bash code based on the already existing optimization-test --- src/compiler/Makefile.glsl.am | 3 ++- src/compiler/glsl/tests/warnings-test | 31 +++ 2

[Mesa-dev] [Bug 94291] llvmpipe tests fail if built on skylake i7-6700k

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94291 --- Comment #5 from Timo Aaltonen --- llvm-3.8 misdetects skylake features, this is fixed in 3.9-snapshot.. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the

Re: [Mesa-dev] [PATCH 2/4] radeonsi: Set range metadata on calls to llvm.SI.tid

2016-04-20 Thread Michel Dänzer
On 20.04.2016 02:52, Tom Stellard wrote: > The range metadata tells LLVM the range of expected values for this intrinsic, > so it can do some additional optimizations on the result. > --- > src/gallium/drivers/radeonsi/si_shader.c | 29 ++--- > 1 file changed, 26

[Mesa-dev] [Bug 94291] llvmpipe tests fail if built on skylake i7-6700k

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94291 --- Comment #6 from Jose Fonseca --- It's not the first time LLVM misidentifies modern CPUs. I thought that all the logic in src/gallium/auxiliary/gallivm/lp_bld_misc.cpp for setting +/-foo mattrs would save us from this

Re: [Mesa-dev] [PATCH mesa v2 1/2] nouveau: codegen: Use FILE_MEMORY_BUFFER for buffers

2016-04-20 Thread Hans de Goede
Hi, On 15-04-16 00:29, Samuel Pitoiset wrote: On 04/12/2016 12:04 PM, Hans de Goede wrote: Hi, On 08-04-16 18:14, Samuel Pitoiset wrote: On 04/08/2016 12:17 PM, Hans de Goede wrote: Hi, On 23-03-16 23:10, Samuel Pitoiset wrote: Are you sure this won't break compute shaders on fermi?

Re: [Mesa-dev] [PATCH] configure.ac: fix the --disable-llvm-shared-libs build

2016-04-20 Thread Jose Fonseca
On 19/04/16 19:39, Emil Velikov wrote: Hi Chuck, Thanks for chipping in. On 19 April 2016 at 15:47, Chuck Atkins wrote: This still doesn't quite give what you want. One can also have an llvm with component shared libs. So there's three different options for llvm

Re: [Mesa-dev] [PATCH 4/4] radeonsi: Implement ddx/ddy on VI using ds_bpermute

2016-04-20 Thread Marek Olšák
Patches 1-3: Reviewed-by: Marek Olšák Patch 4: Acked-by: Marek Olšák Marek On Tue, Apr 19, 2016 at 7:52 PM, Tom Stellard wrote: > The ds_bpermute instruction allows threads to transfer data directly > to or from the vgprs of

Re: [Mesa-dev] [PATCH] loader: add a libdrm case for loader_get_device_name_for_fd

2016-04-20 Thread Emil Velikov
On 20 April 2016 at 03:27, Jonathan Gray wrote: > On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote: >> On 19 April 2016 at 04:03, Jonathan Gray wrote: >> > Any objections to this? >> > >> Absolutely none. I simply missed it. >> >> > On Mon, Dec 21,

Re: [Mesa-dev] [PATCH 1/4] radeonsi: use CE suballocator for CP DMA realignment.

2016-04-20 Thread Marek Olšák
On Wed, Apr 20, 2016 at 1:29 AM, Bas Nieuwenhuizen wrote: > I retract patch 1 and 2. Large scratch buffers are nice, but the > hardware only supports a 32-bit offset into it. You can still allocate a smaller scratch buffer. This should limit the number of waves in hw.

[Mesa-dev] [PATCH] st/va: hardlink driver instances to gallium_drv_video.so

2016-04-20 Thread Stefan Dirsch
Removes the need to set LIBVA_DRIVER_NAME=gallium for supported targets and is consistent with vdpau and general gallium drivers. Author: Jimmy Berry Signed-off-by: Jimmy Berry --- src/gallium/targets/va/Makefile.am | 14 ++ 1 file

[Mesa-dev] [PATCH] dri2: Check for dummyContext to see if the glx_context is valid

2016-04-20 Thread Stefan Dirsch
According to the comments in src/glx/glxcurrent.c __glXGetCurrentContext() always returns a valid pointer. If no context is made current, it will contain dummyContext. Thus a test for NULL will always fail. Patch author: Egbert Eich

Re: [Mesa-dev] [PATCH] egl: android: add dma-buf fd support

2016-04-20 Thread Rob Clark
On Tue, Apr 19, 2016 at 11:02 PM, Rob Herring wrote: >> It would ofc be good to make sure we don't break things for >> android-x86, but I think they would stick on dri2 + pre-atomic state >> w/ compile time build decisions until enough of the desktop gpu's >> support atomic..

Re: [Mesa-dev] [PATCH 3/5] egl: implement EGL part of interop interface (v2)

2016-04-20 Thread Marek Olšák
On Wed, Apr 20, 2016 at 4:56 PM, Aaron Watry wrote: > Hi Marek, > > This patch ended up breaking make check for me (src/egl/egl-symbols-check). > > Also, while bisecting, it occurred to me that you pushed v2 of the series, > while you had previously sent a v3 to the mesa-dev

[Mesa-dev] [Bug 94291] llvmpipe tests fail if built on skylake i7-6700k

2016-04-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94291 --- Comment #8 from Timo Aaltonen --- Actually it wasn't avx512, that was the first one I tried :) It's enabled also on 3.7 and that version works fine. Only one that was added in 3.8 is PKU, but dropping just that didn't

[Mesa-dev] [PATCH 2/4] st/mesa: check return value of begin/end_query

2016-04-20 Thread Nicolai Hähnle
From: Nicolai Hähnle They can only indicate out of memory conditions, since the other error conditions are caught earlier. --- src/mesa/state_tracker/st_cb_queryobj.c | 55 - 1 file changed, 33 insertions(+), 22 deletions(-) diff --git

[Mesa-dev] [PATCH 09/10] radeonsi: shorten slot masks to 32 bits

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_blit.c| 12 +-- src/gallium/drivers/radeonsi/si_descriptors.c | 110 +- src/gallium/drivers/radeonsi/si_pipe.h| 4 +- src/gallium/drivers/radeonsi/si_state.h | 4 +- 4

[Mesa-dev] [PATCH 04/10] radeonsi: rework polygon stippling to use constant buffer instead of texture

2016-04-20 Thread Marek Olšák
From: Marek Olšák add it to the RW_BUFFERS descriptor array now the slot masks don't have to have 64 bits --- src/gallium/drivers/radeonsi/si_descriptors.c | 23 + src/gallium/drivers/radeonsi/si_pipe.c| 2 - src/gallium/drivers/radeonsi/si_pipe.h|

[Mesa-dev] [PATCH 07/10] radeonsi: move default tess level constant buffer to RW buffers

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c| 19 +++ src/gallium/drivers/radeonsi/si_shader.h| 6 ++ src/gallium/drivers/radeonsi/si_state.c | 4 ++-- src/gallium/drivers/radeonsi/si_state.h | 1 +

[Mesa-dev] [PATCH 05/10] radeonsi: move clip plane constant buffer to RW buffers

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_descriptors.c | 14 +++--- src/gallium/drivers/radeonsi/si_shader.c | 5 +++-- src/gallium/drivers/radeonsi/si_state.c | 3 ++- src/gallium/drivers/radeonsi/si_state.h | 4 4 files

Re: [Mesa-dev] [PATCH 3/5] egl: implement EGL part of interop interface (v2)

2016-04-20 Thread Aaron Watry
Hi Marek, This patch ended up breaking make check for me (src/egl/egl-symbols-check). Also, while bisecting, it occurred to me that you pushed v2 of the series, while you had previously sent a v3 to the mesa-dev list. Was pushing v2 intentional? --Aaron On Tue, Mar 8, 2016 at 4:52 PM, Marek

Re: [Mesa-dev] [PATCH] st/mesa: Use correct size for compute CAPs.

2016-04-20 Thread Nicolai Hähnle
Reviewed-by: Nicolai Hähnle On 20.04.2016 08:35, Bas Nieuwenhuizen wrote: Some CAPs are stored as 64-bit value while Mesa stores the related constant as 32-bit value. Signed-off-by: Bas Nieuwenhuizen ---

Re: [Mesa-dev] [PATCH 1/3] dri/common: add MESA_FORMAT_R8G8B8{A8, X8}_UNORM formats as supported configs

2016-04-20 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák You might want to wait for others to comment. Mare On Tue, Apr 19, 2016 at 9:38 PM, Rob Herring wrote: > Add MESA_FORMAT_R8G8B8A8_UNORM and MESA_FORMAT_R8G8B8X8_UNORM formats as > these are the preferred formats

Re: [Mesa-dev] [PATCH] Update Doxygen for Windows users

2016-04-20 Thread Rhys Kidd
On 20 April 2016 at 01:46, Elie TOURNIER wrote: > Now Windows users have the same doxygen files than *nix users > Not tested (I don't have a windows) > > Signed-off-by: Elie TOURNIER > --- > doxygen/doxy.bat | 7 +++ > 1 file changed, 7

[Mesa-dev] [PATCH 4/4] radeon: handle query buffer allocation and mapping failures

2016-04-20 Thread Nicolai Hähnle
From: Nicolai Hähnle Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94984 --- src/gallium/drivers/radeon/r600_query.c | 40 ++--- src/gallium/drivers/radeon/r600_query.h | 2 +- 2 files changed, 33 insertions(+), 9 deletions(-) diff

[Mesa-dev] [PATCH 06/10] radeonsi: move sample positions constant buffer to RW buffers

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 4 ++-- src/gallium/drivers/radeonsi/si_state.c | 4 ++-- src/gallium/drivers/radeonsi/si_state.h | 1 + 3 files changed, 5 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 02/10] radeonsi: make RW buffer descriptor array global, not per shader stage

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_descriptors.c | 50 +-- src/gallium/drivers/radeonsi/si_pipe.h| 2 +- 2 files changed, 25 insertions(+), 27 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c

[Mesa-dev] [PATCH 00/10] RadeonSI: cleanup RW shader slots

2016-04-20 Thread Marek Olšák
Hi, This moves all private shader resources to the RW buffer bindings, including all driver constant buffers, and the poly stipple image (which is converted into a constant buffer). RW buffer descriptors are made global, not per shader stage, so all shaders receive the same pointer. Finally,

[Mesa-dev] [PATCH 01/10] radeonsi: rename and rearrange RW buffer slots

2016-04-20 Thread Marek Olšák
From: Marek Olšák - use an enum - use a unique slot number regardless of the shader stage (the per-stage slots will go away for RW buffers) --- src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++--- src/gallium/drivers/radeonsi/si_shader.c| 13 ++

[Mesa-dev] [PATCH 03/10] radeonsi: generalize si_set_constant_buffer

2016-04-20 Thread Marek Olšák
From: Marek Olšák this will be used in the next commit --- src/gallium/drivers/radeonsi/si_descriptors.c | 27 +-- 1 file changed, 17 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c

[Mesa-dev] [PATCH 10/10] radeonsi: decrease GS copy shader user SGPRs to 2

2016-04-20 Thread Marek Olšák
From: Marek Olšák const buffers are no longer used since the clip plane const buffer was moved to RW buffers --- src/gallium/drivers/radeonsi/si_shader.c | 4 ++-- src/gallium/drivers/radeonsi/si_shader.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 08/10] radeonsi: clean up shader resource limit definitions

2016-04-20 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.c | 4 ++-- src/gallium/drivers/radeonsi/si_shader.c | 6 +++--- src/gallium/drivers/radeonsi/si_state.h | 25 +++-- 3 files changed, 12 insertions(+), 23 deletions(-) diff --git

Re: [Mesa-dev] [android-x86-devel] [PATCH] android: fix glcpp building error

2016-04-20 Thread Rob Herring
On Tue, Apr 19, 2016 at 8:45 PM, Mauro Rossi wrote: > LOCAL_C_INCLUDES needs an additional path to fix the following build error: > > external/mesa/src/compiler/glsl/glcpp/glcpp-lex.l:30:25: fatal error: > glcpp-parse.h: No such file or directory > #include

Re: [Mesa-dev] [PATCH] swr: fix resource backed constant buffers

2016-04-20 Thread Chuck Atkins
While i never got this error in the first place, I can attest that it certainly hasn't broken any use in paraview, so... Tested-by: Chuck Atkins - Chuck On Tue, Apr 19, 2016 at 1:45 AM, Markus Wick wrote: > Am 2016-04-19 01:12, schrieb Tim Rowley:

Re: [Mesa-dev] [PATCH] st/mesa: Use correct size for compute CAPs.

2016-04-20 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Wed, Apr 20, 2016 at 3:35 PM, Bas Nieuwenhuizen wrote: > Some CAPs are stored as 64-bit value while Mesa stores > the related constant as 32-bit value. > > Signed-off-by: Bas Nieuwenhuizen

[Mesa-dev] [PATCH] glsl: Use correct mode for split components.

2016-04-20 Thread Bas Nieuwenhuizen
The mode should stay the same as the original struct. In particular, shared should not be changed to temporary. Signed-off-by: Bas Nieuwenhuizen --- src/compiler/glsl/opt_structure_splitting.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH] configure.ac: fix the --disable-llvm-shared-libs build

2016-04-20 Thread Emil Velikov
On 19 April 2016 at 20:59, Tom Stellard wrote: > On Tue, Apr 19, 2016 at 07:39:13PM +0100, Emil Velikov wrote: >> Hi Chuck, >> >> Thanks for chipping in. >> >> On 19 April 2016 at 15:47, Chuck Atkins wrote: >> > This still doesn't quite give what you

Re: [Mesa-dev] [PATCH 3/5] i965/vec4: Use nir_intrinsic_base in the load_uniform implementation

2016-04-20 Thread Iago Toral
Patched 3-5 are: Reviewed-by: Iago Toral Quiroga On Mon, 2016-04-18 at 19:04 -0700, Jason Ekstrand wrote: > We shouldn't be reading the const_index directly > --- > src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff

Re: [Mesa-dev] [PATCH] loader: add a libdrm case for loader_get_device_name_for_fd

2016-04-20 Thread Jonathan Gray
On Wed, Apr 20, 2016 at 01:33:24PM +0100, Emil Velikov wrote: > On 20 April 2016 at 03:27, Jonathan Gray wrote: > > On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote: > >> On 19 April 2016 at 04:03, Jonathan Gray wrote: > >> > Any objections to this? >

  1   2   >