Re: [Mesa-dev] [PATCH] wayland-drm: Implement wl_buffer.damage in old versions of Wayland

2012-04-12 Thread Kristian Hoegsberg
On Wed, Apr 11, 2012 at 05:07:56PM +0100, Neil Roberts wrote: > Commit 272bc48976 removed the damage implementation for the > wl_buffer_interface because that has been removed from git master of > Wayland. However this breaks building with the 0.85 branch of Wayland > because it would end up initia

[Mesa-dev] [Bug 48545] LLVMpipe glReadPixels Firefox hits the slow path (WebGL rendering)

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48545 Brian Paul changed: What|Removed |Added AssignedTo|mesa-dev@lists.freedesktop. |brian.e.p...@gmail.com |o

Re: [Mesa-dev] [PATCH 00/21] Gallium: making user vertex buffers optional and other goodness

2012-04-12 Thread Brian Paul
On Wed, Apr 11, 2012 at 9:38 AM, Marek Olšák wrote: > Hi everyone, > > This series adds these optional features to st/mesa: > - uploading user vertex buffers > - translating unsupported vertex formats into floats > - vertex data with unaligned buffer_offset, src_offset, or stride is > transformed

[Mesa-dev] [PATCH 7/7] tests/glsl: Add test for uniform initialization by the linker

2012-04-12 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- Makefile |1 + configure.ac |1 + tests/Makefile.am |2 +- tests/glsl/Makefile.am| 22 + tests/glsl/copy_con

[Mesa-dev] [PATCH 6/7] mesa: Use initializers to configure samplers

2012-04-12 Thread Ian Romanick
From: Ian Romanick Now that the linker handles initializers of samplers just like any other uniform, a bunch of this annoying code is unnecessary. Signed-off-by: Ian Romanick --- src/mesa/main/ff_fragment_shader.cpp | 41 +++-- 1 files changed, 9 insertions(+), 32

[Mesa-dev] [PATCH 5/7] ir_to_mesa: Don't set initial uniform values again

2012-04-12 Thread Ian Romanick
From: Ian Romanick This work is now done by the linker, so we don't need to keep doing it here. Signed-off-by: Ian Romanick --- src/mesa/program/ir_to_mesa.cpp | 104 --- 1 files changed, 0 insertions(+), 104 deletions(-) diff --git a/src/mesa/program/ir_t

[Mesa-dev] [PATCH 4/7] ir_to_mesa: Propagate initial values in _mesa_associate_uniform_storage

2012-04-12 Thread Ian Romanick
From: Ian Romanick The linker may have set initial values for uniforms. Propagate these values to the driver's backing storage when it is first associated. Signed-off-by: Ian Romanick --- src/mesa/program/ir_to_mesa.cpp |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff

[Mesa-dev] [PATCH 2/7] glsl: Initialize samplers to 0, propagate sampler values to the gl_program

2012-04-12 Thread Ian Romanick
From: Ian Romanick The spec requires that samplers be initialized to 0. Since this differs from the 1-to-1 mapping of samplers to texture units assumed by ARB assembly shaders (and the gl_program structure), be sure to propagate this date from the gl_shader_program to the gl_program. Signed-off

[Mesa-dev] [PATCH 3/7] glsl: Propagate sampler uniform initializers to gl_shader_program::SamplerUnits

2012-04-12 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glsl/link_uniform_initializers.cpp |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/src/glsl/link_uniform_initializers.cpp b/src/glsl/link_uniform_initializers.cpp index bbd26df..2c12cf8 100644 --- a/src/gls

[Mesa-dev] [PATCH 1/7] glsl: Set initial values for uniforms in the linker

2012-04-12 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glsl/Makefile.sources |1 + src/glsl/link_uniform_initializers.cpp | 162 src/glsl/link_uniforms.cpp |2 + src/glsl/linker.h |3 + 4 files changed,

[Mesa-dev] [PATCH 0/7] Refactor uniform initializer handling

2012-04-12 Thread Ian Romanick
This was a bit of follow-on work that I had wanted to do back when I did all the other uniform rework. Setting the initial values for uniforms (that have initializers in the GLSL code) is moved from ir_to_mesa to the linker. In addition, support for initializers on samplers is added. This gives

Re: [Mesa-dev] [PATCH 00/21] Gallium: making user vertex buffers optional and other goodness

2012-04-12 Thread Marek Olšák
On Thu, Apr 12, 2012 at 8:01 PM, Jose Fonseca wrote: > > > - Original Message - >> On Thu, Apr 12, 2012 at 10:26 AM, Christian König >> wrote: >> > >> > It looks like a step into the right direction to me, cause AFAIK we >> > need >> >> I thought so too, but if radeon drivers are the only

[Mesa-dev] Gallium optimization: add the user pointer into pipe_vertex_buffer and pipe_index_buffer

