Re: [Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Brian Paul
On 11/28/2012 02:14 PM, Jose Fonseca wrote: - Original Message - On 11/28/2012 12:39 PM, Brian Paul wrote: Add a DEBUG_FREED_MEMORY option to help catch use-after-free errors. Add debug_memory_check() function which can be periodically called to check that all known blocks are good. -

[Mesa-dev] [PATCH] docs: Mark some features in GL3.txt as done for r600

2012-11-28 Thread Jason Wood
From: Jason Wood These extensions are advertised as available on r600 hardware. If this patch is accepted, please feel free to commit this for me as I do not have access. - Jason Wood --- a/docs/GL3.txt 2012-11-28 14:13:06.654726848 -0700 +++ b/docs/GL3.txt 2012-11-28 14:16:18.671718865 -0700 @

Re: [Mesa-dev] [PATCH] glcpp: Reject token pasting operator in GLES

2012-11-28 Thread Ian Romanick
On 11/28/2012 11:21 AM, Matt Turner wrote: The GLSL ES 3.0 spec (Section 12.17) says: "GLSL ES 1.00 removed token pasting and other functionality." Do we have piglit tests? 'make check' tests? I assume this passes the GLES3 conformance test without regressing anything. This should be marke

Re: [Mesa-dev] [PATCH] vbo: move code after declarations to fix MSVC errors

2012-11-28 Thread Ian Romanick
On 11/27/2012 06:57 PM, Kenneth Graunke wrote: On 11/27/2012 12:58 PM, Brian Paul wrote: --- src/mesa/vbo/vbo_attrib_tmp.h | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/mesa/vbo/vbo_attrib_tmp.h b/src/mesa/vbo/vbo_attrib_tmp.h index de1b451..adb22d7

Re: [Mesa-dev] [PATCH 4/4] llvmpipe: Eliminate color buffer swizzling.

2012-11-28 Thread Roland Scheidegger
Am 28.11.2012 21:57, schrieb jfons...@vmware.com: > From: José Fonseca > > Now dead code. > > Also had to remove the show_tiles/show_subtiles because now the color > buffers are always stored in their native format, so there is no longer > an easy way to paint the tile sizes. > > Depth-stencil

[Mesa-dev] [PATCH 2/5] AMDGPU: Remove unused macros

2012-11-28 Thread Tom Stellard
From: Tom Stellard --- lib/Target/AMDGPU/AMDGPUInstrInfo.cpp | 4 +- lib/Target/AMDGPU/AMDILCFGStructurizer.cpp | 29 ++-- lib/Target/AMDGPU/AMDILISelDAGToDAG.cpp| 1 - lib/Target/AMDGPU/AMDILISelLowering.cpp| 1 - lib/Target/AMDGPU/AMDILUtilityFunctions.h | 75 -

[Mesa-dev] [PATCH 3/5] AMDGPU: Fix 4-space indentation

2012-11-28 Thread Tom Stellard
From: Tom Stellard --- lib/Target/AMDGPU/AMDGPUCodeEmitter.h | 48 +- lib/Target/AMDGPU/AMDILDeviceInfo.cpp | 96 +-- lib/Target/AMDGPU/AMDILSIDevice.cpp | 2 +- 3 files changed, 73 insertions(+), 73 deletions(-) diff --git a/lib/Target/AMDGPU/

[Mesa-dev] [PATCH 4/5] AMDGPU: Remove or document commented out code

2012-11-28 Thread Tom Stellard
From: Tom Stellard --- lib/Target/AMDGPU/AMDILCFGStructurizer.cpp | 97 +++--- lib/Target/AMDGPU/AMDILISelDAGToDAG.cpp| 2 +- lib/Target/AMDGPU/AMDILIntrinsicInfo.cpp | 2 - lib/Target/AMDGPU/SIInstructions.td| 4 ++ 4 files changed, 13 insertions(+), 92

[Mesa-dev] [PATCH 5/5] R600: Fold immediates into ALU instructions when possible v2

2012-11-28 Thread Tom Stellard
From: Tom Stellard v2: - Fold the immediates using the SelectionDAG --- lib/Target/AMDGPU/AMDILISelDAGToDAG.cpp | 94 + lib/Target/AMDGPU/R600InstrInfo.cpp | 16 - lib/Target/AMDGPU/R600InstrInfo.h | 7 ++ test/CodeGen/R600/fcmp-cn

[Mesa-dev] [PATCH 4/4] st/dri: implement new driver hook flush_with_flags

2012-11-28 Thread Marek Olšák
v2: added documentation for dri_flush as per Brian's request --- .../state_trackers/dri/common/dri_drawable.c | 110 ++-- .../state_trackers/dri/common/dri_drawable.h |6 ++ src/gallium/state_trackers/dri/common/dri_screen.h |1 + src/gallium/state_trackers/dri

Re: [Mesa-dev] [PATCH] radeonsi: Bitcast result of packf16 intrinsic to float for export intrinsic.

2012-11-28 Thread Christian König
On 27.11.2012 19:55, Michel Dänzer wrote: From: Michel Dänzer Fixes 4 piglit tests and prevents many more from crashing. Signed-off-by: Michel Dänzer Reviewed & Tested-by: Christian König --- src/gallium/drivers/radeonsi/radeonsi_shader.c |6 +- 1 file changed, 5 insertions(+)

Re: [Mesa-dev] [PATCH 3/4] llvmpipe: Only advertise unswizzled formats.

2012-11-28 Thread Dave Airlie
On Thu, Nov 29, 2012 at 6:57 AM, wrote: > From: José Fonseca > > Update llvmpipe_is_format_supported and llvmpipe_is_format_unswizzled > so that only the formats that we can render without swizzling are > advertised. > > We can still render all D3D10 required formats except > PIPE_FORMAT_R11G11B

Re: [Mesa-dev] [PATCH 1/4] glapi: Move ARB_base_instance to the correct location

2012-11-28 Thread Fredrik Höglund
On Wednesday 28 November 2012, Matt Turner wrote: > It's #107, it shouldn't be added after the #116 comment. > --- > src/mapi/glapi/gen/gl_API.xml |8 +--- > 1 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml > i

Re: [Mesa-dev] [PATCH] docs: Mark some features in GL3.txt as done for r600

2012-11-28 Thread Marek Olšák
Pushed. Next time please use "git send-email" or "git format-patch", and not "git diff" or anything else. Marek On Wed, Nov 28, 2012 at 11:01 PM, Jason Wood wrote: > From: Jason Wood > > These extensions are advertised as available on r600 hardware. > > If this patch is accepted, please feel fr

Re: [Mesa-dev] [PATCH] mesa: move some helper functions from fboobject.c to glformats.c

2012-11-28 Thread Ian Romanick
On 11/28/2012 10:07 AM, Marek Olšák wrote: It looks reasonable enough, but... why? Is there a follow-up patch that uses it them in some other file? --- src/mesa/main/fbobject.c | 127 +++-- src/mesa/main/glformats.c | 113

Re: [Mesa-dev] [PATCH 1/8] i965/vs: Move some functions from brw_vec4_emit.cpp to brw_vec4.cpp.

2012-11-28 Thread Eric Anholt
Kenneth Graunke writes: > This leaves only the final code generation stage in brw_vec4_emit.cpp, > moving the payload setup, run(), and brw_vs_emit functions to brw_vec4.cpp. > > The fragment shader backend puts these functions in brw_fs.cpp, so this > patch also helps with consistency. This ser

Re: [Mesa-dev] [PATCH] mesa: move some helper functions from fboobject.c to glformats.c

2012-11-28 Thread Marek Olšák
On Thu, Nov 29, 2012 at 1:23 AM, Ian Romanick wrote: > On 11/28/2012 10:07 AM, Marek Olšák wrote: > > It looks reasonable enough, but... why? Is there a follow-up patch that > uses it them in some other file? Brian suggested it just in case somebody else needs to use the functions. Marek __

Re: [Mesa-dev] [PATCH] mesa: move some helper functions from fboobject.c to glformats.c

2012-11-28 Thread Brian Paul
On 11/28/2012 05:23 PM, Ian Romanick wrote: On 11/28/2012 10:07 AM, Marek Olšák wrote: It looks reasonable enough, but... why? Is there a follow-up patch that uses it them in some other file? I suggested this to Marek in case we need to use these elsewhere someday and since that's where we al

[Mesa-dev] [PATCH V3 0/7] i965: add support for ARB_texture_cube_map_array

2012-11-28 Thread Chris Forbes
No significant functional changes since V2: - Moved enable into existing block - Fixed various weird whitespace ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH V3 1/7] i965: Add various plumbing for cubemap arrays

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_tex_layout.c| 4 src/mesa/drivers/dri/i965/brw_wm_sampler_state.c | 3 ++- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 1 + src/mesa/drivers/dri/i965/gen7_sampler_state.c| 3 ++- src/mesa/drivers/dri/i965/gen7

