[Mesa-dev] [PATCH] radeonsi: Fix memory leak in si_shader_select.

2013-02-19 Thread Vinson Lee
Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee --- src/gallium/drivers/radeonsi/si_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index d20e3ff..7f76cb5 100644 --- a/src/ga

Re: [Mesa-dev] [PATCH] mesa: Don't install glEvalMesh in the beginend dispatch table

2013-02-19 Thread Eric Anholt
Ian Romanick writes: > From: Ian Romanick > > NOTE: This is a candidate for the 9.1 branch. > > Signed-off-by: Ian Romanick > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59740 > Cc: Eric Anholt I had "make a GL 1.1 testcase like gl-1.0/beginend" on the back burner because of this,

[Mesa-dev] [Bug 61153] New: [softpipe] piglit interpolation-noperspective-gl_BackColor-flat-vertex regression

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61153 Priority: medium Bug ID: 61153 Keywords: regression CC: bri...@vmware.com Assignee: mesa-dev@lists.freedesktop.org Summary: [softpipe] piglit interpolation-no

[Mesa-dev] [PATCH 4/4] i965/vs/gen7: Allow MATH instructions to have MRF as a destination.

2013-02-19 Thread Matt Turner
total instructions in shared programs: 346873 -> 346847 (-0.01%) instructions in affected programs: 364 -> 338 (-7.14%) (All affected shaders are from Lightsmark) --- src/mesa/drivers/dri/i965/brw_vec4.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/driv

[Mesa-dev] [PATCH 3/4] i965/fs/gen7: Allow MATH instructions to have MRF as a destination.

2013-02-19 Thread Matt Turner
total instructions in shared programs: 1376297 -> 1375626 (-0.05%) instructions in affected programs: 35977 -> 35306 (-1.87%) --- src/mesa/drivers/dri/i965/brw_fs.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/driv

[Mesa-dev] [PATCH 2/4] i965/gen7: Relax restrictions on fake MRFs.

2013-02-19 Thread Matt Turner
Gen6 has write-only MRF registers, and for ease of implementation we paritition off 16 general purposes registers to act as MRFs on Gen7. Knowing that our Gen7 MRFs are actually GRFs, we can potentially do things we can't do with real MRFs: - read from them; - return values directly to them

[Mesa-dev] [PATCH 1/4] i965/fs: Remove duplicate scan_inst->mlen check.

2013-02-19 Thread Matt Turner
Is already checked 20 lines below. --- src/mesa/drivers/dri/i965/brw_fs.cpp |5 - 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index bdb6616..56358f7 100644 --- a/src/mesa/drivers/dri/i965/brw_fs

Re: [Mesa-dev] [PATCH 3/9] glsl: Convert mix() to use a new ir_triop_lrp opcode.

2013-02-19 Thread Roland Scheidegger
Not much to say about the code (the theory sounds sane) but I was wondering about the comment. Why did glsl implement this really as x * (1 - a) + y * a? The usual way for lerp would be (y - x) * a + x, i.e. two ops for most gpus (sub+mad, or sub+mul+add). But I'm wondering if that sacrifices preci

[Mesa-dev] [PATCH 4/4] mesa: Reduce memory usage for reg alloc with many graph nodes (part 2).

2013-02-19 Thread Eric Anholt
After the previous fix that almost removes an allocation of 4*n^2 bytes, we can use a bitset to reduce another allocation from n^2 bytes to n^2/8 bytes. Between the previous commit and this one, the peak heap size for an oglconform ARB_fragment_program max instructions test on i965 goes from 4GB t

[Mesa-dev] [PATCH 1/4] i965/fs: Improve live variables calculation performance.

2013-02-19 Thread Eric Anholt
We can execute way fewer instructions by doing our boolean manipulation on an "int" of bits at a time, while also reducing our working set size. Reduces compile time of L4D2's slowest shader from 4s to 1.1s (-72.4% +/- 0.2%, n=10) --- .../drivers/dri/i965/brw_fs_live_variables.cpp | 44

[Mesa-dev] [PATCH 3/4] mesa: Reduce the memory usage for reg alloc with many graph nodes (part 1)

2013-02-19 Thread Eric Anholt
We were allocating an adjacency_list entry for every possible interference that could get created, but that usually doesn't happen. We can save a lot of memory by resizing the array on demand. --- src/mesa/program/register_allocate.c | 14 +- 1 file changed, 13 insertions(+), 1 delet

[Mesa-dev] [PATCH 2/4] i965/fs: Improve CSE performance by expiring some available expressions.

2013-02-19 Thread Eric Anholt
We're already walking the list, and we can easily know when something has no reason to be in the list any longer, so take a brief extra step to reduce our worst-case runtime (an oglconform test that emits the maximum instructions in a fragment program). I don't actually know what the worst-case ru

