Re: [Mesa-dev] [RFC] [PATCH 1/1] mesa: setup_glsl_generate_mipmap(): meta program compile failed

2012-10-03 Thread Matt Turner
> mesa: setup_glsl_generate_mipmap(): meta program compile failed This is not a commit title. After reading the title, I should have at least some idea of what the commit does. Pasting the error tells me nothing. ___ mesa-dev mailing list mesa-dev@lists.

[Mesa-dev] [RFC] [PATCH 1/1] mesa: setup_glsl_generate_mipmap(): meta program compile failed

2012-10-03 Thread Oliver McFadden
Discovered while attempting to run GLBenchMark 2.5 with test GLB25_TriangleTexVertexLitTestC24Z16 on an ES2.0 context. I have changed the `if' condition in setup_glsl_generate_mipmap() such that it is consistent throughout the function. This required swapping the code blocks of the `if-else' stat

Re: [Mesa-dev] [PATCH] mesa: don't enable glVertexPointer() when using API_OPENGLES2.

2012-10-03 Thread Oliver McFadden
On Wed, Oct 03, 2012 at 02:47:48PM -0700, Paul Berry wrote: > This function is only present in GLES1 and in the OpenGL compatibility > profile. > > Fixes the following "make check" failure: > > [--] 1 test from DispatchSanity_test > [ RUN ] DispatchSanity_test.GLES2 > Mes

[Mesa-dev] [Bug 55599] New: LTO build breakage

2012-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55599 Priority: medium Bug ID: 55599 Assignee: mesa-dev@lists.freedesktop.org Summary: LTO build breakage Severity: critical Classification: Unclassified OS: Linux (All)

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Oliver McFadden
On Wed, Oct 03, 2012 at 07:45:12AM -0600, Brian Paul wrote: > On 10/03/2012 02:37 AM, Oliver McFadden wrote: > > On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: > >> On 10/02/2012 04:00 AM, Oliver McFadden wrote: > >>> This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. >

[Mesa-dev] [Bug 53179] libOSMesa / libdricore not linked well

2012-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53179 --- Comment #5 from Matt Turner --- Patch committed to 9.0: http://cgit.freedesktop.org/mesa/mesa/commit/?h=9.0&id=c8669c7ba7a01780d4cfd1cfc6ab8cc6f7fc2510 -- You are receiving this mail because: You are the assignee for the bug. _

[Mesa-dev] [PATCH] r600g: add in-place DB decompression and texturing with DB tiling

2012-10-03 Thread Marek Olšák
The decompression is done in-place and only the compressed tiles are decompressed. Note: R6xx-R7xx can do that only with Z16 and Z32F. The texture unit is programmed to use non-displayable tiling and depth ordering of samples, so that it can fetch the texture in the native DB format. The latest v

[Mesa-dev] [PATCH] mesa: don't enable glVertexPointer() when using API_OPENGLES2.

2012-10-03 Thread Paul Berry
This function is only present in GLES1 and in the OpenGL compatibility profile. Fixes the following "make check" failure: [--] 1 test from DispatchSanity_test [ RUN ] DispatchSanity_test.GLES2 Mesa warning: couldn't open libtxc_dxtn.so, software DXTn compression/decom

Re: [Mesa-dev] [PATCH 19/19] mesa/tests: Sanity check the ES2 dispatch table

2012-10-03 Thread Paul Berry
On 1 October 2012 17:07, Oliver McFadden wrote: > On Fri, Sep 28, 2012 at 02:15:18PM -0700, Ian Romanick wrote: > > On 09/28/2012 01:55 PM, Paul Berry wrote: > > > On 5 September 2012 15:09, Ian Romanick > > > wrote: > > > > > > From: Ian Romanick > >

[Mesa-dev] [Bug 53179] libOSMesa / libdricore not linked well

2012-10-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53179 --- Comment #4 from Matt Turner --- Patch committed to master: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1762ec28db4bfb85eeb6e61377839a3889f77216 -- You are receiving this mail because: You are the assignee for the bug. _

[Mesa-dev] [PATCH 2/2] r600g: ensure PERFECT_ZPASS+NOOP_CULL_DISABLE are 0 for blits+decompression

2012-10-03 Thread Marek Olšák
When an occlusion query was active, the derived DB state wasn't changed for u_blitter even though all the occlusion queries were suspended. It's fixed by moving the state update into the emit functions, which are called whenever queries are stopped or suspended. --- src/gallium/drivers/r600/r600_

[Mesa-dev] [PATCH 1/2] r600g: don't modify pipe_resource in resource_copy_region, fixing race condition

2012-10-03 Thread Marek Olšák
pipe_resource can be shared between contexts, we shouldn't modify its description. Instead, let's use the resource "views" (sampler views and surfaces), where we can freely change almost any property of a resource. --- src/gallium/drivers/r600/evergreen_state.c | 21 ++- src/gallium/drivers/r600

[Mesa-dev] [PATCH] intel: Fix intel_texsubimage_tiled_memcpy to skip GL_EXT_unpack_subimage case

2012-10-03 Thread Rob Bradford
From: Rob Bradford 413c49141 added an optimisation to improve the performance of teximage under a limited set of circumstances. If GL_EXT_unpack_subimage has been used then we we must also skip this optimisation since the optimised codepath does not take the packing values into consideration. Si

Re: [Mesa-dev] [git pull] 8.0 fixes

