Re: [Mesa-dev] [PATCH 05/12] gallivm: Use LLVM global context.

2014-05-19 Thread Mathias Fröhlich
Hi Jose, I tried to get my local llvm install again to a point where I can see backtrace information, but still failed to get valgrind/massif to print these nice backtraces. All of the llvm addresses are not resolved so far. But a appart from that which gl example/test program do you run to see

[Mesa-dev] [PATCH 2/4] meta: Drop unnecessary early returns in _mesa_meta_BlitFramebuffer.

2014-05-19 Thread Kenneth Graunke
These aren't necessary - all of the following code is predicated on mask being non-zero, so no code will get executed anyway. Signed-off-by: Kenneth Graunke kenn...@whitecape.org Cc: 10.2 mesa-sta...@lists.freedesktop.org --- src/mesa/drivers/common/meta_blit.c | 8 1 file changed, 8

[Mesa-dev] [PATCH 4/4] meta: Avoid _swrast_BlitFramebuffer in the meta CopyTexSubImage code.

2014-05-19 Thread Kenneth Graunke
This is a replacement for bd44ac8b5ca08016bb064b37edaec95eccfdbcd5 that should actually work. Fixes Piglit's copyteximage-border on swrast, as well as one of es3conform's packed_pixels_pixelstore test. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78546 Bugzilla:

[Mesa-dev] [PATCH 1/4] Revert i965: Don't _swrast_BlitFramebuffer when doing CopyTexSubImage.

2014-05-19 Thread Kenneth Graunke
This reverts commit bd44ac8b5ca08016bb064b37edaec95eccfdbcd5. Fixes: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78842 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78843 Re-breaks: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77705 but that will be fixed properly

[Mesa-dev] [PATCH 3/4] meta: Split _swrast_BlitFramebuffer out of the meta blit path.

2014-05-19 Thread Kenneth Graunke
Separating the software fallbacks from the rest of the meta path (which is usually hardware accelerated) gives callers better control over their blitting options. For example, i965 might want to try meta blit, hardware blits, then swrast as a last resort. Splitting it makes that possible. This

[Mesa-dev] [Bug 78843] [swrast] piglit copyteximage 1D regression

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78843 Kenneth Graunke kenn...@whitecape.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Mesa-dev] [Bug 78546] [swrast] piglit copyteximage-border regression

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78546 Kenneth Graunke kenn...@whitecape.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Mesa-dev] [Bug 78842] [swrast] piglit fcc-read-after-clear copy rb regression

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78842 Kenneth Graunke kenn...@whitecape.org changed: What|Removed |Added Status|NEW |ASSIGNED

Re: [Mesa-dev] [PATCH] mesa: Expose GL_OES_texture_float and GL_OES_texture_half_float.

2014-05-19 Thread Rogovin, Kevin
Hi, Each of the four extensions are right now set to be advertised if and only if a GL context would advertise GL_ARB_texture_float: { GL_OES_texture_float, o(ARB_texture_float), ES2,2005 }, { GL_OES_texture_half_float,

Re: [Mesa-dev] [PATCH 05/12] gallivm: Use LLVM global context.

2014-05-19 Thread Michel Dänzer
On 19.05.2014 15:03, Mathias Fröhlich wrote: I tried to get my local llvm install again to a point where I can see backtrace information, but still failed to get valgrind/massif to print these nice backtraces. All of the llvm addresses are not resolved so far. You may want to try some or all

[Mesa-dev] [Bug 78773] Doom3 BFG doesnt start

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78773 Tapani Pälli lem...@gmail.com changed: What|Removed |Added CC||lem...@gmail.com ---

[Mesa-dev] [Bug 78773] Doom3 BFG doesnt start

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78773 --- Comment #4 from Tapani Pälli lem...@gmail.com --- Jan, could you please add full log output from doom3 when you run it. -- You are receiving this mail because: You are the assignee for the bug.

[Mesa-dev] [PATCH] i965/fbo: Only try stencil meta blits on gen = 8

2014-05-19 Thread Topi Pohjolainen
I don't have an ILK at hand but the fix should be trivial. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78872 Cc: 10.2 mesa-sta...@lists.freedesktop.org Signed-off-by: Topi Pohjolainen topi.pohjolai...@intel.com --- src/mesa/drivers/dri/i965/intel_fbo.c | 6 -- 1 file changed, 4

[Mesa-dev] [Bug 78888] New: test_eu_compact.c:54:3: error: implicit declaration of function ‘brw_disasm’ [-Werror=implicit-function-declaration]

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=7 Priority: medium Bug ID: 7 Keywords: regression CC: kenn...@whitecape.org Assignee: mesa-dev@lists.freedesktop.org Summary: test_eu_compact.c:54:3: error: implicit

Re: [Mesa-dev] [PATCH] mesa: Expose GL_OES_texture_float and GL_OES_texture_half_float.

2014-05-19 Thread Rogovin, Kevin
Hi It should be possible to adapt some of the existing float texture tests to run on ES mode without too much effort. Oh dear, the test makes the GL API convert between 16 and 32 bit float formats. Also it does not appear to test filtering either. Would it be prudent to make 4 tests: one for

[Mesa-dev] [PATCH] i965: rename brw_disasm to brw_disassemble_inst in test_eu_compact

2014-05-19 Thread Tapani Pälli
(forgotten from commit 4b04152d) Signed-off-by: Tapani Pälli tapani.pa...@intel.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=7 --- src/mesa/drivers/dri/i965/test_eu_compact.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH] i965: rename brw_disasm to brw_disassemble_inst in test_eu_compact

