Re: [Mesa-dev] [PATCH] glx/dri: Initialize api even if num_attribs == 0.

2012-01-10 Thread Kenneth Graunke
On 01/10/2012 11:47 PM, Vadim Girlin wrote: On Tue, 2012-01-10 at 23:36 -0800, Kenneth Graunke wrote: Both dri2_create_context_attribs and drisw_create_context_attribs call dri2_convert_glx_attribs, expecting it to fill in *api on success. However, when num_attribs == 0, it was returning true w

Re: [Mesa-dev] [PATCH] glx/dri: Initialize api even if num_attribs == 0.

2012-01-10 Thread Vadim Girlin
On Tue, 2012-01-10 at 23:36 -0800, Kenneth Graunke wrote: > Both dri2_create_context_attribs and drisw_create_context_attribs call > dri2_convert_glx_attribs, expecting it to fill in *api on success. > > However, when num_attribs == 0, it was returning true without setting > *api, causing the call

[Mesa-dev] [PATCH] glx/dri: Initialize api even if num_attribs == 0.

2012-01-10 Thread Kenneth Graunke
Both dri2_create_context_attribs and drisw_create_context_attribs call dri2_convert_glx_attribs, expecting it to fill in *api on success. However, when num_attribs == 0, it was returning true without setting *api, causing the caller to use an uninitialized value. Cc: Vadim Girlin Signed-off-by:

Re: [Mesa-dev] [PATCH] glx/dri2: fix use of uninitialized variable "api"

2012-01-10 Thread Kenneth Graunke
On 01/10/2012 09:32 PM, Vadim Girlin wrote: Signed-off-by: Vadim Girlin --- Fixes unigine heaven startup problem for me. src/glx/dri2_glx.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c index 10b6f52..b4efa40 100644 --- a/sr

Re: [Mesa-dev] [PATCH] glx/dri2: fix use of uninitialized variable "api"

2012-01-10 Thread Stéphane Marchesin
What about initializing "api" earlier in dri2_convert_glx_attribs instead? i.e. before the first return. That should fix it also and is cleaner. Stéphane On Tue, Jan 10, 2012 at 21:32, Vadim Girlin wrote: > Signed-off-by: Vadim Girlin > --- > > Fixes unigine heaven startup problem for me. > >

Re: [Mesa-dev] [PATCH] configure.ac: use AC_PROG_SED

2012-01-10 Thread Kenneth Graunke
On 01/10/2012 10:16 PM, Matt Turner wrote: AC_PROG_PATH was used in the original configure.ac (dca1b796b) and I can't see any reason AC_PROG_SED wasn't used. It looks like AC_PROG_SED was added in 2.59b, and wasn't in the original 2.59. Presumably that's why, though it could've been an oversi

Re: [Mesa-dev] [PATCH] configure.ac: add and use AC_PROG_BISON/FLEX macros

2012-01-10 Thread Kenneth Graunke
On 01/10/2012 10:16 PM, Matt Turner wrote: Defining these macros lets us use AC_PROG_YACC/LEX which makes automake easier while still specifically requiring bison and flex. Based on bison.m4 and flex.m4 found in LLVM's tree. Signed-off-by: Matt Turner --- configure.ac |9 - m4/bi

Re: [Mesa-dev] Getting patches applied

2012-01-10 Thread Kenneth Graunke
On 01/10/2012 10:16 PM, Kurt Roeckx wrote: Hi, I've send a few patches to the bug tracker, but get no reaction to them. They're very simple and fix problems found by piglit on i830/i915. I wonder why they don't get applied. Should I send them to the list instead? Ah! This is the first time

Re: [Mesa-dev] [PATCH] configure.ac: add and use AC_PROG_BISON/FLEX macros

2012-01-10 Thread Matt Turner
On Wed, Jan 11, 2012 at 1:16 AM, Matt Turner wrote: > Defining these macros lets us use AC_PROG_YACC/LEX which makes automake > easier while still specifically requiring bison and flex. > > Based on bison.m4 and flex.m4 found in LLVM's tree. > > Signed-off-by: Matt Turner > --- >  configure.ac |

[Mesa-dev] Getting patches applied

2012-01-10 Thread Kurt Roeckx
Hi, I've send a few patches to the bug tracker, but get no reaction to them. They're very simple and fix problems found by piglit on i830/i915. I wonder why they don't get applied. Should I send them to the list instead? Anyway, you can find them here: https://bugs.freedesktop.org/show_bug.cgi

[Mesa-dev] [PATCH] configure.ac: don't set HAVE_GTEST twice

2012-01-10 Thread Matt Turner
Signed-off-by: Matt Turner --- configure.ac |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 08a0209..d36901b 100644 --- a/configure.ac +++ b/configure.ac @@ -69,7 +69,6 @@ AC_PATH_PROG([GTESTCONFIG], [gtest-config]) if test "x$GTESTCONFI

