[Mesa-dev] [Bug 38092] Mesa outputs warning wih Unigine sanctuary (Mesa warning: glDraw[Range]Elements())

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38092 --- Comment #1 from Pavel Ondračka 2011-06-08 23:47:03 PDT --- > I had no warning before if I remember correctly. I will try to bisect. Don't waste time, those warnings were present from the moment Sanctuary started to work last year. See for e

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #4 from Alexandre Demers 2011-06-08 23:43:25 PDT --- It seems related to a wrapper I use to let Diablo 2 use Glide over OpenGL. All other configuration (Direct 2D and Direct 3D) are OK and don't produce this error. http://www.svenswr

[Mesa-dev] [Bug 38092] New: Mesa outputs warning wih Unigine sanctuary (Mesa warning: glDraw[Range]Elements())

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38092 Summary: Mesa outputs warning wih Unigine sanctuary (Mesa warning: glDraw[Range]Elements()) Product: Mesa Version: git Platform: Other OS/Version: All Status: NEW

Re: [Mesa-dev] [PATCH] faster logbase2() - again

2011-06-08 Thread Roland Scheidegger
Ok I've pushed this. The mesa part was bogus though (can't use PIPE_CC_GCC there). Technically, the non-builtin versions are bogus now though if unsigned isn't 32bit (on ILP64 it is 64bit - I believe some MIPS/ALPHA systems might use that). But only if the values fed to them are actually larger tha

[Mesa-dev] [PATCH] i965: Add support for GL_FIXED vertex attributes.

2011-06-08 Thread Eric Anholt
This sadly requires work in the VS to rescale them, because the hardware doesn't support this format natively. Fixes arb_es2_compatibility-fixed-type and gtf/fixed_data_type. --- src/mesa/drivers/dri/i965/brw_draw_upload.c |5 + src/mesa/drivers/dri/i965/brw_vs.c | 11 +

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #3 from Brian Paul 2011-06-08 15:14:27 PDT --- Hmmm, can you go in with gdb and poke around a bit? Get the call stack, print *program, etc? It would be easy to silence this warning, but I'd like to know the root cause first. -- C

[Mesa-dev] [Bug 38085] Mesa: User error: GL_INVALID_ENUM in glTexParameter(param=0x2901)

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38085 --- Comment #3 from Brian Paul 2011-06-08 15:13:04 PDT --- 0x8920 = GL_FRAGMENT_SHADER_ATI which is part of the GL_ATI_fragment_shader extension. The app probably isn't properly checking if the extension is supported by the driver. I don't thi

Re: [Mesa-dev] [PATCH 1/3] i965/gen6: Refactor SF setup a bit to handle overrides in one place.

2011-06-08 Thread Kenneth Graunke
For the series: Acked-by: Kenneth Graunke gen7_sf_state.c will need updating as well. Would you mind doing that, or should I? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] glsl: Fix incorrect optimization of instructions before discard statements.

2011-06-08 Thread Kenneth Graunke
On 06/08/2011 11:27 AM, Eric Anholt wrote: The function was named "find_unconditional_discard", but didn't actually check that the discard statement found was unconditional. Fixes piglit glsl-fs-discard-04. --- src/glsl/opt_discard_simplification.cpp | 20 +--- 1 files change

[Mesa-dev] [Bug 38085] Mesa: User error: GL_INVALID_ENUM in glTexParameter(param=0x2901)

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38085 --- Comment #2 from Alexandre Demers 2011-06-08 14:40:38 PDT --- I could try to find the culprit commit, I was not having this issue before and I'm still using the same application (no update, same version). I'll try to find it out. Also, I rece

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #2 from Alexandre Demers 2011-06-08 14:37:13 PDT --- With the latest code, here is the result: Mesa 7.11-devel implementation error: Unexpected program target 0x0 in destroy_program_variants_cb() Please report at bugs.freedesktop.or

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Kenneth Graunke
We've been (gradually) switching to (bool/true/false) in the Intel driver code. I definitely prefer that. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 38085] Mesa: User error: GL_INVALID_ENUM in glTexParameter(param=0x2901)

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38085 --- Comment #1 from Brian Paul 2011-06-08 14:20:10 PDT --- Looks like an application bug to me. My guess is the app is trying to set GL_REPEAT wrap mode for a GL_TEXTURE_RECTANGLE target. -- Configure bugmail: https://bugs.freedesktop.org/use