2014-05-19 Thread Pohjolainen, Topi
On Mon, May 19, 2014 at 10:31:56AM +0300, Tapani P?lli wrote: (forgotten from commit 4b04152d) Signed-off-by: Tapani Pälli tapani.pa...@intel.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=7 --- src/mesa/drivers/dri/i965/test_eu_compact.c | 2 +- 1 file changed, 1

[Mesa-dev] [Bug 78888] test_eu_compact.c:54:3: error: implicit declaration of function ‘brw_disasm’ [-Werror=implicit-function-declaration]

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=7 Vinson Lee v...@freedesktop.org changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [PATCH] define GL_OES_standard_derivatives if extension is supported

2014-05-19 Thread kevin . rogovin
From: Kevin Rogovin kevin.rogo...@intel.com Define the macro GL_OES_standard_derivatives as 1 if the extension GL_OES_standard_derivatives is supported. --- src/glsl/glcpp/glcpp-parse.y | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/glsl/glcpp/glcpp-parse.y

[Mesa-dev] [Bug 78892] New: configure: error: Could not find clang internal header stddef.h in /usr/lib64/llvm/clang/3.4 Use --with-clang-libdir to specify the correct path to the clang libraries.

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78892 Priority: medium Bug ID: 78892 Assignee: mesa-dev@lists.freedesktop.org Summary: configure: error: Could not find clang internal header stddef.h in /usr/lib64/llvm/clang/3.4 Use

Re: [Mesa-dev] [PATCH] mesa: Expose GL_OES_texture_float and GL_OES_texture_half_float.

2014-05-19 Thread Marek Olšák
You are complicating it. If we followed the specification to the letter, the driver would have to advertise OpenGL 1.1 instead of 2.1. The fact r300 cannot filter floating-point textures is documented by the vendor and game developers (especially those who targeted D3D9) knew about it. For

[Mesa-dev] [PATCH] docs: update the prerequisites section

2014-05-19 Thread Brian Paul
SCons is required for Windows. Add links to flex/bison for Windows. Reorder items and improve formatting. --- docs/install.html | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/docs/install.html b/docs/install.html index 5061ede..f12425f 100644 ---

[Mesa-dev] [Bug 78914] New: Front/Backfaces do not cover the same pixels when rasterized

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78914 Priority: medium Bug ID: 78914 Assignee: mesa-dev@lists.freedesktop.org Summary: Front/Backfaces do not cover the same pixels when rasterized Severity: normal

[Mesa-dev] [PATCH] targets/opencl: Fix (static) linking with LLVM (v2)

2014-05-19 Thread Kai Wasserbäch
Without this, I get linking failures (static linking). The static linking is sort of required for me, because otherwise Steam and applications using the Steam runtime regularily fail because my LLVM was compiled and linked against a newer libgcc_s, libstdc++, etc. and uses features from those

Re: [Mesa-dev] [PATCH] targets/opencl: Fix (static) linking with LLVM

2014-05-19 Thread Kai Wasserbäch
Michel Dänzer schrieb am 19.05.2014 04:12: On 18.05.2014 18:37, Kai Wasserbäch wrote: And instead of just not starting, my X starts crashing, whenever libGL fails to load a (32 bit) driver. FWIW, some potential alternatives for avoiding the X crashes: With current xserver Git master, you

Re: [Mesa-dev] [PATCH] llvmpipe: do IR counting for shader cache management after optimization.

