Re: [Mesa-dev] [PATCH] mesa: Fix error generation for glClearBuffer{i ui}v with GL_DEPTH or GL_STENCIL

2011-11-04 Thread Eric Anholt
On Thu, 3 Nov 2011 15:29:29 -0700, Ian Romanick i...@freedesktop.org wrote: From: Ian Romanick ian.d.roman...@intel.com The spec says Only ClearBufferiv should be used to clear stencil buffers. and Only ClearBufferfv should be used to clear depth buffers. However, on the following page it

[Mesa-dev] [Bug 41571] libglapi.so.0: undefined symbol: is_selinux_enabled

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41571 --- Comment #5 from Chia-I Wu olva...@gmail.com 2011-11-04 01:36:19 PDT --- Created attachment 53140 -- https://bugs.freedesktop.org/attachment.cgi?id=53140 link with -lselinux Does this patch help? You need to re-run autogen.sh. --

[Mesa-dev] [Bug 42517] Abort with llvmpipe renderer: ir_swizzle @ 0x2e8f0a0 specifies a channel not present in the value.

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42517 Vinson Lee v...@vmware.com changed: What|Removed |Added AssignedTo|mesa-dev@lists.freedesktop. |i...@freedesktop.org

[Mesa-dev] [Bug 42586] New: [PNV]mesa demo/arbocclude fail to show the animation displayed

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42586 Bug #: 42586 Summary: [PNV]mesa demo/arbocclude fail to show the animation displayed Classification: Unclassified Product: Mesa Version: 7.11 Platform: All

[Mesa-dev] Performance glxSwapBuffers 32 bit vs. 64 bit

2011-11-04 Thread Theiss, Ingo
Hi everyone, this is my first post to a mailing list here at freedesktop.org and I hope this is the right place for my question/problem. I am using VirtualGL (http://www.virtualgl.org) for full 3D hardware accelerated remote OpenGL applications with latest mesa from git (compiled for both 32

[Mesa-dev] [Bug 42461] Render error when there is no vertex shader

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42461 zhao jian jian.j.z...@intel.com changed: What|Removed |Added Status|RESOLVED|VERIFIED --- Comment

[Mesa-dev] glReadPixels broken w/ latest MapRenderbuffer changes

2011-11-04 Thread Brian Paul
The pure, non-DRI swrast driver dies in all glReadPixels calls when reading from the front/back color buffers. So basically all piglit tests fail. I think that the problem is there is no rb-Data for these surfaces; they're XImages/Pixmaps. The xlib driver needs to have

Re: [Mesa-dev] [PATCH] pp: Reorder calloc to avoid memory leak on error path.

2011-11-04 Thread Brian Paul
On 11/03/2011 10:00 PM, Vinson Lee wrote: Fixes Coverity resource leak defect. --- src/gallium/auxiliary/postprocess/pp_mlaa.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/postprocess/pp_mlaa.c

Re: [Mesa-dev] [PATCH] g3dvl: Fix memory leaks on error paths.

2011-11-04 Thread Brian Paul
On 11/03/2011 10:12 PM, Vinson Lee wrote: Fixes Coverity resource leak defect. --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c

[Mesa-dev] [PATCH] mesa/st: Be nice with the stack -- use malloc for large structures/arrays.

2011-11-04 Thread jfonseca
From: José Fonseca jfons...@vmware.com st_translate_program's data was taking 51K off the stack, causing stack overflow on Windows. --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 52 +-- 1 files changed, 40 insertions(+), 12 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] st/mesa: set geometry shader to NULL when doing internal drawing

2011-11-04 Thread Marek Olšák
The code expects the geometry shader to be NULL. We don't have geometry shaders now, but it's good to be prepared. --- src/gallium/auxiliary/util/u_blit.c | 25 + src/gallium/auxiliary/util/u_gen_mipmap.c | 16

[Mesa-dev] [PATCH 2/2] gallium/cso_cache: remove one call to pipe_sampler_view_reference

2011-11-04 Thread Marek Olšák
--- src/gallium/auxiliary/cso_cache/cso_context.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/cso_cache/cso_context.c b/src/gallium/auxiliary/cso_cache/cso_context.c index fdd40fc..b91fe1a 100644 ---

Re: [Mesa-dev] [PATCH 1/2] swrast: update program type assertion

2011-11-04 Thread Ian Romanick
On 11/03/2011 06:58 PM, Brian Paul wrote: Fixes bogus failed assertion when using NV_fragment_program, such as with demos/fplight.c Note: This is a candidate for the 7.11 branch. --- src/mesa/swrast/s_context.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] r600g: precalculate semantic indices for SPI setup