[Mesa-dev] [PATCH V3 2/7] i965: vs: Add fixup for textureSize with cube array samplers

2012-11-28 Thread Chris Forbes
V3: Fixed weird whitespace Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp index edb66ea..bfae35c 10064

[Mesa-dev] [PATCH V3 3/7] i965: fs: fix gen6 math operands in one place

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs.cpp | 48 +--- 1 file changed, 23 insertions(+), 25 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index cc5a790..cdfa46f 100644 --- a/src/mesa

[Mesa-dev] [PATCH V3 4/7] i965: fs: expand IMM math arguments to a temp too

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index cdfa46f..f63a776 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/

[Mesa-dev] [PATCH V3 5/7] i965: fs: Add fixup for textureSize on Gen6/7

2012-11-28 Thread Chris Forbes
V2: Moved up into emit(ir_texture *) to avoid duplication and fix ordering for Gen7; Gen6 math quirks moved into previous patches. Tested on Gen6 only; passes all the cube_map_array piglits. V3: Fixed weird whitespace Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp

[Mesa-dev] [PATCH V3 6/7] i965: expose ARB_texture_cube_map_array

2012-11-28 Thread Chris Forbes
V3: Put enable in an existing block rather than making a new one for no good reason. Signed-off-by: Chris Forbes --- docs/GL3.txt | 2 +- src/mesa/drivers/dri/intel/intel_extensions.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/GL3.t

