[Mesa-dev] [PATCH] r600g: fix UINT_TO_FLT on evergreen

2012-01-17 Thread Vadim Girlin
It should be processed as trans-only. Signed-off-by: Vadim Girlin --- src/gallium/drivers/r600/r600_shader.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c index 95c6d1e..8bdda8b 100644 --

Re: [Mesa-dev] [PATCH 5/7] automake: src/mesa/drivers/dri/nouveau

2012-01-17 Thread Gaetan Nadon
On 12-01-17 03:58 PM, Matt Turner wrote: > +AM_CPPFLAGS = \ > + -I$(top_srcdir)/include \ > + -I$(top_srcdir)/src/ \ > + -I$(top_srcdir)/src/mapi \ > + -I$(top_srcdir)/src/mesa/ \ > + -I../common \ Why does this one cannot be written properly based on top_srcdir? > + $(DEFIN

Re: [Mesa-dev] [PATCH 3/7] automake: src/mesa/drivers/dri/radeon

2012-01-17 Thread Gaetan Nadon
On 12-01-17 03:58 PM, Matt Turner wrote: > +AM_CPPFLAGS = \ > + -DRADEON_R100 \ > + -I$(top_srcdir)/include \ > + -I$(top_srcdir)/src/ \ > + -I$(top_srcdir)/src/mapi \ > + -I$(top_srcdir)/src/mesa/ \ > + -I../common \ > + -Iserver \ > + $(DEFINES) \ > + $(ASM_FLA

Re: [Mesa-dev] [PATCH 1/7] i965: use AM_CPPFLAGS instead of AM_CFLAGS/CXXFLAGS

2012-01-17 Thread Gaetan Nadon
On 12-01-17 03:58 PM, Matt Turner wrote: > --- > src/mesa/drivers/dri/i965/Makefile.am |4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/Makefile.am > b/src/mesa/drivers/dri/i965/Makefile.am > index 5512381..93937b1 100644 > --- a/src/mesa/

[Mesa-dev] [Bug 44618] Cross-compilation broken by glsl builtin_compiler

2012-01-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44618 --- Comment #7 from Thierry Reding 2012-01-17 22:56:18 PST --- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51882(In reply to comment #6) > (In reply to comment #5) > > I guess I should be reporting that to GCC, right? > > Right. Done: http://g

[Mesa-dev] [PATCH] i965/vs: Take attributes into account when deciding urb_entry_size.

2012-01-17 Thread Kenneth Graunke
Both the VF and VS share space in the URB. First, the VF stores attributes (shader inputs) there. The VS then reads the attributes, executes, and reuses the space to store varyings (shader outputs). Thus, we need to calculate the amount of URB space necessary for inputs, outputs, and pick whiche

Re: [Mesa-dev] [PATCH] glsl: Fix 'control reaches end of non-void function' warning.

2012-01-17 Thread Kenneth Graunke
On 01/16/2012 09:10 PM, Vinson Lee wrote: Fix this GCC warning on non-debug builds. glsl_types.cpp: In member function 'gl_texture_index glsl_type::sampler_index() const': glsl_types.cpp:157: warning: control reaches end of non-void function Signed-off-by: Vinson Lee --- src/glsl/glsl_types.cp

Re: [Mesa-dev] [PATCH] Revert "mesa: check depth, stencil formats (not depths) in glBlitFramebuffer"

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 05:27 PM, Ian Romanick wrote: On 01/17/2012 05:12 PM, Chad Versace wrote: This reverts commit 3f1fab06844f696de44d9a56e83ff62e8ea576bd. This loosens the format validation in glBlitFramebuffer. When blitting depth bits, don't require an exact match between the depth formats; only r

Re: [Mesa-dev] [PATCH 1/3] i965/gen7: Remove stale comment.

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 04:57 PM, Eric Anholt wrote: This field is actually set up above. --- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c b/src/mesa/drivers/dri/i965/gen7_wm_

Re: [Mesa-dev] [PATCH 2/3] i965/gen7: Set up texture horizontal alignment field.

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 04:57 PM, Eric Anholt wrote: This is required for Z16 support, which is the first thing to be textured with horizontal alignment of 8. --- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers

Re: [Mesa-dev] [PATCH] i965/vs: Fix URB entry size calculations on Ivybridge.

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 04:46 PM, Eric Anholt wrote: On Tue, 17 Jan 2012 21:33:09 -0800, Kenneth Graunke wrote: This is inspired by the BSpec, vol2a 3D Pipeline - Overview: 3D Pipeline / 3D Pipeline State Overview / Push Constant URB Allocation. In particular, the last paragraph. The new VS change fixe

Re: [Mesa-dev] [PATCH] mesa: Set default access flags based on the run-time API

2012-01-17 Thread Brian Paul
On Tue, Jan 17, 2012 at 5:29 PM, Ian Romanick wrote: > From: Ian Romanick > > The default access flags for OpenGL ES (via GL_OES_map_buffer) and > desktop OpenGL are different.  The code previously tried to handle > this, but the decision was made at compile time.  Since the same > driver binary

Re: [Mesa-dev] [PATCH] i965/gen5: Fix rendering of depth buffers without stencil

2012-01-17 Thread Eric Anholt
On Tue, 17 Jan 2012 16:03:56 -0800, Chad Versace wrote: > Fixes the following OGLConform tests on gen5: > depth-stencil(misc.state_on.depth_int) > fbo_db_ARBfp(basic.OnlyDepthBuffDrawBufferRender) > > The problem was that, if the depth buffer's Mesa format was X8_Z24, then > we emitted t

Re: [Mesa-dev] [PATCH] i965: Bump Ivybridge's fake MRF range to g112-127 instead of g111-126.

2012-01-17 Thread Eric Anholt
On Tue, 17 Jan 2012 19:43:52 -0800, Kenneth Graunke wrote: > When I originally implemented the hack to use GRFs 111+ as fake MRFs, I > did so purely to avoid rewriting all the code that dealt with MRFs. > However, it turns out that a similar hack is actually required. > > Newly discovered langua

Re: [Mesa-dev] [PATCH 2/7] automake: src/mesa/drivers/dri/i915

2012-01-17 Thread Ian Romanick
On 01/17/2012 03:09 PM, Matt Turner wrote: On Tue, Jan 17, 2012 at 5:51 PM, Ian Romanick wrote: On 01/17/2012 12:58 PM, Matt Turner wrote: --- a/configure.ac +++ b/configure.ac @@ -1292,6 +1292,11 @@ case $DRI_DIRS in PKG_CHECK_MODULES([INTEL], [libdrm_intel>= $LIBDRM_INTEL_REQUIRED])

Re: [Mesa-dev] [PATCH 3/3] i965: Add support for Z16 depth formats.

2012-01-17 Thread Chad Versace
On 01/17/2012 04:57 PM, Eric Anholt wrote: > v2: Don't flag the format as being HiZ ready (there's DRI2 handshake > pain to go through). I don't want to enable HiZ on z16 until we can do extensive test coverage on that visual. So, we need to ensure that QA is running oglc with that visual. And

Re: [Mesa-dev] [PATCH 5/7] automake: src/mesa/drivers/dri/nouveau

2012-01-17 Thread Matt Turner
On Tue, Jan 17, 2012 at 7:32 PM, Gaetan Nadon wrote: > On 12-01-17 03:58 PM, Matt Turner wrote: > > +AM_CPPFLAGS = \ > + -I$(top_srcdir)/include \ > + -I$(top_srcdir)/src/ \ > + -I$(top_srcdir)/src/mapi \ > + -I$(top_srcdir)/src/mesa/ \ > + -I../common \ > > Why does this one c

Re: [Mesa-dev] [PATCH 1/3] i965/gen7: Remove stale comment.

2012-01-17 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/17/2012 04:57 PM, Eric Anholt wrote: > This field is actually set up above. > --- > src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |1 - > 1 files changed, 0 insertions(+), 1 deletions(-) For the series: Reviewed-by: Chad Versace -B

Re: [Mesa-dev] [PATCH 1/7] i965: use AM_CPPFLAGS instead of AM_CFLAGS/CXXFLAGS

2012-01-17 Thread Matt Turner
On Tue, Jan 17, 2012 at 7:21 PM, Gaetan Nadon wrote: > On 12-01-17 03:58 PM, Matt Turner wrote: >> --- >>  src/mesa/drivers/dri/i965/Makefile.am |    4 +--- >>  1 files changed, 1 insertions(+), 3 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/i965/Makefile.am >> b/src/mesa/drivers/dri/i965

Re: [Mesa-dev] [PATCH] Revert "mesa: check depth, stencil formats (not depths) in glBlitFramebuffer"

2012-01-17 Thread Ian Romanick
On 01/17/2012 05:12 PM, Chad Versace wrote: This reverts commit 3f1fab06844f696de44d9a56e83ff62e8ea576bd. This loosens the format validation in glBlitFramebuffer. When blitting depth bits, don't require an exact match between the depth formats; only require that the two formats have the same num

[Mesa-dev] [PATCH] Revert "mesa: check depth, stencil formats (not depths) in glBlitFramebuffer"

2012-01-17 Thread Chad Versace
This reverts commit 3f1fab06844f696de44d9a56e83ff62e8ea576bd. This loosens the format validation in glBlitFramebuffer. When blitting depth bits, don't require an exact match between the depth formats; only require that the two formats have the same number of depth bits. Ditto for stencil. Fixes P

Re: [Mesa-dev] [PATCH] i965/gen5: Fix rendering of depth buffers without stencil

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 04:03 PM, Chad Versace wrote: Fixes the following OGLConform tests on gen5: depth-stencil(misc.state_on.depth_int) fbo_db_ARBfp(basic.OnlyDepthBuffDrawBufferRender) The problem was that, if the depth buffer's Mesa format was X8_Z24, then we emitted the hardware format D24

Re: [Mesa-dev] [PATCH] i965/vs: Fix URB entry size calculations on Ivybridge.

2012-01-17 Thread Eric Anholt
On Tue, 17 Jan 2012 21:33:09 -0800, Kenneth Graunke wrote: > This is inspired by the BSpec, vol2a 3D Pipeline - Overview: > 3D Pipeline / 3D Pipeline State Overview / Push Constant URB Allocation. > In particular, the last paragraph. > > The new VS change fixes vertex scrambling in GLBenchmark P

[Mesa-dev] [PATCH 2/3] i965/gen7: Set up texture horizontal alignment field.

2012-01-17 Thread Eric Anholt
This is required for Z16 support, which is the first thing to be textured with horizontal alignment of 8. --- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c b/src/mesa/d

[Mesa-dev] [PATCH 3/3] i965: Add support for Z16 depth formats.

2012-01-17 Thread Eric Anholt
v2: Don't flag the format as being HiZ ready (there's DRI2 handshake pain to go through). Fixes piglit gl-3.0-required-sized-texture-formats --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/mesa/drivers/dri/i965/

