[Mesa-dev] [Bug 47261] piglit fbo-luminance-alpha regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47261 Vinson Lee changed: What|Removed |Added CC||bri...@vmware.com, |

[Mesa-dev] [Bug 47261] New: piglit fbo-luminance-alpha regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47261 Bug #: 47261 Summary: piglit fbo-luminance-alpha regression Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: Linux (All) St

[Mesa-dev] [Bug 47259] piglit fbo-blending-formats regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47259 Vinson Lee changed: What|Removed |Added CC||bri...@vmware.com, |

[Mesa-dev] [Bug 47259] New: piglit fbo-blending-formats regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47259 Bug #: 47259 Summary: piglit fbo-blending-formats regression Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: Linux (All) S

Re: [Mesa-dev] [PATCH 2/2] i915: set SPRITE_POINT_ENABLE bit only for points

2012-03-12 Thread Yuanhan Liu
On Mon, Mar 12, 2012 at 12:30:20PM -0700, Eric Anholt wrote: > On Mon, 12 Mar 2012 16:04:01 +0800, Yuanhan Liu > wrote: > > Signed-off-by: Yuanhan Liu > > Is there a reason for this change? What test does it fix? No big reason and it doesn't fix anyting so far. I made this patch for: 1. There

[Mesa-dev] [PATCH 9/9] glsl: Add GLSL 1.40 textureSize() implementations for sampler2DRect.

2012-03-12 Thread Eric Anholt
By setting lod to 0 in the builtin function implementation, we avoid needing to update all the visitors to ignore LOD in this case, when the hardware drivers actually want to ask for LOD 0 for rectangular textures. Fixes piglit spec/GLSL-1.40/textureSize-*Rect. --- src/glsl/builtins/profiles/140.

[Mesa-dev] [PATCH 7/9] glsl: Add support for integer sampler2DRect variants in GLSL 1.40.

2012-03-12 Thread Eric Anholt
--- src/glsl/builtin_types.h | 12 src/glsl/glsl_types.cpp | 15 +-- src/glsl/glsl_types.h|2 ++ 3 files changed, 27 insertions(+), 2 deletions(-) diff --git a/src/glsl/builtin_types.h b/src/glsl/builtin_types.h index cc99b1b..890c121 100644 --- a/src/glsl/bu

[Mesa-dev] [PATCH 8/9] glsl: Set up generated builtin functions handling for GLSL 1.40.

2012-03-12 Thread Eric Anholt
Otherwise, when we go to use ir_reader on the generated code, we won't have the types present. --- src/glsl/builtins/tools/generate_builtins.py |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/glsl/builtins/tools/generate_builtins.py b/src/glsl/builtins/tools/gene

[Mesa-dev] [PATCH 6/9] glsl: Drop ftransform() from GLSL 1.40 profile.

2012-03-12 Thread Eric Anholt
This is the one builtin function claimed to be dropped due to the ARB_compatibility split. Fixes piglit spec/GLSL-1.40/compiler/ftransform.vert --- src/glsl/builtins/profiles/140.vert |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/src/glsl/builtins/profiles/140.vert b/

[Mesa-dev] [PATCH 2/9] i965/fs: Jump from discard statements to the end of the program when done.

2012-03-12 Thread Eric Anholt
>From the GLSL 1.30 spec: The discard keyword is only allowed within fragment shaders. It can be used within a fragment shader to abandon the operation on the current fragment. This keyword causes the fragment to be discarded and no updates to any buffers will occur. Control fl

[Mesa-dev] [PATCH 4/9] glsl: When failing to compile some builtins, print the error.

2012-03-12 Thread Eric Anholt
This makes the process slightly more debuggable, though it would be nice if the build just failed immediately instead. --- src/glsl/builtins/tools/generate_builtins.py |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/src/glsl/builtins/tools/generate_builtins.py b/sr

[Mesa-dev] GLSL 1.40 progress: hooking up builtin functions

2012-03-12 Thread Eric Anholt
This is a continuation of the GLSL 1.40 work, which now gets the newly-submitted textureSize(*2DRect) tests working, with a brief detour to fix the GPU hangs due to the discard-exiting-the-shader test. I want to rework that test, actually, because in my first attempt at fixing it I only exited the