[Mesa-dev] [PATCH V3 7/7] mesa: expose ARB_texture_cube_map_array in core contexts as well

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes Reviewed-by: Dave Airlie --- src/mesa/main/extensions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index d5a7886..fa5e1aa 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/exten

Re: [Mesa-dev] [PATCH 0/4] DRI2: Fixing the triple-flush for Gallium v2

2012-11-28 Thread Marek Olšák
FYI, I am going to push this series in 2 days if there is no other review. There are 2 libGL patches that might interest some non-gallium folks too. Marek On Fri, Nov 23, 2012 at 8:35 PM, Marek Olšák wrote: > This series consolidates the 3 gallium context flushes done by libGL in > SwapBuffers

[Mesa-dev] [Bug 57372] x11-libs/libxcb media-libs/mesa segfault in __glXGetString

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57372 Matt Turner changed: What|Removed |Added CC||i...@freedesktop.org -- You are receiving

[Mesa-dev] [PATCH] gallium/postprocess: share pipe_context and cso_context with the state tracker

2012-11-28 Thread Marek Olšák
Using one context instead of two is more efficient and we can skip another context flush. --- src/gallium/auxiliary/postprocess/postprocess.h|6 ++- src/gallium/auxiliary/postprocess/pp_init.c| 13 ++- src/gallium/auxiliary/postprocess/pp_program.c | 20 -- src/

[Mesa-dev] [Bug 57372] x11-libs/libxcb media-libs/mesa segfault in __glXGetString

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57372 --- Comment #9 from Richard Freeman --- Ok, it turns out that I can eliminate this issue in xserver 1.13.0 if I revert two commits: ed6daa15a7dcf8dba930f67401f4c1c8ca2e6fac (mentioned above) bcbf95b1bafa6ffe724768b9309295e2fdb4b860 Author: Jon TU

[Mesa-dev] [Bug 57372] x11-libs/libxcb media-libs/mesa segfault in __glXGetString

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57372 Richard Freeman changed: What|Removed |Added CC||jon.tur...@dronecode.org.uk -- You ar

Re: [Mesa-dev] [PATCH 2/3] glcpp: Support #elif(expression) with no intervening space.

2012-11-28 Thread Carl Worth
Carl Worth writes: > (For the subsequent __LINE__ and __FILE__ I think we can do better, and > I'll attempt that next.) I've coded this up, and I think the support is quite reasonable. After I coded it, I simplified some repeated code a bit, and then I started exploring some token-pasting code w

[Mesa-dev] [PATCH 1/7] glcpp: Add support for __LINE__ and __FILE__ macros

2012-11-28 Thread Carl Worth
These tokens are easy to expand by just looking at the current, tracked location values, (and no need to look anything up in the hash table). Add a test which verifies __LINE__ with several values, (and verifies __FILE__ for the single value of 0). Our testing framework isn't so[hissticated enough

[Mesa-dev] [PATCH 2/7] glcpp: Factor out a tiny bit of repeated code.

2012-11-28 Thread Carl Worth
This function is getting a little long too read. Simplify it by pulling up one assignment from every condition. --- src/glsl/glcpp/glcpp-parse.y |5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y index d928044..5b3

[Mesa-dev] [PATCH 4/7] glcpp: Extend the invalid-paste test

2012-11-28 Thread Carl Worth
The current code lets a few invalid pastes through, such as an string pasted onto the end of an integer. Extend the invalid-paste test to catch some of these. --- src/glsl/glcpp/tests/082-invalid-paste.c |5 + src/glsl/glcpp/tests/082-invalid-paste.c.expected | 15 ++

[Mesa-dev] [PATCH 3/7] glcpp: More factoring-out of common code to simplify things.

2012-11-28 Thread Carl Worth
This time creating a new _token_list_create_with_one_integer function modeled after the existing _token_list_create_with_one_space function (both implemented with new _token_list_create_with_one_ival). --- src/glsl/glcpp/glcpp-parse.y | 46 ++ 1 file chang

[Mesa-dev] [PATCH 5/7] glcpp: Flag invalid pastes for integer followed by non-digits

2012-11-28 Thread Carl Worth
As recently tested in the additions to the invalid paste test, it is illegal to paste a non-digit sequence onto the end of an integer. The 082-invalid-paste test should now pass again. --- src/glsl/glcpp/glcpp-parse.y | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) di

[Mesa-dev] [PATCH 7/7] glcpp: Add support for pasting of INTEGER tokens.

2012-11-28 Thread Carl Worth
By generalizing the current code designed to paste string tokens of various types. With this commit, the recently added 114-paste-integer-tokens test now passes. --- src/glsl/glcpp/glcpp-parse.y | 61 -- 1 file changed, 47 insertions(+), 14 deletions(-) d

[Mesa-dev] [PATCH 6/7] glcpp: Add test involving token pasting of INTEGER tokens.

