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

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38085 --- Comment #6 from Brian Paul 2011-06-10 06:44:06 PDT --- It's best to ssh in from another computer and do your debugging from there. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this ma

[Mesa-dev] [Bug 37274] Crash in draw_llvm_shader23 (r300g, rs690, in warzone2100)

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37274 José Fonseca changed: What|Removed |Added CC||jfons...@vmware.com --- Comment #3 from J

[Mesa-dev] [Bug 37274] Crash in draw_llvm_shader23 (r300g, rs690, in warzone2100)

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37274 --- Comment #4 from Brian Paul 2011-06-10 07:25:33 PDT --- This might be a CPU-caps-dependent issue. Can you do "export GALLIUM_DUMP_CPU=1" then run any GL test/app? That'll print your CPU's features. BTW, a short-hand for disassembling in gd

[Mesa-dev] [Bug 37862] Mesa 7.11-devel implementation error: _mesa_texstore_null() is called

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37862 --- Comment #8 from Pepi 2011-06-10 07:37:58 PDT --- Thanks Benjamin, But I'm not sure did everything went right?: The command: # LD_LIBRARY_PATH="/usr/local/lib" LIBGL_DRIVERS_PATH="/usr/local/lib/dri" glxinfo Gives me: . . . OpenGL renderer

[Mesa-dev] [Bug 38129] Account request

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38129 Bryan Cain changed: What|Removed |Added CC||bryancain3+...@gmail.com -- Configure bugm

[Mesa-dev] [Bug 38129] Account request

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38129 Brian Paul changed: What|Removed |Added Product|Mesa|freedesktop.org Component|Other

Re: [Mesa-dev] Upcoming Mesa releases

2011-06-10 Thread Marek Olšák
On Tue, Jun 7, 2011 at 9:13 PM, Ian Romanick wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > We've accumulated a lot of fixes in the 7.10 branch, so it seems like we > should do a 7.10.3 release soon.  How do Friday (6/10) or Monday (6/13) > sound? I am for Monday. There are still so

[Mesa-dev] [PATCH] st/mesa: remove asserts in st_texture_image_copy

2011-06-10 Thread Marek Olšák
This is for 7.10 only. The fix in master looks too complex to be cherry-picked. The assertions fail when generating mipmaps for NPOT textures. This fixes: - fbo-generatemipmap-formats (all of them) --- src/mesa/state_tracker/st_texture.c |4 1 files changed, 0 insertions(+), 4 deletions

[Mesa-dev] [Bug 37862] Mesa 7.11-devel implementation error: _mesa_texstore_null() is called

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37862 --- Comment #9 from Benjamin Bellec 2011-06-10 10:57:20 PDT --- So try with $ LD_LIBRARY_PATH="/usr/local/lib64" LIBGL_DRIVERS_PATH="/usr/local/lib64/dri" glxinfo But, anyway, when you execute autogen, you have a summary of what you will compil

[Mesa-dev] [PATCH] Fix format not a string literal error with -Werror=format-security

2011-06-10 Thread Eugeni Dodonov
From: Eugeni Dodonov A trivial fix for error: format not a string literal and no format arguments with compiling with -Werror=format-security flags. --- src/mesa/drivers/dri/i965/brw_fs.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs

Re: [Mesa-dev] [PATCH] Fix format not a string literal error with -Werror=format-security