[Mesa-dev] [PATCH 1/3] i965/gen7: Remove stale comment.

2012-01-17 Thread Eric Anholt
This field is actually set up above. --- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c index aed0b5d..25d4a3e 100644 -

[Mesa-dev] [PATCH] mesa: Set default access flags based on the run-time API

2012-01-17 Thread Ian Romanick
From: Ian Romanick The default access flags for OpenGL ES (via GL_OES_map_buffer) and desktop OpenGL are different. The code previously tried to handle this, but the decision was made at compile time. Since the same driver binary can be used for both OpenGL ES and desktop OpenGL, the decision m

Re: [Mesa-dev] [PATCH] intel: Return if pointer to intel_context is null

2012-01-17 Thread Chad Versace
On 01/17/2012 02:00 PM, Anuj Phogat wrote: > It is better to test if(intel == NULL) and simply return in that case. > > Signed-off-by: Anuj Phogat > --- > src/mesa/drivers/dri/intel/intel_screen.c | 14 +++--- > 1 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/src/mes

Re: [Mesa-dev] [PATCH] glsl: Fix 'control reaches end of non-void function' warning.

2012-01-17 Thread Chad Versace
On 01/16/2012 09:10 PM, Vinson Lee wrote: > Fix this GCC warning on non-debug builds. > glsl_types.cpp: In member function 'gl_texture_index > glsl_type::sampler_index() const': > glsl_types.cpp:157: warning: control reaches end of non-void function > > Signed-off-by: Vinson Lee > --- > src/glsl

