[Mesa-dev] [PATCH 01/13] i965: Don't recalculate partial_clear inside brw_fast_clear_depth

2017-06-18 Thread Ian Romanick
From: Ian Romanick textdata bss dec hex filename 7155954 256860 37332 7450146 71ae22 32-bit i965_dri.so before 7155858 256860 37332 7450050 71adc2 32-bit i965_dri.so after 6789395 328056 50704 7168155 6d609b 64-bit i965_dri.so before

[Mesa-dev] [PATCH 09/13] i915: On Gen <= 3 there is no multisampling

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 34 +-- src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 10 +++- src/mesa/drivers/dri/i915/intel_tex_layout.c | 3

[Mesa-dev] [PATCH 11/13] i915: Remove unused fields intel_mipmap_tree::logical_(width|height|depth)0

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 3 --- src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 8 2 files changed, 11 deletions(-) diff --git

[Mesa-dev] [PATCH 08/13] i915: Trivial code reformatting

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 110 +- 1 file changed, 54 insertions(+), 56 deletions(-) diff --git a/src/mesa/drivers/dri/i915/intel_mipmap_tree.c

[Mesa-dev] [PATCH 07/13] i915, i965: Don't condition use of GLSL clear on the current API

2017-06-18 Thread Ian Romanick
From: Ian Romanick Meta always sets the API to API_OPENGL_COMPAT, so the current API setting is irrelevant. textdata bss dec hex filename 7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so before 7154978 256860 37332 7449170 71aa52 32-bit

[Mesa-dev] [PATCH 06/13] meta: Coalesce the GLSL and FF paths in meta_clear

2017-06-18 Thread Ian Romanick
From: Ian Romanick textdata bss dec hex filename 7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so before 7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so after 6788475 328056 50704 7167235 6d5d03 64-bit i965_dri.so before

[Mesa-dev] [PATCH 02/13] meta: Remove support for clearing integer buffers

2017-06-18 Thread Ian Romanick
From: Ian Romanick Since i965 no longer uses this function for clearing color buffers, there is no driver left that will ever support integer textures and use _mesa_meta_glsl_Clear. As a side note, the has_integer_textures check was rubbish anyway because meta always

[Mesa-dev] [PATCH 04/13] meta: Stop frobbing MatrixMode

2017-06-18 Thread Ian Romanick
From: Ian Romanick textdata bss dec hex filename 7155246 256860 37332 7449438 71ab5e 32-bit i965_dri.so before 7155058 256860 37332 7449250 71aaa2 32-bit i965_dri.so after 6788683 328056 50704 7167443 6d5dd3 64-bit i965_dri.so before

[Mesa-dev] [PATCH 03/13] mesa: Add matrix utility functions to load matrices

2017-06-18 Thread Ian Romanick
From: Ian Romanick These are basically DSA versions of glLoadIdentity() and glLoadMatrix(). textdata bss dec hex filename 7155026 256860 37332 7449218 71aa82 32-bit i965_dri.so before 7155246 256860 37332 7449438 71ab5e 32-bit i965_dri.so

[Mesa-dev] [PATCH 05/13] meta: Use same vertex coordinates for GLSL and FF clears

2017-06-18 Thread Ian Romanick
From: Ian Romanick textdata bss dec hex filename 7155058 256860 37332 7449250 71aaa2 32-bit i965_dri.so before 7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so after 6788611 328056 50704 7167371 6d5d8b 64-bit i965_dri.so before

[Mesa-dev] [PATCH 12/13] i915: On Gen <= 3 there is no W-tiling

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 5 ++--- src/mesa/drivers/dri/i915/intel_regions.c | 21 ++--- src/mesa/drivers/dri/i915/intel_regions.h | 5 ++---

[Mesa-dev] [PATCH 13/13] i915: On Gen <= 3 there are no array textures

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 5 - src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 2 -- src/mesa/drivers/dri/i915/intel_tex.c | 4

[Mesa-dev] [PATCH 10/13] i915: Remove unused field intel_mipmap_tree::array_spacing_lod0

2017-06-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 9 - 1 file changed, 9 deletions(-) diff --git a/src/mesa/drivers/dri/i915/intel_mipmap_tree.h

[Mesa-dev] [PATCH 14/17] i965: Silence unused parameter warning

2017-06-18 Thread Ian Romanick
From: Ian Romanick brw_blorp.c: In function ‘brw_blorp_clear_depth_stencil’: brw_blorp.c:913:53: warning: unused parameter ‘partial_clear’ [-Wunused-parameter] GLbitfield mask, bool partial_clear)

[Mesa-dev] [PATCH 16/17] i965: Silence uninitialized varible warnings