[Mesa-dev] [PATCH 3/9] glsl: Let the builtin compiler process GLSL 1.40 shaders.

2012-03-12 Thread Eric Anholt
This is required to put the new 1.40 builtins in place, since they require new types. --- src/glsl/main.cpp |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/glsl/main.cpp b/src/glsl/main.cpp index e174224..d43bf1a 100644 --- a/src/glsl/main.cpp +++ b/src/glsl/main.

[Mesa-dev] [PATCH 1/9] i965: Add disasm for gen6+ UIP/JIP on BREAK/CONT/HALT.

2012-03-12 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_disasm.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c b/src/mesa/drivers/dri/i965/brw_disasm.c index 187bc0a..aebfa73 100644 --- a/src/mesa/drivers/dri/i965/brw_disasm.c +++ b/src/mesa/drivers/

[Mesa-dev] Doing 8.0.1 release?

2012-03-12 Thread Jakob Bornecrantz
Hi all We well over due for a 8.0.1 release, so I thought we do it aggressively this week. A quick rc tomorrow and a release on Thursday or Friday? Is that okay with people, comments please? Cheers, Jakob. ___ mesa-dev mailing list mesa-dev@lists.freede

Re: [Mesa-dev] [PATCH 2/2] i915: set SPRITE_POINT_ENABLE bit only for points

2012-03-12 Thread Eric Anholt
On Mon, 12 Mar 2012 16:04:01 +0800, Yuanhan Liu wrote: > Signed-off-by: Yuanhan Liu Is there a reason for this change? What test does it fix? The hardware docs I have say "This bit controls the generation of texture coordinates at the corners of point primitives. When ENABLED, the corners of

[Mesa-dev] [Bug 47248] autogen missing dependency on flex and bison, causes infinite loop in glsl build

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47248 Darxus changed: What|Removed |Added CC||dar...@chaosreigns.com -- Configure bugmail: h

[Mesa-dev] [Bug 47248] New: autogen missing dependency on flex and bison, causes infinite loop in glsl build

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47248 Bug #: 47248 Summary: autogen missing dependency on flex and bison, causes infinite loop in glsl build Classification: Unclassified Product: Mesa Version: git Platform

Re: [Mesa-dev] [PATCH] mesa: fix _mesa_DebugMessageCallbackARB() to silence warnings

2012-03-12 Thread Jose Fonseca
Ah, yes. Missed this with my changes. Thanks. Joe - Original Message - > --- > src/mesa/main/errors.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c > index 2256100..fcf873f 100644 > --- a/src/mesa/main/er

[Mesa-dev] [PATCH] mesa: fix _mesa_DebugMessageCallbackARB() to silence warnings

2012-03-12 Thread Brian Paul
--- src/mesa/main/errors.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c index 2256100..fcf873f 100644 --- a/src/mesa/main/errors.c +++ b/src/mesa/main/errors.c @@ -718,10 +718,10 @@ _mesa_DebugMessageControlARB(GLenum s

Re: [Mesa-dev] [PATCH] mesa: Fix the cause of piglit test fbo-array failure

2012-03-12 Thread Brian Paul
On 03/12/2012 12:00 PM, Anuj Phogat wrote: Handle the special case of glFramebufferTextureLayer() for which we pass teximage = 0 internally in framebuffer_texture(). This patch makes failing piglit test fbo-array, fbo-depth-array to pass. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=471

[Mesa-dev] [Bug 47175] piglit fbo-alphatest-formats regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47175 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

Re: [Mesa-dev] [PATCH 3/3] glsl: Update builtin variables for GLSL 1.40.

2012-03-12 Thread Kenneth Graunke
On 03/12/2012 09:50 AM, Eric Anholt wrote: On Sun, 11 Mar 2012 23:59:35 -0700, Kenneth Graunke wrote: On 03/09/2012 01:27 PM, Eric Anholt wrote: Mostly this is a matter of removing variables that have been moved to the compatibility profile. There's one addition: gl_InstanceID is present in

[Mesa-dev] [Bug 47241] docs/autoconf.html is invalid

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47241 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Mesa-dev] [PATCH] mesa: Fix the cause of piglit test fbo-array failure