2011-11-04 Thread Vadim Girlin
There is no need to duplicate semantic mapping which is done in hw, so get rid of r600_find_vs_semantic_index. TGSI name/sid pair is mapped to the 8-bit semantic index for SPI. Signed-off-by: Vadim Girlin vadimgir...@gmail.com --- This patch and the next one were tested on the evergreen, there

[Mesa-dev] [PATCH 2/2] r600g: move SPI setup to PS setup

2011-11-04 Thread Vadim Girlin
SPI semantic indices for PS/VS are now static, so we don't need to update spi config for every shaders combination. We can move the functionality of r600_spi_update to r600(evergreen)_pipe_shader_ps. Flatshade state is now controlled by the global FLAT_SHADE_ENA flag instead of updating

Re: [Mesa-dev] [PATCH 1/2] swrast: update program type assertion

2011-11-04 Thread Brian Paul
On 11/04/2011 11:15 AM, Ian Romanick wrote: On 11/03/2011 06:58 PM, Brian Paul wrote: Fixes bogus failed assertion when using NV_fragment_program, such as with demos/fplight.c Note: This is a candidate for the 7.11 branch. --- src/mesa/swrast/s_context.c | 4 +++- 1 files changed, 3

[Mesa-dev] [PATCH] t_vertex: fix mipmap generation on rv100 swtcl.

2011-11-04 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com the meta mipmap generator on rv100 is passing a s,t,r coordinate, but r100 is ancient so has no r handling in hw, so we have to pass a s,t,q with q set to 1. /me dares someone to review this :) Signed-off-by: Dave Airlie airl...@redhat.com ---

Re: [Mesa-dev] [PATCH 1/2] st/mesa: set geometry shader to NULL when doing internal drawing

2011-11-04 Thread Jose Fonseca
What about checking has_geometry_shader inside cso module, and make cso_save_geometry_shader no-ops when it's false? Seems less code and more reusable. Jose - Original Message - The code expects the geometry shader to be NULL. We don't have geometry shaders now, but it's good to be

Re: [Mesa-dev] [PATCH 2/2] gallium/cso_cache: remove one call to pipe_sampler_view_reference

2011-11-04 Thread Jose Fonseca
Looks ok to me. Jose - Original Message - --- src/gallium/auxiliary/cso_cache/cso_context.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/cso_cache/cso_context.c b/src/gallium/auxiliary/cso_cache/cso_context.c index

Re: [Mesa-dev] [PATCH] t_vertex: fix mipmap generation on rv100 swtcl.

2011-11-04 Thread Roland Scheidegger
Am 04.11.2011 18:49, schrieb Dave Airlie: From: Dave Airlie airl...@redhat.com the meta mipmap generator on rv100 is passing a s,t,r coordinate, but r100 is ancient so has no r handling in hw, so we have to pass a s,t,q with q set to 1. /me dares someone to review this :)

[Mesa-dev] [Bug 42540] glxgears segfaults on POWER / Xvnc

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42540 --- Comment #2 from hugh.m.bri...@gmail.com 2011-11-04 12:53:06 PDT --- José Fonseca Thanks very much for your help. I dont know if real X works; I dont have root access. I asked the admin to install a few debug libraries. Here is the result:

Re: [Mesa-dev] [PATCH 1/2] r600g: precalculate semantic indices for SPI setup

2011-11-04 Thread Alex Deucher
On Fri, Nov 4, 2011 at 1:24 PM, Vadim Girlin vadimgir...@gmail.com wrote: There is no need to duplicate semantic mapping which is done in hw, so get rid of r600_find_vs_semantic_index. TGSI name/sid pair is mapped to the 8-bit semantic index for SPI. Pushed. Thanks! Note that the hw can use

Re: [Mesa-dev] [PATCH 1/3] glsl: Assign transform feedback varying slots in linker.

2011-11-04 Thread Paul Berry
Sorry I missed the first round of feedback on these patches. I hope my comments aren't coming too late. On 1 November 2011 12:48, Marek Olšák mar...@gmail.com wrote: From: Dan McCabe zen3d.li...@gmail.com Modify the linker to assign additional slots for varying variables used by transform

Re: [Mesa-dev] [PATCH 2/3] glsl: translate transform feedback varyings into low-level representation

2011-11-04 Thread Paul Berry
On 1 November 2011 12:48, Marek Olšák mar...@gmail.com wrote: This adds a function that takes an array of varyings from glTranformFeedbackVaryingsEXT and generates gl_transform_feedback_info, which is supposed to be consumed by drivers. Useful for ir_to_mesa and glsl_to_tgsi. v2: - changes