2017-06-18 Thread Ian Romanick
From: Ian Romanick brw_blorp.c:1007:4: warning: ‘num_layers’ may be used uninitialized in this function [-Wmaybe-uninitialized] blorp_clear_depth_stencil(, _surf, _surf, ^

[Mesa-dev] [PATCH 17/17] intel: Silence many unused parameter warnings in release builds

2017-06-18 Thread Ian Romanick
From: Ian Romanick In file included from brw_context.h:40:0, from brw_blorp.c:32: ../../../../../src/intel/compiler/brw_compiler.h: In function ‘brw_stage_has_packed_dispatch’: ../../../../../src/intel/compiler/brw_compiler.h:1150:61: warning: unused

Re: [Mesa-dev] [PATCH v1 5/7] android: enable texture-float

2017-06-18 Thread Ian Romanick
On 06/16/2017 06:25 AM, Rob Clark wrote: > On Fri, Jun 16, 2017 at 9:18 AM, Emil Velikov > wrote: >> On 15 June 2017 at 21:47, Robert Foss wrote: >>> From: Rob Herring >>> >>> This is required by freedreno at least for GLES3

Re: [Mesa-dev] [PATCH] mesa: rename MESA format names to have the same names as PIPE formats

2017-06-18 Thread DeLap, Russell
Sent from my homecjf ghgrtyyytk th ttuithibygvgbuujjiiggti ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH mesa 2/2] egl: turn one more boolean `int` into a `bool`

2017-06-18 Thread Eric Engestrom
Same as the previous commit, but this one was split out because it's a bit more complicated: this field is given as a pointer to a function, so the function had to be changed as well, and the function was use in a bunch of places, which needed updating as well. Signed-off-by: Eric Engestrom

[Mesa-dev] [PATCH mesa 1/2] egl: turn boolean `int`s into `bool`s

2017-06-18 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/egl/drivers/dri2/egl_dri2.c | 2 +- src/egl/drivers/dri2/egl_dri2.h | 17 + src/egl/drivers/dri2/platform_drm.c | 6 +++--- src/egl/drivers/dri2/platform_wayland.c | 20 ++--

Re: [Mesa-dev] [PATCH v1 5/7] android: enable texture-float

2017-06-18 Thread Ian Romanick
On 06/15/2017 01:47 PM, Robert Foss wrote: > From: Rob Herring > > This is required by freedreno at least for GLES3 support. > > See docs/patents.txt for information about turning this on for s/w > renderers. If I'm not mistaken (I'm not a lawyer, and I don't play one on TV),

Re: [Mesa-dev] [PATCH v3 0/7] mesa/st: glsl_to_tgsi: improved temp-reg lifetime estimation

2017-06-18 Thread Gert Wollny
It seems some of the tests I added require a a too long life-time of the registers so that the actual aim of the patch is no longer achieved. Comments on the code and algorithm structure are welcome, but I have to rethink the tests, because they seem to be too focused on corner-cases that may

Re: [Mesa-dev] [PATCH v3 0/7] mesa/st: glsl_to_tgsi: improved temp-reg lifetime estimation

2017-06-18 Thread Dieter Nützel
Hello Gert, starting with short log from 'git am ...' /opt/mesa> git am ~/Dokumente/Software/DRI-Mesa/Radeon/Mesa/mesa-st-glsl_to_tgsi-improved-temp-reg-lifetime-estimation.mbox Wende an: mesa/st: glsl_to_tgsi move some helper classes to extra files .git/rebase-apply/patch:561: new blank line

Re: [Mesa-dev] [PATCH 1/2] glsl: allow inputs in interface blocks in interpolateAt*

2017-06-18 Thread Ian Romanick
On 06/16/2017 01:37 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Since interface blocks are simply considered groupings of input variables, > this is allowed. > > var->data.must_be_shader_input is now determined on-the-fly after lowering > interface blocks,

[Mesa-dev] [PATCH v3 3/7] mesa/st: glsl_to_tgsi: implement new temporary register lifetime tracker

2017-06-18 Thread Gert Wollny
This patch adds a class for tracking the life times of temporary registers in the glsl to tgsi translation. The algorithm runs in three steps: First, in order to minimize the number of needed memory allocations the program is scanned to evaluate the number of scopes. Then, the program is scanned

[Mesa-dev] [PATCH v3 7/7] mesa/st: glsl_to_tgsi: tie in new temporary register merge approach

2017-06-18 Thread Gert Wollny
This patch ties in the new temporary register lifetime estiamtion and rename mapping evaluation. In order to enable it, the evironment variable MESA_GLSL_TO_TGSI_NEW_MERGE must be set. Performance to compare between the current and the new implementation were measured by running the shader-db in

[Mesa-dev] [PATCH v3 2/7] mesa: Propagate c++11 CXXFLAGS from LLVM_CXXFLAGS to mesa/