[Mesa-dev] [PATCH] i965/gen5: Fix rendering of depth buffers without stencil

2012-01-17 Thread Chad Versace
Fixes the following OGLConform tests on gen5: depth-stencil(misc.state_on.depth_int) fbo_db_ARBfp(basic.OnlyDepthBuffDrawBufferRender) The problem was that, if the depth buffer's Mesa format was X8_Z24, then we emitted the hardware format D24_UNORM_X8. But, on gen5, D24_UNORM_S8 must be em

Re: [Mesa-dev] [PATCH] st/dri: Remove useless flush front.

2012-01-17 Thread Stéphane Marchesin
On Tue, Jan 17, 2012 at 15:47, Stéphane Marchesin wrote: > On Wed, Jan 11, 2012 at 10:13, Jose Fonseca wrote: >> - Original Message - >>> Where else would the flush go? >> >> Maybe one of the callers already invoked glFlush, or something like that. >> >>> I'll look into fixing it differen

Re: [Mesa-dev] [PATCH 2/7] automake: src/mesa/drivers/dri/i915

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 03:09 PM, Matt Turner wrote: On Tue, Jan 17, 2012 at 5:51 PM, Ian Romanick wrote: On 01/17/2012 12:58 PM, Matt Turner wrote: --- a/configure.ac +++ b/configure.ac @@ -1292,6 +1292,11 @@ case $DRI_DIRS in PKG_CHECK_MODULES([INTEL], [libdrm_intel>= $LIBDRM_INTEL_REQUIRED])

Re: [Mesa-dev] [PATCH] st/dri: Remove useless flush front.

2012-01-17 Thread Stéphane Marchesin
On Wed, Jan 11, 2012 at 10:13, Jose Fonseca wrote: > - Original Message - >> Where else would the flush go? > > Maybe one of the callers already invoked glFlush, or something like that. > >> I'll look into fixing it differently >> but if the flush is anywhere in the makecurrent path, the s

Re: [Mesa-dev] [PATCH] i965: Add .gitignore file to exclude automake build artifacts from git.