2014-05-19 Thread Jose Fonseca
Looks good to me. Jose - Original Message - From: Roland Scheidegger srol...@vmware.com 2ea923cf571235dfe573c35c3f0d90f632bd86d8 had the side effect of IR counting now being done after IR optimization instead of before. Some quick analysis shows that there's roughly 1.5 times more

[Mesa-dev] [PATCH 0/7] r600g/compute: Some cleanup patches

2014-05-19 Thread Bruno Jiménez
Hi, Firstly, I shall introduce myself (at least more formally than just sending some patches). My name is Bruno Jiménez, I'm studying physics at Zaragoza's University (Spain) and I am participating in this year's Google Summer of Code, where I will try to improve the compute_memory_pool, solve an

[Mesa-dev] [PATCH 1/7] r600g/compute: Fixing a typo and some indentation

2014-05-19 Thread Bruno Jiménez
--- src/gallium/drivers/r600/compute_memory_pool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src/gallium/drivers/r600/compute_memory_pool.c index 2f0d4c8..ccbb211 100644 ---

[Mesa-dev] [PATCH 4/7] r600g/compute: Tidy a bit compute_memory_finalize_pending

2014-05-19 Thread Bruno Jiménez
Explanation of the changes, as requested by Tom Stellard: Let's take need after is calculated as item-size_in_dw+2048 - (pool-size_in_dw - allocated) BEFORE: If need is positive or 0: we calculate need += 1024 - (need % 1024), which is like cealing to the nearest multiple of 1024,

[Mesa-dev] [PATCH 2/7] r600g/compute: Adding checks for NULL after CALLOC

2014-05-19 Thread Bruno Jiménez
--- src/gallium/drivers/r600/compute_memory_pool.c | 8 1 file changed, 8 insertions(+) diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src/gallium/drivers/r600/compute_memory_pool.c index ccbb211..7143545 100644 --- a/src/gallium/drivers/r600/compute_memory_pool.c +++

[Mesa-dev] [PATCH 6/7] r600g/compute: align items correctly

2014-05-19 Thread Bruno Jiménez
Now, items whose size is a multiple of 1024 dw won't leave 1024 dw between itself and the following item The rest of the cases is left as it was --- src/gallium/drivers/r600/compute_memory_pool.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 3/7] r600g/compute: Add more NULL checks

2014-05-19 Thread Bruno Jiménez
In this case, NULL checks are added to compute_memory_grow_pool, so it returns -1 when it fails. This makes necesary to handle such cases in compute_memory_finalize_pending when it is needed to grow the pool --- src/gallium/drivers/r600/compute_memory_pool.c | 30 --

[Mesa-dev] [PATCH 7/7] r600g/compute: Use %u as the unsigned format

2014-05-19 Thread Bruno Jiménez
This fixes an issue when running cl-program-bitcoin-phatk piglit test where some of the inputs have negative values --- src/gallium/drivers/r600/evergreen_compute.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/evergreen_compute.c

[Mesa-dev] [PATCH 5/7] r600g/compute: Cleanup of compute_memory_pool.h

2014-05-19 Thread Bruno Jiménez
Removed compute_memory_defrag declaration because it seems to be unimplemented. I think that this function would have been the one that solves the problem with fragmentation that compute_memory_finalize_pending has. Also removed comments that are already at compute_memory_pool.c ---

Re: [Mesa-dev] [PATCH 04/10] glsl/linker: initialize explicit uniform locations

2014-05-19 Thread Ian Romanick
On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch initializes the UniformRemapTable for explicit locations. This needs to happen before optimizations to make sure all inactive uniforms get their explicit locations correctly. Signed-off-by: Tapani Pälli tapani.pa...@intel.com ---

Re: [Mesa-dev] [PATCH 06/10] mesa: support inactive uniforms in glUniform* functions

2014-05-19 Thread Ian Romanick
On 04/09/2014 02:56 AM, Tapani Pälli wrote: Support inactive uniforms that have explicit location set in glUniform* functions. Signed-off-by: Tapani Pälli tapani.pa...@intel.com --- src/mesa/main/uniform_query.cpp | 15 +++ 1 file changed, 15 insertions(+) diff --git

Re: [Mesa-dev] [PATCH 05/10] glsl/linker: assign explicit uniform locations

2014-05-19 Thread Ian Romanick
On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch refactors the existing uniform processing so explicit locations are taken in to account during variable processing. These locations are temporarily stored in gl_uniform_storage before actual locations are set. The 'remap_location' variable

Re: [Mesa-dev] [PATCH 08/10] glsl: parser changes for GL_ARB_explicit_uniform_location

2014-05-19 Thread Ian Romanick
On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch adds a preprocessor define for the extension and stores explicit location data for uniforms during AST-HIR conversion. It also sets layout token to be available when having the extension in place. Signed-off-by: Tapani Pälli

Re: [Mesa-dev] [PATCH 09/10] Enable GL_ARB_explicit_uniform_location in the drivers.

2014-05-19 Thread Ian Romanick
Either this patch should: - Delete the extension enable flag - Change the table in extensions.c to use dummy_true or The next patch needs to not say all drivers that support GLSL. I think we should just enable it everywhere. On 04/09/2014 02:56 AM, Tapani Pälli wrote: Signed-off-by: Tapani

Re: [Mesa-dev] [PATCH 00/10] GL_ARB_explicit_uniform_location v2

2014-05-19 Thread Ian Romanick
Patches 1, 2, and 7 are Reviewed-by: Ian Romanick ian.d.roman...@intel.com I sent out comments for the rest. On 04/09/2014 02:56 AM, Tapani Pälli wrote: Hi; Patches implement the extension, no Piglit regressions and all the tests for the extension pass. Location initialization and

Re: [Mesa-dev] [PATCH 03/10] mesa: add new enum MAX_UNIFORM_LOCATIONS

2014-05-19 Thread Ian Romanick
On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch adds new implementation dependent value required by the GL_ARB_explicit_uniform_location extension. Default value for user assignable locations is calculated as sum of MaxUniformComponents for each stage. Signed-off-by: Tapani Pälli

[Mesa-dev] glsl: ideas how to improve dead code elimination?

2014-05-19 Thread Aras Pranckevicius
Hi, When Mesa's GLSL compiler is faced with a code like this: // vec4 packednormal exists vec3 normal; normal.xy = packednormal.wy * 2.0 - 1.0; normal.z = sqrt(1.0 - dot(normal.xy, normal.xy)); // now do not use the normal at all anywhere Then the dead code elimination pass

Re: [Mesa-dev] [PATCH] mesa: Expose GL_OES_texture_float and GL_OES_texture_half_float.

2014-05-19 Thread Ian Romanick
On 05/19/2014 06:39 AM, Marek Olšák wrote: You are complicating it. If we followed the specification to the letter, the driver would have to advertise OpenGL 1.1 instead of 2.1. The fact r300 cannot filter floating-point textures is documented by the vendor and game developers (especially

Re: [Mesa-dev] [PATCH 1/4] Revert i965: Don't _swrast_BlitFramebuffer when doing CopyTexSubImage.

2014-05-19 Thread Ian Romanick
Thanks for the quick fix. :) Series is Reviewed-by: Ian Romanick ian.d.roman...@intel.com On 05/18/2014 11:12 PM, Kenneth Graunke wrote: This reverts commit bd44ac8b5ca08016bb064b37edaec95eccfdbcd5. Fixes: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78842 Bugzilla:

Re: [Mesa-dev] [PATCH] docs: update the prerequisites section

2014-05-19 Thread Ian Romanick
LGTM Reviewed-by: Ian Romanick ian.d.roman...@intel.com On 05/19/2014 07:17 AM, Brian Paul wrote: SCons is required for Windows. Add links to flex/bison for Windows. Reorder items and improve formatting. --- docs/install.html | 15 --- 1 file changed, 12 insertions(+), 3

[Mesa-dev] [PATCH 00/23] i965: Instruction compaction improvements.

2014-05-19 Thread Matt Turner
Available from git://people.freedesktop.org/~mattst88/mesa compaction Highlights - Print disassembly after instruction compaction, while still having control-flow graph information and higher-level IR annotations. - Three improvements to instruction compaction increase

[Mesa-dev] [PATCH 02/23] i965: Pass in start_offset to brw_compact_instructions().

2014-05-19 Thread Matt Turner
Let's us avoid recompacting the SIMD8 instructions when we compact the SIMD16 program. --- src/mesa/drivers/dri/i965/brw_blorp_clear.cpp| 2 +- src/mesa/drivers/dri/i965/brw_clip.c | 2 +- src/mesa/drivers/dri/i965/brw_eu.h | 2 +-

[Mesa-dev] [PATCH 03/23] i965/fs: Don't hardcode DEBUG_WM in generic fs code.

2014-05-19 Thread Matt Turner
Similar to Paul's commit e9fa3a944 except brw_fs_generator's debug_flag is for DEBUG_WM and DEBUG_BLORP. --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp| 13 +++-- src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp | 17 - src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h

[Mesa-dev] [PATCH 01/23] i965/cfg: Make DO instruction begin a basic block.

2014-05-19 Thread Matt Turner
The DO instruction doesn't exist on Gen6+. Since before this commit, DO always ended a basic block, if it also happened to start one (e.g., a while loop inside an if statement) the block containing only the DO would actually contain no hardware instructions. Pre-Gen6's WHILE instructions jumps to

[Mesa-dev] [PATCH 05/23] i965: Add annotation data structure and support code.

2014-05-19 Thread Matt Turner
Will be used to print disassembly after jump targets are set and instructions are compacted, while still retaining higher-level IR annotations and basic block information. An array of 'struct annotation' will live along side the generated assembly. The generators will populate the array with

[Mesa-dev] [PATCH 14/23] i965: Support compacted instructions with immediate sources.

2014-05-19 Thread Matt Turner
Note the weirdness with src1 subregs. The compacted immediate fields are uncompacted to bits [127:96] and the high five bits of the subreg mapping maps to bits [100:96]. Number of compacted instructions: 790085 - 817752 (3.50%) --- src/mesa/drivers/dri/i965/brw_eu_compact.c | 83

[Mesa-dev] [PATCH 17/23] i965: Switch types D-UD when possible to allow compaction.

2014-05-19 Thread Matt Turner
Number of compacted instructions: 827404 - 833045 (0.68%) --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 20 1 file changed, 20 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index 1810233..ab00d7c 100644 ---

[Mesa-dev] [PATCH 06/23] i965/fs: Make patch_discard_jumps_to_fb_writes return bool.

2014-05-19 Thread Matt Turner
... to tell us whether it emitted any code. Will be used to determine whether we need to skip an annotation for it. --- src/mesa/drivers/dri/i965/brw_fs.h | 4 ++-- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 5 +++-- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 5 +++-- 3

[Mesa-dev] [PATCH 11/23] i965: Rename next_ip() - next_offset().

2014-05-19 Thread Matt Turner
That we were comparing its return value with offsets should have been a clue. :) Make it take a void *store in preparation for making the function useful elsewhere. --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 63 + 1 file changed, 33 insertions(+), 30

[Mesa-dev] [PATCH 22/23] i965: Emit compaction stats without walking the assembly.

2014-05-19 Thread Matt Turner
The instruction count does not include padding NOPs, but the compaction stats do. --- src/mesa/drivers/dri/i965/brw_eu_compact.c | 19 --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 8 src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 7 +++ 3 files

[Mesa-dev] [PATCH 15/23] i965: Emit ARF:UD for non-present src1 on Gen6+.

2014-05-19 Thread Matt Turner
Enables the next commits to compact more instructions. --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 28 ++-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index

[Mesa-dev] [PATCH 18/23] i965: Print IR annotations only with INTEL_DEBUG=annotation.

2014-05-19 Thread Matt Turner
Running shader-db without INTEL_DEBUG=annotation reduces the runtime from ~90 to ~80 seconds on my machine. It also reduces the disk space consumed by the .out files from 660 MB (676 on disk) to 343 MB (358 on disk). --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp| 6 --

[Mesa-dev] [PATCH 07/23] i965/fs: Print disassembly after compaction.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_fs.h | 4 +- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 156 - 2 files changed, 77 insertions(+), 83 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h index

[Mesa-dev] [PATCH 19/23] i965/fs: Use next_insn_offset rather than nr_insn.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 4 ++-- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp index

[Mesa-dev] [PATCH 08/23] i965/vec4: Print disassembly after compaction.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_vec4.h | 4 +- src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 109 +-- 2 files changed, 66 insertions(+), 47 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h b/src/mesa/drivers/dri/i965/brw_vec4.h index

[Mesa-dev] [PATCH 13/23] i965: Use next_offset() in instruction compaction code.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_eu_compact.c | 20 +++- 1 file changed, 3 insertions(+), 17 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_eu_compact.c b/src/mesa/drivers/dri/i965/brw_eu_compact.c index 40d1fc2..f6f055f 100644 ---

[Mesa-dev] [PATCH 12/23] i965: Move next_offset() to brw_eu.h for use elsewhere.