2017-06-18 Thread Gert Wollny
For the new register renaming approach C++11 is wanted. --- src/mesa/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/Makefile.am b/src/mesa/Makefile.am index 53f311d2a9..3339926d93 100644 --- a/src/mesa/Makefile.am +++ b/src/mesa/Makefile.am @@ -101,7

[Mesa-dev] [PATCH v3 5/7] mesa/st: glsl_to_tgsi: add register renamame mapping evaluator

2017-06-18 Thread Gert Wollny
The remapping evaluator first sorts the temporary registers ascending based on their first life time instruction, and then uses a binary search to find merge canidates. For the initial sorting it uses std::sort because qsort is quite slow in comparison. By removing the define USE_STL_SORT in

[Mesa-dev] [PATCH v3 4/7] mesa/st: glsl_to_tgsi: add tests for the new temporary lifetime tracker

2017-06-18 Thread Gert Wollny
This patch adds a set of unit tests for the new lifetime tracker. --- configure.ac | 1 + src/mesa/Makefile.am | 2 +- src/mesa/state_tracker/tests/Makefile.am | 41 + .../tests/test_glsl_to_tgsi_lifetime.cpp

[Mesa-dev] [PATCH v3 1/7] mesa/st: glsl_to_tgsi move some helper classes to extra files

2017-06-18 Thread Gert Wollny
To prepare the implementation of a temp register lifetime tracker some of the classes are moved into seperate header/implementation files to make them accessible from other files. Specifically these are: class st_src_reg; class st_dst_reg; class glsl_to_tgsi_instruction; struct

[Mesa-dev] [PATCH v3 6/7] mesa/st: glsl_to_tgsi: Add test set for evaluation of rename mapping

2017-06-18 Thread Gert Wollny
The patch adds tests for the register rename mapping evaluation. --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 71 -- 1 file changed, 66 insertions(+), 5 deletions(-) diff --git a/src/mesa/state_tracker/tests/test_glsl_to_tgsi_lifetime.cpp

[Mesa-dev] [PATCH v3 0/7] mesa/st: glsl_to_tgsi: improved temp-reg lifetime estimation

2017-06-18 Thread Gert Wollny
Dear all, following the comments of Emil and Nicolai I've updated the patch set. Changes with respect to the old version are: - split the changes into more patches - correct formatting errors - remove the use of the STL with one exception though: since in st_glsl_to_tgsi.cpp std::sort is

Re: [Mesa-dev] [PATCH v2 2/3] mesa/st: glsl_to_tgsi Implement a new lifetime tracker for temporaries

2017-06-18 Thread Gert Wollny
Hello Dieter, > W'll start with your attachment > https://bugs.freedesktop.org/attachment.cgi?id=131683 This one is a dirty hack that works around the use of too many registers mostly by allowing to allocate more than the allowed limit and let the bytecode optimizer sort it out (there is also

Re: [Mesa-dev] [PATCH v2 2/3] mesa/st: glsl_to_tgsi Implement a new lifetime tracker for temporaries

2017-06-18 Thread Dieter Nützel
Hello Gert, (hello Nicolai, ;-)) do you have some 'work in progress' ready. Then I'll put my Turks XT back in and test LS2015 (Farming Simulator 2015) under Wine. It show missing details (the driver (the player) and all the other people walking around) with current Mesa git code. I get the

Re: [Mesa-dev] [PATCH v2 2/3] mesa/st: glsl_to_tgsi Implement a new lifetime tracker for temporaries

2017-06-18 Thread Gert Wollny
Hello Nicolai, Am Sonntag, den 18.06.2017, 12:05 +0200 schrieb Nicolai Hähnle: >    > >   if HAVE_XLIB_GLX > >   SUBDIRS += drivers/x11 > > @@ -101,7 +101,7 @@ AM_CFLAGS = \ > >    $(VISIBILITY_CFLAGS) \ > >    $(MSVC2013_COMPAT_CFLAGS) > >   AM_CXXFLAGS = \ > > - $(LLVM_CFLAGS) \ > > + 

Re: [Mesa-dev] [PATCH 3/5] mesa: add KHR_no_error support for glGetTexture*HandleARB()

2017-06-18 Thread Samuel Pitoiset
On 06/15/2017 11:59 PM, Timothy Arceri wrote: On 14/06/17 19:27, Samuel Pitoiset wrote: It would be nice to have a no_error path for _mesa_test_texobj_completeness() because this function doesn't only test if the texture is complete. Are you sure? I don't see any error checks in there.

Re: [Mesa-dev] [PATCH v2 2/3] mesa/st: glsl_to_tgsi Implement a new lifetime tracker for temporaries

2017-06-18 Thread Nicolai Hähnle
On 16.06.2017 11:32, Gert Wollny wrote: This patch adds new classes and tests to implement a tracker for the life time of temporary registers for the register renaming stage of glsl_to_tgsi. The tracker aims at estimating the shortest possible life time for each register. The code base requires