[Mesa-dev] [PATCH] glsl_to_tgsi: emit multi-level structs and arrays properly.

2012-12-09 Thread Dave Airlie
This follow the code from the i965 driver, and emits the structs and arrays recursively. This fixes an assert in the two UBO tests fs-struct-copy-complicated and vs-struct-copy-complicated These tests now pass on softpipe, with no regressions. Signed-off-by: Dave Airlie --- src/mesa/state_trac

Re: [Mesa-dev] set.c vs. hash_table.c

2012-12-09 Thread Eric Anholt
Brian Paul writes: > Hi Jordan, > > set.c and hash_table.c look a LOT alike. Could the functions in set.c > be implemented in terms of the hash_table.c functions? It seems > little silly to have so much duplicated code. The idea of the set code is to save most of the memory in the case that

Re: [Mesa-dev] [PATCH] mesa syncobj: don't store a pointer to the set_entry

2012-12-09 Thread Eric Anholt
Jordan Justen writes: > The set_entry pointer can become invalid if the set table > is re-hashed. > > This likely will fix > https://bugs.freedesktop.org/show_bug.cgi?id=58012 > (Regression since 56e95d3c) Reviewed-by: Eric Anholt pgpRhlMJQnE0s.pgp Description: PGP signature _

Re: [Mesa-dev] [PATCH 8/8] i965/fs: Improve performance of shaders that start out with a discard.

2012-12-09 Thread Kenneth Graunke
On 12/07/2012 02:08 PM, Eric Anholt wrote: I had tried this in the past, but ran into trouble with applications that sample from undiscarded pixels in the same subspan. To fix that issue, only jump to the end for an entire subspan at a time. Improves GLbenchmark 2.7 (1024x768) performance by 7.

Re: [Mesa-dev] [PATCH 7/8] i965/fs: Rewrite discards to use a flag subreg to track discarded pixels.

2012-12-09 Thread Kenneth Graunke
On 12/07/2012 02:08 PM, Eric Anholt wrote: This makes much more sense on gen6+, and will also prove useful for early exit of shaders on discard. --- src/mesa/drivers/dri/i965/brw_defines.h|1 - src/mesa/drivers/dri/i965/brw_fs.cpp |8 +++ src/mesa/drivers/dri

Re: [Mesa-dev] [PATCH 7/7] i965/fs: Add empirically-determined instruction latencies for gen7.

2012-12-09 Thread Kenneth Graunke
On 12/07/2012 02:58 PM, Eric Anholt wrote: The limited performance testing I've done on this hasn't shown any statistically significant differences yet. --- .../dri/i965/brw_fs_schedule_instructions.cpp | 150 +++- 1 file changed, 147 insertions(+), 3 deletions(-) diff -

Re: [Mesa-dev] [PATCH 3/7] i965/fs: Before reg alloc, schedule instructions to reduce live ranges.

2012-12-09 Thread Kenneth Graunke
On 12/07/2012 02:58 PM, Eric Anholt wrote: This came from an idea by Ben Segovia. 16-wide pixel shaders are very important for latency hiding on i965, so we want to try really hard to get them. If scheduling an instruction makes some set of instructions available, those are probably the ones th

[Mesa-dev] [PATCH 12/12] mesa: minor indentation fixes in texcompress_etc.c

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_etc.c | 34 +- 1 files changed, 17 insertions(+), 17 deletions(-) diff --git a/src/mesa/main/texcompress_etc.c b/src/mesa/main/texcompress_etc.c index 57c42c4..7ad4dde 100644 --- a/src/mesa/main/texcompress_etc.c +

[Mesa-dev] [PATCH 11/12] mesa: remove old swrast-based compressed texel fetch code

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_etc.c | 244 +- src/mesa/main/texcompress_etc.h | 46 --- src/mesa/main/texcompress_fxt1.c | 32 - src/mesa/main/texcompress_fxt1.h |8 -- src/mesa/main/texcompress_rgtc.c | 132 ---

[Mesa-dev] [PATCH 10/12] swrast: use new core Mesa compressed texel fetch functions

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/swrast/s_context.h |4 + src/mesa/swrast/s_texfetch.c | 193 +++--- 2 files changed, 110 insertions(+), 87 deletions(-) diff --git a/src/mesa/swrast/s_context.h b/src/mesa/swrast/s_context.h index 18353c4..f3f188e 100644 --

