Re: [Mesa-dev] [PATCH] vbo: count min/max_index before vbo->draw_prims

2011-12-24 Thread Liu Aleaxander
On Sun, Dec 25, 2011 at 8:03 AM, Eric Anholt wrote: > On Thu, 22 Dec 2011 18:55:50 +0800, Yuanhan Liu > wrote: >> For the case that index data is stored in element array buffer object, >> and user called glMultiDrawElements, count the min/max_index before >> calling vbo->draw_prims. vbo_get_minm

Re: [Mesa-dev] [PATCH 21/22] swrast: stop using _DepthBuffer in triangle code

2011-12-24 Thread Eric Anholt
On Thu, 22 Dec 2011 10:43:57 -0700, Brian Paul wrote: > On Wed, Dec 21, 2011 at 2:16 PM, Eric Anholt wrote: > > On Sun, 18 Dec 2011 20:08:26 -0700, Brian Paul wrote: > >> The only consequence is we can only use the occlusion_zless_16_triangle() > >> function with MESA_FORMAT_Z16. > > > > I'm not

Re: [Mesa-dev] [PATCH] vbo: count min/max_index before vbo->draw_prims

2011-12-24 Thread Eric Anholt
On Thu, 22 Dec 2011 18:55:50 +0800, Yuanhan Liu wrote: > For the case that index data is stored in element array buffer object, > and user called glMultiDrawElements, count the min/max_index before > calling vbo->draw_prims. vbo_get_minmax_index() isn't friendly to this > case. So do it while bui

Re: [Mesa-dev] [PATCH] i965 gen6: Fix incorrect order of dwords in gen6_update_sol_indices()

2011-12-24 Thread Eric Anholt
On Fri, 23 Dec 2011 14:02:10 -0800, Paul Berry wrote: > When updating SOL indices, we were accidentally putting the starting > index in dword 1 and the SVBI number to increment in dword 2--these > should be reversed. Usually both of these values are zero, so we > didn't see any problem. However,

[Mesa-dev] [PATCH] gallium/u_pack: fix l8/i8 pack color ub

2011-12-24 Thread Dave Airlie
From: Dave Airlie just noticed this in passing, not sure it actually fixes any issus. Signed-off-by: Dave Airlie --- src/gallium/auxiliary/util/u_pack_color.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/util/u_pack_color.h b/src/gallium/auxi

Re: [Mesa-dev] [PATCH 2/2] i965: Create mock implementation of GL_OES_EGL_image_external

2011-12-24 Thread Eric Anholt
On Wed, 21 Dec 2011 18:34:19 -0800, Chad Versace wrote: > In Android IceCreamSandwich, SurfaceFlinger requires GL_OES_image_external > for basic compositing tasks. Without the extension, SurfaceFlinger fails > to start. > > Despite the incompleteness of the extension's implementation introduced

Re: [Mesa-dev] [PATCH 4/4] glu: Fix build of static GLU libraries

2011-12-24 Thread Alexander von Gluck
On 24.12.2011 07:35, Brian Paul wrote: On Fri, Dec 23, 2011 at 4:25 PM, Alexander von Gluck wrote: * Several platforms try to build static libGLU.a  however, the libGLU makefile only will build a shared .so * Haiku links libGLU into libGL, so this makes the process easier ---  configs/haiku  

Re: [Mesa-dev] [PATCH 2/4] makefile: Add Haiku makefile build support

2011-12-24 Thread Alexander von Gluck
On 24.12.2011 07:30, Brian Paul wrote: On Fri, Dec 23, 2011 at 4:24 PM, Alexander von Gluck wrote: ---  Makefile      |    1 +  acinclude.m4  |    2 +-  configs/haiku |   66 +  3 files changed, 68 insertions(+), 1 deletions(-)  create mod

Re: [Mesa-dev] [PATCH 1/4] mklib: Add Haiku build support

2011-12-24 Thread Alexander von Gluck
On 24.12.2011 07:29, Brian Paul wrote: On Fri, Dec 23, 2011 at 4:24 PM, Alexander von Gluck wrote: ---  bin/mklib |   37 +  1 files changed, 37 insertions(+), 0 deletions(-) diff --git a/bin/mklib b/bin/mklib index 70bd1a2..93dd989 100755 --- a/bin/mklib ++

[Mesa-dev] [PATCH] vbo: signal _NEW_ARRAY when transitioning between glBegin/End, glDrawArrays

2011-12-24 Thread Brian Paul
From: Brian Paul This fixes a regression seen with the isosurf demo when switching between glBegin/End and glDrawArrays (do it several times). The problem was the driver wasn't getting _NEW_ARRAY when the arrays were subtly changed: (vertex3f, normal3f) vs. (normal3f, vertex3f). This patch fixe

Re: [Mesa-dev] [PATCH] vbo: signal _NEW_ARRAY when transitioning between glBegin/End, glDrawArrays

2011-12-24 Thread Brian Paul
2011/12/23 Mathias Fröhlich : > > Hi, > > On Thursday, December 22, 2011 18:30:44 Brian Paul wrote: >> I'm not sure if "playback_vertex_list" is more like DRAW_BEGIN_END or >> DRAW_ARRAYS. >> Maybe add a DRAW_DISPLAY_LIST enum value? > > It's more like begin/end I think. > The begin/end code just s

Re: [Mesa-dev] [PATCH] Clean up GL3 status

2011-12-24 Thread Brian Paul
On Sat, Dec 24, 2011 at 7:19 AM, Ferry Huberts wrote: > > > On 24-12-11 14:45, Brian Paul wrote: >> On Fri, Dec 23, 2011 at 1:19 PM, Matt Turner wrote: >>> The (swrast, i965, gallium, r600g) tuples are inconsistent and >>> confusing. If swrast, i965, and gallium support something, let's simply >>

Re: [Mesa-dev] [PATCH] Clean up GL3 status

2011-12-24 Thread Ferry Huberts
On 24-12-11 14:45, Brian Paul wrote: > On Fri, Dec 23, 2011 at 1:19 PM, Matt Turner wrote: >> The (swrast, i965, gallium, r600g) tuples are inconsistent and >> confusing. If swrast, i965, and gallium support something, let's simply >> say DONE without qualifying it. > > Hmm, I'm in favor of cle

Re: [Mesa-dev] [PATCH] Move the format and type check before select_tex_image.

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 6:51 AM, wrote: > From: Jian Zhao > > Move the format and type check before select_tex_image, or it will fail to > report > the mismatch error if the teximage is null. > > Reported-by: Anuj Phogat > Signed-off-by: Jian Zhao > Reviewed-by: Yuanhan Liu > --- >  src/mesa

Re: [Mesa-dev] [PATCH] vl: Fix inverted logic in vlc checks

2011-12-24 Thread Christian König
Sound it it was a good idea to look into my mailbox today. Thanks, patch is committed. Have a nice x-mas, and a happy new year, Christian. On 24.12.2011 12:33, Maarten Lankhorst wrote: Reported-by: Andy Furniss Signed-off-by: Maarten Lankhorst --- diff --git a/src/gallium/auxiliary/vl/vl_vlc

[Mesa-dev] [Bug 44039] Mesa 7.12-devel implementation error: unexpected format 0x822e in _mesa_choose_tex_format()

2011-12-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44039 --- Comment #1 from Brian Paul 2011-12-24 06:38:44 PST --- Can you run with gdb, set a breakpoint on _mesa_problem() and get a stack trace when this error is reported? I don't know how this would be related to a divide by zero error. Which dri

[Mesa-dev] [Bug 44113] git mesa gallium/state_trackers/egl fails to compile

2011-12-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44113 Jos van Wolput changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

Re: [Mesa-dev] [PATCH] Clean up GL3 status

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 1:19 PM, Matt Turner wrote: > The (swrast, i965, gallium, r600g) tuples are inconsistent and > confusing. If swrast, i965, and gallium support something, let's simply > say DONE without qualifying it. Hmm, I'm in favor of cleaning up status info where we can, but we're loo

Re: [Mesa-dev] [PATCH 4/4] glu: Fix build of static GLU libraries

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 4:25 PM, Alexander von Gluck wrote: > > * Several platforms try to build static libGLU.a >  however, the libGLU makefile only will build a shared .so > * Haiku links libGLU into libGL, so this makes the process easier > --- >  configs/haiku        |    3 +++ >  src/glu/sgi/

Re: [Mesa-dev] [PATCH 3/4] build: Haiku build fixes, BeOS code removal

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 4:25 PM, Alexander von Gluck wrote: > > * Use generic pthread barrier on Haiku. > * Re-add debug_printf define for Haiku. > * Add mapi headers to r300 build (to not rely on makedepend?) > * Remove NDEBUG define of union, as the union is used >  regardless of NDEFINE setting

Re: [Mesa-dev] [PATCH 2/4] makefile: Add Haiku makefile build support

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 4:24 PM, Alexander von Gluck wrote: > > --- >  Makefile      |    1 + >  acinclude.m4  |    2 +- >  configs/haiku |   66 > + >  3 files changed, 68 insertions(+), 1 deletions(-) >  create mode 100644 configs/haiku > >

Re: [Mesa-dev] [PATCH 1/4] mklib: Add Haiku build support

2011-12-24 Thread Brian Paul
On Fri, Dec 23, 2011 at 4:24 PM, Alexander von Gluck wrote: > > --- >  bin/mklib |   37 + >  1 files changed, 37 insertions(+), 0 deletions(-) > > diff --git a/bin/mklib b/bin/mklib > index 70bd1a2..93dd989 100755 > --- a/bin/mklib > +++ b/bin/mklib > @@ -959,6

[Mesa-dev] [PATCH 2/2] vl: Merge remaining members

2011-12-24 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- Flush explicitly ends the current frame so having a separate begin/endframe is unneeded, and decode_bitstream always has to operate on a full picture. src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c |4 src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 1

[Mesa-dev] [Bug 44101] [PATCH] egl_g3d_api.c:176:10: error: too few arguments to function ‘gctx- >stapi-> create_context’

2011-12-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44101 José Fonseca changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Mesa-dev] [PATCH 1/2] vl: Remove decode buffers

2011-12-24 Thread Maarten Lankhorst
It's still a curiosity used by vl_mpeg12_bitstream only. Signed-off-by: Maarten Lankhorst --- Version n+1: split up, and fixed to apply with new vl_vlc, although for testing you might want to grab the 'vl: Fix inverted logic in vlc checks' first. src/gallium/auxiliary/vl/vl_decoder.c

[Mesa-dev] [PATCH] vl: Fix inverted logic in vlc checks

2011-12-24 Thread Maarten Lankhorst
Reported-by: Andy Furniss Signed-off-by: Maarten Lankhorst --- diff --git a/src/gallium/auxiliary/vl/vl_vlc.h b/src/gallium/auxiliary/vl/vl_vlc.h index ff0c4b5..baaa48a 100644 --- a/src/gallium/auxiliary/vl/vl_vlc.h +++ b/src/gallium/auxiliary/vl/vl_vlc.h @@ -215,7 +215,7 @@ vl_vlc_bits_left(s