2011-06-10 Thread Kenneth Graunke
On 06/10/2011 11:26 AM, Eugeni Dodonov wrote: From: Eugeni Dodonov A trivial fix for error: format not a string literal and no format arguments with compiling with -Werror=format-security flags. --- src/mesa/drivers/dri/i965/brw_fs.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(

Re: [Mesa-dev] [PATCH 2/4] intel: Clean up intel_render_texture with a rename and a helper function.

2011-06-10 Thread Kenneth Graunke
On 06/07/2011 11:47 AM, Eric Anholt wrote: The "newImage" isn't particularly new -- it might be the same texture that was attached to the same attachment point before. This function also gets called when just rebinding back to an FBO with a texture attachment. --- src/mesa/drivers/dri/intel/in

[Mesa-dev] [PATCH 02/11] intel: Add the MESA_FORMAT as a field of the miptree.

2011-06-10 Thread Eric Anholt
We only had internal_format before, which is way more irritating to work with. --- src/mesa/drivers/dri/intel/intel_fbo.c |1 + src/mesa/drivers/dri/intel/intel_mipmap_tree.c |8 ++-- src/mesa/drivers/dri/intel/intel_mipmap_tree.h |3 +++ src/mesa/drivers/dri/intel/intel

[Mesa-dev] intel prep for compressed mipmap generation

2011-06-10 Thread Eric Anholt
Here's a bunch of cleanup of our miptree handling to try to make dealing with compressed textures more sane. <3 gl_format. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 01/11] intel: Fix 2x2 and 1x1 compressed teximages from _mesa_generate_mipmap()

2011-06-10 Thread Eric Anholt
Generally image uploads to a the region occur at TexImage time, but that's not the case for fallback _mesa_generate_mipmap(), and in this path we were forgetting to align the width when dividing height. We were just leaving out parts of the compressed block at 2x2 and 1x1 levels. Fixes gen-compre

[Mesa-dev] [PATCH 03/11] intel: Add block alignment for RGTC textures.

2011-06-10 Thread Eric Anholt
We were using the default 4x2 alignment instead of the 4x4 required for non-FXT compressed textures. --- src/mesa/drivers/dri/i965/brw_tex_layout.c |2 +- src/mesa/drivers/dri/intel/intel_mipmap_tree.c |6 +-- src/mesa/drivers/dri/intel/intel_tex_layout.c | 42 +-

[Mesa-dev] [PATCH 04/11] i915: Drop dead argument to translate_texture_format().

2011-06-10 Thread Eric Anholt
--- src/mesa/drivers/dri/i915/i830_texstate.c |5 ++--- src/mesa/drivers/dri/i915/i915_texstate.c |4 +--- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i915/i830_texstate.c b/src/mesa/drivers/dri/i915/i830_texstate.c index 3298dbb..d4af5e5 100644 ---

[Mesa-dev] [PATCH 07/11] intel: Use the gl_format to get the base_format for miptree create.

2011-06-10 Thread Eric Anholt
One less argument to this insanely long function call. --- src/mesa/drivers/dri/intel/intel_fbo.c |1 - src/mesa/drivers/dri/intel/intel_mipmap_tree.c |2 +- src/mesa/drivers/dri/intel/intel_mipmap_tree.h |1 - src/mesa/drivers/dri/intel/intel_tex_image.c|1 - src/me

[Mesa-dev] [PATCH 05/11] intel: Make the intel_miptree_match_image format check more specific.

2011-06-10 Thread Eric Anholt
We don't care just about the internalFormat/cpp/compressed, but about the specific format chosen. We have no support for format translations as part of texture validation, and furthermore it has restrictions in the GL specification. However, we should be making consistent decisions for this check

[Mesa-dev] [PATCH 08/11] intel: Calculate compress_byte in intel_miptree_create.

2011-06-10 Thread Eric Anholt
One less argument and thing to get wrong. --- src/mesa/drivers/dri/intel/intel_fbo.c |7 ++- src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 18 +++--- src/mesa/drivers/dri/intel/intel_mipmap_tree.h |4 +--- src/mesa/drivers/dri/intel/intel_tex_image.c|

[Mesa-dev] [PATCH 09/11] intel: Drop the cpp argument to intel_miptree_create().

2011-06-10 Thread Eric Anholt
--- src/mesa/drivers/dri/intel/intel_fbo.c |5 + src/mesa/drivers/dri/intel/intel_mipmap_tree.c |9 - src/mesa/drivers/dri/intel/intel_mipmap_tree.h |1 - src/mesa/drivers/dri/intel/intel_tex_image.c|4 src/mesa/drivers/dri/intel/intel_tex_validate.c

[Mesa-dev] [PATCH 10/11] intel: Drop dead preinitialization of align_w, align_h.

2011-06-10 Thread Eric Anholt
--- src/mesa/drivers/dri/intel/intel_tex_layout.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_tex_layout.c b/src/mesa/drivers/dri/intel/intel_tex_layout.c index 7493b42..2002777 100644 --- a/src/mesa/drivers/dri/intel/intel_tex_layo

[Mesa-dev] [PATCH 06/11] intel: Drop the internal_format field of the mipmap tree.

2011-06-10 Thread Eric Anholt
This has been replaced with the gl_format now. --- src/mesa/drivers/dri/intel/intel_fbo.c |1 - src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 10 +++--- src/mesa/drivers/dri/intel/intel_mipmap_tree.h |3 --- src/mesa/drivers/dri/intel/intel_tex_image.c|4 +--- s

[Mesa-dev] [PATCH 11/11] intel: Fix miptree height alignment for compressed NPOT textures.

2011-06-10 Thread Eric Anholt
This is effectively just "round up when dividing by 4" compared to the previous code. Fixes the broken stripe at the top of fbo-generatemipmap-formats GL_EXT_texture_compression_rgtc. --- src/mesa/drivers/dri/intel/intel_tex_layout.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-)