[Mesa-dev] [PATCH 09/12] mesa: reimplement _mesa_decompress_image() using new tex fetch code

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress.c | 110 +++ 1 files changed, 7 insertions(+), 103 deletions(-) diff --git a/src/mesa/main/texcompress.c b/src/mesa/main/texcompress.c index 9ad6a8b..33c580a 100644 --- a/src/mesa/main/texcompress.c +++ b/sr

[Mesa-dev] [PATCH 08/12] mesa: added _mesa_get_compressed_fetch_func()

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress.c | 33 + src/mesa/main/texcompress.h |3 +++ 2 files changed, 36 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texcompress.c b/src/mesa/main/texcompress.c index 372a483..9ad6a8b 100644 --- a/src/mes

[Mesa-dev] [PATCH 07/12] mesa: add new texel fetch code for etc formats

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_etc.c | 274 +++ src/mesa/main/texcompress_etc.h |6 + 2 files changed, 280 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texcompress_etc.c b/src/mesa/main/texcompress_etc.c index 73d2fa4..c8bf6ea

[Mesa-dev] [PATCH 06/12] mesa: add new texel fetch code for rgtc formats

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_rgtc.c | 161 ++ src/mesa/main/texcompress_rgtc.h |5 + 2 files changed, 166 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texcompress_rgtc.c b/src/mesa/main/texcompress_rgtc.c index 5773459..de71

[Mesa-dev] [PATCH 05/12] mesa: add new texel fetch code for fxt formats

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_fxt1.c | 42 ++ src/mesa/main/texcompress_fxt1.h |3 ++ 2 files changed, 45 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texcompress_fxt1.c b/src/mesa/main/texcompress_fxt1.c index eeed788..6a0f

[Mesa-dev] [PATCH 04/12] mesa: add new texel fetch code for dxt formats

2012-12-09 Thread Brian Paul
From: Brian Paul --- src/mesa/main/texcompress_s3tc.c | 106 +- src/mesa/main/texcompress_s3tc.h |5 ++ 2 files changed, 110 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc.c b/src/mesa/main/texcompress_s3tc.c index 476b998..959

[Mesa-dev] [PATCH 03/12] mesa: add compressed_fetch_func typedef

2012-12-09 Thread Brian Paul
From: Brian Paul This is a first step in removing the swrast-related code in core Mesa's texture compression files. --- src/mesa/main/texcompress.h |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texcompress.h b/src/mesa/main/texcompress.h index 359

[Mesa-dev] [PATCH 02/12] swrast: merge get_texel_fetch_func() and set_fetch_functions()

2012-12-09 Thread Brian Paul
From: Brian Paul No real need for separate functions anymore. --- src/mesa/swrast/s_texfetch.c | 46 ++--- 1 files changed, 20 insertions(+), 26 deletions(-) diff --git a/src/mesa/swrast/s_texfetch.c b/src/mesa/swrast/s_texfetch.c index 9117885..c133eac 100

[Mesa-dev] [PATCH 01/12] swrast: make _mesa_get_texel_fetch_func() static

2012-12-09 Thread Brian Paul
From: Brian Paul Not called from any other file. --- src/mesa/swrast/s_texfetch.c |8 src/mesa/swrast/s_texfetch.h |3 --- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/src/mesa/swrast/s_texfetch.c b/src/mesa/swrast/s_texfetch.c index 86b01a0..9117885 100644 ---

[Mesa-dev] R: Re: Gallium versioning, currently at 0.4