[Mesa-dev] [PATCH] configure.ac: use AC_PROG_SED

2012-01-10 Thread Matt Turner
AC_PROG_PATH was used in the original configure.ac (dca1b796b) and I can't see any reason AC_PROG_SED wasn't used. Signed-off-by: Matt Turner --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index c0abeb9..08a0209 100644 --- a

[Mesa-dev] [PATCH] configure.ac: add and use AC_PROG_BISON/FLEX macros

2012-01-10 Thread Matt Turner
Defining these macros lets us use AC_PROG_YACC/LEX which makes automake easier while still specifically requiring bison and flex. Based on bison.m4 and flex.m4 found in LLVM's tree. Signed-off-by: Matt Turner --- configure.ac |9 - m4/bison.m4 | 16 m4/flex.m4

[Mesa-dev] [PATCH] glx/dri2: fix use of uninitialized variable "api"

2012-01-10 Thread Vadim Girlin
Signed-off-by: Vadim Girlin --- Fixes unigine heaven startup problem for me. src/glx/dri2_glx.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c index 10b6f52..b4efa40 100644 --- a/src/glx/dri2_glx.c +++ b/src/glx/dri2_glx.c @@ -2

[Mesa-dev] [PATCH] mesa: Make ffs/ffsl conditions match popcount ones.

2012-01-10 Thread Alexander von Gluck
- Ensure mesa code uses the _mesa_ffs(l) functions consistantly instead of jumping around between ffs and _mesa_ffs - This makes ffs/ffsl behave more like the popcount code - Better detects and handles edge cases of missing ffs/ffsl - Use builtin ffs/ffsl more often as it may provide perfor

Re: [Mesa-dev] [PATCH] mesa: fix situations where ffs or ffsl may not be defined

2012-01-10 Thread Alexander von Gluck
On 10.01.2012 21:53, Brian Paul wrote: On Mon, Jan 9, 2012 at 2:39 PM, Alexander von Gluck wrote: ---  src/mesa/main/imports.h |   16  1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/src/mesa/main/imports.h b/src/mesa/main/imports.h index b7e8743..522ffeb 10064

Re: [Mesa-dev] [PATCH] mesa: fix situations where ffs or ffsl may not be defined

2012-01-10 Thread Brian Paul
On Mon, Jan 9, 2012 at 2:39 PM, Alexander von Gluck wrote: > > --- >  src/mesa/main/imports.h |   16 >  1 files changed, 12 insertions(+), 4 deletions(-) > > diff --git a/src/mesa/main/imports.h b/src/mesa/main/imports.h > index b7e8743..522ffeb 100644 > --- a/src/mesa/main/import

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

2012-01-10 Thread Brian Paul
On Tue, Jan 3, 2012 at 8:59 PM, Yuanhan Liu wrote: > On Wed, Jan 04, 2012 at 11:20:07AM +0800, Yuanhan Liu wrote: >> On Tue, Jan 03, 2012 at 08:25:31PM +0100, Roland Scheidegger wrote: >> > Ah index scanning... >> > I don't like that this will map/unmap the ib once for each prim, >> Me either :) >

[Mesa-dev] [PATCH] st/dri: Remove useless flush front.

2012-01-10 Thread Stéphane Marchesin
In the following scenario: - CreateContext C1 - MakeCurrent C1 - DestroyContext C1 (does not actually destroy the first context, postponed until the next MakeCurrent) - CreateContext C2 - MakeCurrent C2 MakeCurrent will call flush on a context which might not even have had a front buffer, leadin

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

2012-01-10 Thread Yuanhan Liu
On Wed, Jan 04, 2012 at 07:23:24PM +0100, Roland Scheidegger wrote: > Am 04.01.2012 04:59, schrieb Yuanhan Liu: > > On Wed, Jan 04, 2012 at 11:20:07AM +0800, Yuanhan Liu wrote: > >> On Tue, Jan 03, 2012 at 08:25:31PM +0100, Roland Scheidegger wrote: > >>> Ah index scanning... > >>> I don't like tha

[Mesa-dev] [PATCH 3/3] xlib: use X error handler to catch XShmQueryVersion() failure

2012-01-10 Thread Brian Paul
From: Brian Paul This is a follow-on to the previous commits. It seems that XShmQueryVersion() can trigger an X error after the first X connection is closed and we start using a new connection. Fixes a bug found by Wayne E. Robertz. NOTE: This is a candidate for the 7.11 branch. --- src/mesa/

[Mesa-dev] [PATCH 2/3] st/glx/xlib: call register_with_display() in Fake_glXChooseFBConfig()

2012-01-10 Thread Brian Paul
From: Brian Paul This is the same fix as the previous commit, except it's for the gallium glx/xlib state tracker. NOTE: This is a candidate for the 7.11 branch. --- src/gallium/state_trackers/glx/xlib/glx_api.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/galli