[Mesa-dev] [PATCH 1/2] st/mesa: set geometry shader to NULL when doing internal drawing

2011-11-04 Thread Marek Olšák
The code expects the geometry shader to be NULL. We don't have geometry shaders now, but it's good to be prepared. v2: check for support in the cso context --- src/gallium/auxiliary/cso_cache/cso_context.c | 17 - src/gallium/auxiliary/util/u_blit.c |6 ++

Re: [Mesa-dev] [PATCH 1/2] st/mesa: set geometry shader to NULL when doing internal drawing

2011-11-04 Thread Jose Fonseca
- Original Message - The code expects the geometry shader to be NULL. We don't have geometry shaders now, but it's good to be prepared. v2: check for support in the cso context --- src/gallium/auxiliary/cso_cache/cso_context.c | 17 -

Re: [Mesa-dev] glReadPixels broken w/ latest MapRenderbuffer changes

2011-11-04 Thread Eric Anholt
On Fri, 04 Nov 2011 07:31:54 -0600, Brian Paul bri...@vmware.com wrote: The pure, non-DRI swrast driver dies in all glReadPixels calls when reading from the front/back color buffers. So basically all piglit tests fail. I think that the problem is there is no rb-Data for these surfaces;

Re: [Mesa-dev] [PATCH 1/2] r600g: precalculate semantic indices for SPI setup

2011-11-04 Thread Marek Olšák
FYI, this commit completely breaks RV670. glxgears is completely black, other apps are either black or rendered incorrectly (mostly one-colored, the clear color I guess). Marek On Fri, Nov 4, 2011 at 6:24 PM, Vadim Girlin vadimgir...@gmail.com wrote: There is no need to duplicate semantic

[Mesa-dev] GL_EXT_texture_integer on i965

2011-11-04 Thread Eric Anholt
Here's a patch series to get GL_EXT_texture_integer partially working on i965, hidden under the GL 3.0 override. There's a bunch of support I think isn't finished yet, mostly because testing is in a bad state. A few things I noticed while looking at the spec: Test Per-fragment operations that

[Mesa-dev] [PATCH 04/10] i965: Add mapping from MESA_FORMAT to BRW_SURFACEFORMAT for integer.

2011-11-04 Thread Eric Anholt
Like non-integer, we can't render to RGB textures, so we promote them to RGBA. --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 24 ++ 1 files changed, 24 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c

[Mesa-dev] [PATCH 01/10] i965: Re-disable EXT_texture_float pre-gen5.

2011-11-04 Thread Eric Anholt
Texture filtering is not supported on floating-point textures until gen5, which showed up as failures in texwrap in particular. --- src/mesa/drivers/dri/intel/intel_extensions.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH 02/10] docs: Note EXT_texture_array on i965.

2011-11-04 Thread Eric Anholt
--- docs/relnotes-7.12.html |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/docs/relnotes-7.12.html b/docs/relnotes-7.12.html index a92278e..06aeda0 100644 --- a/docs/relnotes-7.12.html +++ b/docs/relnotes-7.12.html @@ -41,7 +41,7 @@ tbd

[Mesa-dev] [PATCH 03/10] intel: Expose GL_EXT_texture_integer when GL 3.0 override is set.

2011-11-04 Thread Eric Anholt
This will let the feature be incrementally developed, hidden behind the flag we're all using as we work on GL 3.0 support. --- src/mesa/drivers/dri/intel/intel_extensions.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git

[Mesa-dev] [PATCH 05/10] i965: Enable ChooseTexFormat for supported GL_EXT_texture_integer formats.

2011-11-04 Thread Eric Anholt
We don't get to use the 3-component formats, but faking of RGB with RGBA seems to be working. --- src/mesa/drivers/dri/intel/intel_context.c | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_context.c

[Mesa-dev] [PATCH 06/10] i965/fs: Add support for user-defined out variables.

2011-11-04 Thread Eric Anholt
Before, I was tracking the ir_variable * found for gl_FragColor or gl_FragData[]. Instead, when visiting those variables, set up an array of per-render-target fs_regs to copy the output data from. This cleans up the color emit path, while making handling of multiple user-defined out variables

[Mesa-dev] [PATCH 08/10] meta: Add support for glClear() to integer color buffers.

2011-11-04 Thread Eric Anholt
This requires using a new fragment shader to get the integer color output, and a new vertex shader because #version has to match between the two. --- src/mesa/drivers/common/meta.c | 108 ++-- 1 files changed, 104 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 09/10] mesa: Add support for unpacking 32-bit integer formats to int spans.