2014-05-19 Thread Matt Turner
Also perform arithmetic on char* rather than void* since the latter is a GNU C extension not available in C++. --- src/mesa/drivers/dri/i965/brw_eu.h | 12 src/mesa/drivers/dri/i965/brw_eu_emit.c | 11 --- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git

[Mesa-dev] [PATCH 10/23] i965/gen8/vec4: Print disassembly after compaction.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_vec4.h | 3 +- src/mesa/drivers/dri/i965/gen8_vec4_generator.cpp | 103 +- 2 files changed, 63 insertions(+), 43 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h b/src/mesa/drivers/dri/i965/brw_vec4.h index

[Mesa-dev] [PATCH 16/23] i965: Emit 0.0:F sources with type VF instead.

2014-05-19 Thread Matt Turner
Number of compacted instructions: 817752 - 827404 (1.18%) --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 16 1 file changed, 16 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index d8efa01..1810233 100644 ---

[Mesa-dev] [PATCH 23/23] i965/gen8: Print number of instructions directly.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 5 + src/mesa/drivers/dri/i965/gen8_vec4_generator.cpp | 3 +++ 2 files changed, 8 insertions(+) diff --git a/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp b/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp index 0ac00f9..90743ee

[Mesa-dev] [PATCH 09/23] i965/gen8/fs: Print disassembly after compaction.

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_fs.h | 3 +- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 138 +++- 2 files changed, 65 insertions(+), 76 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h index

[Mesa-dev] [PATCH 20/23] i965/fs: Loop over instruction lists and generate code.

2014-05-19 Thread Matt Turner
Small code reduction. Will let us move the program header code into a common place in generate_assembly(). --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 56 ++--- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 46 +--- 2 files changed, 42

[Mesa-dev] [PATCH 04/23] i965/fs+blorp: Remove left over dump_file arguments.

2014-05-19 Thread Matt Turner
Were used by the blorp unit test programs. --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp| 20 src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp | 4 ++-- src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h | 2 +- src/mesa/drivers/dri/i965/brw_fs.h | 5 ++---

[Mesa-dev] [PATCH 21/23] i965: Print shader header in generate_assembly().

2014-05-19 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_fs_generator.cpp| 29 ++- src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 17 ++--- src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 29 ++- src/mesa/drivers/dri/i965/gen8_vec4_generator.cpp | 17

[Mesa-dev] [Bug 78771] egl not works on 10.0.x and 10.1.x with black screen

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78771 U. Artie Eoff ullysses.a.e...@intel.com changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH] loader: allow alternative methods for PCI identification.

2014-05-19 Thread Gary Wong
On Wed, May 14, 2014 at 10:39:05PM -0600, Gary Wong wrote: loader_get_pci_id_for_fd() and loader_get_device_name_for_fd() now attempt all available strategies to identify the hardware, instead of conditionally compiling in a single test. The existing libudev and DRM approaches have been

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/4] meta: Drop unnecessary early returns in _mesa_meta_BlitFramebuffer.

2014-05-19 Thread Courtney Goeltzenleuchter
Looks good. Reviewed-by: Courtney Goeltzenleuchter court...@lunarg.com On Mon, May 19, 2014 at 12:12 AM, Kenneth Graunke kenn...@whitecape.orgwrote: These aren't necessary - all of the following code is predicated on mask being non-zero, so no code will get executed anyway. Signed-off-by:

[Mesa-dev] [PATCH 1/2] meta: Refactor _mesa_meta_setup_blit_shader() to avoid duplicate shader code

2014-05-19 Thread Anuj Phogat
Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Anuj Phogat anuj.pho...@gmail.com --- src/mesa/drivers/common/meta.c | 97 +++--- 1 file changed, 44 insertions(+), 53 deletions(-) diff --git a/src/mesa/drivers/common/meta.c

[Mesa-dev] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Anuj Phogat
_mesa_meta_setup_blit_shader() currently generates a fragment shader which, irrespective of the number of draw buffers, writes the color to only one output variable. Current shader rely on an undefined behavior and possibly works by chance. From OpenGL 4.0 spec, page 256: If a fragment shader

Re: [Mesa-dev] [Mesa-stable] [PATCH 4/4] meta: Avoid _swrast_BlitFramebuffer in the meta CopyTexSubImage code.

2014-05-19 Thread Chris Forbes
Series is: Reviewed-by: Chris Forbes chr...@ijw.co.nz On Mon, May 19, 2014 at 6:12 PM, Kenneth Graunke kenn...@whitecape.org wrote: This is a replacement for bd44ac8b5ca08016bb064b37edaec95eccfdbcd5 that should actually work. Fixes Piglit's copyteximage-border on swrast, as well as one of

