Re: [Mesa-dev] [PATCH] gallivm: Change getExtent and readByte to non-const with llvm-3.1.

2012-02-09 Thread Jose Fonseca
I haven't tested LLVM-3.1 myself but seems good to me. Thanks Vinson. Jose - Original Message - Fix build with llvm-3.1svn. llvm-3.1svn r149918 changed BufferMemoryObject::getExtent and BufferMemoryObject::readByte from const member functions to non-const member functions in

[Mesa-dev] Quick EGL questions

2012-02-09 Thread Thomas Hellstrom
Hi! I have a couple of quick EGL questions that I hope someone knows the answer to: 1) I'm trying to test the vmwgfx_dri driver with EGL driver egl_dri2, platform drm, but it doesn't seem like this configuration supports EGL_MESA_screen_surface so basically all relevant EGL demos fail. Is

Re: [Mesa-dev] Quick EGL questions

2012-02-09 Thread Kristian Høgsberg
On Thu, Feb 9, 2012 at 8:51 AM, Thomas Hellstrom thellst...@vmware.com wrote: Hi! I have a couple of quick EGL questions that I hope someone knows the answer to: 1) I'm trying to test the vmwgfx_dri driver with EGL driver egl_dri2, platform drm, but it doesn't seem like this configuration

Re: [Mesa-dev] Quick EGL questions

2012-02-09 Thread Benjamin Franzke
2012/2/9 Thomas Hellstrom thellst...@vmware.com: Hi! I have a couple of quick EGL questions that I hope someone knows the answer to: 1) I'm trying to test the vmwgfx_dri driver with EGL driver egl_dri2, platform drm, but it doesn't seem like this configuration supports

Re: [Mesa-dev] [PATCH 3/3] mesa: replace GET_SHINE_TAB_ENTRY() macro with an inline function

2012-02-09 Thread Brian Paul
On 02/08/2012 11:47 PM, Mathias Fröhlich wrote: Hi Brian, On Wednesday, February 08, 2012 20:13:44 Brian Paul wrote: --- src/mesa/main/light.h| 32 +--- src/mesa/tnl/t_rasterpos.c |3 +-- src/mesa/tnl/t_vb_lighttmp.h | 27

Re: [Mesa-dev] [PATCH 8/9] gallivm: use undef integer in broadcast.

2012-02-09 Thread Jose Fonseca
Dave, The series looks great. Just this one patch seems off: LLVMTypeOf(scalar) should always match bld-elem_type, and it would only match bld-int_elem_type if the type itself is an integer. Therefore it seems that the caller is using an integer scalar with a floating point build context when

Re: [Mesa-dev] [PATCH 6/9] gallivm: enable stores of integer types. (v2)

2012-02-09 Thread Jose Fonseca
I think that we could probably just remove TGSI_TYPE_UNTYPED and just say MOV is TGSI_TYPE_FLOAT. I'm OK either way. Jose - Original Message - From: Dave Airlie airl...@redhat.com Infer from the operand the type of value to store. MOV is untyped but we use the float store path.

Re: [Mesa-dev] [PATCH] meta: replace abort() with _mesa_problem()

2012-02-09 Thread Jose Fonseca
Good idea. Jose - Original Message - --- src/mesa/drivers/common/meta.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c index aa5fef8..0cf1028 100644 --- a/src/mesa/drivers/common/meta.c

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Brian Paul
Anuj, The patch you committed (15986d2), the first version you posted that tests: if (width 2 * border || width maxSize) return GL_FALSE; causes my copy of conform (mustpass.c / proxy texture test) to fail with swrast/softpipe/llvmpipe. It passes w/ NVIDIA's driver. I

[Mesa-dev] [PATCH] mesa: fix proxy texture target initialization

2012-02-09 Thread Brian Paul
The mapping from TEXTURE_x_INDEX to GL_TEXTURE_x was broken in alloc_proxy_textures() because the elements in the targets[] array were in the wrong order. This didn't actually cause any failures since we never really use the proxy texture's Target field. But let's get it right. NOTE: This is a

Re: [Mesa-dev] [PATCH] i965: Fix border color on Ironlake.

2012-02-09 Thread Eric Anholt
On Wed, 8 Feb 2012 19:20:24 -0800, Kenneth Graunke kenn...@whitecape.org wrote: Ironlake appears to check our pointer against the General State Base Address upper bound, rather than ignoring the zero bound as it ought. Unfortunately, since we leave GSBA set to zero, there is no logical

Re: [Mesa-dev] [PATCHv3] r600g: Use a fake reloc to sleep for fences