2012-12-09 Thread Fabio Pedretti
So, is there a plan to do it? >Da: mar...@gmail.com > >The Gallium version has no meaning to me. The renderer string could >just be "ATI RV530". > >Marek > >On Wed, Oct 31, 2012 at 6:05 PM, Fabio Pedretti wrote: >> Is there a reason to keep the '0.4' on Gallium renderer string ( 'Gallium 0.4 >>

[Mesa-dev] [PATCH] vega: remove unused variables

2012-12-09 Thread Fabio Pedretti
diff --git a/src/gallium/state_trackers/vega/path.c b/src/gallium/state_trackers/vega/path.c index 43755f4..31ec719 100644 --- a/src/gallium/state_trackers/vega/path.c +++ b/src/gallium/state_trackers/vega/path.c @@ -1085,10 +1085,8 @@ static INLINE VGubyte normalize_coords(struct path_iter_data *p

[Mesa-dev] [PATCH] nvc0: comment unused nvc0_validate_zcull function

2012-12-09 Thread Fabio Pedretti
diff --git a/src/gallium/drivers/nvc0/nvc0_state_validate.c b/src/gallium/drivers/nvc0/nvc0_state_validate.c index 0f92614..80a8c01 100644 --- a/src/gallium/drivers/nvc0/nvc0_state_validate.c +++ b/src/gallium/drivers/nvc0/nvc0_state_validate.c @@ -3,6 +3,7 @@ #include "nvc0_context.h" +#if 0

[Mesa-dev] [PATCH] nv50: remove unused OpClassStr array

2012-12-09 Thread Fabio Pedretti
diff --git a/src/gallium/drivers/nv50/codegen/nv50_ir_print.cpp b/src/gallium/drivers/nv50/codegen/nv50_ir_print.cpp index ded4f61..ea81e53 100644 --- a/src/gallium/drivers/nv50/codegen/nv50_ir_print.cpp +++ b/src/gallium/drivers/nv50/codegen/nv50_ir_print.cpp @@ -67,26 +67,6 @@ static void init_co

[Mesa-dev] R: Re: [PATCH] silence unused code warnings

2012-12-09 Thread Fabio Pedretti
>Comments below. Thanks, updated patches follow. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] mesa: Restore NULL context check in _mesa_reference_renderbuffer_().

2012-12-09 Thread Ian Romanick
That's not a false alarm... That's supposed to work! libGL and driver versions are supposed to be independent. Anytime they're not is a bug. On Dec 8, 2012, at 8:43 PM, Kenneth Graunke wrote: > On 12/08/2012 05:40 AM, Brian Paul wrote: >> On 12/08/2012 01:10 AM, Kenneth Graunke wrote: >>> Start

Re: [Mesa-dev] [PATCH] mesa syncobj: don't store a pointer to the set_entry

2012-12-09 Thread Stefan Dösinger
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am 2012-12-08 22:24, schrieb Jordan Justen: > This likely will fix > https://bugs.freedesktop.org/show_bug.cgi?id=58012 (Regression > since 56e95d3c) The crashes are gone on r300g and r600g. This is most likely unrelated, but on r600g I get broken re

[Mesa-dev] [PATCH 3/4] AMDGPU: enable S_*N2_* instructions

2012-12-09 Thread Christian König
They seem to work fine. Signed-off-by: Christian König --- lib/Target/AMDGPU/SIInstructions.td |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/Target/AMDGPU/SIInstructions.td b/lib/Target/AMDGPU/SIInstructions.td index e9bbe23..42fa8e6 100644 --- a/lib/Target/

[Mesa-dev] [PATCH 2/4] AMDGPU: BB operand support for SI

2012-12-09 Thread Christian König
Signed-off-by: Christian König --- lib/Target/AMDGPU/AMDGPUMCInstLower.cpp| 10 -- lib/Target/AMDGPU/AMDGPUMCInstLower.h |5 - .../AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp | 10 +- lib/Target/AMDGPU/MCTargetDesc/SIMCCodeEmitter.cpp |6

[Mesa-dev] [PATCH 1/4] AMDGPU: remove nonsense setPrefLoopAlignment

2012-12-09 Thread Christian König
The Align parameter is a power of two, so 16 results in 64K alignment. Additional to that even 16 byte alignment doesn't make any sense, so just remove it. Signed-off-by: Christian König --- lib/Target/AMDGPU/AMDILISelLowering.cpp |1 - 1 file changed, 1 deletion(-) diff --git a/lib/Target/

Re: [Mesa-dev] [PATCH] drisw: fix up context and apis for software context

2012-12-09 Thread Dave Airlie
On 9 Dec 2012 20:32, "Dave Airlie" wrote: > > This ports over from the dri2 code to the drisw bits. It means 3.1 > core contexts now work for softpipe. > Well if soft pipe had msaa they would but its enough to hack so I can test ubo/tbo better. Dave. > Signed-off-by: Dave Airlie > --- > src/me

[Mesa-dev] [PATCH] drisw: fix up context and apis for software context

2012-12-09 Thread Dave Airlie
This ports over from the dri2 code to the drisw bits. It means 3.1 core contexts now work for softpipe. Signed-off-by: Dave Airlie --- src/mesa/drivers/dri/common/drisw_util.c | 16 1 file changed, 16 insertions(+) diff --git a/src/mesa/drivers/dri/common/drisw_util.c b/src/me