2012-04-12 Thread Marek Olšák
Hi, In parallel with my other work, I have been looking for ways to decrease our draw-call overhead and tried adding the user buffer pointer into pipe_vertex_buffer and pipe_index_buffer, which led to a removal of a lot of code from st_draw.c. With that change, I have tested r300g with torcs and i

Re: [Mesa-dev] [PATCH 00/21] Gallium: making user vertex buffers optional and other goodness

2012-04-12 Thread Jose Fonseca
- Original Message - > On Thu, Apr 12, 2012 at 10:26 AM, Christian König > wrote: > > > > It looks like a step into the right direction to me, cause AFAIK we > > need > > I thought so too, but if radeon drivers are the only users of this, > there's not much point in doing it in the stat

Re: [Mesa-dev] [PATCH] glsl: If an "if" has no "then" or "else" code left, remove it.

2012-04-12 Thread Eric Anholt
On Wed, 11 Apr 2012 23:46:03 -0700, Kenneth Graunke wrote: > On 04/11/2012 08:49 PM, Matt Turner wrote: > > On Wed, Apr 11, 2012 at 9:33 PM, Eric Anholt wrote: > >> Cuts 8/1068 instructions from glyphy's fragment shaders on i965. > >> --- > >> src/glsl/opt_if_simplification.cpp |8

Re: [Mesa-dev] [PATCH] glsl: If an "if" has no "then" or "else" code left, remove it.

2012-04-12 Thread Ian Romanick
On 04/11/2012 06:33 PM, Eric Anholt wrote: Cuts 8/1068 instructions from glyphy's fragment shaders on i965. Reviewed-by: Ian Romanick --- src/glsl/opt_if_simplification.cpp |8 1 file changed, 8 insertions(+) diff --git a/src/glsl/opt_if_simplification.cpp b/src/glsl/opt_if

Re: [Mesa-dev] [PATCH] ir_to_mesa: Fix uninitialized member in add_uniform_to_shader.

2012-04-12 Thread Brian Paul
On 04/12/2012 12:11 AM, Vinson Lee wrote: Fix uninitialized scalar field defect reported by Coverity. NOTE: This is a candidate for the 8.0 branch. Signed-off-by: Vinson Lee --- src/mesa/program/ir_to_mesa.cpp |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/pr

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 Alex Deucher changed: What|Removed |Added Attachment #59846|text/x-log |text/plain mime type|

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 Alex Deucher changed: What|Removed |Added Attachment #59845|text/x-log |text/plain mime type|

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 --- Comment #6 from Alex Deucher 2012-04-12 06:11:02 PDT --- (In reply to comment #3) > Er, that's not a patch... Yes, but it's easier to view inline in the browser if you set the mimetype to patch. -- Configure bugmail: https://bugs.freedesk

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 --- Comment #5 from Rena Kunisaki 2012-04-12 03:33:32 PDT --- Created attachment 59846 --> https://bugs.freedesktop.org/attachment.cgi?id=59846 dmesg -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 --- Comment #4 from Rena Kunisaki 2012-04-12 03:32:02 PDT --- Created attachment 59845 --> https://bugs.freedesktop.org/attachment.cgi?id=59845 Xorg log -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You a

[Mesa-dev] [Bug 48535] Spurious GL_INVALID_OPERATION error generated by glColor()

2012-04-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=48535 Rena Kunisaki changed: What|Removed |Added Component|Mesa core |Drivers/DRI/Radeon --- Comment #3 from R

Re: [Mesa-dev] [PATCH 00/21] Gallium: making user vertex buffers optional and other goodness

2012-04-12 Thread Marek Olšák
On Thu, Apr 12, 2012 at 10:26 AM, Christian König wrote: > > It looks like a step into the right direction to me, cause AFAIK we need I thought so too, but if radeon drivers are the only users of this, there's not much point in doing it in the state tracker. I'd like to know about at least one no

Re: [Mesa-dev] [PATCH 00/21] Gallium: making user vertex buffers optional and other goodness

2012-04-12 Thread Christian König
On 11.04.2012 17:38, Marek Olšák wrote: Hi everyone, This series adds these optional features to st/mesa: - uploading user vertex buffers - translating unsupported vertex formats into floats - vertex data with unaligned buffer_offset, src_offset, or stride is transformed such that it's aligned

Re: [Mesa-dev] [PATCH 0/4] v2: force_glsl_extensions_warn option for gallium drivers

2012-04-12 Thread Michel Dänzer
On Don, 2012-04-12 at 04:41 +0400, Vadim Girlin wrote: > v2: Updated with the proposed changes: > > - using the driver name from the driver_descriptor > - using new "struct st_config_options" to pass the options Looks great, thanks! Series is Reviewed-by: Michel Dänzer -- Earthling Michel