2012-01-17 Thread Paul Berry
On 17 January 2012 15:33, Kenneth Graunke wrote: > On 01/17/2012 02:37 PM, Paul Berry wrote: > >> With the conversion to automake in commit >> e326480e4ebe8687948041c2dc5f5b**7595559a2e, several additional build >> artifacts are created: >> >> src/mesa/drivers/dri/i965/.**deps/ >> src/mesa/dr

Re: [Mesa-dev] [PATCH v2] intel: Set depth to 6 for cubemaps

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 02:55 PM, Ian Romanick wrote: From: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41216 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43212 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43250 Signed-off-by: Ian Romanick Reviewed-by: Chad Vers

Re: [Mesa-dev] [PATCH] i965: Add .gitignore file to exclude automake build artifacts from git.

2012-01-17 Thread Kenneth Graunke
On 01/17/2012 02:37 PM, Paul Berry wrote: With the conversion to automake in commit e326480e4ebe8687948041c2dc5f5b7595559a2e, several additional build artifacts are created: src/mesa/drivers/dri/i965/.deps/ src/mesa/drivers/dri/i965/.libs/ src/mesa/drivers/dri/i965/Makefile src/mesa/

Re: [Mesa-dev] [PATCH] mesa: Add condition in glGetTexImage for zero size textures

2012-01-17 Thread Anuj Phogat
On Tue 17 Jan 2012 02:54:41 PM PST, Brian Paul wrote: > On 01/17/2012 02:03 PM, Anuj Phogat wrote: >> TestMipMaps() function in src/OGLconform/textureNPOT.c calls >> glTexImage2D() >> with width = 0. Texture with zero size skips miptree allocation due to a >> condition in function _mesa_store_texim

Re: [Mesa-dev] [PATCH] i965: Add .gitignore file to exclude automake build artifacts from git.

2012-01-17 Thread Matt Turner
On Tue, Jan 17, 2012 at 5:37 PM, Paul Berry wrote: > With the conversion to automake in commit > e326480e4ebe8687948041c2dc5f5b7595559a2e, several additional build > artifacts are created: > >  src/mesa/drivers/dri/i965/.deps/ >  src/mesa/drivers/dri/i965/.libs/ >  src/mesa/drivers/dri/i965/Makefi

Re: [Mesa-dev] [PATCH 1/4] glapi: remove non-existent parameter references

2012-01-17 Thread Brian Paul
On 01/16/2012 04:45 PM, nobled wrote: glGetTexImage, for example, has no width/height/depth parameters. Also, copy some missing parameter info from the original versions of certain functions over to their ARB_robustness counterparts. These look OK to me. There's no real change in the generate

Re: [Mesa-dev] [PATCH 2/7] automake: src/mesa/drivers/dri/i915

2012-01-17 Thread Matt Turner
On Tue, Jan 17, 2012 at 5:51 PM, Ian Romanick wrote: > On 01/17/2012 12:58 PM, Matt Turner wrote: >> --- a/configure.ac >> +++ b/configure.ac >> @@ -1292,6 +1292,11 @@ case $DRI_DIRS in >>      PKG_CHECK_MODULES([INTEL], [libdrm_intel>= $LIBDRM_INTEL_REQUIRED]) >> >>      case $DRI_DIRS in >> +  

Re: [Mesa-dev] renderbuffer-cleanups-v2 branch

2012-01-17 Thread Brian Paul
On 01/16/2012 08:30 PM, Brian Paul wrote: On Mon, Jan 16, 2012 at 4:31 PM, Ian Romanick wrote: On 01/16/2012 01:30 PM, Brian Paul wrote: The renderbuffer-cleanups-v2 branch removes all the old swrast GetRow/PutRow stuff. All swrast rendering is now done through renderbuffer mapping and the f

[Mesa-dev] [PATCH v2] intel: Set depth to 6 for cubemaps

2012-01-17 Thread Ian Romanick
From: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41216 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43212 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43250 Signed-off-by: Ian Romanick Reviewed-by: Chad Versace Cc: Jin Yang --- v2: Rework the change

Re: [Mesa-dev] [PATCH] mesa: Add condition in glGetTexImage for zero size textures

2012-01-17 Thread Brian Paul
On 01/17/2012 02:03 PM, Anuj Phogat wrote: TestMipMaps() function in src/OGLconform/textureNPOT.c calls glTexImage2D() with width = 0. Texture with zero size skips miptree allocation due to a condition in function _mesa_store_teximage3d(). While calling glGetTexImage() it results in assertion fai

Re: [Mesa-dev] [PATCH 2/7] automake: src/mesa/drivers/dri/i915

2012-01-17 Thread Ian Romanick
On 01/17/2012 12:58 PM, Matt Turner wrote: --- configs/linux-dri |2 +- configure.ac |7 +++ src/mesa/drivers/dri/i915/Android.mk |2 +- src/mesa/drivers/dri/i915/Makefile | 30 - src/mesa/drivers

[Mesa-dev] [PATCH] i965: Add .gitignore file to exclude automake build artifacts from git.

