[Mesa-dev] [PATCH] mesa: Check for a negative "size" parameter in glCopyBufferSubData().

2012-06-08 Thread Kenneth Graunke
>From the GL_ARB_copy_buffer spec: "An INVALID_VALUE error is generated if any of readoffset, writeoffset, or size are negative [...]" Fixes oglconform's copybuffer/negative.CNNegativeValues test. Signed-off-by: Kenneth Graunke --- src/mesa/main/bufferobj.c |6 ++ 1 file changed, 6 ins

Re: [Mesa-dev] [PATCH] glsl: glcpp: Allow "#if undefined-macro' to evaluate to false.

2012-06-08 Thread Kenneth Graunke
On 06/08/2012 03:17 PM, Carl Worth wrote: > A strict reading of the GLSL specification would have this be an > error, but we've received reports from users who expect the > preprocessor to interepret undefined macros as 0. This is the standard > behavior of the rpeprocessor for C, and according to

[Mesa-dev] [PATCH] glsl: glcpp: Allow "#if undefined-macro' to evaluate to false.

2012-06-08 Thread Carl Worth
A strict reading of the GLSL specification would have this be an error, but we've received reports from users who expect the preprocessor to interepret undefined macros as 0. This is the standard behavior of the rpeprocessor for C, and according to these user reports is also the behavior of other O

Re: [Mesa-dev] [PATCH] i965: enable ARB_instanced_arrays extension

2012-06-08 Thread Jordan Justen
On Mon, Jun 4, 2012 at 1:31 PM, Eric Anholt wrote: > On Sun, 27 May 2012 21:08:07 -0700, Jordan Justen > wrote: >> Set the step_rate value when drawing to implement >> ARB_instanced_arrays for gen >= 4. > >> @@ -504,7 +513,7 @@ static void brw_prepare_vertices(struct brw_context *brw) >> >>    

[Mesa-dev] [PATCH] mesa: implement glTexImage as glTexSubImage when possible

2012-06-08 Thread Brian Paul
I've now seen a couple apps that replace a whole texture image with a glTexImage() call instead of glTexSubImage(). The later is better because we can skip freeing/allocating texture memory. This patch checks if the new glTexImage's format and size matches the current image. If so, use the TexSu

[Mesa-dev] [PATCH] st/mesa: fix glDrawPixels(GL_DEPTH_COMPONENT) color output

2012-06-08 Thread Brian Paul
When drawing a depth image the fragment shader also needs to emit the current raster color. The new piglit drawpix-z test exercises this. NOTE: This is a candiate for the 8.0 branch. --- src/mesa/state_tracker/st_cb_drawpixels.c | 13 +++-- 1 files changed, 11 insertions(+), 2 deletion

[Mesa-dev] [PATCH 3/3] i965/fs: Allow copy propagation on uniforms.

2012-06-08 Thread Eric Anholt
This is a big win for savage2, hon and yofrankie. 62 new programs for savage2/hon get 16-wide mode, along with one for humus demos and two for tropics. Even a few shaders from tropics see reductions of 15% or more. total instructions in shared programs: 216536 -> 207353 (-4.24%) instructions in

[Mesa-dev] [PATCH 1/3] i965/fs: Move copy propagation test out to a separate function.

2012-06-08 Thread Eric Anholt
It's going to get more complicated in a moment. --- src/mesa/drivers/dri/i965/brw_fs.h |4 .../drivers/dri/i965/brw_fs_copy_propagation.cpp | 22 ++-- 2 files changed, 20 insertions(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/

[Mesa-dev] [PATCH 2/3] i965/fs: Allow copy propagation with source modifiers.

2012-06-08 Thread Eric Anholt
This shaves a few instructions off of a ton of programs. For 12 shaders from tropics and sanctuary, it's enough reduction in register pressure to get 16-wide mode. 7 shaders from heroes of newerth and savage2 are hurt by about 1.1%, where copy propagation of negates ends up preventing coalescing,

[Mesa-dev] [Bug 50754] Building 32 bit mesa on 64 bit OS fails since change for automake

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50754 --- Comment #7 from Alexandre Demers 2012-06-08 11:31:03 PDT --- (In reply to comment #6) > Let's be honest, building 32-bit Mesa on a 64-bit system has been a disaster > for a long time. See bug #41700, bug #45466, and > http://lists.freedeskt

Re: [Mesa-dev] [PATCH 0/2] gitignore cleanup

2012-06-08 Thread Kenneth Graunke
On 06/08/2012 10:55 AM, Andreas Boll wrote: > Hi, > > these are my first patches for mesa. Welcome! :) > Please review and commit. > > Andreas. > > PS: I have some more cleanup patches for docs/, which i'll post soon. Thanks for working on that. They haven't received much attention or care i