Re: [Mesa-dev] [PATCH 1/2] meta: Refactor _mesa_meta_setup_blit_shader() to avoid duplicate shader code

2014-05-19 Thread Matt Turner
On Mon, May 19, 2014 at 1:20 PM, Anuj Phogat anuj.pho...@gmail.com wrote: Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Anuj Phogat anuj.pho...@gmail.com --- src/mesa/drivers/common/meta.c | 97 +++--- 1 file changed, 44 insertions(+), 53

Re: [Mesa-dev] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Matt Turner
On Mon, May 19, 2014 at 1:20 PM, Anuj Phogat anuj.pho...@gmail.com wrote: _mesa_meta_setup_blit_shader() currently generates a fragment shader which, irrespective of the number of draw buffers, writes the color to only one output variable. Current shader rely on an undefined behavior and

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Chris Forbes
On Tue, May 20, 2014 at 8:20 AM, Anuj Phogat anuj.pho...@gmail.com wrote: @@ -247,7 +247,8 @@ _mesa_meta_setup_blit_shader(struct gl_context *ctx, struct blit_shader *shader = choose_blit_shader(target, table); const char *vs_input, *vs_output, *fs_input, *fs_output; const char

Re: [Mesa-dev] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Anuj Phogat
On Mon, May 19, 2014 at 2:45 PM, Matt Turner matts...@gmail.com wrote: On Mon, May 19, 2014 at 1:20 PM, Anuj Phogat anuj.pho...@gmail.com wrote: _mesa_meta_setup_blit_shader() currently generates a fragment shader which, irrespective of the number of draw buffers, writes the color to only one

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Anuj Phogat
On Mon, May 19, 2014 at 3:12 PM, Chris Forbes chr...@ijw.co.nz wrote: On Tue, May 20, 2014 at 8:20 AM, Anuj Phogat anuj.pho...@gmail.com wrote: @@ -247,7 +247,8 @@ _mesa_meta_setup_blit_shader(struct gl_context *ctx, struct blit_shader *shader = choose_blit_shader(target, table); const

[Mesa-dev] [Bug 78914] [llvmpipe] Front/Backfaces do not cover the same pixels when rasterized

2014-05-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78914 Roland Scheidegger srol...@vmware.com changed: What|Removed |Added Summary|Front/Backfaces do not |[llvmpipe]

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Chris Forbes
If you're going to do that, you'd really want to add draw buffer count to the cache key (and i guess this might be the point where you convert the blit shader cache to be a hashtable), to avoid recompiling all the time if the app does two blits with the same target but different draw buffer

Re: [Mesa-dev] [PATCH 16/23] targets/vdpau: convert to static and shared pipe-drivers

2014-05-19 Thread Emil Velikov
On 18/05/14 12:22, Jonathan Gray wrote: [snip] Currently I run my autotools builds like this: export LDFLAGS=-L/usr/local/lib export CPPFLAGS=-I/usr/local/include -I/usr/local/include/libelf export AUTOMAKE_VERSION=1.12 export AUTOCONF_VERSION=2.69 export LEX=/usr/local/bin/gflex

Re: [Mesa-dev] [PATCH] loader: allow alternative methods for PCI identification.

2014-05-19 Thread Emil Velikov
On 15/05/14 05:39, Gary Wong wrote: loader_get_pci_id_for_fd() and loader_get_device_name_for_fd() now attempt all available strategies to identify the hardware, instead of conditionally compiling in a single test. The existing libudev and DRM approaches have been retained, and another simple

Re: [Mesa-dev] glsl: ideas how to improve dead code elimination?

2014-05-19 Thread Matt Turner
On Mon, May 19, 2014 at 10:56 AM, Aras Pranckevicius a...@unity3d.com wrote: Hi, When Mesa's GLSL compiler is faced with a code like this: // vec4 packednormal exists vec3 normal; normal.xy = packednormal.wy * 2.0 - 1.0; normal.z = sqrt(1.0 - dot(normal.xy, normal.xy));

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Anuj Phogat
On Mon, May 19, 2014 at 3:52 PM, Chris Forbes chr...@ijw.co.nz wrote: If you're going to do that, you'd really want to add draw buffer count to the cache key (and i guess this might be the point where you convert the blit shader cache to be a hashtable), to avoid recompiling all the time if

Re: [Mesa-dev] [PATCH 16/23] targets/vdpau: convert to static and shared pipe-drivers

2014-05-19 Thread Jonathan Gray
On Mon, May 19, 2014 at 11:57:58PM +0100, Emil Velikov wrote: On 18/05/14 12:22, Jonathan Gray wrote: [snip] Currently I run my autotools builds like this: export LDFLAGS=-L/usr/local/lib export CPPFLAGS=-I/usr/local/include -I/usr/local/include/libelf export AUTOMAKE_VERSION=1.12

[Mesa-dev] [PATCH] tgsi: add GS_INVOCATIONS to property names array

2014-05-19 Thread Ilia Mirkin
In commit 4be146b1, I neglected to add the new property to the strings array. This leads to the string '(null)' to be printed instead when converting a GS shader to text. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 10.2 mesa-sta...@lists.freedesktop.org ---

Re: [Mesa-dev] [PATCH 04/10] glsl/linker: initialize explicit uniform locations

2014-05-19 Thread Tapani
On 05/19/2014 07:51 PM, Ian Romanick wrote: On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch initializes the UniformRemapTable for explicit locations. This needs to happen before optimizations to make sure all inactive uniforms get their explicit locations correctly. Signed-off-by: Tapani

Re: [Mesa-dev] [PATCH 05/10] glsl/linker: assign explicit uniform locations

2014-05-19 Thread Tapani
On 05/19/2014 08:07 PM, Ian Romanick wrote: On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch refactors the existing uniform processing so explicit locations are taken in to account during variable processing. These locations are temporarily stored in gl_uniform_storage before actual locations

Re: [Mesa-dev] [PATCH 06/10] mesa: support inactive uniforms in glUniform* functions

2014-05-19 Thread Tapani
On 05/19/2014 08:09 PM, Ian Romanick wrote: On 04/09/2014 02:56 AM, Tapani Pälli wrote: Support inactive uniforms that have explicit location set in glUniform* functions. Signed-off-by: Tapani Pälli tapani.pa...@intel.com --- src/mesa/main/uniform_query.cpp | 15 +++ 1 file

[Mesa-dev] ARB_sso layout() + other qualifiers

2014-05-19 Thread Chris Forbes
Hi Ian, When I was writing the `precise` support I found some error cases in the GLSL parser where we reject combinations of layout() with invariant / interpolation / etc qualifiers. This seems to be consistent with the GLSL 1.50 grammar (or, at least, admits all the examples that were given in

Re: [Mesa-dev] [PATCH 08/10] glsl: parser changes for GL_ARB_explicit_uniform_location

2014-05-19 Thread Tapani
On 05/19/2014 08:18 PM, Ian Romanick wrote: On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch adds a preprocessor define for the extension and stores explicit location data for uniforms during AST-HIR conversion. It also sets layout token to be available when having the extension in place.

Re: [Mesa-dev] [PATCH 09/10] Enable GL_ARB_explicit_uniform_location in the drivers.

2014-05-19 Thread Tapani
On 05/19/2014 08:21 PM, Ian Romanick wrote: Either this patch should: - Delete the extension enable flag - Change the table in extensions.c to use dummy_true or The next patch needs to not say all drivers that support GLSL. I think we should just enable it everywhere. OK, I was

Re: [Mesa-dev] [PATCH 03/10] mesa: add new enum MAX_UNIFORM_LOCATIONS

2014-05-19 Thread Tapani
On 05/19/2014 08:26 PM, Ian Romanick wrote: On 04/09/2014 02:56 AM, Tapani Pälli wrote: Patch adds new implementation dependent value required by the GL_ARB_explicit_uniform_location extension. Default value for user assignable locations is calculated as sum of MaxUniformComponents for each

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] meta: Write color values in to 'out' variables for all the draw buffers

2014-05-19 Thread Kenneth Graunke
On 05/19/2014 03:52 PM, Chris Forbes wrote: If you're going to do that, you'd really want to add draw buffer count to the cache key (and i guess this might be the point where you convert the blit shader cache to be a hashtable), to avoid recompiling all the time if the app does two blits with

Re: [Mesa-dev] [PATCH] mesa: Expose GL_OES_texture_float and GL_OES_texture_half_float.

2014-05-19 Thread Rogovin, Kevin
For OpenGL ES, I propose a simpler solution: - don't touch ARB_texture_float at all - add OES_texture_float to gl_extensions - add OES_texture_float_linear to gl_extensions - define OES_texture_half_float as o(OES_texture_float) - define OES_texture_half_float_linear as