2012-03-12 Thread Anuj Phogat
Handle the special case of glFramebufferTextureLayer() for which we pass teximage = 0 internally in framebuffer_texture(). This patch makes failing piglit test fbo-array, fbo-depth-array to pass. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=47126 V4: Removed the duplicated code. Note: Th

Re: [Mesa-dev] [PATCH 1/9] i915: remove occurances of _DD_NEW_x flags

2012-03-12 Thread Eric Anholt
On Sun, 11 Mar 2012 18:34:38 -0600, Brian Paul wrote: > Just use the corresponding _NEW_x flags intead. The _DD_NEW_x flags > will be removed in a following patch. Series is: Reviewed-by: Eric Anholt pgpZKmg4uBziS.pgp Description: PGP signature ___

[Mesa-dev] [Bug 47241] docs/autoconf.html is invalid

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47241 --- Comment #1 from Christopher Yeleighton 2012-03-12 09:58:14 PDT --- Created attachment 58332 --> https://bugs.freedesktop.org/attachment.cgi?id=58332 upgrade markup -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=emai

[Mesa-dev] [Bug 47241] New: docs/autoconf.html is invalid

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47241 Bug #: 47241 Summary: docs/autoconf.html is invalid Classification: Unclassified Product: Mesa Version: 8.0 Platform: x86-64 (AMD64) URL: http://mesa3d.org/autoconf.html

Re: [Mesa-dev] [PATCH 3/3] glsl: Update builtin variables for GLSL 1.40.

2012-03-12 Thread Eric Anholt
On Sun, 11 Mar 2012 23:59:35 -0700, Kenneth Graunke wrote: > On 03/09/2012 01:27 PM, Eric Anholt wrote: > > Mostly this is a matter of removing variables that have been moved to > > the compatibility profile. There's one addition: gl_InstanceID is > > present in the core now. > > > > This fixes

Re: [Mesa-dev] [PATCH 1/9] i915: remove occurances of _DD_NEW_x flags