[Mesa-dev] mesa compressed mipmap generation

2011-06-10 Thread Eric Anholt
Here's my series to finally fix compressed mipmap generation on hw drivers. Take a look at the last commit in particular -- I'd love to do TexImage in the uncompressed path as well so we could get rid of that awful code for handling this non-TexImage upload in the drivers, but that would cost us t

[Mesa-dev] [PATCH 1/4] mesa: Split _mesa_generate_mipmap along compressed/uncompressed lines.

2011-06-10 Thread Eric Anholt
The path taken is wildly different based on this (do we generate from a temporary image, or from level-1's data), and we appear to have stride bugs in the compressed case that are tough to disentangle. This just duplicates the code for the moment, the followon commit will do the actual changes. O

[Mesa-dev] [PATCH 2/4] mesa: Remove compressed code from generate_mipmaps_uncompressed().

2011-06-10 Thread Eric Anholt
--- src/mesa/main/mipmap.c | 118 ++-- 1 files changed, 4 insertions(+), 114 deletions(-) diff --git a/src/mesa/main/mipmap.c b/src/mesa/main/mipmap.c index 2bdb111..45fcc27 100644 --- a/src/mesa/main/mipmap.c +++ b/src/mesa/main/mipmap.c @@ -1892,79 +

[Mesa-dev] [PATCH 4/4] mesa: Switch generate_mipmaps_compressed() to using TexImage2D to upload.

2011-06-10 Thread Eric Anholt
The code was playing fast and loose with rowstrides, which meant that if a driver chose anything different for its alignment requirements, the generated mipmaps came out garbage. Unlike the uncompressed case, we can't generate mipmaps directly into image->Data, so by using TexImage2D we cut out mo

[Mesa-dev] [PATCH 3/4] mesa: Remove uncompressed code from generate_mipmaps_compressed().

2011-06-10 Thread Eric Anholt
--- src/mesa/main/mipmap.c | 177 +--- 1 files changed, 77 insertions(+), 100 deletions(-) diff --git a/src/mesa/main/mipmap.c b/src/mesa/main/mipmap.c index 45fcc27..43eea50 100644 --- a/src/mesa/main/mipmap.c +++ b/src/mesa/main/mipmap.c @@ -1970,79

Re: [Mesa-dev] [PATCH 2/4] intel: Clean up intel_render_texture with a rename and a helper function.

2011-06-10 Thread Eric Anholt
On Fri, 10 Jun 2011 12:58:12 -0700, Kenneth Graunke wrote: > On 06/07/2011 11:47 AM, Eric Anholt wrote: > > The "newImage" isn't particularly new -- it might be the same texture > > that was attached to the same attachment point before. This function > > also gets called when just rebinding back

Re: [Mesa-dev] [PATCH 03/11] intel: Add block alignment for RGTC textures.

2011-06-10 Thread Brian Paul
On 06/10/2011 02:38 PM, Eric Anholt wrote: We were using the default 4x2 alignment instead of the 4x4 required for non-FXT compressed textures. --- src/mesa/drivers/dri/i965/brw_tex_layout.c |2 +- src/mesa/drivers/dri/intel/intel_mipmap_tree.c |6 +-- src/mesa/drivers/dri/intel/i

Re: [Mesa-dev] [PATCH] st/mesa: remove asserts in st_texture_image_copy

2011-06-10 Thread Brian Paul
On 06/10/2011 11:44 AM, Marek Olšák wrote: This is for 7.10 only. The fix in master looks too complex to be cherry-picked. The assertions fail when generating mipmaps for NPOT textures. This fixes: - fbo-generatemipmap-formats (all of them) --- src/mesa/state_tracker/st_texture.c |4

Re: [Mesa-dev] [PATCH 03/11] intel: Add block alignment for RGTC textures.

2011-06-10 Thread Eric Anholt
On Fri, 10 Jun 2011 15:17:12 -0600, Brian Paul wrote: > On 06/10/2011 02:38 PM, Eric Anholt wrote: > > We were using the default 4x2 alignment instead of the 4x4 required > > for non-FXT compressed textures. > > --- > > src/mesa/drivers/dri/i965/brw_tex_layout.c |2 +- > > src/mesa/driv

[Mesa-dev] [PATCH] i965/fs: Check for compilation failure and bail as soon as possible.

2011-06-10 Thread Kenneth Graunke
Prior to this patch, it would happily continue compiling and even optimizing after a failure was detected. This seems wasteful. More importantly, the "message length > 11" failure seems to choke the instruction scheduler, making it somehow use an undefined value and segmentation fault. Signed-of

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

2011-06-10 Thread Chia-I Wu
On Wed, Jun 8, 2011 at 10:07 PM, Benjamin Franzke wrote: > 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

[Mesa-dev] [Bug 38172] New: Mesa build errors using build.sh script

2011-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38172 Summary: Mesa build errors using build.sh script Product: Mesa Version: git Platform: x86 (IA32) OS/Version: Linux (All) Status: NEW Severity: normal Priority: medi

Re: [Mesa-dev] [PATCH 06/11] intel: Drop the internal_format field of the mipmap tree.

2011-06-10 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/10/2011 01:38 PM, Eric Anholt wrote: > This has been replaced with the gl_format now. I just want to make sure I grok this. 2/11 and 6/11 just sap IntenalFormat for TexFormat, right? > --- > src/mesa/drivers/dri/intel/intel_fbo.c |

Re: [Mesa-dev] [PATCH 10/11] intel: Drop dead preinitialization of align_w, align_h.

2011-06-10 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/10/2011 01:38 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/intel/intel_tex_layout.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/intel/intel_tex_layout.c > b/src/mesa/drivers/dri/

Re: [Mesa-dev] [PATCH 1/4] mesa: Split _mesa_generate_mipmap along compressed/uncompressed lines.

2011-06-10 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/10/2011 06:14 PM, Ian Romanick wrote: > On 06/10/2011 01:42 PM, Eric Anholt wrote: >> The path taken is wildly different based on this (do we generate from >> a temporary image, or from level-1's data), and we appear to have >> stride bugs in the

Re: [Mesa-dev] [PATCH] i965/fs: Check for compilation failure and bail as soon as possible.

2011-06-10 Thread Eric Anholt
On Fri, 10 Jun 2011 16:07:31 -0700, Kenneth Graunke wrote: > Prior to this patch, it would happily continue compiling and even > optimizing after a failure was detected. This seems wasteful. > > More importantly, the "message length > 11" failure seems to choke the > instruction scheduler, maki

[Mesa-dev] [PATCH] glxinfo: add support for creating/querying core-profile contexts

2011-06-10 Thread Brian Paul
The -c flag says to try to create a core profile (no legacy features) using glXCreateContextAttribsARB(). A core profile may advertise a different set of extensions than a compatibility profile (though, the only difference with NVIDIA's driver is the presence of GL_ARB_compatibility). Also note t

[Mesa-dev] [PATCH] mesa: move texrender.c to swrast

2011-06-10 Thread Brian Paul
This stuff is really for software rendering, it's not core Mesa. A small step toward pushing the FetchTexel() stuff down into swrast. --- src/mesa/SConscript |2 +- src/mesa/drivers/common/driverfuncs.c|5 +- src/mesa/drivers/dri/intel/intel_fbo.c |8 +- src/

[Mesa-dev] [PATCH] i965/fs: Check for compilation failure and bail before optimizing.

2011-06-10 Thread Kenneth Graunke
Prior to this patch, it would attempt to optimize and allocate registers for the program even if it failed to compile. This seems wasteful. More importantly, the "message length > 11" failure seems to choke the instruction scheduler, making it somehow use an undefined value and segmentation fault