[Mesa-dev] [Bug 38086] Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 --- Comment #1 from Brian Paul 2011-06-08 14:16:37 PDT --- I've just committed a change to print a slightly more helpful error message. Can you rebuild your driver and retest? -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?t

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Dave Airlie
On Thu, Jun 9, 2011 at 12:54 AM, Jose Fonseca wrote: > Personally, I prefer the standard C bool type, given it's a) standard, b) > shorter. Furthermore Microsoft's windows.h already defines boolean: > >  $ grep -r '\' /usr/i586-mingw32msvc/include/ >  /usr/i586-mingw32msvc/include/rpcndr.h:typede

[Mesa-dev] [Bug 38086] New: Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb()

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38086 Summary: Mesa 7.11-devel implementation error: Unexpected program target in destroy_program_variants_cb() Product: Mesa Version: git Platform: Other OS/Version: All

[Mesa-dev] [Bug 38085] New: Mesa: User error: GL_INVALID_ENUM in glTexParameter(param=0x2901)

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38085 Summary: Mesa: User error: GL_INVALID_ENUM in glTexParameter(param=0x2901) Product: Mesa Version: git Platform: All OS/Version: All Status: NEW Severity:

[Mesa-dev] [Bug 36282] 34a5d3b9f4740601708c82093e2114356d749e65: glxgears segfaults when compiled with shared glapi

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=36282 --- Comment #39 from Alexandre Demers 2011-06-08 13:55:29 PDT --- Seems good here too. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the

Re: [Mesa-dev] [PATCH] faster logbase2() - again

2011-06-08 Thread Benjamin Bellec
Le 08/06/2011 22:08, Roland Scheidegger a écrit : > Looks good to me - but skip the special n <= 1 case that's what the > (n|1) input to the __builtin_clz is good for. Ok, I'm not familiar with bit manipulation in fact... > Oh and I didn't notice before but there's an error in the gcc version > ch

Re: [Mesa-dev] [PATCH] faster logbase2() - again

2011-06-08 Thread Roland Scheidegger
Looks good to me - but skip the special n <= 1 case that's what the (n|1) input to the __builtin_clz is good for. Oh and I didn't notice before but there's an error in the gcc version check (as that's defined as (__GNUC__ * 100 + __GNUC_MINOR__) - we're ignoring patchlevel). Though as said there's

[Mesa-dev] [PATCH] meta: Fix glCopyTexImage(GL_LUMINANCE) from non-GL_LUMINANCE source.

2011-06-08 Thread Eric Anholt
glReadPixels() was performing RGB -> L conversion differently from the glTexImage() style conversion appropriate for glCopyTexImage(). Fixes gles2conform copy_texture. --- src/mesa/drivers/common/meta.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/src/mes

[Mesa-dev] [PATCH] faster logbase2() - again