2012-03-12 Thread Jose Fonseca
The series looks good to me AFAICT. Joe - Original Message - > Just use the corresponding _NEW_x flags intead. The _DD_NEW_x flags > will be removed in a following patch. > --- > src/mesa/drivers/dri/i915/intel_tris.h | 12 +--- > 1 files changed, 5 insertions(+), 7 deletions(

[Mesa-dev] [Bug 47175] piglit fbo-alphatest-formats regression

2012-03-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47175 --- Comment #2 from Brian Paul 2012-03-12 09:02:10 PDT --- Actually, I think mesa is right and the test is wrong. The NVIDIA driver fails the same tests as Mesa. It looks like the test is doing the wrong thing when computing the expected resul

Re: [Mesa-dev] [PATCH] mesa: Fix the cause of piglit test fbo-array failure

2012-03-12 Thread Brian Paul
On 03/09/2012 07:07 PM, Anuj Phogat wrote: Handle the special case of glFramebufferTextureLayer() for which we pass teximage = 0 internally in framebuffer_texture(). This patch makes failing piglit test fbo-array, fbo-depth-array to pass. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=471

Re: [Mesa-dev] [PATCH 5/9] gallium: add R8G8_R8B8 and G8R8_B8R8 format

2012-03-12 Thread Roland Scheidegger
Am 12.03.2012 15:00, schrieb Jose Fonseca: > - Original Message - >> On 12.03.2012 13:26, Roland Scheidegger wrote: >>> Hmm actually I think these new pipe formats are a mistake. Why do >>> you need them and don't just use the existing R8G8_B8G8 and >>> G8R8_G8B8? Do you really care if Y e

Re: [Mesa-dev] [PATCH 5/9] gallium: add R8G8_R8B8 and G8R8_B8R8 format

2012-03-12 Thread Jose Fonseca
- Original Message - > On 12.03.2012 13:26, Roland Scheidegger wrote: > > Hmm actually I think these new pipe formats are a mistake. > > Why do you need them and don't just use the existing R8G8_B8G8 and > > G8R8_G8B8? > > Do you really care if Y ends up in G channel instead of R? And if >

Re: [Mesa-dev] [PATCH 5/9] gallium: add R8G8_R8B8 and G8R8_B8R8 format

2012-03-12 Thread Christian König
On 12.03.2012 13:26, Roland Scheidegger wrote: Hmm actually I think these new pipe formats are a mistake. Why do you need them and don't just use the existing R8G8_B8G8 and G8R8_G8B8? Do you really care if Y ends up in G channel instead of R? And if you do why don't you just use a sampler swizzle

Re: [Mesa-dev] [PATCH 5/9] gallium: add R8G8_R8B8 and G8R8_B8R8 format

2012-03-12 Thread Roland Scheidegger
Hmm actually I think these new pipe formats are a mistake. Why do you need them and don't just use the existing R8G8_B8G8 and G8R8_G8B8? Do you really care if Y ends up in G channel instead of R? And if you do why don't you just use a sampler swizzle? The R8G8_B8G8 and G8R8_G8B8 formats are there b

[Mesa-dev] [PATCH 2/2] i915: set SPRITE_POINT_ENABLE bit only for points

2012-03-12 Thread Yuanhan Liu
Signed-off-by: Yuanhan Liu --- src/mesa/drivers/dri/i915/i915_state.c |8 +++- src/mesa/drivers/dri/i915/i915_vtbl.c |9 + 2 files changed, 16 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i915/i915_state.c b/src/mesa/drivers/dri/i915/i915_state.c index 3c

[Mesa-dev] [PATCH 1/2] i915: set SPRITE_POINT_ENABLE bit correctly

2012-03-12 Thread Yuanhan Liu
When SPRITE_POINT_ENABLE bit is set, the texture coord would be replaced, and this is only needed when we called something like glTexEnvi(GL_POINT_SPRITE, GL_COORD_REPLACE, GL_TRUE). And more, we currently handle varying inputs as texture coord, we would be careful when setting this bit and set i

Re: [Mesa-dev] [PATCH] i915: fallback when point sprite is enabled while handling varying inputs

2012-03-12 Thread Yuanhan Liu
On Mon, Mar 12, 2012 at 10:14:06AM +0800, Yuanhan Liu wrote: > On Fri, Mar 09, 2012 at 10:35:33AM -0800, Eric Anholt wrote: > > On Thu, 8 Mar 2012 19:21:23 +0800, Yuanhan Liu > > wrote: > > > From ddd1a9d8f0d82c2f5fcb78a471608a005a6a077c Mon Sep 17 00:00:00 2001 > > > From: Yuanhan Liu > > > Dat

Re: [Mesa-dev] [PATCH 1/7] radeon/r200: remove hyperz/fast clear testing code

2012-03-12 Thread Dave Airlie
On Mon, Mar 12, 2012 at 12:33 AM, Brian Paul wrote: > The flags which were computed were never actually used. > --- >  src/mesa/drivers/dri/r200/r200_ioctl.c     |   10 -- >  src/mesa/drivers/dri/radeon/radeon_ioctl.c |   10 -- >  2 files changed, 0 insertions(+), 20 deletions(-)

Re: [Mesa-dev] Initial pieces of GLSL 1.40

2012-03-12 Thread Kenneth Graunke
On 03/09/2012 01:27 PM, Eric Anholt wrote: For the tests I've run this through, and a list of what I know is remaining, see: http://lists.freedesktop.org/archives/piglit/2012-March/002003.html All but the discard infinite loop test work on i965 with this series. I haven't looked at the Piglit

Re: [Mesa-dev] [PATCH 3/3] glsl: Update builtin variables for GLSL 1.40.

2012-03-12 Thread Kenneth Graunke
On 03/09/2012 01:27 PM, Eric Anholt wrote: Mostly this is a matter of removing variables that have been moved to the compatibility profile. There's one addition: gl_InstanceID is present in the core now. This fixes the new piglit tests for GLSL 1.40 builtins. --- src/glsl/builtin_variables.cp