2012-01-17 Thread Paul Berry
With the conversion to automake in commit e326480e4ebe8687948041c2dc5f5b7595559a2e, several additional build artifacts are created: src/mesa/drivers/dri/i965/.deps/ src/mesa/drivers/dri/i965/.libs/ src/mesa/drivers/dri/i965/Makefile src/mesa/drivers/dri/i965/Makefile.in src/mesa/drivers/

[Mesa-dev] [PATCH] intel: Set depth to 6 for cubemaps

2012-01-17 Thread Ian Romanick
From: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41216 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43212 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43250 Signed-off-by: Ian Romanick Cc: Jin Yang --- src/mesa/drivers/dri/i915/i915_tex_layout.c |

[Mesa-dev] [PATCH] intel: Return if pointer to intel_context is null

2012-01-17 Thread Anuj Phogat
It is better to test if(intel == NULL) and simply return in that case. Signed-off-by: Anuj Phogat --- src/mesa/drivers/dri/intel/intel_screen.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/

[Mesa-dev] [PATCH] i965/vs: Fix URB entry size calculations on Ivybridge.

2012-01-17 Thread Kenneth Graunke
This is inspired by the BSpec, vol2a 3D Pipeline - Overview: 3D Pipeline / 3D Pipeline State Overview / Push Constant URB Allocation. In particular, the last paragraph. The new VS change fixes vertex scrambling in GLBenchmark PRO; the old VS change is just for good measure/symmetry. NOTE: This is

Re: [Mesa-dev] [PATCH] Always build shared glapi

2012-01-17 Thread Kenneth Graunke
On 01/11/2012 04:42 PM, Chia-I Wu wrote: On Thu, Jan 12, 2012 at 7:47 AM, Jakob Bornecrantz wrote: On Thu, Jan 12, 2012 at 12:37 AM, Matt Turner wrote: --- No one on IRC knows why an unshared glapi is useful. Does anyone have a use-case for this? If not, we can drop it to simplify automake

[Mesa-dev] [PATCH] mesa: Add condition in glGetTexImage for zero size textures

2012-01-17 Thread Anuj Phogat
TestMipMaps() function in src/OGLconform/textureNPOT.c calls glTexImage2D() with width = 0. Texture with zero size skips miptree allocation due to a condition in function _mesa_store_teximage3d(). While calling glGetTexImage() it results in assertion failure in intel_map_texture_image() due to null

[Mesa-dev] [PATCH] mesa: Add condition in glGetTexImage for zero size textures

2012-01-17 Thread Anuj Phogat
TestMipMaps() function in src/OGLconform/textureNPOT.c calls glTexImage2D() with width = 0. Texture with zero size skips miptree allocation due to a condition in function _mesa_store_teximage3d(). While calling glGetTexImage() it results in assertion failure in intel_map_texture_image() due to null

[Mesa-dev] [PATCH 7/7] automake: src/mesa/drivers/dri

2012-01-17 Thread Matt Turner
--- configure.ac |2 + src/mesa/drivers/dri/Makefile | 55 --- src/mesa/drivers/dri/Makefile.am | 28 src/mesa/drivers/dri/Makefile.defines | 27 --- src/mesa/drivers/dri/Makefile.targets | 78 --

[Mesa-dev] [PATCH 6/7] automake: src/mesa/drivers/dri/swrast