[Mesa-dev] [PATCH 1/3] xlib: call register_with_display() in Fake_glXChooseFBConfig()

2012-01-10 Thread Brian Paul
From: Brian Paul as we do in Fake_glXChooseVisual(). This registers the MesaGLX extension on the display so we can clean up buffers, etc. when the display connection is closed. Fixes a bug reported by Wayne E. Robertz. NOTE: This is a candidate for the 7.11 branch. --- src/mesa/drivers/x11/fa

Re: [Mesa-dev] [PATCH] [rfc] mesa: don't allow GLSL 1.30 without GL3.

2012-01-10 Thread Christoph Bumiller
On 01/10/2012 10:09 PM, Dave Airlie wrote: > On Tue, Jan 10, 2012 at 7:28 PM, Eric Anholt wrote: >> On Tue, 10 Jan 2012 11:52:57 +, Dave Airlie wrote: >>> From: Dave Airlie >>> >>> Things can get confused if you expose one without the other which can happen >>> if you are missing one or two

Re: [Mesa-dev] [PATCH 1/2] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Dave Airlie
On Tue, Jan 10, 2012 at 7:27 PM, Marek Olšák wrote: > On Tue, Jan 10, 2012 at 7:48 PM, Jakob Bornecrantz wrote: >> - Original Message - >>> These initialization functions weren't initializing all the fields so >>> some had undefined values.  The callers of these functions sometimes >>> us

Re: [Mesa-dev] [PATCH] [rfc] mesa: don't allow GLSL 1.30 without GL3.

2012-01-10 Thread Dave Airlie
On Tue, Jan 10, 2012 at 7:28 PM, Eric Anholt wrote: > On Tue, 10 Jan 2012 11:52:57 +, Dave Airlie wrote: >> From: Dave Airlie >> >> Things can get confused if you expose one without the other which can happen >> if you are missing one or two of the extensions (like say float textures). > > C

Re: [Mesa-dev] [PATCH 0/3] i965/gen7: Enable HiZ

2012-01-10 Thread Eric Anholt
On Mon, 9 Jan 2012 14:30:55 -0800, Chad Versace wrote: > Chad Versace (3): > i965: Fix misnamed GEN7_WM_DEPTH_RESOLVE > i965: Replace references to stencil region size with buffer size > i965/gen7: Enable HiZ Reviewed-by: Eric Anholt pgpvhv76geS4j.pgp Description: PGP signature ___

Re: [Mesa-dev] [PATCH] mesa: fix cubemap depth completeness test

2012-01-10 Thread Eric Anholt
On Tue, 10 Jan 2012 16:14:05 +, Dave Airlie wrote: > From: Dave Airlie > > This fixes the test to allow cube/depth combinations on GL3 > or EXT_gpu_shader4. Reviewed-by: Eric Anholt pgpPxPvQlbXtC.pgp Description: PGP signature ___ mesa-dev mail

Re: [Mesa-dev] [PATCH 2/3] mesa: add _mesa_unpack_ubyte_rgba_row() function

2012-01-10 Thread Brian Paul
On 01/10/2012 11:36 AM, Eirik Byrkjeflot Anonsen wrote: Brian Paul writes: --- src/mesa/main/format_unpack.c | 499 + src/mesa/main/format_unpack.h |3 + 2 files changed, 502 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/format_unpac

Re: [Mesa-dev] [PATCH] [rfc] mesa: don't allow GLSL 1.30 without GL3.

2012-01-10 Thread Eric Anholt
On Tue, 10 Jan 2012 11:52:57 +, Dave Airlie wrote: > From: Dave Airlie > > Things can get confused if you expose one without the other which can happen > if you are missing one or two of the extensions (like say float textures). Can you clarify what "things can get confused" means? pgpptt

Re: [Mesa-dev] [PATCH 1/2] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 7:48 PM, Jakob Bornecrantz wrote: > - Original Message - >> These initialization functions weren't initializing all the fields so >> some had undefined values.  The callers of these functions sometimes >> use a structure assignment to initialize new objects from the

Re: [Mesa-dev] [PATCH 2/3] mesa: add _mesa_unpack_ubyte_rgba_row() function

2012-01-10 Thread Eirik Byrkjeflot Anonsen
Brian Paul writes: > --- > src/mesa/main/format_unpack.c | 499 > + > src/mesa/main/format_unpack.h |3 + > 2 files changed, 502 insertions(+), 0 deletions(-) > > diff --git a/src/mesa/main/format_unpack.c b/src/mesa/main/format_unpack.c > index c723

[Mesa-dev] [PATCH] i965/vs: Emit a MAC instead of a MUL+ADD combination when possible.

2012-01-10 Thread Eric Anholt
Fixes a regression in codegen quality from the old VS backend to the new one, where matrix multiplies were more expensive in instruction count. shader-db results: Total instructions: 58540 -> 55984 815/870 programs affected (93.7%) 57412 -> 54856 instructions in affected programs (4.5% reduction)

Re: [Mesa-dev] [PATCH 1/2] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Brian Paul
On 01/10/2012 11:48 AM, Jakob Bornecrantz wrote: - Original Message - These initialization functions weren't initializing all the fields so some had undefined values. The callers of these functions sometimes use a structure assignment to initialize new objects from these templates so we

[Mesa-dev] [PATCH] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Brian Paul
These initialization functions weren't initializing all the fields so some had undefined values. The callers of these functions sometimes use a structure assignment to initialize new objects from these templates so we'd just propagate the undefined values. That made for some confusing info when d

Re: [Mesa-dev] [PATCH 1/2] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Jakob Bornecrantz
- Original Message - > These initialization functions weren't initializing all the fields so > some had undefined values. The callers of these functions sometimes > use a structure assignment to initialize new objects from these > templates so we'd just propagate the undefined values. Tha

[Mesa-dev] [PATCH 2/2] st/mesa: remove unnecessary memset()/zero call

2012-01-10 Thread Brian Paul
The u_surface_default_template() function does the memset()/zero now. --- src/mesa/state_tracker/st_manager.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/mesa/state_tracker/st_manager.c b/src/mesa/state_tracker/st_manager.c index c0af3ce..b83cb23 100644 --- a/src/

[Mesa-dev] [PATCH 1/2] util: use memset() to initialize surface, sampler_view templates

2012-01-10 Thread Brian Paul
These initialization functions weren't initializing all the fields so some had undefined values. The callers of these functions sometimes use a structure assignment to initialize new objects from these templates so we'd just propagate the undefined values. That made for some confusing info when d

[Mesa-dev] [Bug 44561] Piglit: spec/glsl-1.10/execution/interpolation/interpolation-none-gl_*-vertex fail

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44561 --- Comment #5 from Stephane Marchesin 2012-01-10 10:33:00 PST --- I don't think it's a bug in i915g proper, since we don't handle vertex shaders in there, or any vertex stuff for that matter, but instead rely on draw. I'll try Dave'd draw patch

[Mesa-dev] [Bug 44561] Piglit: spec/glsl-1.10/execution/interpolation/interpolation-none-gl_*-vertex fail

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44561 Christoph Bumiller changed: What|Removed |Added Keywords|regression | --- Comment #4 from Christoph Bumi

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - > On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca > wrote: > > - Original Message - > >> The flag is optional, it doesn't have to implemented by everybody. > >> If > >> we do the uploads in the state tracker, we will also do any > >> required > >> data transfor

[Mesa-dev] [Bug 44561] Piglit: spec/glsl-1.10/execution/interpolation/interpolation-none-gl_*-vertex fail

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44561 --- Comment #3 from Kai 2012-01-10 09:11:19 PST --- (In reply to comment #2) So do I understand you correctly, that this bug is then no regression for r600g and i915g (I'm seeing this on r600g)? If so, the "regression" keyword should probably be

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca wrote: > - Original Message - >> The flag is optional, it doesn't have to implemented by everybody. If >> we do the uploads in the state tracker, we will also do any required >> data transformation so that drivers don't have to do it at all. >

[Mesa-dev] [Bug 44564] Fix build with LLVM -DLLVM_TARGETS_TO_BUILD=`native`

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44564 --- Comment #3 from Vinson Lee 2012-01-10 08:56:58 PST --- (In reply to comment #2) > Anyway, Mesa doesn't need LLVM libraries except in `$LLVM_CONFIG --libs > engine` > to build/run? I tried the SCons build using 'llvm-config --libs engine' a

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - > On 01/10/2012 01:13 PM, Jose Fonseca wrote: > > - Original Message - > >> On 10.01.2012 12:29, Jose Fonseca wrote: > >>> Still catching up on email traffic during holidays... > >>> > >>> I agree that user buffer uploads should be moved out of drivers, > >>>

[Mesa-dev] [Bug 44561] Piglit: spec/glsl-1.10/execution/interpolation/interpolation-none-gl_*-vertex fail

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44561 --- Comment #2 from Christoph Bumiller 2012-01-10 08:26:03 PST --- Like I already mentioned on ML, i915g isn't using the ClipVertex for the clipping calculation, but Position (ClipVertex didn't exist before, thus whenever it was not equal to gl_

Re: [Mesa-dev] [PATCH 1/2] st_extensions: fixup GLSL 1.30 related enables

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 12:49 PM, Dave Airlie wrote: > From: Dave Airlie > > This just fixes up the enables for native integers and EXT_texture_integer > support in st/mesa. > > It also set the MaxClipPlanes to 8. > > We should consider exposing caps for MCP vs MCD, but since core > mesa doesn't

[Mesa-dev] [PATCH] mesa: fix cubemap depth completeness test

2012-01-10 Thread Dave Airlie
From: Dave Airlie This fixes the test to allow cube/depth combinations on GL3 or EXT_gpu_shader4. Signed-off-by: Dave Airlie --- src/mesa/main/texobj.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/mesa/main/texobj.c b/src/mesa/main/texobj.c index 7ee2005.

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Christoph Bumiller
On 01/10/2012 01:13 PM, Jose Fonseca wrote: > - Original Message - >> On 10.01.2012 12:29, Jose Fonseca wrote: >>> Still catching up on email traffic during holidays... >>> >>> I agree that user buffer uploads should be moved out of drivers, >>> but I don't think this is the way to go. >> >

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - > On Tue, Jan 10, 2012 at 12:29 PM, Jose Fonseca > wrote: > > Still catching up on email traffic during holidays... > > > > I agree that user buffer uploads should be moved out of drivers, > > but I don't think this is the way to go. > > > > This "PIPE_TRANSFER_MAP_PER

Re: [Mesa-dev] softpipe GL3 status

2012-01-10 Thread Brian Paul
On 01/10/2012 09:06 AM, Dave Airlie wrote: Something in Gallium is probably deciding that GL_DEPTH_COMPONENT is invalid for a cubemap and marking it as incomplete. Hmmm, I'm not sure what would be preventing cube textures in gallium or the state tracker. Something in Mesa is doing it, texo

Re: [Mesa-dev] softpipe GL3 status

2012-01-10 Thread Dave Airlie
>> >> Something in Gallium is probably deciding that GL_DEPTH_COMPONENT is >> invalid for a cubemap and marking it as incomplete. > > > Hmmm, I'm not sure what would be preventing cube textures in gallium or the > state tracker. Something in Mesa is doing it, texobj.c:_mesa_test_texobj_completene

Re: [Mesa-dev] [PATCH 3/3] vl: replace decode_buffers with auxiliary data field

2012-01-10 Thread Christian König
Hi Maarten, On 08.01.2012 14:09, Maarten Lankhorst wrote: Ok. I wont block it any more then. Do you have a followup patch to get rid of quant matrix and separate reference pictures too? Or should I resend it, I kind of lost my rebased git tree, so if you already have the followup patches it wo

[Mesa-dev] [PATCH 9/9] vl: fix YV12 handling

2012-01-10 Thread Christian König
We actually implemented YV21 instead of YV12, so fix the plane ordering. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 13 +--- src/gallium/auxiliary/vl/vl_video_buffer.c | 37 +++-- src/gallium/auxiliary/vl/vl_video_buffer.h |

[Mesa-dev] [PATCH 8/9] st/vdpau: recreate video buffer if decode doesn't like it

2012-01-10 Thread Christian König
Recreate the video buffer if the decoder can't handle it. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/decode.c | 22 +- 1 files changed, 21 insertions(+), 1 deletions(-) diff --git a/src/gallium/state_trackers/vdpau/decode.c b/src/gallium/state_tra

[Mesa-dev] [PATCH 7/9] st/vdpau: recreate video buffer if format doesn't match

2012-01-10 Thread Christian König
Recreate the video buffer in PutBitsYCbCr if the format doesn't match. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/surface.c | 54 ++--- src/gallium/state_trackers/vdpau/vdpau_private.h |2 +- 2 files changed, 37 insertions(+), 19 deletions(-)

[Mesa-dev] [PATCH 6/9] vl: reintroduce PIPE_VIDEO_CAP_PREFERED_FORMAT

2012-01-10 Thread Christian König
Create the video buffers in the format the driver preffers. This temporary creates problems with decoder less VDPAU video playback. Signed-off-by: Christian König --- src/gallium/drivers/nouveau/nouveau_video.c|2 ++ src/gallium/drivers/nvfx/nvfx_screen.c |2 ++ src/gallium/d

[Mesa-dev] [PATCH 5/9] vl: add h264 infrastructure

2012-01-10 Thread Christian König
No implementation so far, just the defines for VDPAUs picture info structure. Signed-off-by: Christian König --- src/gallium/include/pipe/p_video_state.h | 40 +++ src/gallium/state_trackers/vdpau/decode.c | 61 + 2 files changed, 101 insertions(+

[Mesa-dev] [PATCH 2/9] vl/video_buffer: improve constructor

2012-01-10 Thread Christian König
Add a second extened constructor that takes plane textures for the video buffer. Also provide a function for texture templates. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 112 +--- src/gallium/auxiliary/vl/vl_video_buffer.h | 17 +++

[Mesa-dev] [PATCH 1/9] vl/video_buffer: use template style create params

2012-01-10 Thread Christian König
Just like in the rest of gallium, this reduces the number of parameters significantly. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 27 ++ src/gallium/auxiliary/vl/vl_video_buffer.c | 47 ++-- src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 3/9] vl/video_buffer: add support for interlaced buffers