2012-02-09 Thread Simon Farnsworth
On Wednesday 8 February 2012 18:28:05 Michel Dänzer wrote: On Fre, 2012-02-03 at 17:32 +, Simon Farnsworth wrote: r300g is able to sleep until a fence completes rather than busywait because it creates a special buffer object and relocation that stays busy until the CS containing the

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Ian Romanick
On 02/09/2012 08:14 AM, Brian Paul wrote: Anuj, The patch you committed (15986d2), the first version you posted that tests: if (width 2 * border || width maxSize) return GL_FALSE; causes my copy of conform (mustpass.c / proxy texture test) to fail with swrast/softpipe/llvmpipe. It passes w/

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Anuj Phogat
On Thu, Feb 9, 2012 at 11:02 AM, Ian Romanick i...@freedesktop.org wrote: On 02/09/2012 08:14 AM, Brian Paul wrote: Anuj, The patch you committed (15986d2), the first version you posted that tests: if (width 2 * border || width maxSize) return GL_FALSE; causes my copy of conform

[Mesa-dev] [Bug 45578] main/image.c:1659: _mesa_convert_colors: Assertion `dstType == 0x1403' failed.

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45578 Ian Romanick i...@freedesktop.org changed: What|Removed |Added CC||xunx.f...@intel.com

[Mesa-dev] [PATCH 2/2] mesa/vbo: inline vbo_sizeof_ib_type.

2012-02-09 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Can't see any reason this wouldn't be better off as an inline. Signed-off-by: Dave Airlie airl...@redhat.com --- src/mesa/vbo/vbo.h| 18 -- src/mesa/vbo/vbo_exec_array.c | 16 2 files changed, 16

[Mesa-dev] [PATCH 1/2] st/mesa: don't unreference user attribs up front.

2012-02-09 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com postpone unreferences until end of function, as the ones in use will get naturally dereferenced. Signed-off-by: Dave Airlie airl...@redhat.com --- src/mesa/state_tracker/st_draw.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git

Re: [Mesa-dev] [PATCH/RFC] glsl: Avoid excessive loop unrolling.

2012-02-09 Thread Mathias Fröhlich
Hi Eric, On Tuesday, February 07, 2012 12:25:29 Eric Anholt wrote: Does this also fix our piglit tests for the bug? If so, note it in the commit message, and Ok, included. Reviewed-by: Eric Anholt e...@anholt.net Thanks! Mathias ___ mesa-dev

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Brian Paul
On 02/09/2012 12:15 PM, Anuj Phogat wrote: On Thu, Feb 9, 2012 at 11:02 AM, Ian Romanick i...@freedesktop.org mailto:i...@freedesktop.org wrote: On 02/09/2012 08:14 AM, Brian Paul wrote: Anuj, The patch you committed (15986d2), the first version you posted that

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Brian Paul
On 02/09/2012 01:57 PM, Brian Paul wrote: On 02/09/2012 12:15 PM, Anuj Phogat wrote: On Thu, Feb 9, 2012 at 11:02 AM, Ian Romanick i...@freedesktop.org mailto:i...@freedesktop.org wrote: On 02/09/2012 08:14 AM, Brian Paul wrote: Anuj, The patch you committed (15986d2), the first version you

Re: [Mesa-dev] [PATCH] mesa: fix maximum allowed proxy texture size condition

2012-02-09 Thread Anuj Phogat
On Thu, Feb 9, 2012 at 1:07 PM, Brian Paul bri...@vmware.com wrote: On 02/09/2012 01:57 PM, Brian Paul wrote: On 02/09/2012 12:15 PM, Anuj Phogat wrote: On Thu, Feb 9, 2012 at 11:02 AM, Ian Romanick i...@freedesktop.org mailto:i...@freedesktop.org wrote: On 02/09/2012 08:14 AM, Brian Paul

Re: [Mesa-dev] [PATCH] glx/dri2: use GLX prefix to enable INTEL_swap_event

2012-02-09 Thread Ian Romanick
On 02/08/2012 11:49 AM, Robert Bragg wrote: This adds the GLX_ prefix to the string we pass to __glXEnableDirectExtension() otherwise it doesn't match the name we have in known_glx_extensions[] in glxextensions.c and doesn't get enabled. This mistake wasn't noticed before since

[Mesa-dev] Mesa 8.0 release

2012-02-09 Thread Ian Romanick
Mesa 8.0 has been released. Mesa 8.0 is a new development release. People who are concerned with stability and reliability should stick with a previous release or wait for Mesa 8.0.1. The tag in the GIT repository for Mesa 8.0 is 'mesa-8.0'. Mesa 7.11.1 is available for download at

Re: [Mesa-dev] Mesa 8.0 release

2012-02-09 Thread Brian Paul
On 02/09/2012 03:43 PM, Ian Romanick wrote: Mesa 8.0 has been released. Mesa 8.0 is a new development release. People who are concerned with stability and reliability should stick with a previous release or wait for Mesa 8.0.1. The tag in the GIT repository for Mesa 8.0 is 'mesa-8.0'. Mesa

Re: [Mesa-dev] [PATCH] intel: Also check renderability in intel_renderbuffer_update_wrapper

2012-02-09 Thread Ian Romanick
On 02/06/2012 03:08 PM, Eric Anholt wrote: On Fri, 03 Feb 2012 16:09:25 -0700, Ian Romanicki...@freedesktop.org wrote: On 02/03/2012 02:11 AM, Eric Anholt wrote: On Wed, 1 Feb 2012 12:29:39 -0700, Ian Romanicki...@freedesktop.org wrote: From: Ian Romanickian.d.roman...@intel.com A

[Mesa-dev] [PATCH] meta: Unbind texture from FBO after completeness check

2012-02-09 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com --- src/mesa/drivers/common/meta.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c index aa5fef8..3647395 100644 --- a/src/mesa/drivers/common/meta.c +++

[Mesa-dev] about driconf and experimental flags

2012-02-09 Thread Alexandre Demers
Simple question: is there a reason why many options and features under development or testing are only available by manually setting env variables? The point is, why isn't there an experimental tab under driconf where we could easily enable or disable the different features? Modifying .drirc

[Mesa-dev] [Bug 45420] undefined reference to `LLVMInitializeARMDisassembler'

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45420 --- Comment #9 from Benjamin Herrenschmidt b...@kernel.crashing.org 2012-02-09 19:00:37 PST --- This is broken. This is not valid C :-) #if (LLVM_NATIVE_ARCH == X86 || LLVM_NATIVE_ARCH == X86Target) Doesn't work with the C preprocessor. The