2012-10-03 Thread Andreas Boll
2012/9/17 Ian Romanick : > On 09/17/2012 06:11 PM, Andreas Boll wrote: >> >> 2012/9/10 Andreas Boll : >>> >>> Hi Ian, >>> >>> I've rebased on your 8.0-staging branch. >>> You should probably squash the commit >>> >>>6cabf2532 radeon: fix Base/base typo >>> >>> into commit >>> >>>35af2b6fe r

Re: [Mesa-dev] [PATCH 1/2] i965/vs: Fix unit mismatch in scratch base_offset parameter.

2012-10-03 Thread Eric Anholt
Kenneth Graunke writes: > move_grf_array_access_to_scratch() calculates scratch buffer offsets in > bytes. However, emit_scratch_read/write() expects the base_offset > parameter to be measured in OWords. An OWORD is 4 floats (in powers of two: dword, qword, oword). I forget the stupid name we

[Mesa-dev] [PATCH] i965/vs: Try again when we've successfully spilled a reg.

2012-10-03 Thread Eric Anholt
Without this, we'd spill one reg, then continue on without actually register allocating, then assertion fail when we tried to use a vgrf number as a register number. --- src/mesa/drivers/dri/i965/brw_vec4.h|4 ++-- src/mesa/drivers/dri/i965/brw_vec4_emit.cpp |5

Re: [Mesa-dev] [PATCH 2/2] i965/vs: Implement register spilling.

2012-10-03 Thread Eric Anholt
Kenneth Graunke writes: > To validate this code, I ran piglit -t vs quick.tests with the "go spill > everything" debugging code enabled. There was only one regression: > glsl-vs-unroll-explosion simply ran out of registers. This should be > fine in the real world, since no one actually spills e

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Brian Paul
On 10/03/2012 02:37 AM, Oliver McFadden wrote: On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: On 10/02/2012 04:00 AM, Oliver McFadden wrote: This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=55523 Signed-off-b

Re: [Mesa-dev] [Linaro-mm-sig] [RFC] New dma_buf -> EGLImage EGL extension

2012-10-03 Thread Rob Clark
On Tue, Oct 2, 2012 at 2:10 PM, Maarten Lankhorst wrote: > How do you want to deal with the case where Y' and CbCr are different > hardware buffers? > Could some support for 2d arrays be added in case Y' and CbCr are separated > into top/bottom fields? > How are semi-planar/planar formats handle

Re: [Mesa-dev] [PATCH] mesa: Flag _NEW_VARYING_VP_INPUTS when TexEnv programs are active.

2012-10-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Of course, both patches are correct, because both lead to the texenv program being updated when it should be. The only difference is that this patch leads to _NEW_VARYING_VP_INPUTS being set even for the cases where it doesn't actually have any effect on the texenv progra

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Oliver McFadden
On Wed, Oct 03, 2012 at 11:53:55AM +0200, Andreas Boll wrote: > 2012/10/3 Oliver McFadden : > > On Wed, Oct 03, 2012 at 11:37:33AM +0300, Oliver McFadden wrote: > >> On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: > >> > On 10/02/2012 04:00 AM, Oliver McFadden wrote: > >> > > This

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Andreas Boll
2012/10/3 Oliver McFadden : > On Wed, Oct 03, 2012 at 11:37:33AM +0300, Oliver McFadden wrote: >> On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: >> > On 10/02/2012 04:00 AM, Oliver McFadden wrote: >> > > This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. >> > > >> > > B

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Oliver McFadden
On Wed, Oct 03, 2012 at 11:37:33AM +0300, Oliver McFadden wrote: > On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: > > On 10/02/2012 04:00 AM, Oliver McFadden wrote: > > > This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. > > > > > > Bugzilla: https://bugs.freedesktop.

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Oliver McFadden
On Wed, Oct 03, 2012 at 11:37:33AM +0300, Oliver McFadden wrote: > On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: > > On 10/02/2012 04:00 AM, Oliver McFadden wrote: > > > This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. > > > > > > Bugzilla: https://bugs.freedesktop.

Re: [Mesa-dev] [PATCH 1/2] Revert "i965: Implement guardband clipping on Sandybridge."

2012-10-03 Thread Oliver McFadden
On Tue, Oct 02, 2012 at 06:47:49PM -0700, Kenneth Graunke wrote: > On 10/02/2012 04:00 AM, Oliver McFadden wrote: > > This reverts commit 85cd30406ff8414bc6ac6de1064d059a4c564160. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=55523 > > Signed-off-by: Oliver McFadden > > CC: Kenne

Re: [Mesa-dev] [PATCH 1/6] i965/fs: Make the register allocation class_sizes[] choice static.

2012-10-03 Thread Kenneth Graunke
On 10/02/2012 07:52 PM, Eric Anholt wrote: Based on split_virtual_grfs(), we choose the same set every time, so set it in stone. This will help us avoid regenerating the somewhat expensive class/register set setup every compile. --- src/mesa/drivers/dri/i965/brw_fs.h|1 +

Re: [Mesa-dev] [PATCH] mesa: Flag _NEW_VARYING_VP_INPUTS when TexEnv programs are active.

2012-10-03 Thread Kenneth Graunke
On 10/01/2012 08:11 PM, Kenneth Graunke wrote: The idea here is to not flag _NEW_VARYING_VP_INPUTS when shaders (either GLSL or ARB vp/fp) are in use. If either TNL or TexEnv programs are active, at least one stage is using fixed function. On Pineview, fixes 20 Piglit, 60 oglconforms, and 7 ES