[Mesa-dev] [PATCH] st/mesa: Check after _mesa_create_context()

2011-07-12 Thread Emil Velikov
In some cases _mesa_create_context() can return NULL an in the mesa state tracker, we do not concider the case, which may cause issues within st_create_context_priv() This patch adds a simple check (similar to the one in the dri drivers) v2 Do not zero-out the funcs table Signed-off-by: Emil

[Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Egon Ashrafinia
Hello guys. 1 month ago, we talked about merging glsl-to-tgsi to master but it still not happend. What about now? I could compile and test it a bit. It works. Anyone who could do it? What does Bryan Cain say about it? Greetings ___ mesa-dev mailing

[Mesa-dev] Patch: Fix building mesa with llvm 3.0

2011-07-12 Thread Carl-Philip Haensch
Hi, I could not compile mesa with llvm 3.0 from svn. So i made this patch which fixes the issue: http://pastebin.com/1Y19vrFf Is this a proper patch? Or should there be more version checking with #if for it? Thank You ___ mesa-dev mailing list

Re: [Mesa-dev] Patch: Fix building mesa with llvm 3.0

2011-07-12 Thread Vinson Lee
Please try compiling again with mesa master commit e4189f2e2e310276136dc06af20062986920e8e2 or later. The build succeeds here with llvm-3.0svn but there's a missing LLVMAddTypeName symbol during runtime. From:

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Keith Whitwell
On Mon, 2011-07-11 at 18:24 +0200, Christian König wrote: Hi guys, as the subject already indicates: I'm about to merge pipe-video to master and just wanted to ask if anybody has still any objections? After following Jose and Younes discussion on mesa-dev about how to design such an

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Christian König
Hi, as the subject already indicates: I'm about to merge pipe-video to master and just wanted to ask if anybody has still any objections? Yes [I am a watchman of compile/build errors (I know I am an old bitcher but code quality rules)]:

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Christian König
Am Dienstag, den 12.07.2011, 13:06 +0200 schrieb Henri Verbeet: 2011/7/11 Christian König deathsim...@vodafone.de: So what do you guys think about it? I didn't look much at the rest of the changes, but some of the changes for r600g (diff attached) look a bit questionable. Could you take a

Re: [Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)

2011-07-12 Thread Dan Nicholson
Hi Jose, On Sat, Jul 9, 2011 at 6:03 AM, Jose Fonseca jfons...@vmware.com wrote: I heard no concerns so I went ahead and made a branch where: - I removed GLUT - derived Mesa tarballs' file list from git ls-files. http://cgit.freedesktop.org/mesa/mesa/log/?h=robust-tarballs I've confirmed

[Mesa-dev] [PATCH 3/3] configure.ac: don't build gallium driver libs just to see if there are no errors

2011-07-12 Thread Marek Olšák
I think the past are those times when the gallium interface was changed all the time. Now it is not, so there is no reason to always compile the libs if they are not needed. No idea why some hardware drivers need to contain whole softpipe and llvmpipe. --- configure.ac | 23

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Henri Verbeet
2011/7/12 Christian König deathsim...@vodafone.de: it works with my available hardware (no piglit regressions). The changes to the winsys code is about making a bo optional, even when the reg informations says it isn't. This is useful for registers where only a subset of the bits needs to be

Re: [Mesa-dev] [PATCH 1/3] Rename swrastg_dri to swrast_dri

2011-07-12 Thread Jose Fonseca
I'm in favor for this FWIW. Jose - Original Message - I prefer it this way and it has been suggested earlier by others too. Opinions? --- src/egl/drivers/dri2/egl_dri2.c |5 - src/gallium/targets/dri-swrast/Makefile |2 +-

Re: [Mesa-dev] Is it a memory leak?

2011-07-12 Thread Brian Paul
On 07/12/2011 06:38 AM, Zhenbo Xu wrote: Hi all, I have been detecting memory leaks in mesa for SPEC2000, and I found a kind of memory leaks shown in the following code gl_text_image* image_to_texture(...) { ... textImage = gl_alloc_texttrue_image(...); if (!textImage) return

Re: [Mesa-dev] [PATCH 1/3] Rename swrastg_dri to swrast_dri

2011-07-12 Thread Brian Paul
On 07/12/2011 07:27 AM, Marek Olšák wrote: I prefer it this way and it has been suggested earlier by others too. Opinions? OK by me. -Brian ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] st/mesa: Check after _mesa_create_context()

2011-07-12 Thread Brian Paul
On 07/11/2011 08:42 AM, Emil Velikov wrote: In some cases _mesa_create_context() can return NULL an in the mesa state tracker, we do not concider the case, which may cause issues within st_create_context_priv() This patch adds a simple check (similar to the one in the dri drivers) v2 Do not

Re: [Mesa-dev] Is it a memory leak?

2011-07-12 Thread Brian Paul
On 07/12/2011 08:40 AM, Zhenbo Xu wrote: I found the following bugs in mesa(v100) for SPEC CPU2000 v1.3 leak bug 1: //in file textimage.c at line 345 gl_text_image* image_to_texture(...) { ... textImage = gl_alloc_texttrue_image(...); if (!textImage) return NULL; ... switch

Re: [Mesa-dev] Is it a memory leak?

2011-07-12 Thread Zhenbo Xu
Thank you for your replies! 在 2011年7月12日 下午10:47,Brian Paul bri...@vmware.com写道: On 07/12/2011 08:40 AM, Zhenbo Xu wrote: I found the following bugs in mesa(v100) for SPEC CPU2000 v1.3 leak bug 1: //in file textimage.c at line 345 gl_text_image* image_to_texture(...) { ...

[Mesa-dev] [Bug 39147] egl_gallium.so: errors when linking

2011-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39147 --- Comment #1 from Chia-I Wu olva...@gmail.com 2011-07-12 08:00:34 PDT --- Created an attachment (id=49010) View: https://bugs.freedesktop.org/attachment.cgi?id=49010 Review: https://bugs.freedesktop.org/review?bug=39147attachment=49010 fix

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Younes Manton
2011/7/12 Keith Whitwell kei...@vmware.com: I'm a bit unsure about what's the best approach here, though at this stage I'm happy with your approach and don't think it needs to be changed before any merge. But speaking in general terms, individual planes map well onto 8-bit single-component

[Mesa-dev] [Bug 39158] [bisected]piglit glx/glx-swap-event_async failed

2011-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39158 Ian Romanick i...@freedesktop.org changed: What|Removed |Added Component|Drivers/DRI/i965|GLX

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Keith Whitwell
On Tue, 2011-07-12 at 11:13 -0400, Younes Manton wrote: 2011/7/12 Keith Whitwell kei...@vmware.com: I'm a bit unsure about what's the best approach here, though at this stage I'm happy with your approach and don't think it needs to be changed before any merge. But speaking in general

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Jose Fonseca
- Original Message - Hi guys, as the subject already indicates: I'm about to merge pipe-video to master and just wanted to ask if anybody has still any objections? After following Jose and Younes discussion on mesa-dev about how to design such an abstraction layer I took another

Re: [Mesa-dev] [PATCH 3/3] configure.ac: don't build gallium driver libs just to see if there are no errors

2011-07-12 Thread Marek Olšák
On Tue, Jul 12, 2011 at 4:28 PM, Brian Paul bri...@vmware.com wrote:  On 07/12/2011 07:27 AM, Marek Olšák wrote: I think the past are those times when the gallium interface was changed all the time. Now it is not, so there is no reason to always compile the libs if they are not needed. No

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Christian König
Am Dienstag, den 12.07.2011, 15:44 +0200 schrieb Henri Verbeet: 2011/7/12 Christian König deathsim...@vodafone.de: it works with my available hardware (no piglit regressions). The changes to the winsys code is about making a bo optional, even when the reg informations says it isn't. This is

[Mesa-dev] [Bug 39147] egl_gallium.so: errors when linking

2011-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39147 --- Comment #2 from Laurent carlier lordhea...@gmail.com 2011-07-12 08:59:17 PDT --- The patch fixes the problem. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: ---

[Mesa-dev] [Bug 39147] egl_gallium.so: errors when linking

2011-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39147 --- Comment #3 from Laurent carlier lordhea...@gmail.com 2011-07-12 09:11:40 PDT --- ... and applying only the first hunk seems enough. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this

[Mesa-dev] [Bug 39158] [bisected]piglit glx/glx-swap-event_async failed

2011-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39158 Ian Romanick i...@freedesktop.org changed: What|Removed |Added AssignedTo|mesa-dev@lists.freedesktop.

Re: [Mesa-dev] [PATCH 1/3] Rename swrastg_dri to swrast_dri

2011-07-12 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 06:27 AM, Marek Olšák wrote: I prefer it this way and it has been suggested earlier by others too. Opinions? So what happens to people that don't build any of gallium but do occasionally compare with a software driver? Does this make

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/11/2011 09:53 AM, Egon Ashrafinia wrote: Hello guys. 1 month ago, we talked about merging glsl-to-tgsi to master but it still not happend. What about now? I could compile and test it a bit. It works. Anyone who could do it? What does Bryan

Re: [Mesa-dev] About merging pipe-video to master

2011-07-12 Thread Henri Verbeet
2011/7/12 Christian König deathsim...@vodafone.de: Am Dienstag, den 12.07.2011, 15:44 +0200 schrieb Henri Verbeet: 2011/7/12 Christian König deathsim...@vodafone.de: +           // TODO get BLEND_CLAMP state from rasterizer state Is this comment still accurate? Yes it is, the very first

Re: [Mesa-dev] [PATCH 1/3] Rename swrastg_dri to swrast_dri

2011-07-12 Thread Jose Fonseca
- Original Message - -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 06:27 AM, Marek Olšák wrote: I prefer it this way and it has been suggested earlier by others too. Opinions? So what happens to people that don't build any of gallium but do occasionally

Re: [Mesa-dev] [PATCH 1/3] Rename swrastg_dri to swrast_dri

2011-07-12 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 10:04 AM, Jose Fonseca wrote: - Original Message - -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 06:27 AM, Marek Olšák wrote: I prefer it this way and it has been suggested earlier by others too.

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Corbin Simpson
I'd like to see it merged eventually for my GSoC student's work, which will probably depend on it, but waiting until August will probably be alright. ~ C. On Wed, Jul 13, 2011 at 1:00 AM, Ian Romanick i...@freedesktop.org wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/11/2011

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Lauri Kasanen
On Wed, 13 Jul 2011 01:18:16 +0800 Corbin Simpson mostawesomed...@gmail.com wrote: I'd like to see it merged eventually for my GSoC student's work, which will probably depend on it, but waiting until August will probably be alright. No, the post-processing doesn't depend on the glsl-to-tgsi

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Bryan Cain
On 07/11/2011 11:53 AM, Egon Ashrafinia wrote: Hello guys. 1 month ago, we talked about merging glsl-to-tgsi to master but it still not happend. What about now? I could compile and test it a bit. It works. Anyone who could do it? What does Bryan Cain say about it? Hi Egon, Last month, I

[Mesa-dev] Case where opt_copy_propagation doesn't do its job

2011-07-12 Thread Bryan Cain
It appears that the copy propagation pass in the GLSL compiler (in opt_copy_propagation.cpp) doesn't do copy propagation when the components of a variable are initialized separately, like this: (declare (temporary ) vec4 vec_ctor) (assign (w) (var_ref vec_ctor) (constant float (1.00)) )

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-12 Thread Egon Ashrafinia
Am 12.07.2011 21:44, schrieb Bryan Cain: On 07/11/2011 11:53 AM, Egon Ashrafinia wrote: Hello guys. 1 month ago, we talked about merging glsl-to-tgsi to master but it still not happend. What about now? I could compile and test it a bit. It works. Anyone who could do it? What does Bryan Cain

Re: [Mesa-dev] Case where opt_copy_propagation doesn't do its job

2011-07-12 Thread Eric Anholt
On Tue, 12 Jul 2011 15:44:12 -0500, Bryan Cain bryanca...@gmail.com wrote: It appears that the copy propagation pass in the GLSL compiler (in opt_copy_propagation.cpp) doesn't do copy propagation when the components of a variable are initialized separately, like this: (declare (temporary )

[Mesa-dev] [PATCH 03/12] intel: Move intel_draw_buffers() code into each driver.

2011-07-12 Thread Eric Anholt
The illusion of shared code here wasn't fooling anybody. It was tempting to keep i830 and i915 still shared, but I think I actually want to make them diverge shortly. --- src/mesa/drivers/dri/i915/i830_vtbl.c | 202 src/mesa/drivers/dri/i915/i915_vtbl.c |

[Mesa-dev] [PATCH 01/12] intel: Use the post-execution batchbuffer contents for dumping.

2011-07-12 Thread Eric Anholt
We were missing out on all the relocation changes by dumping what we subdata()ed in instead of what's there after the kernel finished with it. --- src/mesa/drivers/dri/intel/intel_batchbuffer.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[Mesa-dev] [PATCH 02/12] intel: Clarify the depthRb == stencilRb logic.

2011-07-12 Thread Eric Anholt
--- src/mesa/drivers/dri/intel/intel_fbo.c | 31 +++ 1 files changed, 15 insertions(+), 16 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c b/src/mesa/drivers/dri/intel/intel_fbo.c index 1669af2..55bcc75 100644 ---

[Mesa-dev] [PATCH 05/12] i965: Remove empty brw_set_draw_region.

2011-07-12 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_vtbl.c | 14 -- 1 files changed, 0 insertions(+), 14 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c index 1609938..5f67239 100644 --- a/src/mesa/drivers/dri/i965/brw_vtbl.c +++

[Mesa-dev] [PATCH 06/12] i965: Remove unused region calculations in brw_update_draw_buffer().

2011-07-12 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_vtbl.c | 62 + 1 files changed, 2 insertions(+), 60 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c index 5f67239..831c3e0 100644 --- a/src/mesa/drivers/dri/i965/brw_vtbl.c

[Mesa-dev] [PATCH 04/12] i965: Remove FALLBACK() from brw_update_draw_region().

2011-07-12 Thread Eric Anholt
The 965 driver doesn't use these for deciding on fallbacks. --- src/mesa/drivers/dri/i965/brw_vtbl.c | 16 1 files changed, 0 insertions(+), 16 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c index ef1d4f6..1609938 100644

[Mesa-dev] [PATCH 07/12] i965: Remove i915 paths from brw_update_draw_buffers().

2011-07-12 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_vtbl.c | 48 -- 1 files changed, 11 insertions(+), 37 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c index 831c3e0..55dbd4f 100644 ---

[Mesa-dev] [PATCH 09/12] i915: Disable the depth test whenever we don't have a depth buffer.

2011-07-12 Thread Eric Anholt
We were disabling it once at the moment we changed draw buffers, but later enabling of depth test could turn it back on. Fixes fbo-nodepth-test. Note that ctx-DrawBuffer has to be checked because during context create we get called while it's still unset. However, we know we'll get an

[Mesa-dev] [PATCH 10/12] i915: Make stencil test for no-stencil handling match depth test.

2011-07-12 Thread Eric Anholt
i915_update_draw_buffers() already handles the fallback bit for missing stencil region, so here we just need to handle whether the GL thinks we have stencil data or not (and disable the test if so). --- src/mesa/drivers/dri/i915/i915_state.c | 32 +++-

[Mesa-dev] [PATCH 08/12] i915: Remove i965 paths from i915_update_drawbuffer() and i830's too.

2011-07-12 Thread Eric Anholt
--- src/mesa/drivers/dri/i915/i830_vtbl.c | 46 +++- src/mesa/drivers/dri/i915/i915_vtbl.c | 47 +++-- 2 files changed, 20 insertions(+), 73 deletions(-) diff --git a/src/mesa/drivers/dri/i915/i830_vtbl.c

[Mesa-dev] [PATCH 11/12] i915: Fix incorrect depth scaling when enabling/disabling depth buffers.

2011-07-12 Thread Eric Anholt
We were updating our new viewport using the old buffers' _WindowMap.m. We can do less math and avoid using that deprecated matrix by just folding the viewport calculation right in to the driver. Fixes piglit fbo-depthtex. --- src/mesa/drivers/dri/i915/i915_state.c | 43

[Mesa-dev] [PATCH 12/12] i915: Add support for HW rendering with no color draw buffer.

2011-07-12 Thread Eric Anholt
This is useful for shadow map generation. Tested with glsl-bug-22603, which rendered the depth textures with fallbacks before. --- src/mesa/drivers/dri/i915/i915_vtbl.c | 98 + 1 files changed, 50 insertions(+), 48 deletions(-) diff --git

Re: [Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)

2011-07-12 Thread Jose Fonseca
- Original Message - Hi Jose, On Sat, Jul 9, 2011 at 6:03 AM, Jose Fonseca jfons...@vmware.com wrote: I heard no concerns so I went ahead and made a branch where: - I removed GLUT - derived Mesa tarballs' file list from git ls-files.

Re: [Mesa-dev] Case where opt_copy_propagation doesn't do its job

2011-07-12 Thread Bryan Cain
On 07/12/2011 04:36 PM, Eric Anholt wrote: On Tue, 12 Jul 2011 15:44:12 -0500, Bryan Cain bryanca...@gmail.com wrote: It appears that the copy propagation pass in the GLSL compiler (in opt_copy_propagation.cpp) doesn't do copy propagation when the components of a variable are initialized