2012-01-10 Thread Christian König
Add the infrastructure, but not the decode implementation. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 35 ++- src/gallium/auxiliary/vl/vl_video_buffer.h |2 +- src/gallium/include/pipe/p_video_decoder.h |1 + 3 files changed,

Re: [Mesa-dev] [PATCH 03/12] rbug: Silence warning

2012-01-10 Thread Jakob Bornecrantz
On Tue, Jan 10, 2012 at 2:40 AM, Jakob Bornecrantz wrote: > On Tue, Jan 10, 2012 at 12:01 AM, Alan Coopersmith > wrote: >> On 01/ 9/12 02:53 PM, Jakob Bornecrantz wrote: >>> >>> -   struct pipe_shader_state pss = { 0 }; >>> +   struct pipe_shader_state pss; >>> +   memset(&pss, 0, sizeof(pss)); >

Re: [Mesa-dev] [PATCH 4/4] softpipe: clipdistance support (v1)

2012-01-10 Thread Brian Paul
The check-in message should probably be "draw: clipdistance support (v1)", not "softpipe". On 01/10/2012 06:55 AM, Dave Airlie wrote: From: Dave Airlie Add support for using the clipdistance instead of clip plane. Passes all piglit clipdistance tests. Signed-off-by: Dave Airlie --- src/ga

Re: [Mesa-dev] [PATCH 06/12] mesa: Ignores

2012-01-10 Thread Jakob Bornecrantz
2012/1/10 Michel Dänzer : > On Mon, 2012-01-09 at 23:53 +0100, Jakob Bornecrantz wrote: >> Signed-off-by: Jakob Bornecrantz >> --- >>  src/mesa/.gitignore |    1 + >>  1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/src/mesa/.gitignore b/src/mesa/.gitignore >> index ce83eaf..0c

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 12:29 PM, Jose Fonseca wrote: > Still catching up on email traffic during holidays... > > I agree that user buffer uploads should be moved out of drivers, but I don't > think this is the way to go. > > This "PIPE_TRANSFER_MAP_PERMANENTLY" means the driver relinquishes the

[Mesa-dev] [PATCH] r600g: add support for virtual address space on cayman v10

2012-01-10 Thread j . glisse
From: Jerome Glisse Virtual address space put the userspace in charge of their GPU address space. It's up to userspace to bind bo into the virtual address space. Command stream can them be executed using the IB_VM chunck. This patch add support for this configuration. It doesn't remove the 64K i

Re: [Mesa-dev] Mesa (master): gallium: introduce GLSL based interpolation rules. (v2)

2012-01-10 Thread Brian Paul
On 01/10/2012 08:34 AM, Brian Paul wrote: On 01/10/2012 04:56 AM, Dave Airlie wrote: Module: Mesa Branch: master Commit: 67e3cbf1632e361220234013147331e4618b70cb URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=67e3cbf1632e361220234013147331e4618b70cb Author: Dave Airlie Date: Mon Jan 9 1

Re: [Mesa-dev] Mesa (master): gallium: introduce GLSL based interpolation rules. (v2)

2012-01-10 Thread Brian Paul
On 01/10/2012 04:56 AM, Dave Airlie wrote: Module: Mesa Branch: master Commit: 67e3cbf1632e361220234013147331e4618b70cb URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=67e3cbf1632e361220234013147331e4618b70cb Author: Dave Airlie Date: Mon Jan 9 15:57:02 2012 + gallium: introduc

Re: [Mesa-dev] [PATCH 3/4] tgsi_scan: add support to count number of output clip distances

2012-01-10 Thread Jose Fonseca
Dave, - Original Message - > From: Dave Airlie > > Just add support to the scanner to count the number of clip > distances. > > Signed-off-by: Dave Airlie > --- > src/gallium/auxiliary/tgsi/tgsi_scan.c |4 > src/gallium/auxiliary/tgsi/tgsi_scan.h |1 + > 2 files changed,

Re: [Mesa-dev] [PATCH] mesa/clear: fix crashes with illegal clear tests.

2012-01-10 Thread Jose Fonseca
- Original Message - > From: Dave Airlie > > Mesa shouldn't call into the drivers if there are no renderbuffers > bound to the attachments for the buffers to be cleared. > > Fixes a number of the clearbuffer-* tests on softpipe. > > Signed-off-by: Dave Airlie > --- > src/mesa/main/c

[Mesa-dev] [PATCH 4/4] softpipe: clipdistance support (v1)

2012-01-10 Thread Dave Airlie
From: Dave Airlie Add support for using the clipdistance instead of clip plane. Passes all piglit clipdistance tests. Signed-off-by: Dave Airlie --- src/gallium/auxiliary/draw/draw_cliptest_tmp.h | 25 ++-- src/gallium/auxiliary/draw/draw_context.c |6 src/gallium/