2011-06-08 Thread Benjamin Bellec
On a Roland Scheidegger idea! Benjamin diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c index 3e42911..da2e72f 100644 --- a/src/mesa/main/teximage.c +++ b/src/mesa/main/teximage.c @@ -86,6 +86,12 @@ _mesa_free_texmemory(void *m) static GLuint logbase2(GLuint n) { +#if defined(PI

[Mesa-dev] generating _glapi_table from a dlfcn handle

2011-06-08 Thread Jeremy Huddleston
In moving src/glx/apple over to glapi this past week, I used our dispatch table code from xserver/hw/xquartz/GL/indirect.c as the model for creating our _glapi_table. Our table should contain pointers into OSX's OpenGL implementation, so the call is just passed along to that implementation. C

[Mesa-dev] [PATCH] intel: Fix mipmap and format handling of blit glCopyPixels().

2011-06-08 Thread Eric Anholt
Fixes fbo-mipmap-copypix. --- src/mesa/drivers/dri/intel/intel_pixel_copy.c | 98 +--- 1 files changed, 53 insertions(+), 45 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_pixel_copy.c b/src/mesa/drivers/dri/intel/intel_pixel_copy.c index e83f1bf..88258d5 100644

[Mesa-dev] [PATCH] glsl: Fix incorrect optimization of instructions before discard statements.

2011-06-08 Thread Eric Anholt
The function was named "find_unconditional_discard", but didn't actually check that the discard statement found was unconditional. Fixes piglit glsl-fs-discard-04. --- src/glsl/opt_discard_simplification.cpp | 20 +--- 1 files changed, 17 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [Bug 36282] 34a5d3b9f4740601708c82093e2114356d749e65: glxgears segfaults when compiled with shared glapi

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=36282 --- Comment #38 from Alexandre Demers 2011-06-08 10:22:07 PDT --- I'll test it later today when I'll get home. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You

[Mesa-dev] [Bug 36282] 34a5d3b9f4740601708c82093e2114356d749e65: glxgears segfaults when compiled with shared glapi

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=36282 --- Comment #37 from ojab 2011-06-08 10:18:33 PDT --- glxgears works fine for me (r600g) with mesa aea2236 compiled with --enable-shared-glapi. Looks like fixed. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ---

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Marek Olšák
Boolean is also more prone to bugs. I've got this habit from C++, I usually convert int to bool by a simple assignment (without the "!= 0" part). That doesn't work with boolean defined as char, because the upper bits are always truncated. Marek On Wed, Jun 8, 2011 at 4:54 PM, Jose Fonseca wrote:

Re: [Mesa-dev] [PATCH] faster util_next_power_of_two() function

2011-06-08 Thread Roland Scheidegger
Looks ok to me. I think it might actually be ok to omit the version check, we use __builtin_popcount without it too (and that seems to be gcc 3.4 too, so presumably gcc 3.4 is a requirement). So what about __builtin_clz optimized logbase2 :-) I think that should just be 31 - __builtin_clz(n | 1) I

[Mesa-dev] [Bug 36282] 34a5d3b9f4740601708c82093e2114356d749e65: glxgears segfaults when compiled with shared glapi

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=36282 --- Comment #36 from Chia-I Wu 2011-06-08 08:22:38 PDT --- The symptom I had was the same as Marc's. I've pushed several changes to fix it and to avoid future breakage. Please test again. -- Configure bugmail: https://bugs.freedesktop.org/use

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Jakob Bornecrantz
On Wed, Jun 8, 2011 at 4:59 PM, Brian Paul wrote: > On 06/08/2011 08:54 AM, Jose Fonseca wrote: >> >> Personally, I prefer the standard C bool type, given it's a) standard, b) >> shorter. Furthermore Microsoft's windows.h already defines boolean: >> >>   $ grep -r '\' /usr/i586-mingw32msvc/include

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Brian Paul
On 06/08/2011 08:54 AM, Jose Fonseca wrote: Personally, I prefer the standard C bool type, given it's a) standard, b) shorter. Furthermore Microsoft's windows.h already defines boolean: $ grep -r '\' /usr/i586-mingw32msvc/include/ /usr/i586-mingw32msvc/include/rpcndr.h:typedef unsigned ch

Re: [Mesa-dev] s/bool/boolean/

2011-06-08 Thread Jose Fonseca
Personally, I prefer the standard C bool type, given it's a) standard, b) shorter. Furthermore Microsoft's windows.h already defines boolean: $ grep -r '\' /usr/i586-mingw32msvc/include/ /usr/i586-mingw32msvc/include/rpcndr.h:typedef unsigned char boolean; which can create havoc depending on

Re: [Mesa-dev] Mesa (master): mga: enable GL_ARB_vertex_array_object extension

2011-06-08 Thread Brian Paul
On 06/07/2011 06:37 PM, Ian Romanick wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/07/2011 03:35 PM, Brian Paul wrote: Module: Mesa Branch: master Commit: 1d71bbed3bbeb99d954cc137e4923c5db6e063c8 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1d71bbed3bbeb99d954cc137e492

Re: [Mesa-dev] [PATCH] faster util_next_power_of_two() function

