Re: [Mesa-dev] [PATCH] Finish _HAVE_FULL_GL removal

2012-10-16 Thread Kenneth Graunke
On 10/16/2012 07:39 PM, Matt Turner wrote: --- doxygen/main.doxy |2 +- src/mesa/main/api_exec.c | 34 -- src/mesa/main/enums.h | 12 src/mesa/main/mfeatures.h |4 4 files changed, 1 insertions(+), 51 deletions(-)

[Mesa-dev] [PATCH] Finish _HAVE_FULL_GL removal

2012-10-16 Thread Matt Turner
--- doxygen/main.doxy |2 +- src/mesa/main/api_exec.c | 34 -- src/mesa/main/enums.h | 12 src/mesa/main/mfeatures.h |4 4 files changed, 1 insertions(+), 51 deletions(-) diff --git a/doxygen/main.doxy b/doxygen/main.doxy

Re: [Mesa-dev] [PATCH] i965/fs: Reduce the interference between payload regs and virtual GRFs.

2012-10-16 Thread Kenneth Graunke
On 10/16/2012 06:05 PM, Eric Anholt wrote: Kenneth Graunke writes: On 10/15/2012 04:06 PM, Eric Anholt wrote: Improves performance of the Lightsmark penumbra shadows scene by 15.7% +/- 1.0% (n=15), by eliminating register spilling. (tested by smashing the list of scenes to have all other scen

Re: [Mesa-dev] [PATCH] i965/fs: Reduce the interference between payload regs and virtual GRFs.

2012-10-16 Thread Eric Anholt
Kenneth Graunke writes: > On 10/15/2012 04:06 PM, Eric Anholt wrote: >> Improves performance of the Lightsmark penumbra shadows scene by 15.7% +/- >> 1.0% (n=15), by eliminating register spilling. (tested by smashing the list >> of >> scenes to have all other scenes have 0 duration -- includes a

Re: [Mesa-dev] [PATCH 1/8] util/blitter: silence assorted MSVC warnings

2012-10-16 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Wed, Oct 17, 2012 at 2:35 AM, Brian Paul wrote: > From: Brian Paul > > Fix signed/unsigned comparison warnings and float/int assignment warnings. > --- > src/gallium/auxiliary/util/u_blitter.c | 27 ++- > src/gallium/auxiliary/util/u

[Mesa-dev] [PATCH 8/8] st/mesa: silence MSVC signed/unsigned comparison warning

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/mesa/state_tracker/st_extensions.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/state_tracker/st_extensions.c b/src/mesa/state_tracker/st_extensions.c index 08a41c3..0560f3a 100644 --- a/src/mesa/state_tracker/st_extensions.c +++ b

[Mesa-dev] [PATCH 7/8] st/mesa: silence MSVC double/unsigned assignment warning

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/mesa/state_tracker/st_atom_msaa.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_msaa.c b/src/mesa/state_tracker/st_atom_msaa.c index ea9eb9a..9baa4fc 100644 --- a/src/mesa/state_tracker/st_atom_msaa.c +++ b/src

[Mesa-dev] [PATCH 6/8] tgsi: silence MSVC signed/unsigned comparison warnings

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/gallium/auxiliary/tgsi/tgsi_dump.c |2 +- src/gallium/auxiliary/tgsi/tgsi_exec.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_dump.c b/src/gallium/auxiliary/tgsi/tgsi_dump.c index c202033..3e6f76a 100644 -

[Mesa-dev] [PATCH 5/8] util: fix MSVC signed/unsigned comparison warning in u_upload_mgr.c code

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/gallium/auxiliary/util/u_upload_mgr.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/util/u_upload_mgr.c b/src/gallium/auxiliary/util/u_upload_mgr.c index b62973d..ee1c688 100644 --- a/src/gallium/auxiliary/util/u_upload_

[Mesa-dev] [PATCH 4/8] util: fix MSVC signed/unsigned comparison warning in u_vbuf.c code

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/gallium/auxiliary/util/u_vbuf.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/util/u_vbuf.c b/src/gallium/auxiliary/util/u_vbuf.c index 52db294..1cc83c3 100644 --- a/src/gallium/auxiliary/util/u_vbuf.c +++ b/src/gallium/

[Mesa-dev] [PATCH 3/8] util: fix MSVC double/float conversion warning in u_format_r11g11b10f.h

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/gallium/auxiliary/util/u_format_r11g11b10f.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/util/u_format_r11g11b10f.h b/src/gallium/auxiliary/util/u_format_r11g11b10f.h index bd64b28..b883b31 100644 --- a/src/gallium/aux

[Mesa-dev] [PATCH 2/8] draw: silence MSVC signed/unsigned comparison warnings

2012-10-16 Thread Brian Paul
From: Brian Paul --- src/gallium/auxiliary/draw/draw_context.c |2 +- src/gallium/auxiliary/draw/draw_gs.c|4 ++-- src/gallium/auxiliary/draw/draw_pipe_stipple.c |2 +- src/gallium/auxiliary/draw/draw_pt_vsplit_tmp.h |8 4 files changed, 8 insertions(+

[Mesa-dev] [PATCH 1/8] util/blitter: silence assorted MSVC warnings

2012-10-16 Thread Brian Paul
From: Brian Paul Fix signed/unsigned comparison warnings and float/int assignment warnings. --- src/gallium/auxiliary/util/u_blitter.c | 27 ++- src/gallium/auxiliary/util/u_blitter.h | 16 2 files changed, 22 insertions(+), 21 deletions(-) diff --gi

[Mesa-dev] [PATCH 1/2] i965: Don't flush the batch immediately on EndQuery.

2012-10-16 Thread Eric Anholt
The theory I had when I wrote the code was that you wanted to minimize latency on your queries because the app was going to ask soon. Only, it turns out that everybody batches up their queries and asks for the results later (often after the next SwapBuffers!), so this was a pessimization. Until n

[Mesa-dev] [PATCH 2/2] i965: Stop flushing the batch on timestamp queries, too.

2012-10-16 Thread Eric Anholt
Given the usecase we have of trying to measure timestamps across individual draw calls, flushing will totally mess up what people are trying to measure. --- src/mesa/drivers/dri/i965/brw_queryobj.c |1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c b/sr

[Mesa-dev] [Bug 55817] Torchlight: Texture renders as garbage

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55817 --- Comment #9 from Tim Allen --- (In reply to comment #6) > - resave the original texture without compression. I was actually thinking of bug 55445 here; it turns out that decompressing the texture does *not* solve the issue. I extracted the p

[Mesa-dev] [Bug 55817] Torchlight: Texture renders as garbage

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55817 Alex Deucher changed: What|Removed |Added Attachment #68654|text/plain |image/png mime type|

[Mesa-dev] [Bug 55817] Torchlight: Texture renders as garbage

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55817 Alex Deucher changed: What|Removed |Added Attachment #68653|text/plain |image/png mime type|

[Mesa-dev] [Bug 55817] Torchlight: Texture renders as garbage

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55817 --- Comment #8 from Tim Allen --- Created attachment 68654 --> https://bugs.freedesktop.org/attachment.cgi?id=68654&action=edit The original scene with the problem texture decompressed (closeup). -- You are receiving this mail because: You ar

[Mesa-dev] [Bug 55817] Torchlight: Texture renders as garbage

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55817 --- Comment #7 from Tim Allen --- Created attachment 68653 --> https://bugs.freedesktop.org/attachment.cgi?id=68653&action=edit The original scene with the problem texture decompressed. -- You are receiving this mail because: You are the assi

Re: [Mesa-dev] [PATCH 2/2] mesa/tests: Add ES1.1 dispatch table sanity test

2012-10-16 Thread Paul Berry
On 16 October 2012 11:17, Ian Romanick wrote: > From: Ian Romanick > > This test actually depends on FEATURE_ES1 because > _mesa_create_exec_table_es1 doesn't exist without it. > Reviewed-by: Paul Berry > > Signed-off-by: Ian Romanick > Cc: Paul Berry > --- > src/mesa/main/tests/dispatch_

Re: [Mesa-dev] [PATCH 1/3] i965/vs: Simplify emit_scratch_write() prototype.

2012-10-16 Thread Kenneth Graunke
On 10/16/2012 01:56 PM, Eric Anholt wrote: Both callers used (effectively) inst->dst as the argument, so just reference it. --- src/mesa/drivers/dri/i965/brw_vec4.h|1 - src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp |2 +- src/mesa/drivers/dri/i965/brw_vec4_visit

Re: [Mesa-dev] [PATCH 2/6] mesa: remove #if _HAVE_FULL_GL checks

2012-10-16 Thread Brian Paul
On 10/16/2012 03:19 PM, Paul Berry wrote: On 22 September 2012 17:45, Brian Paul mailto:brian.e.p...@gmail.com>> wrote: From: Brian Paul mailto:bri...@vmware.com>> This is basically more of the "remove FEATURE_x" clean-up. Did this patch ever land? Apparently not. Feel free to push

Re: [Mesa-dev] [PATCH 2/6] mesa: remove #if _HAVE_FULL_GL checks

2012-10-16 Thread Paul Berry
On 22 September 2012 17:45, Brian Paul wrote: > From: Brian Paul > > This is basically more of the "remove FEATURE_x" clean-up. > Did this patch ever land? I would love to see the _HAVE_FULL_GL macro go away. (Also, _HAVE_FULL_GL is used in src/mesa/main/api_exec.c. Any particular reason why

[Mesa-dev] [PATCH 3/3] i965/vs: Trim the swizzle of the scratch write temporary.

2012-10-16 Thread Eric Anholt
This fixes confusion by the upcoming live variable analysis which saw e.g. use of temp.w when only temp.xyz were initialized in the basic block, and concluded that temp.w must have come from outside of the block (even though it was never initialized anywhere). --- src/mesa/drivers/dri/i965/brw_vec

[Mesa-dev] [PATCH 1/3] i965/vs: Simplify emit_scratch_write() prototype.

2012-10-16 Thread Eric Anholt
Both callers used (effectively) inst->dst as the argument, so just reference it. --- src/mesa/drivers/dri/i965/brw_vec4.h|1 - src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp |2 +- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 11 +-- 3 files changed,

[Mesa-dev] [PATCH 2/3] i965/vs: Do the temporary allocation in emit_scratch_write().

2012-10-16 Thread Eric Anholt
Both callers were doing basically the same thing, just written differently. --- src/mesa/drivers/dri/i965/brw_vec4.h |1 - .../drivers/dri/i965/brw_vec4_reg_allocate.cpp | 11 +- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 21 ++-- 3 files

Re: [Mesa-dev] [PATCH 1/2] glapi: Add es2="3.0" attributes to XML.

2012-10-16 Thread Paul Berry
On 16 October 2012 13:53, Matt Turner wrote: > On Mon, Oct 15, 2012 at 3:44 PM, Matt Turner wrote: > > - > > + > > > > > > > > -- > > This should be added to . I'll fix > that before pushing. (Thanks Paul!) > With that fixed, this patch is Reviewed-by: Pau

Re: [Mesa-dev] [PATCH 1/2] glapi: Add es2="3.0" attributes to XML.

2012-10-16 Thread Matt Turner
On Mon, Oct 15, 2012 at 3:44 PM, Matt Turner wrote: > - > + > > > > -- This should be added to . I'll fix that before pushing. (Thanks Paul!) ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.fre

Re: [Mesa-dev] [PATCH 2/2] mesa/tests: Add ES1.1 dispatch table sanity test

2012-10-16 Thread Ian Romanick
On 10/16/2012 12:04 PM, Matt Turner wrote: On Tue, Oct 16, 2012 at 11:17 AM, Ian Romanick wrote: From: Ian Romanick This test actually depends on FEATURE_ES1 because _mesa_create_exec_table_es1 doesn't exist without it. Signed-off-by: Ian Romanick Cc: Paul Berry --- src/mesa/main/tests/d

Re: [Mesa-dev] [PATCH] i965/fs: Reduce the interference between payload regs and virtual GRFs.

2012-10-16 Thread Kenneth Graunke
On 10/15/2012 04:06 PM, Eric Anholt wrote: Improves performance of the Lightsmark penumbra shadows scene by 15.7% +/- 1.0% (n=15), by eliminating register spilling. (tested by smashing the list of scenes to have all other scenes have 0 duration -- includes additional rendering of scene descriptio

Re: [Mesa-dev] [PATCH 1/5] intel: Add a macro for printing a debug warning once.

2012-10-16 Thread Eric Anholt
Jason Wood writes: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 10/15/2012 10:41 AM, Eric Anholt wrote: >> Jason Wood writes: >> >>> -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 >>> >>> On 10/12/2012 04:30 PM, Eric Anholt wrote: There are a number of places where some obscure

Re: [Mesa-dev] [PATCH 2/3] mesa: Remove PROG_EMIT_VERTEX and PROG_END_PRIMITIVE opcodes.

2012-10-16 Thread Brian Paul
On 10/16/2012 12:57 PM, Kenneth Graunke wrote: These were only used for geometry shader support back in the days before the new GLSL compiler. Future geometry shader support will not use these. --- src/mesa/program/prog_instruction.c | 2 -- src/mesa/program/prog_instruction.h | 2 -

Re: [Mesa-dev] [PATCH 2/2] mesa/tests: Add ES1.1 dispatch table sanity test

2012-10-16 Thread Matt Turner
On Tue, Oct 16, 2012 at 11:17 AM, Ian Romanick wrote: > From: Ian Romanick > > This test actually depends on FEATURE_ES1 because > _mesa_create_exec_table_es1 doesn't exist without it. > > Signed-off-by: Ian Romanick > Cc: Paul Berry > --- > src/mesa/main/tests/dispatch_sanity.cpp | 207 > +++

Re: [Mesa-dev] [PATCH] i965/fs: Fix segfault when using INTEL_DEBUG=perf with non-GLSL.

2012-10-16 Thread Eric Anholt
Kenneth Graunke writes: > Now that ARB programs and fixed function are routed through the new > backend, shader might be NULL. Don't do INTEL_DEBUG=perf support in > that case, since it relies on shader->compiled_once. > > Since INTEL_DEBUG=perf wasn't previously supported, this maintains the >

[Mesa-dev] [PATCH 3/3] mesa/vbo: Remove vp_mode enum in favor of a boolean.

2012-10-16 Thread Kenneth Graunke
Now that there are only two options, an enum seems unnecessary. --- src/mesa/vbo/vbo_context.h| 17 +++-- src/mesa/vbo/vbo_exec.h | 7 --- src/mesa/vbo/vbo_exec_array.c | 8 ++-- src/mesa/vbo/vbo_exec_draw.c | 10 +++--- src/mesa/vbo/vbo_save_draw.c | 10 +++--

[Mesa-dev] [PATCH 2/3] mesa: Remove PROG_EMIT_VERTEX and PROG_END_PRIMITIVE opcodes.

2012-10-16 Thread Kenneth Graunke
These were only used for geometry shader support back in the days before the new GLSL compiler. Future geometry shader support will not use these. --- src/mesa/program/prog_instruction.c | 2 -- src/mesa/program/prog_instruction.h | 2 -- src/mesa/program/prog_print.c| 6 ---

[Mesa-dev] [PATCH 1/3] mesa: Kill unused VERT_ATTRIB_GENERIC_NV and VERT_BIT_GENERIC_NV macros.

2012-10-16 Thread Kenneth Graunke
--- src/mesa/main/mtypes.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index c3378cd..60d5048 100644 --- a/src/mesa/main/mtypes.h +++ b/src/mesa/main/mtypes.h @@ -144,9 +144,6 @@ typedef enum * VERT_ATTRIB_TEX * include the c

[Mesa-dev] [PATCH 2/2] mesa/tests: Add ES1.1 dispatch table sanity test

2012-10-16 Thread Ian Romanick
From: Ian Romanick This test actually depends on FEATURE_ES1 because _mesa_create_exec_table_es1 doesn't exist without it. Signed-off-by: Ian Romanick Cc: Paul Berry --- src/mesa/main/tests/dispatch_sanity.cpp | 207 1 file changed, 207 insertions(+) diff --g

[Mesa-dev] [PATCH 1/2] mesa/tests: Compile ES2 test regardless of FEATURE_ES2 setting

2012-10-16 Thread Ian Romanick
From: Ian Romanick The relevant ES2 code is always in Mesa. Always building the tests ensures that things aren't accidentally broken when people don't build with --enable-es2. Signed-off-by: Ian Romanick --- src/mesa/main/tests/dispatch_sanity.cpp | 4 1 file changed, 4 deletions(-) dif

Re: [Mesa-dev] [PATCH 17/23 v2] st/mesa: Remove the PROG_PARAM_BIT_CYL_WRAP flag.

2012-10-16 Thread Brian Paul
On 10/15/2012 01:05 PM, Kenneth Graunke wrote: Nobody ever set the flag, which makes this dead code. v2: Leave the ureg_DECL_fs_input_cyl function in place, even though it's unused, since VMWare uses it for their internal projects. --- src/mesa/program/prog_parameter.h| 8 ---

[Mesa-dev] [Bug 55998] Pretty huge slowdown in mesa 9.0

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55998 --- Comment #9 from Arthur Nascimento --- Perhaps, but SNA was mentioned and I only found it on xf86-video-intel. I must say I do experience problems with this setup - just not the 1/5th screen tearing or the slowdown. One of the problems - rand

[Mesa-dev] [Bug 55998] Pretty huge slowdown in mesa 9.0

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55998 --- Comment #8 from ValdikSS --- (In reply to comment #7) COMMONOPTS="--prefix=/usr \ --sysconfdir=/etc \ --with-dri-driverdir=/usr/lib/xorg/modules/dri \ --with-gallium-drivers=r300,r600,radeonsi,nouveau,svga,swrast \ --with-

[Mesa-dev] [Bug 55998] Pretty huge slowdown in mesa 9.0

2012-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55998 --- Comment #7 from Arthur Nascimento --- I'm on SNA as well. How about comparing the ./configure options? For Mesa 9.0: ./configure --prefix=/usr --host=x86_64-pc-linux-gnu --build=x86_64-pc-linux-gnu --mandir=/usr/share/man --infodir=/usr/shar

Re: [Mesa-dev] [PATCH] mesa/es: Enable GL_EXT_map_buffer_range

2012-10-16 Thread Fredrik Höglund
On Tuesday 16 October 2012, Paul Berry wrote: > On 15 October 2012 09:50, Ian Romanick wrote: > > > On 10/10/2012 12:31 PM, Fredrik Höglund wrote: > > > >> This extension is functionally the same as GL_ARB_map_buffer_range. > >> > > > > Reviewed-by: Ian Romanick > > > FYI, this patch conflicts