[Mesa-dev] [PATCH 3/4] tgsi_scan: add support to count number of output clip distances

2012-01-10 Thread Dave Airlie
From: Dave Airlie Just add support to the scanner to count the number of clip distances. Signed-off-by: Dave Airlie --- src/gallium/auxiliary/tgsi/tgsi_scan.c |4 src/gallium/auxiliary/tgsi/tgsi_scan.h |1 + 2 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/galliu

[Mesa-dev] [PATCH 2/4] draw/softpipe: add clip vertex support. (v2)

2012-01-10 Thread Dave Airlie
From: Dave Airlie softpipe always clipped using the position vector, however for unclipped vertices it stored the position in window coordinates, however when position and clipping are separated, we need to store the clip-space position and the clip-space vertex clip, so we can interpolate both s

[Mesa-dev] [PATCH 1/4] tgsi/softpipe: add VertexID support.

2012-01-10 Thread Dave Airlie
From: Dave Airlie This required changing the system value semantics, so we stored a system value per vertex, instance id is the only other system value we currently support, so I span it across the channels. This passes the 3 vertexid-* piglit tests + lots of instanceid tests. Signed-off-by: Da

[Mesa-dev] [PATCH] softpipe: allow softpipe to set shader params depending on runtime llvm (v2)

2012-01-10 Thread Dave Airlie
From: Dave Airlie If draw isn't using llvm we can support vertex texture and integers, These will be fixed up later, but for now allow this check to happen at run-time. v2: since 3e22c7a25321554a32fa6254485912fd53deff3a we can ask draw for a non-llvm context. Just track if ask and set the vars

Re: [Mesa-dev] [PATCH 2/2] draw/softpipe: allow softpipe to set shader params depending on runtime llvm

2012-01-10 Thread Dave Airlie
On Tue, Jan 10, 2012 at 1:08 PM, Jakob Bornecrantz wrote: > On Tue, Jan 10, 2012 at 12:49 PM, Dave Airlie wrote: >> From: Dave Airlie >> >> If draw isn't using llvm we can support vertex texture and integers, >> These will be fixed up later, but for now allow this check to happen >> at run-time.

Re: [Mesa-dev] [PATCH 2/2] draw/softpipe: allow softpipe to set shader params depending on runtime llvm

2012-01-10 Thread Jakob Bornecrantz
On Tue, Jan 10, 2012 at 12:49 PM, Dave Airlie wrote: > From: Dave Airlie > > If draw isn't using llvm we can support vertex texture and integers, > These will be fixed up later, but for now allow this check to happen > at run-time. With my draw_create_no_llvm change this logic no longer holds tr

[Mesa-dev] [PATCH] mesa/clear: fix crashes with illegal clear tests.

2012-01-10 Thread Dave Airlie
From: Dave Airlie Mesa shouldn't call into the drivers if there are no renderbuffers bound to the attachments for the buffers to be cleared. Fixes a number of the clearbuffer-* tests on softpipe. Signed-off-by: Dave Airlie --- src/mesa/main/clear.c | 15 +++ 1 files changed, 11

Re: [Mesa-dev] Mesa (master): st/mesa: add support for clip vertex.

2012-01-10 Thread Kai Wasserbäch
Dear Stéphane, Stéphane Marchesin schrieb am 10.01.2012 04:52: > This regresses interpolation-none-gl_FrontColor-flat-vertex.shader_test > piglit test on i915g. this looks like bz#44561 [0] to me. If so we should take the discussion there to have it in one place. Kind regards, Kai Wasserbäch [0

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - > On 10.01.2012 12:29, Jose Fonseca wrote: > > Still catching up on email traffic during holidays... > > > > I agree that user buffer uploads should be moved out of drivers, > > but I don't think this is the way to go. > > I don't. If the state tracker uploads user buf

[Mesa-dev] [PATCH] [rfc] mesa: don't allow GLSL 1.30 without GL3.

2012-01-10 Thread Dave Airlie
From: Dave Airlie Things can get confused if you expose one without the other which can happen if you are missing one or two of the extensions (like say float textures). Signed-off-by: Dave Airlie --- src/mesa/main/version.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --

[Mesa-dev] [PATCH 1/2] st_extensions: fixup GLSL 1.30 related enables

2012-01-10 Thread Dave Airlie
From: Dave Airlie This just fixes up the enables for native integers and EXT_texture_integer support in st/mesa. It also set the MaxClipPlanes to 8. We should consider exposing caps for MCP vs MCD, but since core mesa doesn't care yet maybe we can wait for now. Signed-off-by: Dave Airlie ---

[Mesa-dev] [PATCH 2/2] draw/softpipe: allow softpipe to set shader params depending on runtime llvm

2012-01-10 Thread Dave Airlie
From: Dave Airlie If draw isn't using llvm we can support vertex texture and integers, These will be fixed up later, but for now allow this check to happen at run-time. Signed-off-by: Dave Airlie --- src/gallium/auxiliary/draw/draw_context.c | 10 ++ src/gallium/auxiliary/draw/draw_c

[Mesa-dev] [PATCH 3/4] nvfx: drop render temporaries code

2012-01-10 Thread Lucas Stach
This code is unneeded now, we don't use render temps any more. Signed-off-by: Lucas Stach --- src/gallium/drivers/nvfx/nvfx_fragtex.c |4 - src/gallium/drivers/nvfx/nvfx_miptree.c | 18 +--- src/gallium/drivers/nvfx/nvfx_resource.h | 30 +-- src/gallium/drivers/nvfx/nvfx_surface.c

[Mesa-dev] [PATCH 2/4] nvfx: rework state_fb code to get rid of render temps

2012-01-10 Thread Lucas Stach
This commit rewrites a lot of the state_fb code to support rendering to targets not aligned to 64 byte. This allows us to drop the render temporaries as unaligned targets are the only use-case where they are really needed. The temporaries code was used for a lot of things more, but apparently thos

[Mesa-dev] [PATCH 4/4] nvfx: random cleanups of the state validation code

2012-01-10 Thread Lucas Stach
Signed-off-by: Lucas Stach --- src/gallium/drivers/nvfx/nvfx_state_emit.c | 49 --- 1 files changed, 22 insertions(+), 27 deletions(-) diff --git a/src/gallium/drivers/nvfx/nvfx_state_emit.c b/src/gallium/drivers/nvfx/nvfx_state_emit.c index e2cfb76..a959015 100644 ---

[Mesa-dev] [PATCH 1/4] nvfx: say no to stream output caps

2012-01-10 Thread Lucas Stach
nvfx doesn't support any kind of stream out, so silence the unused cap warnings. Signed-off-by: Lucas Stach --- src/gallium/drivers/nvfx/nvfx_screen.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/src/gallium/drivers/nvfx/nvfx_screen.c b/src/gallium/drivers/nvfx/

[Mesa-dev] [PATCH 0/4] nvfx: rework render temps code and fixes

2012-01-10 Thread Lucas Stach
This patch series silences some unknown cap warnings and fixes up coding style (patch 1+4). The most important part of this series are the two patches in the middle. They rework the state_fb code, so that we are able to render to not 64 byte aligned targets, as this is the only re

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Christoph Bumiller
On 10.01.2012 12:29, Jose Fonseca wrote: > Still catching up on email traffic during holidays... > > I agree that user buffer uploads should be moved out of drivers, but I don't > think this is the way to go. I don't. If the state tracker uploads user buffers and presents them to the driver as n

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. This "PIPE_TRANSFER_MAP_PERMANENTLY" means the driver relinquishes the ability to transform this data in any way before reashing the G

Re: [Mesa-dev] Mesa (master): st/mesa: add support for clip vertex.

2012-01-10 Thread Christoph Bumiller
On 10.01.2012 04:52, Stéphane Marchesin wrote: > Hi Dave, > > This regresses interpolation-none-gl_FrontColor-flat-vertex.shader_test > piglit test on i915g. That's likely. You're probably not using the ClipVertex but instead gl_Position for clipping since the prior was not identifiable before. T

[Mesa-dev] [Bug 44564] Fix build with LLVM -DLLVM_TARGETS_TO_BUILD=`native`

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44564 --- Comment #2 from ojab 2012-01-10 02:44:58 PST --- Anyway, Mesa doesn't need LLVM libraries except in `$LLVM_CONFIG --libs engine` to build/run? -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiv

[Mesa-dev] [Bug 44618] New: Cross-compilation broken by glsl builtin_compiler

2012-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44618 Bug #: 44618 Summary: Cross-compilation broken by glsl builtin_compiler Classification: Unclassified Product: Mesa Version: git Platform: Other OS/Version: All Status:

Re: [Mesa-dev] [PATCH 00/14] Fix some linker stuff, kill one more user of ir_to_mesa

2012-01-10 Thread Kenneth Graunke
On 01/06/2012 04:49 PM, Ian Romanick wrote: Patches 01/14 through 03/14 was previously sent to the list. It tickled a pre-existing bug (fixed by patch recently posted to the list by Eric) on some Intel chipsets. Patches 04/14 through 07/14 make the linker do some sampler tracking that was previ

Re: [Mesa-dev] [PATCH 09/14] i965: Remove brw_fs_precompile

2012-01-10 Thread Kenneth Graunke
On 01/09/2012 05:26 PM, Eric Anholt wrote: On Fri, 6 Jan 2012 16:50:00 -0800, "Ian Romanick" wrote: From: Ian Romanick There are several things that could cause the fragment shader precompile to fail. Looking for calls to 'fail' in brw_fs_visitor.cpp finds three classes of things: * Tryi