2011-06-08 Thread Chris Bandy
On 06/08/2011 06:38 AM, Benjamin Bellec wrote: > Here is the v4 patch. > > Benjamin As an uninformed bystander, I have some nitpicks that may just be coding style. >From the patch: + if (x <= 1) + return 1; I think it would make more sense to move this above the #ifdef and not duplicat

Re: [Mesa-dev] [PATCH] faster util_next_power_of_two() function

2011-06-08 Thread Benjamin
2011/6/8 Chris Bandy > On 06/08/2011 06:38 AM, Benjamin Bellec wrote: > > Here is the v4 patch. > > > > Benjamin > > As an uninformed bystander, I have some nitpicks that may just be coding > style. > > From the patch: > > + if (x <= 1) > + return 1; > > > I think it would make more sense

[Mesa-dev] [Bug 36238] Mesa release files don't contain scons control files

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

[Mesa-dev] [Bug 31590] Black space between colors on mole hill example

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=31590 Brian Paul changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

Re: [Mesa-dev] [PATCH] mapi: Fix tls with shared/es-glapi on x86-64

2011-06-08 Thread Benjamin Franzke
2011/6/7 Chia-I Wu : > How is that the case?  It seems the symbol is not used elsewhere. > Right, seems i tried solve the wrong side of the problem. x86_64_entry_start is declared similar to a global static variable in the asm code, so it will be bound local. But in entry_get_public its declared

[Mesa-dev] [Bug 36282] 34a5d3b9f4740601708c82093e2114356d749e65: glxgears segfaults when compiled with shared glapi

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=36282 --- Comment #35 from Chia-I Wu 2011-06-08 06:50:12 PDT --- 34a5d3b9f4740601708c82093e2114356d749e65 is indeed the bad commit. The bug is triggered when libglapi and libdricore use different struct glapi_table. This happens when shared glapi is

Re: [Mesa-dev] [PATCH] faster util_next_power_of_two() function

2011-06-08 Thread Benjamin Bellec
Le 06/06/2011 23:54, Roland Scheidegger a écrit : > Also I believe this builtin requires gcc 3.4 - not sure though if the > rest of the code compiles on older gcc. > > Roland Yes I checked, __builtin_clz() appeared first time in GCC 3.4.0 documentation. Benjamin _

Re: [Mesa-dev] [PATCH] faster util_next_power_of_two() function

2011-06-08 Thread Benjamin Bellec
Le 06/06/2011 23:54, Roland Scheidegger a écrit : > Am 06.06.2011 23:18, schrieb Tormod Volden: >> On Sun, Jun 5, 2011 at 1:14 AM, Benjamin Bellec wrote: >>> So here is a v2 patch with a builtin GCC optimization which is the >>> fastest (thx Matt to point me to this solution). >>> >> >> From patch:

[Mesa-dev] [Bug 31590] Black space between colors on mole hill example

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=31590 --- Comment #13 from José Fonseca 2011-06-08 02:35:23 PDT --- Patches look OK to me, Brian. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee fo

[Mesa-dev] [PATCH 1/2] egl_dri2: add dri2_load_driver_swrast

2011-06-08 Thread Chia-I Wu
From: Chia-I Wu Refactor dri2_load_driver and add dri2_load_driver_swrast for loading swrast DRI driver. --- src/egl/drivers/dri2/egl_dri2.c | 56 +-- src/egl/drivers/dri2/egl_dri2.h |3 ++ src/egl/drivers/dri2/platform_x11.c |4 +-- 3 files chan

[Mesa-dev] [PATCH 2/2] egl_dri2: try swrastg_dri if swrast_dri fails

2011-06-08 Thread Chia-I Wu
From: Chia-I Wu --- src/egl/drivers/dri2/egl_dri2.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 2559ac1..ad70a05 100644 --- a/src/egl/drivers/dri2/egl_dri2.c +++ b/src/egl/drivers/dri2/egl_d

[Mesa-dev] [Bug 37911] supertuxkart crashes with r300g

2011-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37911 --- Comment #2 from Fabio Pedretti 2011-06-08 01:21:53 PDT --- B(In reply to comment #1) > That seems to be a build failure. The one who maintains the repository should > rebuild the binaries (start with git clean -fdX). > > Supertuxkart works