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

2011-08-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40448 Summary: Unigine Sanctuary: window disappear when changing settings Product: Mesa Version: git Platform: Other URL: http://unigine.com/products/sanctuary/ OS/Ver

Re: [Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-29 Thread Christian König
Am Montag, den 29.08.2011, 00:36 -0400 schrieb Younes Manton: [snip] > Well, that was what the last discussion was all about, whether or not > decode buffers should be handled internally by each driver or > externally. It was decided to handle it externally, to simplify life > for drivers that want

Re: [Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

2011-08-29 Thread Christian König
Am Sonntag, den 28.08.2011, 19:13 +0200 schrieb Maarten Lankhorst: > Hey, > > On 08/28/2011 05:04 PM, Christian König wrote: > > Hi Maarten, > > > > could you put this into SetDecoderStatus instead? This makes the picture > > structure also available in end_frame. > Erm it doesn't make sense there

Re: [Mesa-dev] [PATCH] winsys/radeon: Create async thread only once

2011-08-29 Thread Christian König
I haven't really tested this, so please speak up if anybody finds any regression, but I'm going to push this if there are no objections. Beside being a real killer for performance, the whole "[New Thread 0x7fffe60fb700 (LWP 5212)]/[Thread 0x7fffe60fb700 (LWP 5212) exited]" in gdb is quite annoying

Re: [Mesa-dev] [PATCH 7/7] g3dvl: Rewrite the mpeg 1&2 bitstream parser

2011-08-29 Thread Christian König
Am Sonntag, den 28.08.2011, 18:55 +0100 schrieb Andy Furniss: > deathsim...@vodafone.de wrote: > > From: Christian König > > > > Based on work of Maarten Lankhorst this time. > > I am getting new artifacts on r600 and softpipe (ignoring all the other > issues it has) when using -vc ffmpeg12vdpau,

[Mesa-dev] [Bug 40401] Cogs is slow

2011-08-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40401 --- Comment #11 from Tobias Jakobi 2011-08-29 02:03:49 PDT --- (In reply to comment #10) > But the problem maybe is in the same component for the Intel drivers side. i965 is not using gallium, so no. -- Configure bugmail: https://bugs.freedesk

Re: [Mesa-dev] [PATCH] winsys/radeon: Create async thread only once

2011-08-29 Thread Michel Dänzer
On Sam, 2011-08-27 at 22:57 +0200, Maarten Lankhorst wrote: > @@ -402,33 +420,33 @@ static void radeon_drm_cs_flush(struct radeon_winsys_cs > *rcs, unsigned flags) > > radeon_drm_cs_sync_flush(cs); > > +/* Flip command streams. */ > +tmp = cs->csc; > +cs->csc = cs->cst; > +

Re: [Mesa-dev] OpenGL to DirectX with Mesa3D

2011-08-29 Thread Jose Fonseca
- Original Message - > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 08/25/2011 10:55 AM, Anonimo Veneziano wrote: > > Hi all > > I am a programmer but not very skilled in 3D graphics (so pardon me > > if I > > write something wrong). > > I need to build my own opengl32.dll to r

[Mesa-dev] [PATCH] r600g: Make unaligned 3D textures work on +evergreen

2011-08-29 Thread deathsimple
From: Christian König The layersize calculation is slightly different on +evergreen. This makes mpeg2 video decoding and piglits texture-packed-formats test work correctly on this hardware. --- src/gallium/drivers/r600/r600_texture.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-

[Mesa-dev] [Bug 40401] Cogs is slow

2011-08-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40401 --- Comment #12 from imamdxl8...@gmail.com 2011-08-29 04:54:26 PDT --- (In reply to comment #11) > (In reply to comment #10) > > But the problem maybe is in the same component for the Intel drivers side. > i965 is not using gallium, so no. Sorry,

Re: [Mesa-dev] [PATCH] driconf: Added german translation

2011-08-29 Thread Lauri Kasanen
On Sun, 28 Aug 2011 14:55:56 +0200 Carl-Philip Haensch wrote: > > It looks to me you wrote the translations in the result file; the > > Makefile says you should be editing the .po file, and options.h > > would autogenerate from that. > > > > - Lauri > > > > Whats the proper tool to auto gen

[Mesa-dev] [PATCH 1/2] tgsi: add support for texture offsets to the TGSI IR.

2011-08-29 Thread Dave Airlie
From: Dave Airlie This adds tokens for texture offsets, to store 4 * swizzled vec 3 for use in TXF and other opcodes. It also contains TGSI exec changes for softpipe to use this code, along with GLSL->TGSI support for TXF. Signed-off-by: Dave Airlie --- src/gallium/auxiliary/tgsi/tgsi_build.c

[Mesa-dev] [PATCH 2/2] gallium: add caps for MIN/MAX texel offsets.

2011-08-29 Thread Dave Airlie
From: Dave Airlie As per Brian's suggestion, add caps for drivers that support texture offsets to advertise a min/max via TGSI, also use it in the state tracker. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/r600_pipe.c |6 ++ src/gallium/drivers/softpipe/sp_screen.c |

Re: [Mesa-dev] [PATCH 00/13] Fix include style accross Mesa

2011-08-29 Thread Brian Paul
On 08/27/2011 09:51 AM, Kai Wasserbäch wrote: [Please CC me on all replies, I'm not subscribed to mesa-dev.] Dear mesa-dev list, the following 13 patches were prompted by Brian Paul's e-mail [0] about g3dvl. Neither patch should have functional impact and thus be safe to apply. The first patch

Re: [Mesa-dev] OpenGL to DirectX with Mesa3D

2011-08-29 Thread Anonimo Veneziano
Thanks all for your answers and suggestions! I will have a try on the possible solutions/ways/components you all cited. Otherwise I think I will have to wait ANGLE to grow up and cover more features. By the way, ANGLE is OpenGL ES -> DX not OpenGL, so I think I need also a OpenGL -> OpenGL ES tr

Re: [Mesa-dev] split-up maptextureimage series

2011-08-29 Thread Brian Paul
On 08/24/2011 12:08 PM, Eric Anholt wrote: OK, that last patchbomb had some issues. For sanity, I'd like to get some of the non-controversial stuff in before polishing off the remainder. This is the part of my branch not related to GetTexImage(). It's also the stuff that should be reasonably w

Re: [Mesa-dev] [PATCH 2/2] mesa: Remove all mention of MESA_FORMAT_CI8

2011-08-29 Thread Brian Paul
On 08/27/2011 12:47 AM, Ian Romanick wrote: From: Ian Romanick Nothing in Mesa supports color-index textures, and most of the other infrastructure that could allow such support has already been removed. This puts the final nail in the coffin. Reviewed-by: Brian Paul Just one small thing: if

Re: [Mesa-dev] [PATCH 2/2] mesa: Remove all mention of MESA_FORMAT_CI8

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 07:38 AM, Brian Paul wrote: > On 08/27/2011 12:47 AM, Ian Romanick wrote: >> From: Ian Romanick >> >> Nothing in Mesa supports color-index textures, and most of the other >> infrastructure that could allow such support has already been re

Re: [Mesa-dev] llvm-config on a biarch machine

2011-08-29 Thread Brian Paul
On 08/24/2011 09:34 AM, Kevin DeKorte wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On a biarch machine like Fedora llvm-config defaults to outputting the values for a 64bit build. With a recent patch to llvm-devel there are now arch specific versions of llvm-config and this patch should

Re: [Mesa-dev] [PATCH] mesa, glsl_to_tgsi: fixes for native integers

2011-08-29 Thread Eric Anholt
On Fri, 26 Aug 2011 21:58:12 -0500, Bryan Cain wrote: > This fixes all of the piglit regressions in softpipe when native integers are > enabled. > --- > src/mesa/main/uniforms.c |8 + > src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 45 > ++-- >

Re: [Mesa-dev] [PATCH] Initial Common Subexpression Elimination implementation.

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/17/2011 12:29 PM, vljn wrote: > +class box:public exec_node For the record, I'm going to NAK with extreme prejudice any patch that includes this crap. I'm not trying to be a jerk, but I've explained at least twice why this is unacceptable. >

Re: [Mesa-dev] [PATCH] Initial Common Subexpression Elimination implementation.

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 10:02 AM, Ian Romanick wrote: > On 07/17/2011 12:29 PM, vljn wrote: > >> +class box:public exec_node > > For the record, I'm going to NAK with extreme prejudice any patch that > includes this crap. I'm not trying to be a jerk, but I've

Re: [Mesa-dev] [PATCH] mesa, glsl_to_tgsi: fixes for native integers

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 09:31 AM, Eric Anholt wrote: > On Fri, 26 Aug 2011 21:58:12 -0500, Bryan Cain wrote: >> This fixes all of the piglit regressions in softpipe when native integers are >> enabled. >> --- >> src/mesa/main/uniforms.c |

Re: [Mesa-dev] [PATCH] mesa, glsl_to_tgsi: fixes for native integers

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/27/2011 08:44 AM, Bryan Cain wrote: > On 08/27/2011 05:39 AM, Christoph Bumiller wrote: >> On 27.08.2011 04:58, Bryan Cain wrote: >>> This fixes all of the piglit regressions in softpipe when native integers >>> are >>> enabled. >>> --- >>> src

Re: [Mesa-dev] split-up maptextureimage series

2011-08-29 Thread Eric Anholt
On Mon, 29 Aug 2011 08:32:41 -0600, Brian Paul wrote: Non-text part: multipart/mixed > On 08/24/2011 12:08 PM, Eric Anholt wrote: > > OK, that last patchbomb had some issues. For sanity, I'd like to get > > some of the non-controversial stuff in before polishing off the > > remainder. This is th

Re: [Mesa-dev] [PATCH] glsl: Use a separate div_to_mul_rcp lowering flag for integers.

2011-08-29 Thread Eric Anholt
On Sun, 28 Aug 2011 20:51:10 -0500, Bryan Cain wrote: > On 08/28/2011 07:38 PM, Eric Anholt wrote: > > On Sat, 27 Aug 2011 20:18:55 -0700, Kenneth Graunke > > wrote: > >> From: Bryan Cain > >> > >> Using multiply and reciprocal for integer division involves potentially > >> lossy floating point

Re: [Mesa-dev] [PATCH] glsl: Use a separate div_to_mul_rcp lowering flag for integers.

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/28/2011 06:51 PM, Bryan Cain wrote: > On 08/28/2011 07:38 PM, Eric Anholt wrote: >> On Sat, 27 Aug 2011 20:18:55 -0700, Kenneth Graunke >> wrote: >>> From: Bryan Cain >>> >>> Using multiply and reciprocal for integer division involves potentia

[Mesa-dev] [PATCH v3] glsl: Use a separate div_to_mul_rcp lowering flag for integers.

2011-08-29 Thread Kenneth Graunke
From: Bryan Cain Using multiply and reciprocal for integer division involves potentially lossy floating point conversions. This is okay for older GPUs that represent integers as floating point, but undesirable for GPUs with native integer division instructions. TGSI, for example, has UDIV/IDIV

Re: [Mesa-dev] [PATCH 15/21] mesa: Enable vertex programs by default, remove checks

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/27/2011 05:39 AM, Marek Olšák wrote: > Hi Ian, > > I was trying to enable NV_vertex_program without regressions in > st/mesa before, but these piglit tests were always failing and are still > failing with your patch: > > nv-add > nv-arl > nv-in

Re: [Mesa-dev] [PATCH v3] glsl: Use a separate div_to_mul_rcp lowering flag for integers.

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 10:52 AM, Kenneth Graunke wrote: > From: Bryan Cain > > Using multiply and reciprocal for integer division involves potentially > lossy floating point conversions. This is okay for older GPUs that > represent integers as floating point

[Mesa-dev] [PATCH 1/3] mesa: Remove API facing bits of EXT_paletted_texture and EXT_shared_texture_palette

2011-08-29 Thread Ian Romanick
From: Ian Romanick This was also discussed at XDS 2010. However, actually making the change was delayed because several drivers still exposed these extensions to significant benefit (e.g., tdfx). Now that those drivers have been removed, this code can be removed as well. --- src/mesa/main/colo

[Mesa-dev] [PATCH 2/3] mesa: Remove dd_function_table::CopyColorTable, ::CopyColorSubTable, and ::UpdateTexturePalette

2011-08-29 Thread Ian Romanick
From: Ian Romanick There's nothing left that can call any of these functions. This also removes the meta-ops code that implemented the first two. --- src/mesa/drivers/common/driverfuncs.c |5 --- src/mesa/drivers/common/meta.c| 55 - src/mesa/driver

[Mesa-dev] [PATCH 3/3] mesa: Remove all mention of GL_COLOR_INDEX*_EXT

2011-08-29 Thread Ian Romanick
From: Ian Romanick These enums were only valid with the paletted texture extensions. This allows a couple other trivial clean-ups. --- src/mesa/main/image.c | 27 --- src/mesa/main/image.h |3 --- src/mesa/main/texgetimage.c |8 ++-- 3 files chan

Re: [Mesa-dev] [PATCH 1/3] mesa: Remove API facing bits of EXT_paletted_texture and EXT_shared_texture_palette

2011-08-29 Thread Brian Paul
On 08/29/2011 12:12 PM, Ian Romanick wrote: From: Ian Romanick This was also discussed at XDS 2010. However, actually making the change was delayed because several drivers still exposed these extensions to significant benefit (e.g., tdfx). Now that those drivers have been removed, this code ca

Re: [Mesa-dev] [PATCH 1/3] mesa: Remove API facing bits of EXT_paletted_texture and EXT_shared_texture_palette

2011-08-29 Thread Brian Paul
On 08/29/2011 12:26 PM, Brian Paul wrote: On 08/29/2011 12:12 PM, Ian Romanick wrote: From: Ian Romanick This was also discussed at XDS 2010. However, actually making the change was delayed because several drivers still exposed these extensions to significant benefit (e.g., tdfx). Now that thos

[Mesa-dev] [PATCH 0/3] make: A few small cleanups

2011-08-29 Thread Chad Versace
Chad Versace (3): make: Remove duplicate defintion of COMMON_SOURCES in Radeon makefiles make: Remove duplicate occurence of driverfuncs.c make: Factor out source lists from drivers/dri/common into Makefile.sources src/mesa/drivers/dri/Makefile.defines| 21 +++-

[Mesa-dev] [PATCH 1/3] make: Remove duplicate defintion of COMMON_SOURCES in Radeon makefiles

2011-08-29 Thread Chad Versace
Remove defintion of COMMON_SOURCES from {r300,r660}/Makefile. The defintion is a duplicate of that found in src/mesa/drivers/dri/Makefile.defines. CC: Ian Romanick Signed-off-by: Chad Versace --- src/mesa/drivers/dri/r300/Makefile |9 - src/mesa/drivers/dri/r600/Makefile |9

[Mesa-dev] [PATCH 2/3] make: Remove duplicate occurence of driverfuncs.c

2011-08-29 Thread Chad Versace
driverfuncs.o is already contained in libmesa.a, so remove it from the following source lists: src/mesa/drivers/dri/Makefiles.defines:COMMON_SOURCES. src/mesa/drivers/dri/swrast/Makefile:SWRAST_COMMON_SOURCES v2 - [idr] Also remove driverfuncs.c from swrast/Makefile. - [idr] Fix commi

[Mesa-dev] [PATCH 3/3] make: Factor out source lists from drivers/dri/common into Makefile.sources

2011-08-29 Thread Chad Versace
In order that the Autoconf and Android build can share the same source lists, move the lists from src/mesa/drivers/dri/Makefile.defines into src/mesa/drivers/dri/common/Makefile.sources I would like for Android to just reuse Makefile.defines, but the file is unsuitable for reuse. v2

Re: [Mesa-dev] [PATCH 2/2] i965: Avoid generating MOVs for most ir_assignment handling.

2011-08-29 Thread Kenneth Graunke
On 08/28/2011 05:31 PM, Eric Anholt wrote: > On Sat, 27 Aug 2011 03:23:28 -0700, Kenneth Graunke > wrote: >> This is a port of vec4_visitor::try_rewrite_rhs_to_dst to fs_visitor. >> >> Not only is this technique less invasive and more robust, it also >> generates better code. Over and above the

Re: [Mesa-dev] [PATCH 0/8] i965, mesa: Build libmesa_dricore and i965_dri for Android

2011-08-29 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 v2 of this series has been posted. - -- Chad Versace c...@chad-versace.us On 08/26/2011 06:53 PM, Chad Versace wrote: > This series builds libmesa_dricore.so and i965_dri.so for Android. Along the > way, a few cleanups are necessary. > > This just

[Mesa-dev] [v2 PATCH 0/5] i965, mesa: Build libmesa_dricore and i965_dri for Android

2011-08-29 Thread Chad Versace
This series builds libmesa_dricore.a and i965_dri.so for Android. Along the way, a few cleanups are necessary. Note: This series depends on another series I posted to mesa-dev: [PATCH 0/3] make: A few small cleanups This just gets the driver to build. I have not tested it yet. v2 Chan

[Mesa-dev] [PATCH 1/5] mesa: Build libmesa_dricore.a for Android

2011-08-29 Thread Chad Versace
libmesa_dricore.a is analogous to the libmesa.a built by the Autoconf build. v2 - Change libmesa_dricore from a shared library to a static library. I decided to make this change after realizing that libmesa_dricore.so indiscriminately exported all its symbols. CC: Ian Romanick CC: Chia-

[Mesa-dev] [PATCH 2/5] make: Document imported variables

2011-08-29 Thread Chad Versace
In src/mesa/Android.mk, it is non-trivial to determine which variables are imported by `include sources.mak`. So document them. v2 - [olv] Add X86_sources to list of documented variables. CC: Chia-I Wu Signed-off-by: Chad Versace --- src/mesa/Android.mk |6 ++ 1 files changed, 6 i

[Mesa-dev] [PATCH 3/5] dri: Build libmesa_dri_common for Android

2011-08-29 Thread Chad Versace
libmesa_dri_common is a static library that contains the sources in src/mesa/drivers/dri/common. Each DRI driver should link to it. v2 - [olv] In toplevel Android.mk, replace subdir src/mesa/drivers/dri/common with src/mesa/drivers/dri, and add the subdir only if MESA_BUILD_CLASSIC. - [olv]

[Mesa-dev] [PATCH 4/5] i965: Fix Android build by removing relative includes

2011-08-29 Thread Chad Versace
Replace each occurence of #include "../glsl/*.h" with #include "glsl/*.h" v2 - [idr] Eliminated spurious whitespace. CC: Ian Romanick Signed-off-by: Chad Versace --- src/mesa/drivers/dri/i965/brw_clip.c |2 +- src/mesa/drivers/dri/i965/brw_context.c|

[Mesa-dev] [PATCH 5/5] i965: Build i965_dri.so for Android

2011-08-29 Thread Chad Versace
Compile tested only. v2 - [olv] Update comments to include i965 in list of classic drivers. CC: Ian Romanick CC: Chia-I Wu Signed-off-by: Chad Versace --- Android.mk |4 +- src/mesa/drivers/dri/Android.mk | 35 +++ src/mesa/drivers/dri

Re: [Mesa-dev] [PATCH 15/21] mesa: Enable vertex programs by default, remove checks

2011-08-29 Thread Marek Olšák
On Mon, Aug 29, 2011 at 7:59 PM, Ian Romanick wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 08/27/2011 05:39 AM, Marek Olšák wrote: >> Hi Ian, >> >> I was trying to enable NV_vertex_program without regressions in >> st/mesa before, but these piglit tests were always failing and a

[Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Bryan Cain
I somehow didn't notice this until now, but why are there no Gallium "set" opcodes for integers that are equivalent to the SGT or SLE opcodes for floats? Does it have to do with available hardware features? Bryan ___ mesa-dev mailing list mesa-dev@lists

Re: [Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Brian Paul
On Mon, Aug 29, 2011 at 1:49 PM, Bryan Cain wrote: > I somehow didn't notice this until now, but why are there no Gallium > "set" opcodes for integers that are equivalent to the SGT or SLE opcodes > for floats?  Does it have to do with available hardware features? It's probably just an oversight

Re: [Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Zack Rusin
On Monday, August 29, 2011 03:49:03 PM Bryan Cain wrote: > I somehow didn't notice this until now, but why are there no Gallium > "set" opcodes for integers that are equivalent to the SGT or SLE opcodes > for floats? Does it have to do with available hardware features? It's largely because D3D10

Re: [Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Zack Rusin
On Monday, August 29, 2011 04:02:08 PM Zack Rusin wrote: > Either way though if GL needs those ops then, like Brian mentioned, it'd be > a good idea to add them. Actually I think it seems easier to just flip the ops rather than add new instructions, i.e. change stuff like a > b, to b < a, and a

Re: [Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 01:41 PM, Zack Rusin wrote: > On Monday, August 29, 2011 04:02:08 PM Zack Rusin wrote: >> Either way though if GL needs those ops then, like Brian mentioned, it'd be >> a good idea to add them. > > Actually I think it seems easier to ju

Re: [Mesa-dev] [PATCH 20/21] mesa: Remove vertex program extension enable flags

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/27/2011 06:47 AM, Brian Paul wrote: > On Sat, Aug 27, 2011 at 1:57 AM, Ian Romanick wrote: >> From: Ian Romanick >> >> The following extensions are always enabled, and drivers do not have >> to option to disable them: >> >>GL_ARB_vertex_pro

Re: [Mesa-dev] [PATCH 15/21] mesa: Enable vertex programs by default, remove checks

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 12:46 PM, Marek Olšák wrote: > On Mon, Aug 29, 2011 at 7:59 PM, Ian Romanick wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 08/27/2011 05:39 AM, Marek Olšák wrote: >>> Hi Ian, >>> >>> I was trying to enable NV_vertex

[Mesa-dev] [PATCH 01/20] glsl: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/glsl/ast_to_hir.cpp |6 ++ src/glsl/builtin_stubs.cpp|1 + src/glsl/glcpp/glcpp.c|1 + src/glsl/ir_function_detect_recursion.cpp |4 src/glsl/lower_jumps.cpp |1 + src/g

[Mesa-dev] [PATCH 02/20] glsl: Silence "ast_to_hir.cpp:1984:25: warning: comparison of unsigned expression >= 0 is always true"

2011-08-29 Thread Ian Romanick
From: Ian Romanick ast_type_qualifier::location should have been a signed integer from the beginning, and the giant comment in apply_type_qualifier_to_variable explains why. --- src/glsl/ast.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/glsl/ast.h b/src/glsl/ast

[Mesa-dev] [PATCH 03/20] intel: Silence many "intel_batchbuffer.h:97:39: warning: comparison between signed and unsigned integer expressions"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/drivers/dri/intel/intel_batchbuffer.h |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_batchbuffer.h b/src/mesa/drivers/dri/intel/intel_batchbuffer.h index fb4134d..90dc0ed 100644 --- a/src/mesa/drivers/

[Mesa-dev] [PATCH 04/20] intel: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/drivers/dri/intel/intel_buffer_objects.c | 47 ++--- 1 files changed, 31 insertions(+), 16 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_buffer_objects.c b/src/mesa/drivers/dri/intel/intel_buffer_objects.c index d908975..0acc3a1 10

[Mesa-dev] [PATCH 05/20] intel: Silence "warning: unused parameter ‘intel’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The intel_context was not used in any of these functions, so remove it. --- src/mesa/drivers/dri/intel/intel_buffer_objects.c | 10 -- src/mesa/drivers/dri/intel/intel_buffer_objects.h |3 +-- src/mesa/drivers/dri/intel/intel_tex_validate.c |5 ++--- 3 file

[Mesa-dev] [PATCH 06/20] intel: Silence "warning: unused parameter ‘fb’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The gl_framebuffer was not used in intel_draw_buffer, so remove it. --- src/mesa/drivers/dri/intel/intel_buffers.c |2 +- src/mesa/drivers/dri/intel/intel_buffers.h |2 +- src/mesa/drivers/dri/intel/intel_context.c |4 ++-- src/mesa/drivers/dri/intel/intel_fbo.c

[Mesa-dev] [PATCH 07/20] intel: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The intel_context and tiling parameters were not used by any if the i9[14]5_miptree_layout or the functions they call, and the tiling parameter was not used by brw_miptree_layout. Remove the unnecessary parameters. --- src/mesa/drivers/dri/i915/i915_tex_layout.c| 40 +++

[Mesa-dev] [PATCH 09/20] intel: Silence "warning: unused parameter ‘depth0’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The depth0 parameter was not used in intel_miptree_create_for_region, so remove it. All of the places that call this function, pass 1 for that parameter, and the place where it looks like it should have been used (the call to intel_miptree_create_internal) also had 1 hard code

[Mesa-dev] [PATCH 08/20] intel: Silence "warning: unused parameter ‘target’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The GLenum target parameter was not used in intel_copy_texsubimage, so remove it. --- src/mesa/drivers/dri/intel/intel_fbo.c |2 +- src/mesa/drivers/dri/intel/intel_tex.h |1 - src/mesa/drivers/dri/intel/intel_tex_copy.c |5 ++--- 3 files changed, 3 inser

[Mesa-dev] [PATCH 11/20] intel: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick Trivially silence the compiler by adding '(void) foo;' for each unused parameter. These parameters could not be removed. They are part of interface used elsewhere in Mesa, and some of the other customers actually use these parameters. --- src/mesa/drivers/dri/intel/intel_tex

[Mesa-dev] [PATCH 10/20] intel: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The internalFormat, format, and type parameters were not used by either try_pbo_upload or try_pbo_zcopy, so remove them. The width parameter was also not used by try_pbo_zcopy (because it doesn't actually copy anything), so remove it too. --- src/mesa/drivers/dri/intel/intel_

[Mesa-dev] [PATCH 13/20] dri_util: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The parameters can't be removed because they are part of the DRI ABI. --- src/mesa/drivers/dri/common/dri_util.c | 12 +++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/common/dri_util.c b/src/mesa/drivers/dri/common/dri_util.

[Mesa-dev] [PATCH 12/20] intel: Silence "intel/intel_fbo.h:105:4: warning: comparison of unsigned expression < 0 is always false"

2011-08-29 Thread Ian Romanick
From: Ian Romanick The test was of an enum, attIndex, which should be unsigned. The explicit check for < 0 was replaced with a cast to unsigned in an assertion that attIndex is less than the size of the array it will be used to index. --- src/mesa/drivers/dri/intel/intel_fbo.h |6 +++--- 1

[Mesa-dev] [PATCH 14/20] dri: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/drivers/dri/common/depthtmp.h |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/common/depthtmp.h b/src/mesa/drivers/dri/common/depthtmp.h index 81bec9c..c6e2bb7 100644 --- a/src/mesa/drivers/dri/common/depthtmp.

[Mesa-dev] [PATCH 15/20] swrast-dri: Silence several "warning: unused parameter"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/drivers/dri/swrast/swrast.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c index 719b406..6d4f5fc 100644 --- a/src/mesa/drivers/dri/swrast/swrast.c

[Mesa-dev] [PATCH 17/20] spantmp2: Silence many "warning: unused parameter ‘ctx’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick Not all drivers use ctx in LOCAL_VARS, so '(void) ctx;' is added to all the function templates to make GCC happy. --- src/mesa/drivers/dri/common/spantmp2.h | 20 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/common

[Mesa-dev] [PATCH 16/20] mesa: Silence "main/teximage.h:148:72: warning: unused parameter ‘texObj’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/main/teximage.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/teximage.h b/src/mesa/main/teximage.h index 19abc64..8ef1d45 100644 --- a/src/mesa/main/teximage.h +++ b/src/mesa/main/teximage.h @@ -147,6 +147,7 @@ _mesa_lock_

[Mesa-dev] [PATCH 18/20] tnl_dd: Silence "tnl_dd/t_dd_tritmp.h:292:3: warning: suggest braces around empty body in an ‘if’ statement"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/tnl_dd/t_dd_tritmp.h |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/tnl_dd/t_dd_tritmp.h b/src/mesa/tnl_dd/t_dd_tritmp.h index 022f0c6..cb1204d 100644 --- a/src/mesa/tnl_dd/t_dd_tritmp.h +++ b/src/mesa/tnl_dd/t_dd_tritmp.h @@ -

[Mesa-dev] [PATCH 19/20] swrast: Silence many "warning: unused parameter ‘ctx’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick Not all drivers use ctx in LOCAL_VARS, so '(void) ctx;' is added to all the function templates to make GCC happy. --- src/mesa/swrast/s_spantemp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/mesa/swrast/s_spantemp.h b/src/mesa/swrast/s_sp

[Mesa-dev] [PATCH 20/20] nvprogram: Silence "warning: unused parameter ‘ctx’"

2011-08-29 Thread Ian Romanick
From: Ian Romanick --- src/mesa/main/nvprogram.c |2 +- src/mesa/main/nvprogram.h |2 +- src/mesa/program/nvvertparse.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/main/nvprogram.c b/src/mesa/main/nvprogram.c index 7ff7645..a0e89b1 100644

Re: [Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

2011-08-29 Thread Maarten Lankhorst
Hey Christian, On 08/29/2011 10:14 AM, Christian König wrote: > Am Sonntag, den 28.08.2011, 19:13 +0200 schrieb Maarten Lankhorst: >> Hey, >> >> On 08/28/2011 05:04 PM, Christian König wrote: >>> Hi Maarten, >>> >>> could you put this into SetDecoderStatus instead? This makes the picture >>> struc

Re: [Mesa-dev] Mesa (master): glsl_to_tgsi: fix a reference leak on an error path

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 01:31 PM, Marek Olšák wrote: > Module: Mesa > Branch: master > Commit: b2064ff3115afa075021902ecd35f4a5a772dc9e > URL: > http://cgit.freedesktop.org/mesa/mesa/commit/?id=b2064ff3115afa075021902ecd35f4a5a772dc9e > > Author: Marek Olšá

Re: [Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-29 Thread Maarten Lankhorst
Hey, On 08/29/2011 10:08 AM, Christian König wrote: > Am Montag, den 29.08.2011, 00:36 -0400 schrieb Younes Manton: > [snip] >> Well, that was what the last discussion was all about, whether or not >> decode buffers should be handled internally by each driver or >> externally. It was decided to ha

Re: [Mesa-dev] [PATCH 7/7] g3dvl: Rewrite the mpeg 1&2 bitstream parser

2011-08-29 Thread Andy Furniss
Christian König wrote: Thanks to you once more, it was just looking at the bytes left in the buffer, but not the bits! So we ended up not decoding the last 32 bits of a buffer. That seems to have fixed it. I just pushed a fix, and I'm really wondering where the heck do you get all those nic

Re: [Mesa-dev] [PATCH 0/3] make: A few small cleanups

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 11:43 AM, Chad Versace wrote: > Chad Versace (3): > make: Remove duplicate defintion of COMMON_SOURCES in Radeon > makefiles > make: Remove duplicate occurence of driverfuncs.c > make: Factor out source lists from drivers/dri/co

Re: [Mesa-dev] [PATCH 4/5] i965: Fix Android build by removing relative includes

2011-08-29 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/29/2011 11:50 AM, Chad Versace wrote: This patch is Reviewed-by: Ian Romanick I can't really speak to the others in this series. > Replace each occurence of > #include "../glsl/*.h" > with > #include "glsl/*.h" > > v2 > > - [id

Re: [Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-29 Thread Younes Manton
2011/8/29 Maarten Lankhorst : > Hey, > > On 08/29/2011 10:08 AM, Christian König wrote: >> Am Montag, den 29.08.2011, 00:36 -0400 schrieb Younes Manton: >> [snip] >>> Well, that was what the last discussion was all about, whether or not >>> decode buffers should be handled internally by each driver

[Mesa-dev] [Bug 38970] [bisected]piglit glx/glx-pixmap-multi failed

2011-08-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38970 --- Comment #6 from Ian Romanick 2011-08-29 17:31:35 PDT --- This works on recent X servers. I believe that there were some bugs with pixmap tracking on the X server that were fixed around the same time this patch went it. Unless there is some

[Mesa-dev] [PATCH 1/3] Enable GLX_EXT_texture_from_pixmap in software.

2011-08-29 Thread Stéphane Marchesin
From: nobled Signed-off-by: nobled Signed-off-by: Stuart Abercrombie Signed-off-by: Stéphane Marchesin --- src/glx/drisw_glx.c | 83 +- 1 files changed, 81 insertions(+), 2 deletions(-) diff --git a/src/glx/drisw_glx.c b/src/glx/drisw_glx.c i

[Mesa-dev] [PATCH 2/3] Duplicate state_tracker/dri/sw/dri_drawable.c

2011-08-29 Thread Stéphane Marchesin
We need this for the upcoming fix for sw texture_from_pixmap. Signed-off-by: Stuart Abercrombie Signed-off-by: Stéphane Marchesin --- src/gallium/state_trackers/dri/sw/dri_drawable.c | 270 +- 1 files changed, 269 insertions(+), 1 deletions(-) mode change 12 => 100644

[Mesa-dev] [PATCH 3/3] state_trackers/dri/sw: Implement texture_from_pixmap.

2011-08-29 Thread Stéphane Marchesin
Signed-off-by: Stuart Abercrombie Signed-off-by: Stéphane Marchesin --- src/gallium/state_trackers/dri/sw/dri_drawable.c | 50 +++--- 1 files changed, 43 insertions(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/dri/sw/dri_drawable.c b/src/gallium/state_trackers/d

[Mesa-dev] [Bug 38970] [bisected]piglit glx/glx-pixmap-multi failed

2011-08-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38970 --- Comment #7 from Gordon Jin 2011-08-29 17:57:58 PDT --- Ian, thanks for the reply. I agree we can close it if it works with recent xserver. Xun, please verify. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email

Re: [Mesa-dev] [PATCH] winsys/radeon: Create async thread only once

2011-08-29 Thread Marek Olšák
Thanks. I wanted to do this long ago. Reviewed-by: Marek Olšák On Sat, Aug 27, 2011 at 10:57 PM, Maarten Lankhorst wrote: > I noticed that a thread was created for every time async flush was called, so > I moved it and used some semaphores to synch. > > Signed-off-by: Maarten Lankhorst > > --

[Mesa-dev] [PATCH] driconf: Updated German translation

2011-08-29 Thread Carl-Philip Haensch
--- src/mesa/drivers/dri/common/xmlpool/de.po | 141 +++-- 1 files changed, 91 insertions(+), 50 deletions(-) diff --git a/src/mesa/drivers/dri/common/xmlpool/de.po b/src/mesa/drivers/dri/common/xmlpool/de.po index 4ea82f9..8b5a296 100644 --- a/src/mesa/drivers/dri/commo

[Mesa-dev] map-texture-image-v5 branch

2011-08-29 Thread Brian Paul
I've created this branch with the first batch of my reworked map-texture-image patches. I'm having some network/mail problems so git-send-email isn't working for me ATM. Please review/test and I'll merge to master. Thanks. -Brian ___ mesa-dev mailing

Re: [Mesa-dev] [PATCH 3/3] make: Factor out source lists from drivers/dri/common into Makefile.sources

2011-08-29 Thread Chia-I Wu
On Tue, Aug 30, 2011 at 2:43 AM, Chad Versace wrote: > In order that the Autoconf and Android build can share the same source > lists, move the lists from >    src/mesa/drivers/dri/Makefile.defines > into >    src/mesa/drivers/dri/common/Makefile.sources > > I would like for Android to just reuse

Re: [Mesa-dev] [PATCH 1/5] mesa: Build libmesa_dricore.a for Android

2011-08-29 Thread Chia-I Wu
On Tue, Aug 30, 2011 at 2:50 AM, Chad Versace wrote: > libmesa_dricore.a is analogous to the libmesa.a built by the Autoconf > build. > > v2 > > - Change libmesa_dricore from a shared library to a static library. >  I decided to make this change after realizing that libmesa_dricore.so >  indi

Re: [Mesa-dev] [PATCH 2/5] make: Document imported variables

2011-08-29 Thread Chia-I Wu
On Tue, Aug 30, 2011 at 2:50 AM, Chad Versace wrote: > In src/mesa/Android.mk, it is non-trivial to determine which variables are > imported by `include sources.mak`. So document them. > > v2 > > - [olv] Add X86_sources to list of documented variables. Reviewed-by: Chia-I Wu > CC: Chia-I Wu

Re: [Mesa-dev] [PATCH 3/5] dri: Build libmesa_dri_common for Android

2011-08-29 Thread Chia-I Wu
On Tue, Aug 30, 2011 at 2:50 AM, Chad Versace wrote: > libmesa_dri_common is a static library that contains the sources in > src/mesa/drivers/dri/common. Each DRI driver should link to it. > > v2 > > - [olv] In toplevel Android.mk, replace subdir src/mesa/drivers/dri/common >  with src/mesa/d

Re: [Mesa-dev] [PATCH 5/5] i965: Build i965_dri.so for Android

2011-08-29 Thread Chia-I Wu
On Tue, Aug 30, 2011 at 2:50 AM, Chad Versace wrote: > Compile tested only. > > v2 > > - [olv] Update comments to include i965 in list of classic drivers. > > CC: Ian Romanick > CC: Chia-I Wu > Signed-off-by: Chad Versace > --- >  Android.mk                           |    4 +- >  src/mesa/

Re: [Mesa-dev] Missing integer "set" opcodes in Gallium

2011-08-29 Thread Bryan Cain
On 08/29/2011 03:41 PM, Zack Rusin wrote: > On Monday, August 29, 2011 04:02:08 PM Zack Rusin wrote: >> Either way though if GL needs those ops then, like Brian mentioned, it'd be >> a good idea to add them. > Actually I think it seems easier to just flip the ops rather than add new > instruction

Re: [Mesa-dev] [PATCH 02/20] glsl: Silence "ast_to_hir.cpp:1984:25: warning: comparison of unsigned expression >= 0 is always true"

2011-08-29 Thread Kenneth Graunke
On 08/29/2011 02:58 PM, Ian Romanick wrote: > From: Ian Romanick > > ast_type_qualifier::location should have been a signed integer from > the beginning, and the giant comment in > apply_type_qualifier_to_variable explains why. > --- > src/glsl/ast.h |2 +- > 1 files changed, 1 insertions(+)

Re: [Mesa-dev] [PATCH 1/3] mesa: Remove API facing bits of EXT_paletted_texture and EXT_shared_texture_palette

2011-08-29 Thread Kenneth Graunke
On 08/29/2011 11:12 AM, Ian Romanick wrote: > From: Ian Romanick > > This was also discussed at XDS 2010. However, actually making the > change was delayed because several drivers still exposed these > extensions to significant benefit (e.g., tdfx). Now that those > drivers have been removed, t