2011-11-04 Thread Eric Anholt
This is the inverse operation to _mesa_pack_rgba_span_int. The 16-bit code isn't done because of lack of testing and not being sure how sign extension/clamping should be handled between, say, 16-bit int and 32-bit int or uint. --- src/mesa/main/format_unpack.c | 120

[Mesa-dev] [PATCH 07/10] i965: Claim to support rendering to integer FBOs.

2011-11-04 Thread Eric Anholt
We're missing support for the software paths still, but basic rendering is working. --- src/mesa/drivers/dri/intel/intel_span.c |7 ++- src/mesa/drivers/dri/intel/intel_tex_format.c |6 ++ 2 files changed, 12 insertions(+), 1 deletions(-) diff --git

[Mesa-dev] [PATCH 10/10] swrast: Add support for glReadPixels() to integer types.

2011-11-04 Thread Eric Anholt
With this change, i965 passes GL_EXT_texture_integer/fbo_integer_precision_clear --- src/mesa/swrast/s_readpix.c | 19 +-- 1 files changed, 13 insertions(+), 6 deletions(-) diff --git a/src/mesa/swrast/s_readpix.c b/src/mesa/swrast/s_readpix.c index 50422db..54f42db 100644 ---

Re: [Mesa-dev] glReadPixels broken w/ latest MapRenderbuffer changes

2011-11-04 Thread Brian Paul
On 11/04/2011 03:48 PM, Eric Anholt wrote: On Fri, 04 Nov 2011 07:31:54 -0600, Brian Paulbri...@vmware.com wrote: The pure, non-DRI swrast driver dies in all glReadPixels calls when reading from the front/back color buffers. So basically all piglit tests fail. I think that the problem is

Re: [Mesa-dev] [PATCH 1/2] r600g: precalculate semantic indices for SPI setup

2011-11-04 Thread Alex Deucher
On Fri, Nov 4, 2011 at 5:53 PM, Marek Olšák mar...@gmail.com wrote: FYI, this commit completely breaks RV670. glxgears is completely black, other apps are either black or rendered incorrectly (mostly one-colored, the clear color I guess). I knew I should have tested 6xx before I pushed.

[Mesa-dev] [PATCH 1/4] mesa: Stub implementation of glBindFragDataLocation

2011-11-04 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com This just validates the input parameters so far. Fixes piglit's bindfragdata-invalid-parameters test. Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mesa/main/shader_query.cpp | 37 +

[Mesa-dev] [PATCH 2/4] mesa: Keep track of user settings for fragment data locations

2011-11-04 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mesa/main/mtypes.h |9 + src/mesa/main/shader_query.cpp |2 +- src/mesa/main/shaderobj.c |6 ++ 3 files changed, 16 insertions(+), 1 deletions(-) diff

Re: [Mesa-dev] [PATCH 2/3] glsl: translate transform feedback varyings into low-level representation

2011-11-04 Thread Marek Olšák
Hi Paul, I won't comment on the patch 1, because I didn't write it, I was just preserving history. On Fri, Nov 4, 2011 at 10:00 PM, Paul Berry stereotype...@gmail.com wrote: [snip] +bool parse_tfeedback_decl(const void *mem_ctx, const char *input, +                          struct

[Mesa-dev] [Bug 41571] libglapi.so.0: undefined symbol: is_selinux_enabled

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41571 --- Comment #6 from Alexandre Demers alexandre.f.dem...@gmail.com 2011-11-04 16:57:49 PDT --- Yes, I just tested it and it now works fine. Thank you. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are

[Mesa-dev] [PATCH 2/2] mesa: implement DrawTransformFeedback from ARB_transform_feedback2

2011-11-04 Thread Marek Olšák
It's like DrawArrays, but the count is taken from a transform feedback object. This removes DrawTransformFeedback from dd_function_table and adds the same function to GLvertexformat (with the function parameters matching GL). The vbo_draw_func callback has a new parameter struct

[Mesa-dev] [PATCH 1/2] mesa: add missing checks to compute_version

2011-11-04 Thread Marek Olšák
--- src/mesa/main/version.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/version.c b/src/mesa/main/version.c index 17c0600..49cdc30 100644 --- a/src/mesa/main/version.c +++ b/src/mesa/main/version.c @@ -124,9 +124,11 @@ compute_version(struct

[Mesa-dev] [Bug 42610] New: [PATCH] gallivm: build failure due to llvm-trunk api change.

2011-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42610 Bug #: 42610 Summary: [PATCH] gallivm: build failure due to llvm-trunk api change. Classification: Unclassified Product: Mesa Version: git Platform: x86 (IA32)