2012-11-28 Thread Carl Worth
This test file is very similar to test 113-line-and-file-macros but uses token pasting for cleaner quiz answers (without spaces between the digits). This test happens to fail currently because the token paster does not support the pasting of INTEGER tokens. (Note that this test is distinct from te

Re: [Mesa-dev] [PATCH] glcpp: Reject token pasting operator in GLES

2012-11-28 Thread Carl Worth
Matt Turner writes: > "##" { > + if (parser->is_gles) > + glcpp_error(yylloc, yyextra, "Token pasting (##) is illegal in > GLES"); > return PASTE; I'm glad that is_gles flag is proving useful. Thanks for doing this one. Reviewed-by: Carl Worth -Carl -- carl.d.wo...@in

[Mesa-dev] [PATCH 8/7] glcpp: Update README for new support of __LINE__ and __FILE__.

2012-11-28 Thread Carl Worth
Drop these from the known limitations list since support was recently added for these. Also, fix a typo while in the area, (and the oddly missing final newline). --- src/glsl/glcpp/README |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/glsl/glcpp/README b/src/glsl/gl

[Mesa-dev] [Bug 57674] New: [llvmpipe] SIGSEGV src/gallium/drivers/llvmpipe/lp_rast.c:501

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57674 Priority: medium Bug ID: 57674 Keywords: regression CC: jfons...@vmware.com Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] SIGSEGV src/gallium/d

[Mesa-dev] [PATCH] llvmpipe: Fix incorrect sizeof.

2012-11-28 Thread Vinson Lee
Fixes sizeof not portable defects reported by Coverity. Signed-off-by: Vinson Lee --- src/gallium/drivers/llvmpipe/lp_state_fs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/llvmpipe/lp_state_fs.c b/src/gallium/drivers/llvmpipe/lp_state_fs.c index

[Mesa-dev] [PATCH] android: include api_exec.c in generated files list

2012-11-28 Thread Tapani Pälli
Signed-off-by: Tapani Pälli --- src/mesa/Android.gen.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/Android.gen.mk b/src/mesa/Android.gen.mk index 5f0d73c..35f00da 100644 --- a/src/mesa/Android.gen.mk +++ b/src/mesa/Android.gen.mk @@ -32,6 +32,7 @@ intermediates := $(call local-i

Re: [Mesa-dev] [PATCH 4/6] i965: fs: Add fixup for textureSize on Gen7

2012-11-28 Thread Kenneth Graunke
On 11/22/2012 04:35 AM, Chris Forbes wrote: Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visito

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Re-enable Z16 depth buffers.

2012-11-28 Thread Christian König
On 27.11.2012 19:11, Michel Dänzer wrote: From: Michel Dänzer 8 more piglits. Signed-off-by: Michel Dänzer Maybe we should keep the warning in the second patch additionally to the assert, anyway both patches are: Reviewed-by: Christian König --- src/gallium/drivers/radeonsi/si_state

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-11-28 Thread Michel Dänzer
On Fre, 2012-11-23 at 22:23 +0100, Andreas Boll wrote: > > Please help Matt and me to test the automake-gallium4 branch [1] ! > > Here you have a list of things you could test: [...] > - radeonsi make[4]: Entering directory `/home/daenzer/src/mesa-git/mesa/src/gallium/drivers/radeon' GEN

Re: [Mesa-dev] [PATCH 0/8] llvmpipe: Fixes for PowerPC/big-endian machines

2012-11-28 Thread Adhemerval Zanella
On 11/22/2012 07:38 PM, Roland Scheidegger wrote: > Am 22.11.2012 21:33, schrieb Adhemerval Zanella: >> Hi all, >> >> This set of patches fixes all the llvmpipe lp_test_* for PowerPC. The first >> five >> add Altivec intrinsics that fixes all arith testcases, while the remaining >> one >> deals m

Re: [Mesa-dev] [PATCH 8/8] RFC: llvmpipe: Fix format manipulation for big-endian

2012-11-28 Thread Adhemerval Zanella
On 11/22/2012 07:37 PM, Roland Scheidegger wrote: > Am 22.11.2012 21:35, schrieb Adhemerval Zanella: >> This patch fixes various format manipulation for big-endian >> architectures. Basically it works on two fronts: 1) by adjusting >> some algorithms that do shifts/mask on vector or integers and 2)

Re: [Mesa-dev] [PATCH 0/8] llvmpipe: Fixes for PowerPC/big-endian machines

2012-11-28 Thread Adhemerval Zanella
On 11/27/2012 04:28 PM, Jose Fonseca wrote: > > - Original Message - >> Hi all, >> >> This set of patches fixes all the llvmpipe lp_test_* for PowerPC. The >> first five >> add Altivec intrinsics that fixes all arith testcases, while the >> remaining one >> deals mainly with big-endian memo

Re: [Mesa-dev] [PATCH 5/8] PowerPC: clear Altivec NJ bit

2012-11-28 Thread Adhemerval Zanella
On 11/22/2012 07:33 PM, Roland Scheidegger wrote: > Am 22.11.2012 21:34, schrieb Adhemerval Zanella: >> Mostly PowerPC system sets the Altivec NJ bit to 1 so denormal number >> are handled as 0. Initially it was a performance configuration, since >> denormal handling tended to be costly. However it

Re: [Mesa-dev] [PATCH 0/8] llvmpipe: Fixes for PowerPC/big-endian machines

2012-11-28 Thread Adhemerval Zanella
On 11/28/2012 10:44 AM, Adhemerval Zanella wrote: > On 11/27/2012 04:28 PM, Jose Fonseca wrote: >> - Original Message - >>> Hi all, >>> >>> This set of patches fixes all the llvmpipe lp_test_* for PowerPC. The >>> first five >>> add Altivec intrinsics that fixes all arith testcases, while t

[Mesa-dev] [Bug 57044] build failure: make[4]: *** No rule to make target `../../src/mesa/program/libprogram.la', needed by `libmesagallium.la'. Stop.

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57044 --- Comment #6 from Quentin Glidic --- Created attachment 70736 --> https://bugs.freedesktop.org/attachment.cgi?id=70736&action=edit Simple patch to fix libprogram build Attached a patch to fix both build by introducing a new NEED_LIBPROGRAM A

Re: [Mesa-dev] [PATCH 5/8] PowerPC: clear Altivec NJ bit

2012-11-28 Thread Roland Scheidegger
Am 28.11.2012 13:47, schrieb Adhemerval Zanella: > On 11/22/2012 07:33 PM, Roland Scheidegger wrote: >> Am 22.11.2012 21:34, schrieb Adhemerval Zanella: >>> Mostly PowerPC system sets the Altivec NJ bit to 1 so denormal number >>> are handled as 0. Initially it was a performance configuration, sinc

[Mesa-dev] [PATCH 1/4] configure.ac: print LLVM flags

2012-11-28 Thread Marek Olšák
to see what we're mixing with ours --- configure.ac |6 ++ 1 file changed, 6 insertions(+) diff --git a/configure.ac b/configure.ac index 006f4fa..6b0d703 100644 --- a/configure.ac +++ b/configure.ac @@ -2134,6 +2134,12 @@ echo "CFLAGS: $cflags" echo "CXXFLAGS:

[Mesa-dev] [PATCH 2/4] configure.ac: consolidate stripping unwanted LLVM flags

2012-11-28 Thread Marek Olšák
--- configure.ac | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 6b0d703..ec06d37 100644 --- a/configure.ac +++ b/configure.ac @@ -1641,6 +1641,19 @@ AC_ARG_WITH([llvm-prefix], [llvm_prefix=""]) +# Call this insi

[Mesa-dev] [PATCH 3/4] configure.ac: look for whole words in LLVM flags, not prefixes

2012-11-28 Thread Marek Olšák
--- configure.ac | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index ec06d37..c5cc249 100644 --- a/configure.ac +++ b/configure.ac @@ -1644,13 +1644,14 @@ AC_ARG_WITH([llvm-prefix], # Call this inside ` ` to get the return value. #

[Mesa-dev] [PATCH 4/4] configure.ac: remove -fomit-frame-pointer from LLVM flags

2012-11-28 Thread Marek Olšák
--- configure.ac |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index c5cc249..4fb4544 100644 --- a/configure.ac +++ b/configure.ac @@ -1651,7 +1651,8 @@ strip_unwanted_llvm_flags() { -e 's/-Wcovered-switch-default\>//g' \ -e 's

[Mesa-dev] [Bug 57644] [llvmpipe] src/gallium/auxiliary/gallivm/lp_bld_init.c:568:gallivm_verify_function: Assertion `0' failed.

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57644 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH] st/mesa: add support for GL core profiles

2012-11-28 Thread Marek Olšák
The rest of the plumbing was in place already. I have tested this by turning on all GL 3.1 features. The drivers not supporting GL 3.1 will fail to create a core profile as they should. --- src/gallium/state_trackers/dri/common/dri_context.c |4 +++- src/mesa/state_tracker/st_manager.c

[Mesa-dev] [Bug 57044] build failure: make[4]: *** No rule to make target `../../src/mesa/program/libprogram.la', needed by `libmesagallium.la'. Stop.

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57044 --- Comment #7 from Fabio Pedretti --- (In reply to comment #6) > Created attachment 70736 [details] [review] > Simple patch to fix libprogram build Confirmed working. -- You are receiving this mail because: You are the assignee for the bug. _

[Mesa-dev] [PATCH] mesa: move some helper functions from fboobject.c to glformats.c

2012-11-28 Thread Marek Olšák
--- src/mesa/main/fbobject.c | 127 +++-- src/mesa/main/glformats.c | 113 src/mesa/main/glformats.h |6 +++ 3 files changed, 127 insertions(+), 119 deletions(-) diff --git a/src/mesa/main/fbobject.c b/src/mes

[Mesa-dev] [PATCH 1/2] (8.0) glsl: initialise const force glsl extension warning in fake ctx

2012-11-28 Thread Andreas Boll
From: Dave Airlie valgrind complained about an uninitialised value being used in glsl_parser_extras.cpp, and this was the one it was giving out about. Just initialise the value in the fakectx. Signed-off-by: Dave Airlie Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=48057 (cherry picke

[Mesa-dev] [PATCH 2/2] (8.0) mesa: test for GL_EXT_framebuffer_sRGB in glPopAttrib()

2012-11-28 Thread Andreas Boll
From: Brian Paul To avoid spurious GL_INVALID_ENUM errors if the extension isn't supported. (cherry picked from commit 1aee8803f83f7ae24d9c2150c70afff2b1ee4c2f) --- Fixes a regression on 8.0 introduced with bd9f729091bbbce521d4526afd0230da2a7cf350 git://github.com/pineerspacesim/pioneer dies wi

Re: [Mesa-dev] [PATCH 2/3] i965/fs: Don't generate saturates over existing variable values.

2012-11-28 Thread Andreas Boll
2012/11/28 Kenneth Graunke : > On 11/27/2012 12:12 AM, Eric Anholt wrote: >> >> Fixes a crash in http://workshop.chromeexperiments.com/stars/ on i965, >> and the new piglit test glsl-fs-clamp-5. >> We were trying to emit a saturating move into a uniform, which the code >> generator appropriately ch

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-11-28 Thread Andreas Boll
2012/11/28 Michel Dänzer : > On Fre, 2012-11-23 at 22:23 +0100, Andreas Boll wrote: >> >> Please help Matt and me to test the automake-gallium4 branch [1] ! >> >> Here you have a list of things you could test: > [...] >> - radeonsi > > make[4]: Entering directory > `/home/daenzer/src/mesa-git/mesa

Re: [Mesa-dev] [PATCH] st/mesa: add support for GL core profiles

2012-11-28 Thread Andreas Boll
2012/11/28 Marek Olšák : > The rest of the plumbing was in place already. > > I have tested this by turning on all GL 3.1 features. > The drivers not supporting GL 3.1 will fail to create a core profile > as they should. > --- > src/gallium/state_trackers/dri/common/dri_context.c |4 +++- > sr

[Mesa-dev] [PATCH] radeon/llvm: rename if/break operator to improve readability

2012-11-28 Thread Vincent Lejeune
--- lib/Target/AMDGPU/AMDILCFGStructurizer.cpp | 36 +-- .../AMDGPU/MCTargetDesc/R600MCCodeEmitter.cpp | 41 +- lib/Target/AMDGPU/R600ExpandSpecialInstrs.cpp | 2 +- lib/Target/AMDGPU/R600Instructions.td | 11 ++ 4 files chang

[Mesa-dev] [PATCH] r600g: mirror simplification of if/break opcodes

2012-11-28 Thread Vincent Lejeune
--- src/gallium/drivers/r600/r600_shader.c | 44 ++ 1 file changed, 12 insertions(+), 32 deletions(-) diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c index 84821ac..72cb585 100644 --- a/src/gallium/drivers/r600/r600_sha

Re: [Mesa-dev] [PATCH] radeon/llvm: rename if/break operator to improve readability

2012-11-28 Thread Tom Stellard
On Wed, Nov 28, 2012 at 07:57:24PM +0100, Vincent Lejeune wrote: > --- > lib/Target/AMDGPU/AMDILCFGStructurizer.cpp | 36 +-- > .../AMDGPU/MCTargetDesc/R600MCCodeEmitter.cpp | 41 > +- > lib/Target/AMDGPU/R600ExpandSpecialInstrs.cpp | 2 +- >

Re: [Mesa-dev] [PATCH] r600g: mirror simplification of if/break opcodes

2012-11-28 Thread Tom Stellard
On Wed, Nov 28, 2012 at 07:59:33PM +0100, Vincent Lejeune wrote: > --- > src/gallium/drivers/r600/r600_shader.c | 44 > ++ > 1 file changed, 12 insertions(+), 32 deletions(-) > Reviewed-by: Tom Stellard > diff --git a/src/gallium/drivers/r600/r600_shader.c > b

Re: [Mesa-dev] [PATCH 2/2] (8.0) mesa: test for GL_EXT_framebuffer_sRGB in glPopAttrib()

2012-11-28 Thread Kenneth Graunke
On 11/28/2012 10:46 AM, Andreas Boll wrote: From: Brian Paul To avoid spurious GL_INVALID_ENUM errors if the extension isn't supported. (cherry picked from commit 1aee8803f83f7ae24d9c2150c70afff2b1ee4c2f) --- Fixes a regression on 8.0 introduced with bd9f729091bbbce521d4526afd0230da2a7cf350 g

[Mesa-dev] [PATCH] glcpp: Reject token pasting operator in GLES

2012-11-28 Thread Matt Turner
The GLSL ES 3.0 spec (Section 12.17) says: "GLSL ES 1.00 removed token pasting and other functionality." --- src/glsl/glcpp/glcpp-lex.l |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/glsl/glcpp/glcpp-lex.l b/src/glsl/glcpp/glcpp-lex.l index fd28711..a029f62 100644 --

Re: [Mesa-dev] [PATCH] st/mesa: add support for GL core profiles

2012-11-28 Thread Marek Olšák
On Wed, Nov 28, 2012 at 7:55 PM, Andreas Boll wrote: > 2012/11/28 Marek Olšák : >> The rest of the plumbing was in place already. >> >> I have tested this by turning on all GL 3.1 features. >> The drivers not supporting GL 3.1 will fail to create a core profile >> as they should. >> --- >> src/ga

[Mesa-dev] [PATCH] st/mesa: allow forward-compatible contexts and set Const.ContextFlags

2012-11-28 Thread Marek Olšák
--- src/gallium/include/state_tracker/st_api.h |3 +-- src/mesa/state_tracker/st_manager.c| 12 +++- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/gallium/include/state_tracker/st_api.h b/src/gallium/include/state_tracker/st_api.h index 3a11cd4..85de1c2

[Mesa-dev] [Bug 57044] build failure: make[4]: *** No rule to make target `../../src/mesa/program/libprogram.la', needed by `libmesagallium.la'. Stop.

2012-11-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57044 Quentin Glidic changed: What|Removed |Added CC||sardemff7+freedesktop@sarde

[Mesa-dev] [PATCH] i965/fp: Fix segfault on gen4 TXB instructions.

2012-11-28 Thread Eric Anholt
The gen4 simd16 workaround looks at ir->type to determine how much storage to allocate for the simd16 value. In fragment programs, texturing only ever returns float vec4s (unlike GLSL, which can also have scalar floats or vector integers). --- src/mesa/drivers/dri/i965/brw_fs_fp.cpp |2 ++ 1

[Mesa-dev] [PATCH V2 0/7] i965: add support for ARB_texture_cube_map_array

2012-11-28 Thread Chris Forbes
This series enables ARB_texture_cube_map_array on Gen6+. Tested on Gen6, will need someone to try it on Gen7 (particularly the fs behavior, which is different due to quirks in the Gen6 math unit) Changes from V1: * Don't screw up the order of TXS + post fixups in Gen7 * Move Gen6 quirks down into

[Mesa-dev] [PATCH V2 1/7] i965: Add various plumbing for cubemap arrays

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_tex_layout.c| 4 src/mesa/drivers/dri/i965/brw_wm_sampler_state.c | 3 ++- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 1 + src/mesa/drivers/dri/i965/gen7_sampler_state.c| 3 ++- src/mesa/drivers/dri/i965/gen7

[Mesa-dev] [PATCH V2 2/7] i965: vs: Add fixup for textureSize with cube array samplers

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp index edb66ea..59b6758 100644 --- a/src/mesa/drivers/dri

[Mesa-dev] [PATCH V2 3/7] i965: fs: fix gen6 math operands in one place

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs.cpp | 48 +--- 1 file changed, 23 insertions(+), 25 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index f19f275..e81eb60 100644 --- a/src/mesa

[Mesa-dev] [PATCH V2 4/7] i965: fs: expand IMM math arguments to a temp too

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index e81eb60..0144741 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/

[Mesa-dev] [PATCH V2 5/7] i965: fs: Add fixup for textureSize on Gen6/7

2012-11-28 Thread Chris Forbes
V2: Moved up into emit(ir_texture *) to avoid duplication and fix ordering for Gen7; Gen6 math quirks moved into previous patches. Tested on Gen6 only; passes all the cube_map_array piglits. Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 12 1 file c

[Mesa-dev] [PATCH V2 6/7] i965: expose ARB_texture_cube_map_array

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes --- docs/GL3.txt | 2 +- src/mesa/drivers/dri/intel/intel_extensions.c | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index aebba0b..5d6c451 100644 --- a/docs/GL3.txt +++ b/docs/GL3.tx

[Mesa-dev] [PATCH V2 7/7] mesa: expose ARB_texture_cube_map_array in core contexts as well

2012-11-28 Thread Chris Forbes
Signed-off-by: Chris Forbes Reviewed-by: Dave Airlie --- src/mesa/main/extensions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index d5a7886..fa5e1aa 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/exten

[Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Brian Paul
Add a DEBUG_FREED_MEMORY option to help catch use-after-free errors. Add debug_memory_check() function which can be periodically called to check that all known blocks are good. --- src/gallium/auxiliary/os/os_memory_debug.h |3 + src/gallium/auxiliary/util/u_debug_memory.c | 82

Re: [Mesa-dev] [PATCH] st/mesa: add support for GL core profiles

2012-11-28 Thread Brian Paul
On 11/28/2012 10:14 AM, Marek Olšák wrote: The rest of the plumbing was in place already. I have tested this by turning on all GL 3.1 features. The drivers not supporting GL 3.1 will fail to create a core profile as they should. --- src/gallium/state_trackers/dri/common/dri_context.c |4 ++

Re: [Mesa-dev] [PATCH] st/mesa: allow forward-compatible contexts and set Const.ContextFlags

2012-11-28 Thread Brian Paul
On 11/28/2012 12:42 PM, Marek Olšák wrote: --- src/gallium/include/state_tracker/st_api.h |3 +-- src/mesa/state_tracker/st_manager.c| 12 +++- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/gallium/include/state_tracker/st_api.h b/src/gallium/includ

Re: [Mesa-dev] [PATCH] i965/fp: Fix segfault on gen4 TXB instructions.

2012-11-28 Thread Kenneth Graunke
On 11/28/2012 11:46 AM, Eric Anholt wrote: The gen4 simd16 workaround looks at ir->type to determine how much storage to allocate for the simd16 value. In fragment programs, texturing only ever returns float vec4s (unlike GLSL, which can also have scalar floats or vector integers). --- src/mes

Re: [Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Kenneth Graunke
On 11/28/2012 12:39 PM, Brian Paul wrote: Add a DEBUG_FREED_MEMORY option to help catch use-after-free errors. Add debug_memory_check() function which can be periodically called to check that all known blocks are good. --- src/gallium/auxiliary/os/os_memory_debug.h |3 + src/gallium/auxil

[Mesa-dev] [PATCH 1/4] util/u_format: Tighten the meaning of is_array bit to exclude mixed type formats.

2012-11-28 Thread jfonseca
From: José Fonseca This is what we want in practice. The only change is in PIPE_FORMAT_R8SG8SB8UX8U_NORM, which no longer is considered an array format. --- src/gallium/auxiliary/util/u_format.h|2 +- src/gallium/auxiliary/util/u_format_parse.py | 11 ++- 2 files changed,

[Mesa-dev] [PATCH 2/4] util/u_format: Kill util_format_is_array().

2012-11-28 Thread jfonseca
From: José Fonseca It is buggy (it was giving wrong results for some of the formats with padding), and util_format_description::is_array already does precisely what's intended. --- src/gallium/auxiliary/gallivm/lp_bld_format_aos.c |3 +- src/gallium/auxiliary/gallivm/lp_bld_type.h |

[Mesa-dev] [PATCH 3/4] llvmpipe: Only advertise unswizzled formats.

2012-11-28 Thread jfonseca
From: José Fonseca Update llvmpipe_is_format_supported and llvmpipe_is_format_unswizzled so that only the formats that we can render without swizzling are advertised. We can still render all D3D10 required formats except PIPE_FORMAT_R11G11B10_FLOAT, which needs to be implemented in a future oppo

Re: [Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Brian Paul
On 11/28/2012 01:56 PM, Kenneth Graunke wrote: On 11/28/2012 12:39 PM, Brian Paul wrote: Add a DEBUG_FREED_MEMORY option to help catch use-after-free errors. Add debug_memory_check() function which can be periodically called to check that all known blocks are good. --- src/gallium/auxiliary/os/o

Re: [Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Jose Fonseca
- Original Message - > On 11/28/2012 12:39 PM, Brian Paul wrote: > > Add a DEBUG_FREED_MEMORY option to help catch use-after-free > > errors. > > Add debug_memory_check() function which can be periodically called > > to > > check that all known blocks are good. > > --- > > src/gallium/a

Re: [Mesa-dev] [PATCH] util: add more memory debugging features

2012-11-28 Thread Jose Fonseca
Looks good to me. Jose - Original Message - > Add a DEBUG_FREED_MEMORY option to help catch use-after-free errors. > Add debug_memory_check() function which can be periodically called to > check that all known blocks are good. > --- > src/gallium/auxiliary/os/os_memory_debug.h |3 +

[Mesa-dev] [PATCH 1/4] glapi: Move ARB_base_instance to the correct location

2012-11-28 Thread Matt Turner
It's #107, it shouldn't be added after the #116 comment. --- src/mapi/glapi/gen/gl_API.xml |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml index a47316a..4c1329f 100644 --- a/src/mapi/glapi/gen/gl_API.xml

[Mesa-dev] [PATCH 2/4] glapi: Add enums for ARB_ES3_compatibility

2012-11-28 Thread Matt Turner
--- src/mapi/glapi/gen/ARB_ES3_compatibility.xml | 23 +++ src/mapi/glapi/gen/Makefile.am |2 ++ src/mapi/glapi/gen/gl_API.xml|6 ++ 3 files changed, 31 insertions(+), 0 deletions(-) create mode 100644 src/mapi/glapi/gen/ARB_ES3_comp

[Mesa-dev] [PATCH 3/4] mesa: De-duplicate ES2 queries

2012-11-28 Thread Matt Turner
The first hunk of removed code is inside an "apis": ["GL", "GLES", "GL_CORE"] block. GLES exposing ES2_compatibility is nonsense. --- src/mesa/main/get.c |8 +++- src/mesa/main/get_hash_params.py | 16 ++-- 2 files changed, 13 insertions(+), 11 deletions(-) diff

[Mesa-dev] [PATCH 4/4] mesa: Support querying GL_MAX_ELEMENT_INDEX in ES 3

2012-11-28 Thread Matt Turner
The ES 3 spec says that the minumum allowable value is 2^24-1, but the GL 4.3 and ARB_ES3_compatibility specs require 2^32-1, so return 2^32-1. Fixes es3conform's element_index_uint_constants test. --- src/mesa/main/context.c |3 +++ src/mesa/main/get.c |6 ++ sr

  1   2   >