[Mesa-dev] [Bug 44151] glXDestroyWindow after glXDestroyContext -> Invalid read

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44151 Michel Dänzer changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Dave Airlie
> > was doing some piglit comparisons between softpipe and llvmpipe and > noticed the fbo-alphatest-formats test fails on llvmpipe for all the > I8 formats. > > Now I looked at the code generated for > lp_tile_soa.c:lp_tile_i8_unorm_unswizzle_4ub and it references a[i+0] > and a[i+1], if I change i

Re: [Mesa-dev] XCB-only GLX protocol?

2011-12-30 Thread Ingo Krabbe
On Sun, Nov 27, 2011 at 06:13:45PM -0800, Ian Romanick wrote: > All, > > I'm starting to work on GLX_ARB_create_context. This extension and the > layered GLX_ARB_create_context_profile extension add some GLX protocol. > Is there any reason to keep supporting non-XCB protocol? Are there > an

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Jose Fonseca
- Original Message - > > > > was doing some piglit comparisons between softpipe and llvmpipe and > > noticed the fbo-alphatest-formats test fails on llvmpipe for all > > the > > I8 formats. > > > > Now I looked at the code generated for > > lp_tile_soa.c:lp_tile_i8_unorm_unswizzle_4ub and i

[Mesa-dev] [Bug 40448] Unigine Sanctuary: window disappear when changing settings

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40448 Pavel Ondračka changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Mesa-dev] [Bug 42883] Firefox crashes in state_tracker/st_cb_blit.c:87

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42883 Alex Deucher changed: What|Removed |Added AssignedTo|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.

[Mesa-dev] [Bug 44298] SIGSEGV src/mesa/state_tracker/st_cb_blit.c:87

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44298 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Mesa-dev] [Bug 42883] Firefox crashes in state_tracker/st_cb_blit.c:87

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42883 Alex Deucher changed: What|Removed |Added CC||v...@vmware.com --- Comment #4 from Alex

[Mesa-dev] [Bug 44298] SIGSEGV src/mesa/state_tracker/st_cb_blit.c:87

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44298 José Fonseca changed: What|Removed |Added CC||jfons...@vmware.com -- Configure bugmail

Re: [Mesa-dev] [PATCH 0/3] Array object cleanup.

2011-12-30 Thread Brian Paul
2011/12/29 Mathias Fröhlich : > > Hi, > > Following a short series of cleanup following the gl_array_object change. > The most important change is the use ot an ffs based loop to compute > gl_array_object::_MaxElement. This change provides a noticable performance > improovement for my average workl

[Mesa-dev] [Bug 44234] SIGSEGV src/gallium/state_trackers/glx/xlib/glx_api.c:2668

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

Re: [Mesa-dev] [PATCH] gallium: use Haiku provided debug_printf in OS.h

2011-12-30 Thread Brian Paul
On Thu, Dec 29, 2011 at 10:42 AM, Alexander von Gluck wrote: > > --- >  src/gallium/auxiliary/util/u_debug.h |    6 -- >  1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/gallium/auxiliary/util/u_debug.h > b/src/gallium/auxiliary/util/u_debug.h > index b5ea405..448d799 100

Re: [Mesa-dev] [PATCH] glsl: fix usage of potentially undefined data_end union

2011-12-30 Thread Brian Paul
On Thu, Dec 29, 2011 at 9:48 AM, Alexander von Gluck wrote: > > --- >  src/glsl/link_uniforms.cpp |    2 +- >  1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp > index c7de480..b331db7 100644 > --- a/src/glsl/link_uniforms.cp

Re: [Mesa-dev] [PATCH] swrast: Remove dead code in _swrast_clear_depth_buffer()

2011-12-30 Thread Brian Paul
On Thu, Dec 29, 2011 at 11:20 AM, Paul Berry wrote: > This code was generating the gcc warning: > >  variable ‘clearValue’ set but not used [-Wunused-but-set-variable] > --- >  src/mesa/swrast/s_depth.c |    9 - >  1 files changed, 0 insertions(+), 9 deletions(-) > > diff --git a/src/mesa/

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Dave Airlie
>> >> Okay this fixes it here, and even seems reasonable to me. > > Yes. Great catch Dave. > > This should also fix to L8 rendering.  But I belive that in the commit > message, you mean "i8" and not "l8". > > Please add a comment about why order matters in the inv_swizzle() function, > for future

[Mesa-dev] [PATCH 1/2] gallium: dereference are now handled by a separate visitor in glsl_to_tgsi

2011-12-30 Thread Vincent Lejeune
v2: fix glsl-fs-uniform-array-4 piglit test v3: fix glsl-array-bound-02 piglit crash --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 466 ++-- 1 files changed, 307 insertions(+), 159 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_t

