Re: [Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Francisco Jerez
Zoltan Gilian writes: > --- > src/gallium/state_trackers/clover/core/memory.cpp | 2 +- > src/gallium/state_trackers/clover/core/resource.cpp | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/core/memory.cpp > b/src/gallium/state_track

Re: [Mesa-dev] [PATCH 2/3] r600g: fix the CB_SHADER_MASK setup

2015-08-03 Thread Emil Velikov
On 2 August 2015 at 14:36, Marek Olšák wrote: > From: Marek Olšák > > This fixes the single-sample fast clear hang. > > Cc: 10.6 > --- > src/gallium/drivers/r600/evergreen_state.c | 8 > src/gallium/drivers/r600/r600_shader.c | 1 + > 2 files changed, 5 insertions(+), 4 deletions(-

[Mesa-dev] [PATCH] clover: fix image resource depth and array_size

2015-08-03 Thread Zoltan Gilian
--- src/gallium/state_trackers/clover/core/memory.cpp | 2 +- src/gallium/state_trackers/clover/core/resource.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/core/memory.cpp b/src/gallium/state_trackers/clover/core/memory.cpp index 0553

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965/bdw: Fix 3DSTATE_VF_INSTANCING when the edge flag is used

2015-08-03 Thread Neil Roberts
Hi Emil, I posted a v2 of this patch because the v1 wouldn't work in combination with a fix for bug 84677. I think it would be better to wait for review on those two and land both of them instead of this patch. Neither of them are particulary urgent bugs so it's probably not worth worrying about

Re: [Mesa-dev] [PATCH v2 00/78] i965: A new vec4 backend based on NIR

2015-08-03 Thread Eduardo Lima Mitev
On 08/01/2015 02:59 PM, Eduardo Lima Mitev wrote: > On 07/31/2015 10:48 AM, Eduardo Lima Mitev wrote: >> On 07/30/2015 09:48 PM, Jason Ekstrand wrote: >>> >>> On Jul 27, 2015 3:39 PM, "Jason Ekstrand" >> > wrote: On Mon, Jul 27, 2015 at 2:07 PM, Eduardo Lima M

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

2015-08-03 Thread Martin Peres
On 03/08/15 13:50, Tapani Pälli wrote: On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended.

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

2015-08-03 Thread Tapani Pälli
On 08/03/2015 01:48 PM, Martin Peres wrote: On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_q

Re: [Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

2015-08-03 Thread Martin Peres
On 03/08/15 11:10, Tapani Pälli wrote: _mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_query.transform-feedback-types Signed-off-by: T

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

2015-08-03 Thread Timothy Arceri
On Mon, 2015-08-03 at 19:32 +1000, Timothy Arceri wrote: > On Sun, 2015-08-02 at 20:39 -0400, Ilia Mirkin wrote: > > Given that this is a debug-only thing, I doubt perf numbers are that > > interesting. > > > > I have no clue what the diff is between the two hash tables, but if > > one is allegedl

Re: [Mesa-dev] [PATCH] clover: clEnqueue* should block when asked for

2015-08-03 Thread Francisco Jerez
EdB writes: > As a side effect, this fix clRetain/ReleaseEvent Piglit test > --- > src/gallium/state_trackers/clover/api/transfer.cpp | 29 > -- > 1 file changed, 27 insertions(+), 2 deletions(-) > > diff --git a/src/gallium/state_trackers/clover/api/transfer.cpp > b/src/ga

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

2015-08-03 Thread Timothy Arceri
On Mon, 2015-08-03 at 03:54 +0200, Alejandro Seguí Díaz wrote: > I'm starting to contribute and Timothy Arceri told me about this > task. The change was made at that sole file because we talked > about no need to migrate all tables at once, but we discuss this > and I take note to migrate all hash

Re: [Mesa-dev] [PATCH] glsl: replace old hash table with new and faster one

2015-08-03 Thread Timothy Arceri
On Sun, 2015-08-02 at 20:39 -0400, Ilia Mirkin wrote: > Given that this is a debug-only thing, I doubt perf numbers are that > interesting. > > I have no clue what the diff is between the two hash tables, but if > one is allegedly faster than the other, that should be determined, and > we should j

Re: [Mesa-dev] [PATCH] nir: Use a single bit for the dual-source blend index

2015-08-03 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 08/02/2015 08:04 AM, Timothy Arceri wrote: The only values allowed are 0 and 1, and the value is checked before assigning. This is a copy of 8eeca7a56c that seems to have been made to the glsl ir type after it was copied for use in nir but before nir landed. ---

Re: [Mesa-dev] [PATCH] gallium/radeon: suspend timer queries between IBs

2015-08-03 Thread Michel Dänzer
On 01.08.2015 00:15, Marek Olšák wrote: > From: Marek Olšák > > When we are measuring the time spent in a draw call, an unexpected flush > can distort the result. Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast

Re: [Mesa-dev] [PATCH 2/2] radeon/winsys: increase the IB size for VM

2015-08-03 Thread Michel Dänzer
On 01.08.2015 00:16, Marek Olšák wrote: > From: Marek Olšák > > Luckily, there is a kernel query, so use the size from that. > It currently returns 256KB. It can be increased in the kernel. Both patches are Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer |

Re: [Mesa-dev] [PATCH] gallium/radeon: always use the llvm. prefix in intrinsic names

2015-08-03 Thread Michel Dänzer
On 01.08.2015 00:35, Marek Olšák wrote: > From: Marek Olšák [...] > @@ -1176,8 +1176,18 @@ static void emit_frac( > struct lp_build_emit_data * emit_data) > { > LLVMBuilderRef builder = bld_base->base.gallivm->builder; > - > - LLVMValueRef floor = lp_build_intrinsic(buil

[Mesa-dev] [PATCH] mesa: fix name returned for XFB varyings

2015-08-03 Thread Tapani Pälli
_mesa_get_program_resource_name has logic to append '[0]' in name if variable is an array, this should be skipped for XFB varyings that have array index already appended. Fixes: ES31-CTS.program_interface_query.transform-feedback-types Signed-off-by: Tapani Pälli --- src/mesa/main/shader_que

Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement glMemoryBarrierByRegion

2015-08-03 Thread Lofstedt, Marta
> -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On > Behalf Of Matt Turner > Sent: Friday, July 31, 2015 7:48 PM > To: Ilia Mirkin > Cc: mesa-dev@lists.freedesktop.org > Subject: Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement > glMemoryBarrierByRegion

Re: [Mesa-dev] [PATCH] vl/mpeg12: Silence GCC unused-variable warning.

2015-08-03 Thread Christian König
On 01.08.2015 20:45, Vinson Lee wrote: vl/vl_mpeg12_bitstream.c: In function 'decode_slice': vl/vl_mpeg12_bitstream.c:928:19: warning: unused variable 'extra' [-Wunused-variable] unsigned extra = vl_vlc_get_uimsbf(&bs->vlc, 1); ^ Signed-off-by: Vinson Lee Revie

[Mesa-dev] [PATCH v2] mesa: fix ARRAY_SIZE query for GetProgramResourceiv

2015-08-03 Thread Tapani Pälli
Fixes rest of the failures with ES31-CTS.program_interface_query.no-locations v2: make additional check only for GS inputs Signed-off-by: Tapani Pälli --- src/mesa/main/shader_query.cpp | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/shader_query

<    1   2