Re: [Mesa-dev] [PATCH] i965: Avoid segfault in gen6_upload_state

2013-02-19 Thread Ian Romanick
On 02/19/2013 04:27 PM, Carl Worth wrote: This fixes a bug introduced in commit 258453716f001eab1288d99765213 and triggered whenever "rb" is NULL. Fixes bug #59445: [SNB/IVB/HSW Bisected]Oglc draw-buffers2(advanced.blending.none) segfault https://bugs.freedesktop.org/show_bug.c

[Mesa-dev] [Bug 61149] Crash on Intel Sandybridge Mobile with Vertex Buffer Objects and select mode OpenGL rendering

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61149 --- Comment #1 from Antony Riakiotakis --- Attaching full backtrace. Some information that might prove useful, we are using glDrawElements as a draw call. #0 0x7fffe2900a3f in run_vp (ctx=, stage=) at ../../../../../src/mesa/tnl/t_vb_p

[Mesa-dev] [Bug 61149] New: Crash on Intel Sandybridge Mobile with Vertex Buffer Objects and select mode OpenGL rendering

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61149 Priority: medium Bug ID: 61149 Assignee: mesa-dev@lists.freedesktop.org Summary: Crash on Intel Sandybridge Mobile with Vertex Buffer Objects and select mode OpenGL rendering

[Mesa-dev] [PATCH 9/9] i965/vs: Assert that ir_triop_lrp was lowered.

2013-02-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp index ae4cf7d..a2bc9f5 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_vis

[Mesa-dev] [PATCH 0/9] LRP

2013-02-19 Thread Matt Turner
This series adds ir_triop_lrp to the IR. A few patches clear the way since it is the first 3-operand operator. The next patches - emit lrp from GLSL's mix() function; - optimize away the a = 0.0 and 1.0 cases; - add i965 support for emitting the LRP instruction in fragment shaders and fragment

[Mesa-dev] [PATCH 8/9] ir_to_mesa: Translate ir_triop_lrp to OPCODE_LRP.

2013-02-19 Thread Matt Turner
--- src/mesa/program/ir_to_mesa.cpp |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp index 30305d2..5432323 100644 --- a/src/mesa/program/ir_to_mesa.cpp +++ b/src/mesa/program/ir_to_mesa.cpp @@ -1479,7

[Mesa-dev] [PATCH 7/9] i965/fp: Use the LRP instruction for OPCODE_LRP.

2013-02-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 12 1 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_fp.cpp b/src/mesa/drivers/dri/i965/brw_fs_fp.cpp index 5f5f6a9..50e63da 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_fp.cpp +++ b/src/m

[Mesa-dev] [PATCH 6/9] i965/fs: Use the LRP instruction for ir_triop_lrp when possible.

2013-02-19 Thread Matt Turner
From: Kenneth Graunke v2 [mattst88]: - Add BRW_OPCODE_LRP to list of CSE-able expressions. - Fix op_var[] array size. - Rename arguments to emit_lrp to (x, y, a) to clear confusion. - Add LRP function to brw_fs.cpp/.h. - Corrected comment about LRP instruction arguments in emit_lrp

[Mesa-dev] [PATCH 5/9] i965: Add support for emitting the LRP instruction.

2013-02-19 Thread Matt Turner
From: Kenneth Graunke Like MAD, this is another three-source instruction. Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/brw_defines.h |1 + src/mesa/drivers/dri/i965/brw_disasm.c |1 + src/mesa/drivers/dri/i965/brw_eu.h |1 + src/mesa/drivers/dri/i965/brw_eu_emit.c |

[Mesa-dev] [PATCH 4/9] glsl: Optimize ir_triop_lrp(x, y, a) with a = 0.0f or 1.0f

2013-02-19 Thread Matt Turner
--- src/glsl/opt_algebraic.cpp | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp index 75948db..952941e 100644 --- a/src/glsl/opt_algebraic.cpp +++ b/src/glsl/opt_algebraic.cpp @@ -186,12 +186,12 @@ ir_a

[Mesa-dev] [PATCH 3/9] glsl: Convert mix() to use a new ir_triop_lrp opcode.

2013-02-19 Thread Matt Turner
From: Kenneth Graunke Many GPUs have an instruction to do linear interpolation which is more efficient than simply performing the algebra necessary (two multiplies, an add, and a subtract). Pattern matching or peepholing this is more desirable, but can be tricky. By using an opcode, we can at l

[Mesa-dev] [PATCH 2/9] glsl: Rework ir_reader to handle expressions with three operands.