[Mesa-dev] [PATCH 2/2] gallium: improves glsl_to_tgsi generation of ARL

2011-12-30 Thread Vincent Lejeune
This commit should generates less ARL instructions when dealing with indirect addressing. v2: fix glsl-vs-array piglit test --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 60 +--- 1 files changed, 37 insertions(+), 23 deletions(-) diff --git a/src/mesa/state_tracker/s

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Jose Fonseca
- Original Message - > >> > >> Okay this fixes it here, and even seems reasonable to me. > > > > Yes. Great catch Dave. > > > > This should also fix to L8 rendering.  But I belive that in the > > commit message, you mean "i8" and not "l8". > > > > Please add a comment about why order matter

[Mesa-dev] [PATCH] gallium: use Haiku provided debug_printf in OS.h

2011-12-30 Thread Alexander von Gluck
--- src/gallium/auxiliary/util/u_debug.h |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/util/u_debug.h b/src/gallium/auxiliary/util/u_debug.h index b5ea405..3aa6aaf 100644 --- a/src/gallium/auxiliary/util/u_debug.h +++ b/src/gallium/auxiliary

[Mesa-dev] [Bug 44234] SIGSEGV src/gallium/state_trackers/glx/xlib/glx_api.c:2668

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44234 Vinson Lee changed: What|Removed |Added Status|RESOLVED|CLOSED --- Comment #2 from Vinson Lee 2011

Re: [Mesa-dev] [PATCH 2/3] mesa: Introduce enabled bitfield helper functions.

2011-12-30 Thread Eric Anholt
On Thu, 29 Dec 2011 23:13:57 +0100, Mathias Froehlich wrote: > +/** Returns the bitmask of all enabled arrays in fixed function mode. > + * > + * In fixed function mode only the traditional fixed function arrays > + * are available. > + */ > +static inline GLbitfield64 > +_mesa_array_object_get

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Dave Airlie
>> The fbo-blending-formats test now only fails on I8 with DST_ALPHA, >> I'm >> guessing what happens is we have an I8 dst, but we never flush the >> tiles, so we end up using the alpha from the tile as the dst alpha >> value instead of the red value. Okay I've taken your suggestion, and the attac

Re: [Mesa-dev] [PATCH 3/3] mesa: Clean up gl_array_object::_MaxElement computation.

2011-12-30 Thread Eric Anholt
On Thu, 29 Dec 2011 23:14:25 +0100, Mathias Fröhlich wrote: > Use a bitmask approach to compute gl_array_object::_MaxElement. > To make this work reliably depending on the shader type actually used, > make use of the newly introduced typed bitmaks getters. > With this change I gain about 5% draw

Re: [Mesa-dev] [PATCH 2/4] mesa: simplify Driver.TexImage() parameters

2011-12-30 Thread Dave Airlie
On Mon, Dec 26, 2011 at 11:09 PM, Brian Paul wrote: > From: Brian Paul > > As with TexSubImage(), the target, level and texObj values can be obtained > through the texImage pointer. cubemap broke with this, I suspect you do need to pass the target, as the cubemap needs it. Dave. ___

[Mesa-dev] [Bug 43125] [bisected] Start screen in Amnesia too dark after "Rewrite the way uniforms are tracked and handled" commit

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43125 Pavel Ondračka changed: What|Removed |Added URL||http://www.amnesiagame.com/

[Mesa-dev] [PATCH] gallium: improves glsl_to_tgsi generation of ARL

2011-12-30 Thread Vincent Lejeune
This commit should generates less ARL instructions when dealing with indirect addressing. v2: fix glsl-vs-array piglit test v3: fix fs-temp-array-mat2-index-col-rd piglit test --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 64 +--- 1 files changed, 39 insertions(+), 2

[Mesa-dev] [Bug 43125] [bisected] Start screen in Amnesia too dark after "Rewrite the way uniforms are tracked and handled" commit

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43125 Pavel Ondračka changed: What|Removed |Added CC||pavel.ondra...@email.cz -- Configure b

Re: [Mesa-dev] [PATCH 2/2] mesa: remove the dstX/Y/Zoffset params to _mesa_texstore() functions

2011-12-30 Thread Kenneth Graunke
On 12/26/2011 03:10 PM, Brian Paul wrote: > From: Brian Paul > > The were always zero. When doing a sub-texture replacement we account > for the dstX/Y/Zoffsets when we map the texture image. So no need to > pass them into the texstore code anymore. > --- > src/mesa/drivers/dri/intel/intel_tex

Re: [Mesa-dev] llvmpipe vs fbo-alphatest-formats