2012-01-17 Thread Matt Turner
--- configure.ac |8 src/mesa/drivers/dri/swrast/Makefile | 27 src/mesa/drivers/dri/swrast/Makefile.am | 58 ++ src/mesa/drivers/dri/swrast/Makefile.sources | 11 + 4 files changed, 77 insertions(

[Mesa-dev] [PATCH 5/7] automake: src/mesa/drivers/dri/nouveau

2012-01-17 Thread Matt Turner
--- configs/linux-dri |2 +- configure.ac | 15 --- src/mesa/drivers/dri/nouveau/Makefile | 58 src/mesa/drivers/dri/nouveau/Makefile.am | 59 + src/mesa/drivers

[Mesa-dev] [PATCH 4/7] automake: src/mesa/drivers/dri/r200

2012-01-17 Thread Matt Turner
--- configs/freebsd-dri|3 - configs/linux-dri |2 +- configs/linux-dri-debug|6 --- configs/linux-dri-ppc |5 -- configs/linux-dri-x86-64 |3 - configs/linux-dri-xcb

[Mesa-dev] [PATCH 3/7] automake: src/mesa/drivers/dri/radeon

2012-01-17 Thread Matt Turner
--- configs/freebsd-dri |2 +- configs/linux-dri|2 +- configs/linux-dri-debug |2 +- configs/linux-dri-ppc|2 +- configs/linux-dri-x86-64 |2 +- configs/linux-

[Mesa-dev] [PATCH 2/7] automake: src/mesa/drivers/dri/i915

2012-01-17 Thread Matt Turner
--- configs/linux-dri |2 +- configure.ac |7 +++ src/mesa/drivers/dri/i915/Android.mk |2 +- src/mesa/drivers/dri/i915/Makefile | 30 - src/mesa/drivers/dri/i915/Makefile.am | 63

[Mesa-dev] [PATCH 1/7] i965: use AM_CPPFLAGS instead of AM_CFLAGS/CXXFLAGS

2012-01-17 Thread Matt Turner
--- src/mesa/drivers/dri/i965/Makefile.am |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/Makefile.am b/src/mesa/drivers/dri/i965/Makefile.am index 5512381..93937b1 100644 --- a/src/mesa/drivers/dri/i965/Makefile.am +++ b/src/mesa/drivers/dri/

Re: [Mesa-dev] [PATCH] i965: Remove the INTEL_OLD_VS option.

2012-01-17 Thread Eugeni Dodonov
On Tue, Jan 17, 2012 at 10:57, Kenneth Graunke wrote: > Now that we no longer generate Mesa IR from GLSL IR, it's impossible to > use the old vertex shader backend for GLSL programs. There's simply no > Mesa IR to codegen from. > > Any attempt to do so would result in immediate GPU hangs, presuma

Re: [Mesa-dev] [PATCH] Always build shared glapi

2012-01-17 Thread Alexander von Gluck
On 17.01.2012 14:15, Matt Turner wrote: On Wed, Jan 11, 2012 at 6:37 PM, Matt Turner wrote: --- No one on IRC knows why an unshared glapi is useful. Does anyone have a use-case for this? If not, we can drop it to simplify automake work. Okay to commit? Anyone tested it? Haiku uses all sta

Re: [Mesa-dev] llvmpipe lp_build_context

2012-01-17 Thread Jose Fonseca
- Original Message - > > > > That is, lp_bld_arit.c functions such as lp_build_add() are be > > confident that passed the values and types are consistent, and all > > assertions to test that should be preserved. > > > > I hope this makes sense. > > Thanks Jose, this seems sensible and I'

Re: [Mesa-dev] [PATCH 1/4] Always build shared dricore

2012-01-17 Thread Matt Turner
On Sat, Jan 14, 2012 at 11:15 AM, Matt Turner wrote: > There don't seem to be any reasons to prefer unshared-dricore, > and clang and icc both support rpath. > > Signed-off-by: Matt Turner Okay to commit? Anyone tested it? ___ mesa-dev mailing list mes

Re: [Mesa-dev] [PATCH] Always build shared glapi

2012-01-17 Thread Matt Turner
On Wed, Jan 11, 2012 at 6:37 PM, Matt Turner wrote: > --- > No one on IRC knows why an unshared glapi is useful. > > Does anyone have a use-case for this? > > If not, we can drop it to simplify automake work. Okay to commit? Anyone tested it? ___ mesa-d

[Mesa-dev] [PATCH 2/2] scons: Don't set visibility to hidden on Haiku

2012-01-17 Thread Alexander von Gluck
The Haiku swrast driver is out of tree. --- scons/gallium.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scons/gallium.py b/scons/gallium.py index 5a0c6fe..86adc3e 100755 --- a/scons/gallium.py +++ b/scons/gallium.py @@ -350,7 +350,7 @@ def generate(env):

[Mesa-dev] [PATCH 1/2] scons: Add Haiku build support

2012-01-17 Thread Alexander von Gluck
Enables building stock Mesa under the Haiku operating system. --- common.py |2 +- scons/gallium.py |2 ++ src/SConscript |5 +++-- src/gallium/SConscript | 14 ++ src/glu/sgi/SConscript | 16 +++- 5 files changed, 27 insertions(

[Mesa-dev] [PATCH] i965: Bump Ivybridge's fake MRF range to g112-127 instead of g111-126.

2012-01-17 Thread Kenneth Graunke
When I originally implemented the hack to use GRFs 111+ as fake MRFs, I did so purely to avoid rewriting all the code that dealt with MRFs. However, it turns out that a similar hack is actually required. Newly discovered language in the BSpec indicates that SEND instructions with EOT set "should"

Re: [Mesa-dev] [Nouveau] [PATCH 4/4] nvfx: random cleanups of the state validation code

2012-01-17 Thread Lucas Stach
Am Dienstag, den 17.01.2012, 18:22 +0100 schrieb Patrice Mandin: > Le Tue, 10 Jan 2012 12:41:04 +0100 > Lucas Stach a écrit: > > > Signed-off-by: Lucas Stach > > --- > > src/gallium/drivers/nvfx/nvfx_state_emit.c | 49 > > --- > > 1 files changed, 22 insertions(+), 27

[Mesa-dev] [PATCH] mesa: allocate transform_feedback_info::Outputs array dynamically

2012-01-17 Thread Christoph Bumiller
The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS which made it always assert in the linker when TFB was used since the Outputs array was smaller than that maximum. v2: added assertions NOTE: This is a candidate for the 8.0 branch. --- src/glsl/linker.cpp| 64 +++

Re: [Mesa-dev] [PATCH] mesa: allocate transform_feedback_info::Outputs array dynamically

2012-01-17 Thread Paul Berry
On 17 January 2012 09:31, Christoph Bumiller wrote: > The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS > which made it always assert in the linker when TFB was used since > the Outputs array was smaller than that maximum. > > NOTE: This is a candidate for the 8.0 branch. > --- > s

Re: [Mesa-dev] [PATCH 3/4] i965: Convert the build to using automake.

2012-01-17 Thread Eric Anholt
On Thu, 12 Jan 2012 20:52:33 -0500, Matt Turner wrote: > On Thu, Jan 12, 2012 at 7:08 PM, Eric Anholt wrote: > > This does introduce a warning by the automake build system, that the > > missing-symbols test build is non-portable.  That's true -- Mac OS X > > can't take something built as a loadab

[Mesa-dev] [Bug 40612] Mesa-demos: build error against current Mesa git master

2012-01-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40612 Ian Romanick changed: What|Removed |Added AssignedTo|mesa-dev@lists.freedesktop. |k...@bitplanet.net |org

Re: [Mesa-dev] [PATCH 1/2] mesa: use GL_MAP_INVALIDATE_RANGE_BIT in glTexImage paths

2012-01-17 Thread Ian Romanick
On 01/17/2012 07:19 AM, Brian Paul wrote: Update the dd.h docs to indicate that GL_MAP_INVALIDATE_RANGE_BIT can be used with GL_MAP_WRITE_BIT when mapping renderbuffers and texture images. Pass the flag when mapping texture images for glTexImage, glTexSubImage, etc. It's up to drivers whether t

Re: [Mesa-dev] [PATCH 1/6] tgsi: Add output_mode to struct tgsi_opcode_info v2

2012-01-17 Thread Jose Fonseca
Excellent. Thanks. Jose - Original Message - > v2: > - Rename output_type to output_mode > - Add shorthand definitions for TGSI_OUTPUT_* > --- > src/gallium/auxiliary/tgsi/tgsi_info.c | 329 > > src/gallium/auxiliary/tgsi/tgsi_info.h | 34 > 2 fi

Re: [Mesa-dev] [PATCH] mesa: increase size of TFB info Outputs array

2012-01-17 Thread Eric Anholt
On Tue, 17 Jan 2012 17:50:51 +0100, Christoph Bumiller wrote: > On 01/12/2012 10:53 PM, Christoph Bumiller wrote: > > The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS > > which made it always assert in the linker when TFB was used. > > > > Going to push this soon if no one minds

Re: [Mesa-dev] [PATCH] i965: Remove the INTEL_OLD_VS option.

2012-01-17 Thread Eric Anholt
On Tue, 17 Jan 2012 04:57:54 -0800, Kenneth Graunke wrote: > Now that we no longer generate Mesa IR from GLSL IR, it's impossible to > use the old vertex shader backend for GLSL programs. There's simply no > Mesa IR to codegen from. > > Any attempt to do so would result in immediate GPU hangs,

Re: [Mesa-dev] [PATCH] mesa: increase size of TFB info Outputs array

2012-01-17 Thread Christoph Bumiller
On 01/17/2012 05:50 PM, Christoph Bumiller wrote: > On 01/12/2012 10:53 PM, Christoph Bumiller wrote: >> The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS >> which made it always assert in the linker when TFB was used. >> > > Going to push this soon if no one minds ... though maybe

[Mesa-dev] [PATCH] mesa: allocate transform_feedback_info::Outputs array dynamically

2012-01-17 Thread Christoph Bumiller
The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS which made it always assert in the linker when TFB was used since the Outputs array was smaller than that maximum. NOTE: This is a candidate for the 8.0 branch. --- src/glsl/linker.cpp| 57 +

Re: [Mesa-dev] [Nouveau] [PATCH 4/4] nvfx: random cleanups of the state validation code

2012-01-17 Thread Patrice Mandin
Le Tue, 10 Jan 2012 12:41:04 +0100 Lucas Stach a écrit: > Signed-off-by: Lucas Stach > --- > src/gallium/drivers/nvfx/nvfx_state_emit.c | 49 --- > 1 files changed, 22 insertions(+), 27 deletions(-) > > diff --git a/src/gallium/drivers/nvfx/nvfx_state_emit.c > b/src/

Re: [Mesa-dev] [PATCH] mesa: increase size of TFB info Outputs array

2012-01-17 Thread Christoph Bumiller
On 01/12/2012 10:53 PM, Christoph Bumiller wrote: > The nvc0 gallium driver is advertising 128 MAX_INTERLEAVED_COMPS > which made it always assert in the linker when TFB was used. > Going to push this soon if no one minds ... though maybe I should just change the limit in the driver to 64, this s

Re: [Mesa-dev] [PATCH] Complete ARGB8888 naming convention format renames missed by 58dc1b28d1ef4d1931c52b079d304f2e1546329d

2012-01-17 Thread Kristian Høgsberg
On Tue, Jan 17, 2012 at 4:54 AM, Scott Moreau wrote: > Looks like a few renames were missed in > 58dc1b28d1ef4d1931c52b079d304f2e1546329d as mesa failed to build with: Oops. Thanks, pushed. > wayland/native_shm.c: In function ‘wayland_create_shm_buffer’: > wayland/native_shm.c:98:10: error: ‘WL

Re: [Mesa-dev] [PATCH 6/6] gallivm: Add a new interface for doing TGSI->LLVM conversions

2012-01-17 Thread Tom Stellard
On Mon, 2012-01-16 at 11:27 -0800, Jose Fonseca wrote: > Tom, > > Looks good in principle! > > But I need to test this myself before I can be comfortable w/ merging it into > master. > > Just a few quick comments inline. > > Jose > > - Original Message - > > From: Tom Stellard > > >

[Mesa-dev] [PATCH 2/2] st/mesa: pass GL_MAP_INVALIDATE_RANGE_BIT to gallium drivers

2012-01-17 Thread Brian Paul
when mapping renderbuffers or texture images. NOTE: This is a candidate for the 8.0 branch. --- src/mesa/state_tracker/st_cb_fbo.c |2 ++ src/mesa/state_tracker/st_cb_texture.c |2 ++ 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_fbo.c b/

[Mesa-dev] [PATCH 1/2] mesa: use GL_MAP_INVALIDATE_RANGE_BIT in glTexImage paths

2012-01-17 Thread Brian Paul
Update the dd.h docs to indicate that GL_MAP_INVALIDATE_RANGE_BIT can be used with GL_MAP_WRITE_BIT when mapping renderbuffers and texture images. Pass the flag when mapping texture images for glTexImage, glTexSubImage, etc. It's up to drivers whether to actually make use of the flag. NOTE: This

[Mesa-dev] [PATCH] Complete ARGB8888 naming convention format renames missed by 58dc1b28d1ef4d1931c52b079d304f2e1546329d

2012-01-17 Thread Scott Moreau
Looks like a few renames were missed in 58dc1b28d1ef4d1931c52b079d304f2e1546329d as mesa failed to build with: wayland/native_shm.c: In function ‘wayland_create_shm_buffer’: wayland/native_shm.c:98:10: error: ‘WL_SHM_FORMAT_PREMULTIPLIED_ARGB32’ undeclared (first use in this function) wayland/nati

Re: [Mesa-dev] [PATCH] mesa: Add condition in glGetTexImage for zero size textures

2012-01-17 Thread Brian Paul
On 01/16/2012 05:40 PM, Anuj Phogat wrote: TestMipMaps() function in src/OGLconform/textureNPOT.c calls glTexImage2D() with width = 0. Texture with zero size skips miptree allocation due to a condition in function _mesa_store_teximage3d(). While calling glGetTexImage() it results in assertion fai

[Mesa-dev] [PATCH] r600g: rework IDIV/UDIV and implement MOD/UMOD (v2)

2012-01-17 Thread Vadim Girlin
Signed-off-by: Vadim Girlin --- Tested on evergreen. src/gallium/drivers/r600/r600_asm.c|2 + src/gallium/drivers/r600/r600_shader.c | 724 ++-- 2 files changed, 593 insertions(+), 133 deletions(-) diff --git a/src/gallium/drivers/r600/r600_asm.c b/src/ga

[Mesa-dev] [Bug 44239] spring rts crashes with r300g

2012-01-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44239 --- Comment #10 from Pawel 2012-01-17 03:50:31 PST --- I forget to add game seems to be running - I can hear the music and sound when I click somewhere, but what I see is just flickering Spring RTS menu. -- Configure bugmail: https://bugs.freed

[Mesa-dev] [Bug 44239] spring rts crashes with r300g

2012-01-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=44239 --- Comment #9 from Pawel 2012-01-17 03:49:22 PST --- Created attachment 55668 --> https://bugs.freedesktop.org/attachment.cgi?id=55668 dmesg errors when running Spring RTS I reverted to linux-generic (3.0.0-14-generic #23-Ubuntu SMP) and purg

[Mesa-dev] [Bug 43879] (xorg-server 1.11.2-2, xf86-video-ati 6.14.3-1 or git, qt 4.7.4) black pixels instead of transparency from PNGs

2012-01-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43879 --- Comment #6 from Michel Dänzer 2012-01-17 02:30:34 PST --- This seems to be a Qt bug. It works correctly for me with Qt 4.8. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail beca

Re: [Mesa-dev] [PATCH] util: Silence GCC unused-but-set-variable warning.

2012-01-17 Thread Tolga Dalman
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 16.01.2012 19:05, Jose Fonseca wrote: >> Why not just move the definition of frame_pointer into the >> #ifdef PIPE_ARCH_X86 ? > > It would be cleaner, but it would require C99 as the declaration > would appear after code, which would break MSVC.