2013-02-19 Thread Matt Turner
From: Kenneth Graunke Reviewed-by: Matt Turner --- src/glsl/ir_reader.cpp | 45 +++-- 1 files changed, 19 insertions(+), 26 deletions(-) diff --git a/src/glsl/ir_reader.cpp b/src/glsl/ir_reader.cpp index 405e75b..4dec4e8 100644 --- a/src/glsl/ir_reader

[Mesa-dev] [PATCH 1/9] glsl: Consolidate ir_expression constructors that use explicit types.

2013-02-19 Thread Matt Turner
From: Kenneth Graunke Previously, we had separate constructors for one, two, and four operand expressions. This patch consolidates them into a single constructor which uses NULL default parameters. The unary and binary operator constructors had assertions to verify that the caller supplied the

[Mesa-dev] [PATCH] i965: Avoid segfault in gen6_upload_state

2013-02-19 Thread Carl Worth
This fixes a bug introduced in commit 258453716f001eab1288d99765213 and triggered whenever "rb" is NULL. Fixes bug #59445: [SNB/IVB/HSW Bisected]Oglc draw-buffers2(advanced.blending.none) segfault https://bugs.freedesktop.org/show_bug.cgi?id=59445 --- I don't know under what con

Re: [Mesa-dev] [PATCH] meta: Allocate texture before initializing texture coordinates

2013-02-19 Thread Anuj Phogat
On Fri, Feb 15, 2013 at 11:20 AM, Anuj Phogat wrote: > tex->Sright and tex->Ttop are initialized during texture allocation. > This fixes depth buffer blitting failures in khronos conformance tests > when run on desktop GL 3.0. > > Note: This is a candidate for stable branches. > > Signed-off-by: A

Re: [Mesa-dev] [PATCH 2/2] i965: Consign COORD_REPLACE VS hacks to Pre-Gen6.