2011-12-30 Thread Brian Paul
On Fri, Dec 30, 2011 at 1:13 PM, Dave Airlie wrote: >>> The fbo-blending-formats test now only fails on I8 with DST_ALPHA, >>> I'm >>> guessing what happens is we have an I8 dst, but we never flush the >>> tiles, so we end up using the alpha from the tile as the dst alpha >>> value instead of the

[Mesa-dev] [PATCH 1/4] st/mesa: refactor gl_TexImage() code into prep_teximage()

2011-12-30 Thread Brian Paul
From: Brian Paul Preparation for st_TexImage() removal/refactoring. --- src/mesa/state_tracker/st_cb_texture.c | 55 ++-- 1 files changed, 38 insertions(+), 17 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_texture.c b/src/mesa/state_tracker/st_cb_texture.

[Mesa-dev] [PATCH 2/4] st/mesa: remove st_TexImage(), use core Mesa code instead

2011-12-30 Thread Brian Paul
From: Brian Paul The core Mesa code does the equivalent memory allocation, image mapping, storing and unmapping. We just need to call prep_teximage() first to handle the 'surface_based' stuff. The other change is to always use the level=0 mipmap image when accessing individual mipmap level imag

[Mesa-dev] [PATCH 3/4] st/mesa: fix indentation in st_copy_texsubimage()

2011-12-30 Thread Brian Paul
From: Brian Paul --- src/mesa/state_tracker/st_cb_texture.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_texture.c b/src/mesa/state_tracker/st_cb_texture.c index 3be88a0..88800b3 100644 --- a/src/mesa/state_tracker/st_cb_texture.

[Mesa-dev] [PATCH 4/4] st/mesa: remove st_CompressedTexSubImage1D/2D/3D()

2011-12-30 Thread Brian Paul
From: Brian Paul Just use the core Mesa functions instead. --- src/mesa/state_tracker/st_cb_texture.c | 84 +-- 1 files changed, 3 insertions(+), 81 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_texture.c b/src/mesa/state_tracker/st_cb_texture.c index 888

[Mesa-dev] [PATCH] mesa: rework ctx->Driver.CopyTexSubImage() parameters

2011-12-30 Thread Brian Paul
From: Brian Paul Replace target, level parameters with gl_texture_image. Add gl_renderbuffer parameter to indicate source buffer for the copy. This removes some redundant code in the drivers to find the source renderbuffer and the destination texture image (which we already had in _mesa_CopyTexS

[Mesa-dev] [Bug 43125] [bisected] Start screen in Amnesia too dark after "Rewrite the way uniforms are tracked and handled" commit

2011-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43125 --- Comment #7 from Vadim 2011-12-30 16:30:56 UTC --- I guess this patch could help: http://lists.freedesktop.org/archives/mesa-dev/2011-December/016347.html -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- Yo

[Mesa-dev] [PATCH] intel/intel_decode.c: Remove #include "intel_decode.h".

2011-12-30 Thread Johannes Obermayr
--- intel/intel_decode.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/intel/intel_decode.c b/intel/intel_decode.c index dbbcd38..e80e840 100644 --- a/intel/intel_decode.c +++ b/intel/intel_decode.c @@ -28,7 +28,6 @@ #include #include -#include "intel_decode.h" #i

Re: [Mesa-dev] [PATCH] intel/intel_decode.c: Remove #include "intel_decode.h".

2011-12-30 Thread Chad Versace
On 12/30/2011 04:45 PM, Johannes Obermayr wrote: > --- > intel/intel_decode.c |1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/intel/intel_decode.c b/intel/intel_decode.c > index dbbcd38..e80e840 100644 > --- a/intel/intel_decode.c > +++ b/intel/intel_decode.c > @@ -2

Re: [Mesa-dev] [PATCH 2/2] i965: fix the wrong min/max_index for nr_prims > 1

2011-12-30 Thread Yuanhan Liu
On Thu, Dec 29, 2011 at 09:10:03AM +0100, Michel Dänzer wrote: > On Don, 2011-12-29 at 10:03 +0800, Yuanhan Liu wrote: > > On Wed, Dec 28, 2011 at 12:07:08PM -0800, Eric Anholt wrote: > > > On Wed, 28 Dec 2011 13:54:43 +0800, Yuanhan Liu > > > wrote: > > > > The current code would just calculate

[Mesa-dev] [PATCH] vbo: introduce vbo_get_minmax_indices function

2011-12-30 Thread Yuanhan Liu
Introduce vbo_get_minmax_indices() function to handle the min/max index computation for nr_prims(>= 1). The old code just compute the first prim's min/max index; this would results an error rendering if user called functions like glMultiDrawElements(). This patch servers as fixing this issue. As w