Re: [Mesa-dev] [PATCH v5] intel: wait render timeout implementation

2012-06-08 Thread Daniel Vetter
On Thu, Jun 07, 2012 at 09:12:41AM -0700, Ben Widawsky wrote: > int drm_intel_gem_bo_wait(drm_intel_bo *bo, uint64_t timeout_ns) > > This should bump the libdrm version. We're waiting for context support > so we can do both features in one bump. > > v2: don't return remaining timeout amount > use

[Mesa-dev] [Bug 45466] Updated configure.ac check for llvm-config to use 32 version when appropriate

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45466 Ian Romanick changed: What|Removed |Added Depends on||50754 -- Configure bugmail: https://bugs

[Mesa-dev] [Bug 41700] Gallium won't build with --enable-32-bit on a 64-bit system

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41700 Ian Romanick changed: What|Removed |Added Depends on||50754 -- Configure bugmail: https://bugs

[Mesa-dev] [Bug 50754] Building 32 bit mesa on 64 bit OS fails since change for automake

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50754 Ian Romanick changed: What|Removed |Added Blocks||45466, 41700 --- Comment #6 from Ian Roma

[Mesa-dev] [PATCH 2/2] glcpp: .gitignore cleanup

2012-06-08 Thread Andreas Boll
*.o, *.lo and *~ are already in toplevel .gitignore --- src/glsl/glcpp/.gitignore |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/src/glsl/glcpp/.gitignore b/src/glsl/glcpp/.gitignore index 41b34bb..24a7119 100644 --- a/src/glsl/glcpp/.gitignore +++ b/src/glsl/glcpp/.git

[Mesa-dev] [PATCH 1/2] glapi: .gitignore cleanup

2012-06-08 Thread Andreas Boll
remove archaic .cvsignore *.pyo is already in toplevel .gitignore *.pyc is already in toplevel .gitignore --- src/mapi/glapi/.gitignore | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/src/mapi/glapi/.gitignore b/src/mapi/glapi/.gitignore index 21edf4f..ecae56a 1

[Mesa-dev] [PATCH 0/2] gitignore cleanup

2012-06-08 Thread Andreas Boll
Hi, these are my first patches for mesa. Please review and commit. Andreas. PS: I have some more cleanup patches for docs/, which i'll post soon. Andreas Boll (2): glapi: .gitignore cleanup glcpp: .gitignore cleanup src/glsl/glcpp/.gitignore |3 --- src/mapi/glapi/.gitignore | 11 --

Re: [Mesa-dev] [PATCH 0/2] Add vertex id to llvmpipe.

2012-06-08 Thread Jose Fonseca
Oliver, There will be other system values in the future, so instead of passing every value as a different parameter, please define a structure in src/gallium/auxiliary/gallivm/lp_bld_tgsi.h as struct lp_bld_tgsi_system_values { LLVMValueRef facing; LLVMValueRef instance_id; LLVMValueRef v

Re: [Mesa-dev] [PATCH] st/vdpau: fix YCbCr down/up-loads for buffers larger than requested

2012-06-08 Thread Andy Furniss
Christian König wrote: Unfortunately it's not quite right, there's an artifact on r300 s/w decode = thin green lines right and bottom. R600 with the patch doesn't show it as much, but 1080 stuff does have one at the bottom.

[Mesa-dev] [Bug 49504] [Bisected] Mesa master compilation broke when built with --with-llvm-shared-libs

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49504 --- Comment #2 from José Fonseca 2012-06-08 08:13:03 PDT --- I don't see how this commit changes anything. It seems this is a linking bug, that was always there, but latent. And this commit simply slightly changed the order that symbols are lin

[Mesa-dev] [Bug 49504] [Bisected] Mesa master compilation broke when built with --with-llvm-shared-libs

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49504 José Fonseca changed: What|Removed |Added CC||jfons...@vmware.com -- Configure bugmail

Re: [Mesa-dev] [PATCH] st/vdpau: fix YCbCr down/up-loads for buffers larger than requested

2012-06-08 Thread Christian König
Hi Andy & Emeric, On 07.06.2012 16:29, Emeric Grange wrote: 2012/6/7 Andy Furniss mailto:andy...@ukfsn.org>> Andy Furniss wrote: Andy Furniss wrote: Christian König wrote: When the video buffer turns out to be larger than requested by

[Mesa-dev] [Bug 50754] Building 32 bit mesa on 64 bit OS fails since change for automake

2012-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50754 --- Comment #5 from Tapani Pälli 2012-06-08 00:52:46 PDT --- There is a difference in the generated libtool script. When running autogen.sh by manually setting CFLAGS and CXXFLAGS as "-m32" it has : # Compile-time system search path for librarie