2013-02-19 Thread Kenneth Graunke
On 02/16/2013 07:29 AM, Paul Berry wrote: Pre-Gen6, the SF thread requires exact matching between VS output slots (aka VUE slots) and FS input slots, even when the corresponding VS output slot is unused due to being overwritten by point coordinate replacement (glTexEnvi(GL_POINT_SPRITE, GL_COORD_

[Mesa-dev] [PATCH] mesa: Don't install glEvalMesh in the beginend dispatch table

2013-02-19 Thread Ian Romanick
From: Ian Romanick NOTE: This is a candidate for the 9.1 branch. Signed-off-by: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59740 Cc: Eric Anholt --- src/mesa/main/eval.c | 11 --- src/mesa/main/eval.h | 4 +++- src/mesa/main/vtxfmt.c | 10 +- 3 fi

Re: [Mesa-dev] Need bench mark application for Opengles2 on mesa-8.0.4 with Linux

2013-02-19 Thread Ian Romanick
On 02/18/2013 02:31 AM, Ramesh Reddy Emmadi wrote: Hi, Can you please let us know is there any benchmark tool for opengles2 API's in Linux and Windows. Thanks and Regards, Ramesh CAUTION - Disclaimer * This e-mail contains PRIVILEGED AND CONFIDENTIAL INFORMATI

[Mesa-dev] [Bug 59331] piglit arb_uniform_buffer_object-getintegeri_v regression

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59331 Ian Romanick changed: What|Removed |Added CC||xunx.f...@intel.com --- Comment #4 from I

[Mesa-dev] [Bug 59331] piglit arb_uniform_buffer_object-getintegeri_v regression

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59331 Ian Romanick changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH libdrm] freedreno: add freedreno DRM

2013-02-19 Thread Eric Anholt
Rob Clark writes: > From: Rob Clark > > The libdrm_freedreno helper layer for use by xf86-video-freedreno, > fdre (freedreno r/e library and tests for driving gpu), and eventual > gallium driver for the Adreno GPU. This uses the msm gpu driver > from QCOM's android kernel tree. > > Note that cu

Re: [Mesa-dev] [PATCH] llvmpipe: lp_resource_copy cleanup

2013-02-19 Thread Jose Fonseca
- Original Message - > From: Roland Scheidegger > > We don't need to flush resources for each layer, and since we don't actually > care about layer at all in the flush function just drop the parameter. > Also we can use util_copy_box instead of repeated util_copy_rect. > --- > src/gall

[Mesa-dev] [PATCH] gallivm: fix indirect src register fetches requiring bitcast

2013-02-19 Thread sroland
From: Roland Scheidegger For constant and temporary register fetches, the bitcasts weren't done correctly for the indirect case, leading to crashes due to type mismatches. Simply do the bitcasts after fetching (much simpler than fixing up the load pointer for the various cases). This fixes https

[Mesa-dev] [PATCH] llvmpipe: lp_resource_copy cleanup

2013-02-19 Thread sroland
From: Roland Scheidegger We don't need to flush resources for each layer, and since we don't actually care about layer at all in the flush function just drop the parameter. Also we can use util_copy_box instead of repeated util_copy_rect. --- src/gallium/drivers/llvmpipe/lp_flush.c |3 +-

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-19 Thread Vadim Girlin
On 02/19/2013 08:39 PM, Andy Furniss wrote: Vadim Girlin wrote: Could you please test glxgears and other simple mesa demos? It's easier to spot the problems with small apps that don't use a lot of complex shaders. If some of them don't work correctly, please send me the dumps with "R600_DUMP_SH

[Mesa-dev] [PATCH] st/mesa: fix trimming of GL_QUAD_STRIP

2013-02-19 Thread Brian Paul
We sometimes convert GL_QUAD_STRIP prims into GL_TRIANGLE_STRIP, but that changes the results of the u_trim_pipe_prim() call. We need to pass the original primitive type to the trim function. Note that OpenGL's GL_x prim type values match Gallium's PIPE_PRIM_x values. Fixes a failure in the new

[Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread sroland
From: Roland Scheidegger Some parts calculated key size by using shader information, others by using the pipe_vertex_element information. Since it is perfectly valid to have more vertex_elements set than the vertex shader is using those may not be the same, so we weren't copying over all vertex_e

Re: [Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread Roland Scheidegger
Am 19.02.2013 18:54, schrieb Jose Fonseca: > > > - Original Message - >> Am 19.02.2013 15:57, schrieb Jose Fonseca: >>> There may be more vertex elements that used in the shader. But why should >>> the key contain those elements? Won't this cause needless recompilations >>> (e.g., in situ

Re: [Mesa-dev] [PATCH 0/3] radeonsi: Cherry-pick transfer fixes from r600g

2013-02-19 Thread Alex Deucher
On Tue, Feb 19, 2013 at 12:15 PM, Michel Dänzer wrote: > These together get us 11 more little piglits with Marek's > glTex(Sub)Image improvements in st/mesa. > > [PATCH 1/3] radeonsi: use u_box_origin_2d helper function > [PATCH 2/3] radeonsi: add assertions to prevent creation of invalid > [PATCH

Re: [Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread Jose Fonseca
- Original Message - > Am 19.02.2013 15:57, schrieb Jose Fonseca: > > There may be more vertex elements that used in the shader. But why should > > the key contain those elements? Won't this cause needless recompilations > > (e.g., in situations where the state tracker leaves unneeded ele

Re: [Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread Roland Scheidegger
Am 19.02.2013 15:57, schrieb Jose Fonseca: > There may be more vertex elements that used in the shader. But why should the > key contain those elements? Won't this cause needless recompilations (e.g., > in situations where the state tracker leaves unneeded elements from previous > draw?)? I don'

Re: [Mesa-dev] VTK Tests fails with Mesa Swrast passes with OSMesa

2013-02-19 Thread Kevin H. Hobbs
On 02/19/2013 09:51 AM, Brian Paul wrote: > > Looks like lines, in particular, are missing. I don't see any recent > changes to swrast/osmesa that would seem to cause this. > There probably were none. I'm trying to track down long standing issues. > > 1. You do a git-bisect of mesa to find t

[Mesa-dev] [PATCH 3/3] radeonsi: implement 3D transfers

2013-02-19 Thread Michel Dänzer
From: Marek Olšák That means we can map and read multiple slices with one transfer_map call. [ Cherry-picked from r600g commit 1aebb6911e9aa1bd8900868b58d1750ca83a20c7 ] Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/r600_texture.c | 49 + 1 file cha

[Mesa-dev] [PATCH 2/3] radeonsi: add assertions to prevent creation of invalid surfaces

2013-02-19 Thread Michel Dänzer
From: Marek Olšák [ Cherry-picked from r600g commit ef11ed61a0414d0405c3faf7f48fa3f1d083f82e ] Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/r600_blit.c | 15 --- src/gallium/drivers/radeonsi/r600_texture.c | 2 ++ src/gallium/drivers/radeonsi/radeonsi_pipe.h

[Mesa-dev] [PATCH 0/3] radeonsi: Cherry-pick transfer fixes from r600g

2013-02-19 Thread Michel Dänzer
These together get us 11 more little piglits with Marek's glTex(Sub)Image improvements in st/mesa. [PATCH 1/3] radeonsi: use u_box_origin_2d helper function [PATCH 2/3] radeonsi: add assertions to prevent creation of invalid [PATCH 3/3] radeonsi: implement 3D transfers

[Mesa-dev] [PATCH 1/3] radeonsi: use u_box_origin_2d helper function

2013-02-19 Thread Michel Dänzer
From: Marek Olšák [ Cherry-picked from r600g commit b278aba42310e8fa30f2408b9dcd58dbb4901724 ] Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/r600_texture.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/radeonsi/r600_texture.c b/

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #7 from Laurent carlier --- (In reply to comment #6) > Can you make a trace of this issue with apitrace? > https://github.com/apitrace/apitrace You can find it here: http://pkgbuild.com/~lcarlier/traces/hl2_linux.trace.tar.gz -- Y

Re: [Mesa-dev] [PATCH] llvmpipe: fix lp_resource_copy using more than one 3d slice

2013-02-19 Thread Roland Scheidegger
Am 19.02.2013 10:13, schrieb Jose Fonseca: > Thanks for fixing this Roland. > > This is definitely an improvement. I'd recommend a few tweaks (it could even > be as a follow on change): > > - Calling llvmpipe_flush_resource() in a loop is overkill (it will call > llvmpipe_flush() to be called m

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-19 Thread Andy Furniss
Vadim Girlin wrote: Could you please test glxgears and other simple mesa demos? It's easier to spot the problems with small apps that don't use a lot of complex shaders. If some of them don't work correctly, please send me the dumps with "R600_DUMP_SHADERS=2 R600_SB_DUMP=3". All of the mesa de

[Mesa-dev] [RFC] New EGL extension: EGL_EXT_platform_display

2013-02-19 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm seeking feedback on an EGL extension that I'm drafting. The ideas have already been discussed at Khronos meetings to a good reception, but I want feedback from Mesa developers too. Summary - --- The extension, tentatively named EGL_EXT_platfor

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Marek Olšák
On Tue, Feb 19, 2013 at 3:55 PM, Michel Dänzer wrote: > On Die, 2013-02-19 at 15:48 +0100, Marek Olšák wrote: >> On Tue, Feb 19, 2013 at 3:28 PM, Michel Dänzer wrote: >> > On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote: >> >> On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer >> >> wrote: >>

Re: [Mesa-dev] [PATCH v2] configure.ac: Do not check for clock_gettime on MinGW.

2013-02-19 Thread Matt Turner
On Tue, Feb 19, 2013 at 12:55 AM, Vinson Lee wrote: > MinGW does not have clock_gettime. > > Signed-off-by: Vinson Lee > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index 16c2f8c..1e11b4e 100644 > --- a/configure.ac >

Re: [Mesa-dev] [PATCH 1/8] R600/SI: cleanup SIInstrInfo.td and SIInstrFormat.td

2013-02-19 Thread Tom Stellard
Hi Christian, >From now on can you cc llvm-comm...@cs.uiuc.edu when you submit a patch. Thanks, Tom On Tue, Feb 19, 2013 at 02:54:23PM +0100, Christian König wrote: > From: Christian König > > Those two files got mixed up. > > Signed-off-by: Christian König > --- > lib/Target/R600/SIInstrF

Re: [Mesa-dev] [PATCH 4/6] R600: Fix for Unigine when MachineSched is enabled

2013-02-19 Thread Tom Stellard
On Mon, Feb 18, 2013 at 05:27:28PM +0100, Vincent Lejeune wrote: Reviewed-by: Tom Stellard > --- > lib/Target/R600/R600Instructions.td | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/lib/Target/R600/R600Instructions.td > b/lib/Target/R600/R600Instructions.td > index 0a777f1..74106c9 1

Re: [Mesa-dev] [PATCH 3/6] R600: Turn BUILD_VECTOR into Reg_Sequence

2013-02-19 Thread Tom Stellard
On Mon, Feb 18, 2013 at 05:27:27PM +0100, Vincent Lejeune wrote: Reviewed-by: Tom Stellard > --- > lib/Target/R600/AMDILISelDAGToDAG.cpp | 29 + > 1 file changed, 29 insertions(+) > > diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp > b/lib/Target/R600/AMDILISelDA

Re: [Mesa-dev] [PATCH 2/6] R600: CONST_ADDRESS node is not marked as mayLoad anymore

2013-02-19 Thread Tom Stellard
On Mon, Feb 18, 2013 at 05:27:26PM +0100, Vincent Lejeune wrote: > mayLoad complexify scheduling and does not bring any usefull info > as the location is not writeable at all. Reviewed-by: Tom Stellard > --- > lib/Target/R600/R600Instructions.td | 2 +- > 1 file changed, 1 insertion(+), 1 deleti

Re: [Mesa-dev] [PATCH 1/6] R600: Use MUL_IEEE for trig/fdiv intrinsic

2013-02-19 Thread Tom Stellard
On Mon, Feb 18, 2013 at 05:27:25PM +0100, Vincent Lejeune wrote: Reviewed-by: Tom Stellard > --- > lib/Target/R600/R600Instructions.td | 8 > test/CodeGen/R600/fdiv.v4f32.ll | 8 > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/lib/Target/R600/R600Instru

Re: [Mesa-dev] [PATCH 6/6] R600: initial scheduler code

2013-02-19 Thread Tom Stellard
Hi Vincent, >From now on, please cc llvm-comm...@cs.uiuc.edu when you submit a patch. I'm cc'ing that list now. This looks OK to me at first glance, but I would like to test it with compute shaders before you merge it. On Mon, Feb 18, 2013 at 05:27:30PM +0100, Vincent Lejeune wrote: > From: Vad

Re: [Mesa-dev] [PATCH 5/6] R600: Remove LowerConstCopyPass and lower CONST_COPY right after ISel.

2013-02-19 Thread Tom Stellard
On Mon, Feb 18, 2013 at 05:27:29PM +0100, Vincent Lejeune wrote: > Maintaining CONST_COPY Instructions until Pre Emit may prevent some ifcvt case > and taking them in account for scheduling is difficult for no real benefit. > --- > lib/Target/R600/AMDGPU.h| 1 - > lib/Target/R600

Re: [Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread Jose Fonseca
There may be more vertex elements that used in the shader. But why should the key contain those elements? Won't this cause needless recompilations (e.g., in situations where the state tracker leaves unneeded elements from previous draw?)? That is, it seems to be that the key should have the num

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Michel Dänzer
On Die, 2013-02-19 at 15:48 +0100, Marek Olšák wrote: > On Tue, Feb 19, 2013 at 3:28 PM, Michel Dänzer wrote: > > On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote: > >> On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer wrote: > >> > > >> > Really, what I don't understand is why r600g doesn't s

Re: [Mesa-dev] VTK Tests fails with Mesa Swrast passes with OSMesa

2013-02-19 Thread Brian Paul
On 02/15/2013 09:00 AM, Kevin H. Hobbs wrote: I have two machines {bubbles, murron} doing nightly dashboard builds of VTK using nightly Mesa. Each machine does a build of VTK using swrast and one with OSMesa. Many tests pass on both machines when using OSMesa and fail on both machines using swr

[Mesa-dev] [Bug 61091] piglit glsl-fs-texture2drect regression

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61091 --- Comment #1 from Marek Olšák --- glBlitFramebuffer with rectangle textures is also broken with both softpipe and llvmpipe and it has been so for quite a while. I have a piglit test for that. -- You are receiving this mail because: You are th

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Marek Olšák
On Tue, Feb 19, 2013 at 3:28 PM, Michel Dänzer wrote: > On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote: >> On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer wrote: >> > >> > Really, what I don't understand is why r600g doesn't seem affected by >> > this... at least on my RS880 it's passing th

Re: [Mesa-dev] [PATCH] draw: make sure key size is calculated consistently.

2013-02-19 Thread Brian Paul
On 02/18/2013 05:27 PM, srol...@vmware.com wrote: From: Roland Scheidegger Some parts calculated key size by using shader information, others by using the pipe_vertex_element information. Since it is perfectly valid to have more vertex_elements set than the vertex shader is using those may not b

Re: [Mesa-dev] [PATCH 4/8] R600/SI: simplify VOPC_* patterns

2013-02-19 Thread Michel Dänzer
On Die, 2013-02-19 at 14:54 +0100, Christian König wrote: > From: Christian König > > Fixing asm operation names. [...] > diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R600/SIInstrInfo.td > index be791e2..69357ce 100644 > --- a/lib/Target/R600/SIInstrInfo.td > +++ b/lib/Target/R600/

[Mesa-dev] [Bug 61012] alloc_layout_array tx * ty assertion failure when making pbuffer current

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61012 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 59876] glGetTexLevelParameteriv broken for indirect rendering

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59876 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 61026] Segfault in glBitmap when called with PBO source

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61026 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 60938] [softpipe] piglit interpolation-noperspective-gl_BackColor-flat-fixed regression

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60938 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #6 from Brian Paul --- Can you make a trace of this issue with apitrace? https://github.com/apitrace/apitrace -- You are receiving this mail because: You are the assignee for the bug. ___

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Michel Dänzer
On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote: > On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer wrote: > > > > Really, what I don't understand is why r600g doesn't seem affected by > > this... at least on my RS880 it's passing the piglit tests this change > > fixes with radeonsi. So maybe

[Mesa-dev] [Bug 61093] [llvmpipe] lp_surface.c:68:lp_resource_copy: Assertion `src_box->depth == 1' failed.

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61093 José Fonseca changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |srol...@vmware.com |org

[Mesa-dev] [PATCH 8/8] R600/SI: replace SI_V_CNDLT with a pattern

2013-02-19 Thread Christian König
From: Christian König It actually fixes quite a bunch of piglit tests. Signed-off-by: Christian König --- lib/Target/R600/SIISelLowering.cpp | 22 -- lib/Target/R600/SIISelLowering.h |2 -- lib/Target/R600/SIInstructions.td | 12 +--- 3 files changed, 5 i

[Mesa-dev] [PATCH 7/8] R600/SI: add OMOD patterns

2013-02-19 Thread Christian König
From: Christian König Signed-off-by: Christian König --- lib/Target/R600/AMDGPUInstructions.td | 15 +++ lib/Target/R600/SIInstructions.td | 18 ++ 2 files changed, 33 insertions(+) diff --git a/lib/Target/R600/AMDGPUInstructions.td b/lib/Target/R600/AMDGPU

[Mesa-dev] [PATCH 6/8] R600/SI: use patterns for clamp, fabs, fneg

2013-02-19 Thread Christian König
From: Christian König Instead of using custom inserters, it's simpler and should make DAG folding easier. Signed-off-by: Christian König --- lib/Target/R600/SIISelLowering.cpp | 36 lib/Target/R600/SIInstructions.td | 26 ++ 2 f

[Mesa-dev] [PATCH 5/8] R600/SI: sort and cleanup SIInstrInfo.td

2013-02-19 Thread Christian König
From: Christian König Fix code formating and sort/group the classes. Signed-off-by: Christian König --- lib/Target/R600/SIInstrInfo.td | 100 +++- 1 file changed, 58 insertions(+), 42 deletions(-) diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R6

[Mesa-dev] [PATCH 1/8] R600/SI: cleanup SIInstrInfo.td and SIInstrFormat.td

2013-02-19 Thread Christian König
From: Christian König Those two files got mixed up. Signed-off-by: Christian König --- lib/Target/R600/SIInstrFormats.td | 500 +++-- lib/Target/R600/SIInstrInfo.td| 495 +++- 2 files changed, 509 insertions(+), 486 deletion

[Mesa-dev] [PATCH 4/8] R600/SI: simplify VOPC_* patterns

2013-02-19 Thread Christian König
From: Christian König Fixing asm operation names. Signed-off-by: Christian König --- lib/Target/R600/AMDGPUInstructions.td |5 + lib/Target/R600/SIInstrInfo.td| 19 +- lib/Target/R600/SIInstructions.td | 444 +++-- 3 files changed, 213 insertions(

[Mesa-dev] [PATCH 2/8] R600/SI: rework VOP1_* patterns

2013-02-19 Thread Christian König
From: Christian König Fixing asm operation names. Signed-off-by: Christian König --- lib/Target/R600/SIInstrInfo.td | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R600/SIInstrInfo.td index

[Mesa-dev] [PATCH 3/8] R600/SI: rework VOP2_* pattern

2013-02-19 Thread Christian König
From: Christian König Fixing asm operation names. Signed-off-by: Christian König --- lib/Target/R600/SIISelLowering.cpp |3 --- lib/Target/R600/SIInstrInfo.td | 37 ++-- 2 files changed, 18 insertions(+), 22 deletions(-) diff --git a/lib/Target/R600/S

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-19 Thread Vadim Girlin
On 02/19/2013 04:54 PM, Andy Furniss wrote: Vadim Girlin wrote: Testing with rv790 with drm-fixes kernel not much works - etqw runs but in a level 50% of screen is junk. nexuiz menus total junk, didn't test further. xonotic menus OK but gpu lock on starting timedemo. vdpau mpeg2 decode - re

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Marek Olšák
On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer wrote: > On Die, 2013-02-19 at 10:33 +0100, Christian König wrote: >> Am 18.02.2013 20:11, schrieb Roland Scheidegger: >> > Am 18.02.2013 19:14, schrieb Michel Dänzer: >> >> From: Michel Dänzer >> >> >> >> 11 more little piglits. >> >> >> >> NOTE: T

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-19 Thread Andy Furniss
Vadim Girlin wrote: Testing with rv790 with drm-fixes kernel not much works - etqw runs but in a level 50% of screen is junk. nexuiz menus total junk, didn't test further. xonotic menus OK but gpu lock on starting timedemo. vdpau mpeg2 decode - renders 90% junk. heaven 3.0 (on a different p

Re: [Mesa-dev] [PATCH] r600g: don't reserve more stack space than required v3

2013-02-19 Thread Andy Furniss
Vadim Girlin wrote: v3: handle hw-specific cases Signed-off-by: Vadim Girlin --- cc: Andy Furniss Hopefully this should work better on the non-evergreen chips This one seems to work OK for me. ___ mesa-dev mailing list mesa-dev@lists.freedesktop

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Marek Olšák
On Tue, Feb 19, 2013 at 10:33 AM, Christian König wrote: > Am 18.02.2013 20:11, schrieb Roland Scheidegger: > >> Am 18.02.2013 19:14, schrieb Michel Dänzer: >>> >>> From: Michel Dänzer >>> >>> 11 more little piglits. >>> >>> NOTE: This is a candidate for the 9.1 branch. >>> >>> Signed-off-by: Mic

Re: [Mesa-dev] [PATCH mesa] freedreno: gallium driver for adreno

2013-02-19 Thread Marek Olšák
On Mon, Feb 18, 2013 at 7:58 PM, Rob Clark wrote: > On Mon, Feb 18, 2013 at 12:47 PM, Matt Turner wrote: >> On Sun, Feb 17, 2013 at 11:33 AM, Rob Clark wrote: >>> >>> diff --git a/src/gallium/drivers/freedreno/Makefile.am >>> b/src/gallium/drivers/freedreno/Makefile.am >>> new file mode 100644

[Mesa-dev] [Bug 61093] [llvmpipe] lp_surface.c:68:lp_resource_copy: Assertion `src_box->depth == 1' failed.

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61093 --- Comment #1 from Marek Olšák --- The assertion in lp_resource_copy can be fixed easily, but I can't reproduce it. llvmpipe is failing a different assertion here: texsubimage: /home/marek/dev/llvm/include/llvm/CodeGen/SelectionDAGNodes.h:539:

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2013-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #5 from Laurent carlier --- Can reproduce this bug also with mesa-9.2-devel (git) but also 9.0.x with counter strike: Source (steam-linux) gdb backtrace: Breakpoint 2, destroy_program_variants (st=0x8171910, program=0xf52863c0 <_mesa

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Michel Dänzer
On Mon, 2013-02-18 at 20:11 +0100, Roland Scheidegger wrote: > Am 18.02.2013 19:14, schrieb Michel Dänzer: > > From: Michel Dänzer > > > > 11 more little piglits. > > > > NOTE: This is a candidate for the 9.1 branch. > > > > Signed-off-by: Michel Dänzer > > --- > > > > Any ideas why this see

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Michel Dänzer
On Die, 2013-02-19 at 10:33 +0100, Christian König wrote: > Am 18.02.2013 20:11, schrieb Roland Scheidegger: > > Am 18.02.2013 19:14, schrieb Michel Dänzer: > >> From: Michel Dänzer > >> > >> 11 more little piglits. > >> > >> NOTE: This is a candidate for the 9.1 branch. > >> > >> Signed-off-by:

Re: [Mesa-dev] [PATCH] radeonsi: Fix blending using destination alpha factor but non-alpha destination

2013-02-19 Thread Christian König
Am 18.02.2013 20:11, schrieb Roland Scheidegger: Am 18.02.2013 19:14, schrieb Michel Dänzer: From: Michel Dänzer 11 more little piglits. NOTE: This is a candidate for the 9.1 branch. Signed-off-by: Michel Dänzer --- Any ideas why this seems necessary with radeonsi but not with r600g? Mayb

Re: [Mesa-dev] [PATCH] llvmpipe: fix lp_resource_copy using more than one 3d slice

2013-02-19 Thread Jose Fonseca
Thanks for fixing this Roland. This is definitely an improvement. I'd recommend a few tweaks (it could even be as a follow on change): - Calling llvmpipe_flush_resource() in a loop is overkill (it will call llvmpipe_flush() to be called many times needlessly). Please refactor llvmpipe_flush_re

[Mesa-dev] [PATCH v2] configure.ac: Do not check for clock_gettime on MinGW.

2013-02-19 Thread Vinson Lee
MinGW does not have clock_gettime. Signed-off-by: Vinson Lee --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 16c2f8c..1e11b4e 100644 --- a/configure.ac +++ b/configure.ac @@ -500,7 +500,7 @@ AC_CHECK_FUNC([dlopen], [DEFINES="

[Mesa-dev] [PATCH v2] glsl: Initialize parcel_out_uniform_storage member variables.

2013-02-19 Thread Vinson Lee
Fixes uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee --- src/glsl/link_uniforms.cpp | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp index d457e4d..b5bfe13 100644 --- a/src/glsl/li

Re: [Mesa-dev] [PATCH] glsl: Initialize parcel_out_uniform_storage member variables.

2013-02-19 Thread Adrian M Negreanu
Hi, I have tested your changes: [Mesa-dev] [PATCH] glsl: Initialize parcel_out_uniform_storage member variables. Project: mesa (Mesa build tests) Configurations: android linux Tested the patch(es) on top of the

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-19 Thread Vadim Girlin
On 02/18/2013 02:20 PM, Andy Furniss wrote: Stefan Seifert wrote: Hi! Amazing work! I see some 50 % speed ups in FlightGear and even more. While normally 3D clouds tear performance down to an unflyable stutter, with your branch I can fly in densly clouded conditions at usable framerates. I can