[Mesa-dev] [Bug 30484] Mesa 7.9.2-rc2 and git fail to compile

2011-12-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=30484 Jos van Wolput wol...@onsneteindhoven.nl changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] Bug#651370: libgl1-mesa-glx: need close on exec for dri device

2011-12-08 Thread Michel Dänzer
On Mit, 2011-12-07 at 23:43 -0600, David Fries wrote: Package: libgl1-mesa-glx Version: 7.11-6 Severity: normal Tags: patch When dri is used the character device /dev/dri/card0 is opened. That file descriptor should be set with the close on exec file as currently those resources are

[Mesa-dev] [PATCH] glsl: Fix crashes caused by Bison error messages involving '%'.

2011-12-08 Thread Kenneth Graunke
Invalid shaders containing the character % at an unexpected location would cause Bison to call yyerror with a message of: syntax error, unexpected '%' Bison expects yyerror() to take a string, while _mesa_glsl_error() is a printf-style function. This hit the classic printf string escape

[Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Maarten Lankhorst
No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Changes since v2: - fixed borked ureg_MOV, I have no idea how it worked when I tested this on r600 to begin with, it renders correctly now with XvMC

Re: [Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Andy Furniss
Maarten Lankhorst wrote: No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com --- Changes since v2: - fixed borked ureg_MOV, I have no idea how it worked when I tested this on r600 to begin with, it renders

[Mesa-dev] [Bug 43629] New: mesa# gmake freebsd-dri-amd64 breaks

2011-12-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43629 Bug #: 43629 Summary: mesa# gmake freebsd-dri-amd64 breaks Classification: Unclassified Product: Mesa Version: unspecified Platform: x86-64 (AMD64) OS/Version: FreeBSD

[Mesa-dev] [PATCH] mesa/st: Don't modify the context draw/read buffers.

2011-12-08 Thread jfonseca
From: José Fonseca jfons...@vmware.com It sets the wrong values (GL_XXX_LEFT instead of GL_XXX), and no other Mesa driver does this, given that Mesa sets the right draw/read buffers provided the Mesa visual has the doublebuffer flag filled correctly which is the case. ---

[Mesa-dev] [Bug 43629] mesa# gmake freebsd-dri-amd64 breaks

2011-12-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43629 --- Comment #2 from zap...@berentweb.com 2011-12-08 06:17:57 PST --- dri2proto from ports (I assumed ports collection was up-to-date on dri2proto since link is to 1.1.tar.gz on the wiki page:

Re: [Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Maarten Lankhorst
Hey Andy, On 12/08/2011 01:24 PM, Andy Furniss wrote: Maarten Lankhorst wrote: No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com --- Changes since v2: - fixed borked ureg_MOV, I have no idea how it worked

Re: [Mesa-dev] [PATCH] mesa/st: Don't modify the context draw/read buffers.

2011-12-08 Thread Brian Paul
On 12/08/2011 07:00 AM, jfons...@vmware.com wrote: From: José Fonsecajfons...@vmware.com It sets the wrong values (GL_XXX_LEFT instead of GL_XXX), and no other Mesa driver does this, given that Mesa sets the right draw/read buffers provided the Mesa visual has the doublebuffer flag filled

[Mesa-dev] [PATCH 1/2] ARB_Uniform_Buffer_Object: layout(std140) test

2011-12-08 Thread Vincent Lejeune
--- tests/all.tests|4 + tests/spec/CMakeLists.txt |1 + .../arb_uniform_buffer_object/CMakeLists.gl.txt| 17 ++ .../spec/arb_uniform_buffer_object/CMakeLists.txt |1 + .../arb_uniform_buffer_object/standard-layout.c

[Mesa-dev] [PATCH 2/2] ARB_Uniform_Buffer_Object: Draw test

2011-12-08 Thread Vincent Lejeune
--- tests/all.tests|1 + .../arb_uniform_buffer_object/CMakeLists.gl.txt|1 + tests/spec/arb_uniform_buffer_object/draw_test.c | 227 3 files changed, 229 insertions(+), 0 deletions(-) create mode 100644

Re: [Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Andy Furniss
Maarten Lankhorst wrote: Softpipe is working, that's handy - last time I tried it was too borked to be of use, but now I can actually see which errors are software and which are added by R600. My observation for softpipe working correctly only seems to apply for XVMC_MOCOMP, if I try with

Re: [Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Andy Furniss
Andy Furniss wrote: There are minor errors - when I said Softpipe is working I was comparing to the past when it would render mainly junk with a shrunken portion of the vid and later when that went away, it would render with blocks of frame -1 mixed with current frame. Actually with vdpau

[Mesa-dev] [PATCH 0/7] Fix GLX files generated by Mesa for the server

2011-12-08 Thread Ian Romanick
There has been quite a bit of skew between what's in Mesa and what's needed in the xserver. This patch series cleans that up. Most of the changes are quite mundane and just make the code compile inside the xserver. However, the changes in patch 6/7 modify the way the availability and use of

[Mesa-dev] [PATCH 2/7] glapi: Remove mention of nonexistent enums

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com glext.h doesn't have GL_MIN_PROGRAM_TEXEL_OFFSET_EXT or GL_MAX_PROGRAM_TEXEL_OFFSET_EXT. Using them in the XML causes code to be generated for the xserver that won't compile. Use the names that exist instead. Signed-off-by: Ian Romanick

[Mesa-dev] [PATCH 3/7] glapi/glx: Don't send GL API files to the xserver

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com The versions in the xserver and in libGL have diverged enough that the xserver doesn't want these. Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mapi/glapi/gen/Makefile |7 --- 1 files changed, 0 insertions(+), 7 deletions(-)

[Mesa-dev] [PATCH 4/7] glapi/glx: Xserver wants dispatch.h, not glapidispatch.h

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mapi/glapi/gen/Makefile |6 -- src/mapi/glapi/gen/glX_proto_recv.py |2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/mapi/glapi/gen/Makefile

[Mesa-dev] [PATCH 5/7] glapi/glx: Generate dispatch.h so that the xserver can use it

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mapi/glapi/gen/gl_table.py |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/mapi/glapi/gen/gl_table.py b/src/mapi/glapi/gen/gl_table.py index f6182b6..7f3b915

[Mesa-dev] [PATCH 6/7] glapi/glx: Generate glapi_gentable.c so that the xserver can use it

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mapi/glapi/gen/gl_gentable.py | 38 1 files changed, 33 insertions(+), 5 deletions(-) diff --git a/src/mapi/glapi/gen/gl_gentable.py

[Mesa-dev] [PATCH 7/7] glapi/glx: For GLX code, use the existing _X_HIDDEN and _X_INTERNAL defines

2011-12-08 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mapi/glapi/gen/glX_proto_recv.py | 15 --- src/mapi/glapi/gen/glX_proto_send.py | 16 src/mapi/glapi/gen/glX_proto_size.py | 19 +-- 3 files

[Mesa-dev] i965: last ARB_depth_buffer_float prep!

2011-12-08 Thread Eric Anholt
Here's the last stuff I think I need to be happy to push GL_ARB_depth_buffer_float under the GL 3.0 override. Mostly I needed to remove a bunch of code in order to make handling separate stencil/HiZ for Z32_FLOAT_X24S8 sane. I've tested the series on gen6 with hiz enabled and disabled for float

[Mesa-dev] [PATCH 2/8] intel: Move S8 width/height alignment to miptree creation.

2011-12-08 Thread Eric Anholt
We were doing it in the caller in the renderbuffer code, but it was missed in the separate stencil creation for textures. Apparently our testing was using renderbuffers or pre-aligned sizes. --- src/mesa/drivers/dri/intel/intel_fbo.c | 44 +--

[Mesa-dev] [PATCH 4/8] intel: Remove another renderbuffer allocation path.

2011-12-08 Thread Eric Anholt
Now there's the thing that CALLOCs and sets up window system vtable, and the thing that CALLOCs and sets up user renderbuffer vtable. The user renderbuffer vtable gets replaced later by intel_renderbuffer_update_wrapper for wrapped renderbuffers (things with name == ~0). ---

[Mesa-dev] [PATCH 3/8] intel: Make the separate stencil RB storage path match texture more.

2011-12-08 Thread Eric Anholt
There were too many things making intel_renderbuffer *s and tweaking their bits. --- src/mesa/drivers/dri/intel/intel_fbo.c | 128 +--- 1 files changed, 52 insertions(+), 76 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c

[Mesa-dev] [PATCH 1/8] intel: Drop check for wrapped_depth in RB mapping.

2011-12-08 Thread Eric Anholt
This used to be needed because irb-mt would be unset for fake packed depth/stencil, but no longer. --- src/mesa/drivers/dri/intel/intel_fbo.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c b/src/mesa/drivers/dri/intel/intel_fbo.c

[Mesa-dev] [PATCH 7/8] intel: Stop creating the wrapped stencil irb.

2011-12-08 Thread Eric Anholt
There were only two places it was really used at this point, which was in the batchbuffer emit of the separate stencil packets for gen6/7. Just write in the -stencil_mt reference in those two places and ditch all this flailing around with allocation and refcounts. ---

[Mesa-dev] [PATCH 6/8] i965: Drop separate stencil assertions in update_draw_buffer().

2011-12-08 Thread Eric Anholt
The comment said they deserved to be in emit_depthbuffer, and at this point they were all there already. --- src/mesa/drivers/dri/i965/brw_vtbl.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c

[Mesa-dev] [PATCH 5/8] intel: Simplify and touch up the FBO completeness test.

2011-12-08 Thread Eric Anholt
Now that we have miptrees for everything, we can more easily test for !has_separate_stencil completeness. Also, test for whether the stencil rb is the wrong kind of format for separate stencil, or if we are trying to do packed to different images of a single miptree. ---

[Mesa-dev] [PATCH 8/8] intel: Stop creating the wrapped depth irb.

2011-12-08 Thread Eric Anholt
All the operations were just trying to get at irb-wrapped_depth-mt, which is the same as irb-mt now. --- src/mesa/drivers/dri/i965/brw_vtbl.c |1 - src/mesa/drivers/dri/intel/intel_fbo.c | 91 +++- src/mesa/drivers/dri/intel/intel_fbo.h | 27 +- 3

Re: [Mesa-dev] [PATCH 0/7] Fix GLX files generated by Mesa for the server

2011-12-08 Thread Brian Paul
On 12/08/2011 01:47 PM, Ian Romanick wrote: There has been quite a bit of skew between what's in Mesa and what's needed in the xserver. This patch series cleans that up. Most of the changes are quite mundane and just make the code compile inside the xserver. However, the changes in patch 6/7

[Mesa-dev] [PATCH 01/14] i965/fs: Don't swizzle the results of textureSize().

2011-12-08 Thread Kenneth Graunke
Fixes a regression since d2235b0f4681f75d562131d655a6d7b7033d2d8b, in my new textureSize sampler(1DArrayShadow|2DShadow|2DArrayShadow) piglit tests, though I'm not honestly sure how this ever worked. Signed-off-by: Kenneth Graunke kenn...@whitecape.org ---

[Mesa-dev] i965 vertex shader texturing support

2011-12-08 Thread Kenneth Graunke
This series adds vertex shader texturing support to the i965 driver, as required by OpenGL 3.0. Only two things remain: textureGrad() on shadow samplers (which isn't supported in hardware), and coordinate clamping to support proper filtering at texture borders. While I added code for texturing

[Mesa-dev] [PATCH 02/14] i965: Drop support for the GL_MESA_ycbcr_texture extension.

2011-12-08 Thread Kenneth Graunke
We forgot to implement the color conversions in the new fragment shader backend, so YCBCR + GLSL has been broken since Mesa 7.10. Additionally, with the fixed function fragment shader rework, YCBCR + FF is broken in master. The only combination that should still work is YCBCR + ARB_fp.

[Mesa-dev] [PATCH 03/14] i965: Rename texturing ops from FS_OPCODE to SHADER_OPCODE, except TXB.

2011-12-08 Thread Kenneth Graunke
We'll be reusing most of these for the VS shortly. The one exception is TXB (texturing with LOD bias), which is explicitly forbidden in the VS. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_defines.h | 14 +++-

[Mesa-dev] [PATCH 04/14] i965/vs: Add vec4_instruction::is_tex() query.

2011-12-08 Thread Kenneth Graunke
Copy and pasted from fs_inst::is_tex(), but without TXB. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_vec4.cpp | 10 ++ src/mesa/drivers/dri/i965/brw_vec4.h |1 + 2 files changed, 11 insertions(+), 0 deletions(-) diff --git

[Mesa-dev] [PATCH 05/14] i965/vs: Add a new dst_reg constructor for file, number, type, and mask.

2011-12-08 Thread Kenneth Graunke
This will be especially useful for loading texturing parameters, where I need to (for example) reference m3.xzD. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_vec4.h | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) Not strictly

[Mesa-dev] [PATCH 06/14] i965/vs: Implement vec4_visitor::generate_tex().

2011-12-08 Thread Kenneth Graunke
This is the part that takes the vec4_instruction IR and turns it into actual Gen ISA. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_vec4.h|4 ++ src/mesa/drivers/dri/i965/brw_vec4_emit.cpp | 79 +++ 2 files changed,

[Mesa-dev] [PATCH 07/14] i965/vs: Implement vec4_visitor::visit(ir_texture *).

2011-12-08 Thread Kenneth Graunke
This translates the GLSL compiler's IR into vec4_instruction IR, generating code to load coordinates, LOD info, shadow comparitors, and so on into the appropriate message registers. It turns out that the SIMD4x2 parameters are identical on Gen 5-7, and the Gen4 code is similar enough that, unlike

[Mesa-dev] [PATCH 08/14] i965/fs: Factor out texture offset bitfield computation.

2011-12-08 Thread Kenneth Graunke
We'll want to reuse this for the VS, and it's complex enough that I'd rather not cut and paste it. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 19 +-- src/mesa/drivers/dri/i965/brw_shader.cpp | 23

[Mesa-dev] [PATCH 09/14] i965/vs: Add support for texel offsets.

2011-12-08 Thread Kenneth Graunke
The visit() half computes the values to put in the header based on the IR and simply stuffs that in the vec4_instruction; the emit() half uses this to set up the message header. This works out well since emit() can use brw_reg directly and access individual DWords without kludgery.

[Mesa-dev] [PATCH 10/14] i965/fs: Factor out texturing related data from brw_wm_prog_key.

2011-12-08 Thread Kenneth Graunke
The idea is to reuse this for the VS and (in the future) GS as well. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_fs.cpp |4 +- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 26 +++--- src/mesa/drivers/dri/i965/brw_program.h | 54

[Mesa-dev] [PATCH 11/14] i965/fs: Only set brw_wm_prog_key data for samplers used by the WM.

2011-12-08 Thread Kenneth Graunke
This should avoid state-dependent FS recompiles when samplers that are only used by the VS change. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_wm.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[Mesa-dev] [PATCH 12/14] i965/vs: Add texture related data to brw_vs_prog_key.

2011-12-08 Thread Kenneth Graunke
Now that this is all factored out, it's trivial to do. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_vs.c |8 src/mesa/drivers/dri/i965/brw_vs.h |3 +++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git

[Mesa-dev] [PATCH 13/14] i965/vs: Implement EXT_texture_swizzle support for VS texturing.

2011-12-08 Thread Kenneth Graunke
Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- src/mesa/drivers/dri/i965/brw_vec4.h |2 + src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 51 +++- 2 files changed, 52 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h

Re: [Mesa-dev] [PATCH] mesa/st: Don't modify the context draw/read buffers.

2011-12-08 Thread Chia-I Wu
On Fri, Dec 9, 2011 at 2:02 PM, Chia-I Wu o...@lunarg.com wrote: On Thu, Dec 8, 2011 at 10:00 PM,  jfons...@vmware.com wrote: From: José Fonseca jfons...@vmware.com It sets the wrong values (GL_XXX_LEFT instead of GL_XXX), and no other Mesa driver does this, given that Mesa sets the right