[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/

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

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 12 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önigdeathsim...@vodafone.de 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

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; +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 redirect my

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

2011-08-29 Thread deathsimple
From: Christian König deathsim...@vodafone.de 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

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 carl-philip.haen...@mailbox.tu-dresden.de 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

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

2011-08-29 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com 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 airl...@redhat.com ---

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

2011-08-29 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com 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 airl...@redhat.com --- src/gallium/drivers/r600/r600_pipe.c |6 ++

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

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 Romanickian.d.roman...@intel.com 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

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 Romanickian.d.roman...@intel.com Nothing in Mesa supports color-index textures, and most of the other infrastructure that could allow such support has

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 bryanca...@gmail.com 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 bryanca...@gmail.com 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. ---

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

2011-08-29 Thread Eric Anholt
On Mon, 29 Aug 2011 08:32:41 -0600, Brian Paul bri...@vmware.com 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.

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 bryanca...@gmail.com wrote: On 08/28/2011 07:38 PM, Eric Anholt wrote: On Sat, 27 Aug 2011 20:18:55 -0700, Kenneth Graunke kenn...@whitecape.org wrote: From: Bryan Cain bryanca...@gmail.com Using multiply and reciprocal for integer

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 kenn...@whitecape.org wrote: From: Bryan Cain bryanca...@gmail.com Using multiply and reciprocal for

[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 bryanca...@gmail.com 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

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

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 bryanca...@gmail.com Using multiply and reciprocal for integer division involves potentially lossy floating point conversions. This is okay for older GPUs that represent integers

[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 ian.d.roman...@intel.com 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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 |

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 Romanickian.d.roman...@intel.com 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

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 Romanickian.d.roman...@intel.com This was also discussed at XDS 2010. However, actually making the change was delayed because several drivers still exposed these extensions to significant benefit

[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 i...@freedesktop.org Signed-off-by: Chad Versace c...@chad-versace.us --- src/mesa/drivers/dri/r300/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

[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 kenn...@whitecape.org 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

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

[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

[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 o...@lunarg.com Signed-off-by: Chad Versace c...@chad-versace.us --- src/mesa/Android.mk

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 i...@freedesktop.org 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

[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

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 bryanca...@gmail.com 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

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 only

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 = b

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 just

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 i...@freedesktop.org wrote: From: Ian Romanick ian.d.roman...@intel.com The following extensions are always enabled, and drivers do not have to option to

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 i...@freedesktop.org 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- 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

[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 ian.d.roman...@intel.com 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

[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 ian.d.roman...@intel.com --- 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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 +--

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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 ++--

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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. ---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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 ++---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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. ---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

[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 ian.d.roman...@intel.com 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. ---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- 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 ---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- 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 ---

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

[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 ian.d.roman...@intel.com --- 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 @@

[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 ian.d.roman...@intel.com --- 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 +++

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

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

2011-08-29 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- 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

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 structure also

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šák

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 handle it

Re: [Mesa-dev] [PATCH 7/7] g3dvl: Rewrite the mpeg 12 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

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/common

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 ian.d.roman...@intel.com I can't really speak to the others in this series. Replace each occurence of #include ../glsl/*.h with #include glsl/*.h v2

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 m.b.lankho...@gmail.com: 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

[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 i...@freedesktop.org 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.

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

2011-08-29 Thread Stéphane Marchesin
From: nobled nob...@dreamwidth.org Signed-off-by: nobled nob...@dreamwidth.org Signed-off-by: Stuart Abercrombie sabercrom...@chromium.org Signed-off-by: Stéphane Marchesin marc...@chromium.org --- src/glx/drisw_glx.c | 83 +- 1 files changed, 81

[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 sabercrom...@chromium.org Signed-off-by: Stéphane Marchesin marc...@chromium.org --- src/gallium/state_trackers/dri/sw/dri_drawable.c | 270 +- 1 files changed, 269 insertions(+),

[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 gordon@intel.com 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:

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 mar...@gmail.com On Sat, Aug 27, 2011 at 10:57 PM, Maarten Lankhorst m.b.lankho...@gmail.com 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.

[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 ---

[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 c...@chad-versace.us 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

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 c...@chad-versace.us 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

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 c...@chad-versace.us 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

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 c...@chad-versace.us 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  

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 c...@chad-versace.us wrote: Compile tested only. v2 - [olv] Update comments to include i965 in list of classic drivers. CC: Ian Romanick i...@freedesktop.org CC: Chia-I Wu o...@lunarg.com Signed-off-by: Chad Versace c...@chad-versace.us

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 instructions,