[Mesa-dev] [Bug 45420] undefined reference to `LLVMInitializeARMDisassembler'

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45420 Benjamin Herrenschmidt b...@kernel.crashing.org changed: What|Removed |Added Status|RESOLVED

[Mesa-dev] [PATCH] gallivm: Replace architecture test with PIPE_ARCH_*

2012-02-09 Thread Stéphane Marchesin
X86Target is a variable, and therefore isn't defined at compile time. So LLVM_NATIVE_ARCH == X86Target is translated into 0 == 0 and since X86 is first, we always pick it. Therefore we replace the logic with PIPE_ARCH_*. https://bugs.freedesktop.org/show_bug.cgi?id=45420 ---

[Mesa-dev] [PATCH 1/3] mesa: remove unused _mesa_map_ci8_to_rgba8()

2012-02-09 Thread Brian Paul
--- src/mesa/main/pixeltransfer.c | 26 -- src/mesa/main/pixeltransfer.h |6 -- 2 files changed, 0 insertions(+), 32 deletions(-) diff --git a/src/mesa/main/pixeltransfer.c b/src/mesa/main/pixeltransfer.c index 5c167e0..c6172b9 100644 ---

[Mesa-dev] [PATCH 2/3] st/mesa: stop using Map8[] tables in load_color_map_texture()

2012-02-09 Thread Brian Paul
Use the float tables instead. Pixel maps are seldom used so this shouldn't be a big deal. Next, we can get rid of the gl_pixelmap::Map8 array. --- src/mesa/state_tracker/st_atom_pixeltransfer.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git

[Mesa-dev] [PATCH 3/3] mesa: remove unused gl_pixelmap::Map8[] array

2012-02-09 Thread Brian Paul
--- src/mesa/main/mtypes.h |1 - src/mesa/main/pixel.c |2 -- 2 files changed, 0 insertions(+), 3 deletions(-) diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index 521bb39..6066e6f 100644 --- a/src/mesa/main/mtypes.h +++ b/src/mesa/main/mtypes.h @@ -1045,7 +1045,6 @@

[Mesa-dev] [PATCH 2/2] i965/fs: Enable register spilling on gen7 too.

2012-02-09 Thread Eric Anholt
It turns out the same messages work on gen7, we were just being paranoid. Fixes the penumbra shadows mode of Lightsmark since the register allocation fix. --- src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] i965: Report the failure message when failing to compile the fragment shader.

2012-02-09 Thread Eric Anholt
We just abort later, but at least this should result in more informative bug reports. --- src/mesa/drivers/dri/i965/brw_fs.cpp |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index

[Mesa-dev] [Bug 45292] Compilation failure on d3d1x state tracker: ‘ID3D10Include’ has not been declared

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45292 --- Comment #1 from Jeremy Murphy jeremy.william.mur...@gmail.com 2012-02-09 22:43:22 PST --- OK, I have found what is going wrong, it is pretty simple. d3d10.h contains these three lines at the end of the file: #include d3d10misc.h #include

[Mesa-dev] [Bug 45292] Compilation failure on d3d1x state tracker: ‘ID3D10Include’ has not been declared

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45292 --- Comment #2 from Jeremy Murphy jeremy.william.mur...@gmail.com 2012-02-09 22:52:08 PST --- However, I have another thought. The last time that I successfully compiled Mesa against Wine was 16-Dec-2011, which is way after the 'ID3D10Include'

[Mesa-dev] [Bug 45816] New: [IVB]SPECviewperf 11 cause GPU hung

2012-02-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45816 Bug #: 45816 Summary: [IVB]SPECviewperf 11 cause GPU hung Classification: Unclassified Product: Mesa Version: git Platform: x86 (IA32) OS/Version: Linux (All) Status: